-----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 <aflem...@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----- _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
