[U-Boot] [PATCH] corenet: Disable video on P2020DS

[Tom Rini]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 01/25/2013 02:25 PM, Scott Wood wrote:
> On 01/25/2013 12:50:59 PM, Tom Rini wrote:
>> On Fri, Jan 25, 2013 at 10:38:01AM -0600, Andy Fleming wrote:
>> 
>>> The P2020DS build had grown too large, and video support isn't
>>>  enabled in almost any other Freescale board. Disabling it 
>>> allows us to keep building, and provides options for reenabling
>>> it later.
>>> 
>>> Signed-off-by: Andy Fleming <afleming at freescale.com>
>> 
>> Now we may start having dead code around, yes?  Can you perhaps 
>> get away with making this be disable video or something else and
>>  add a P2020DS_video boards.cfg entry or similar?  Thanks!
> 
> There are already 5 P2020DS targets, and there *should* be 8 (why 
> is there no 36BIT version of DDR2, SDCARD, or SPIFLASH?).  This 
> would expand it to 16.  Ideally we would have something like 
> kconfig, but until then I don't see a reasonable alternative to 
> saying that certain config symbols are user-settable by tweaking 
> the board config file.

That's fine, in general.  But does this patch now leave us with
non-build testing video code?  That way lies bitrot, so yes, please
add a 6th target so that when someone needs to hand tweak their
P2020DS setup for this, not that, yes this and not that, oh and video,
they can have some confidence the code still builds.  Or say that
P1020/1022 having video on still too means the code in question is
still used.  That would also be fine.  Thanks.

- -- 
Tom
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iQIcBAEBAgAGBQJRAt5xAAoJENk4IS6UOR1W/cYP/2r3hx6uihLc8BRYfOrHDCov
drjzyOEN9bL/y9vDTVsbYC5HpTWv7ZdsC3JpJ9fWY/ljRAzWuQIYJ6Exvcw7PDEw
6lNdyQXL+NURZ1SreeK0YxdQrTSRtpMn69R+GIEX5Msk6JmZ8Z+qrHYXv8zJm80d
Or6CreG2wk5mm3IWZW+qf9mLIc8SK6uHil8XrXuGPYUSYKFaLpV/9hgUxh3138Dz
OMdUSZZEv+4kfab9nqFgHdfbNmFqrKZsyUZ0Ig+nqDU4/HimasPmud1PmRkGywua
NJP/BYcsMbnjhVzyhLSL3Oj8sPZHTX4668W42ufr4hTpvUoRlMOILE43nqYn3atr
mWCECUPKChR2qXyg7Qnfkj8jiuIEzSJ5FBsBn8T7JldcZhZbOA/uI3xMsXfw57SI
/OrkoOZ3Hcx8LIdCiNhCEoWN6WS/CeSBw1wE2Re1qTKGwMQwtMhi/YrwthB3O6NS
q64gM3Fl5PgzQ7GK+mGIEO/GVgR8Okg7mZG7pF8RjIPQbL9bKBsMJyU7O8Z8DD+2
/OM4Y5Jw8qraN6HK4aOvLYjV2kkkfUQgU9Bo6/SBVndF5FMynMwUdt4P2sC5D4iw
SL6r43XjEQEKbJu/NB3SikI3qAdd8sdzjWvuF3JYAS2iUJ3RBu9wpwGw1kO0QZI5
OE3bYgwhqtt6WeW8Nl89
=ZCH1
-----END PGP SIGNATURE-----