Dear Timur Tabi, In message <1297807109-21948-1-git-send-email-ti...@freescale.com> you wrote: > Clean up the macro defintions used to enable DIU (video) support on the > MPC8610HPCD and the MPC5121ADS so that they look more like the P1022DS, > which is newer. > > Signed-off-by: Timur Tabi <ti...@freescale.com> > --- > include/configs/MPC8610HPCD.h | 12 ++++-------- > include/configs/mpc5121ads.h | 8 ++++---- > 2 files changed, 8 insertions(+), 12 deletions(-) > > diff --git a/include/configs/MPC8610HPCD.h b/include/configs/MPC8610HPCD.h > index 03ee394..8022895 100644 > --- a/include/configs/MPC8610HPCD.h > +++ b/include/configs/MPC8610HPCD.h > @@ -21,12 +21,13 @@ > > #define CONFIG_SYS_TEXT_BASE 0xfff00000 > > -#define CONFIG_FSL_DIU_FB 1 /* FSL DIU */ > > /* video */ > -#undef CONFIG_VIDEO > +/* #define CONFIG_FSL_DIU_FB */
Please do not add dead code. > diff --git a/include/configs/mpc5121ads.h b/include/configs/mpc5121ads.h > index f966325..01dd096 100644 > --- a/include/configs/mpc5121ads.h > +++ b/include/configs/mpc5121ads.h > @@ -46,14 +46,15 @@ > */ > #define CONFIG_E300 1 /* E300 Family */ > #define CONFIG_MPC512X 1 /* MPC512X family */ > -#define CONFIG_FSL_DIU_FB 1 /* FSL DIU */ > > #define CONFIG_SYS_TEXT_BASE 0xFFF00000 > > /* video */ > -#undef CONFIG_VIDEO > +/* #define CONFIG_FSL_DIU_FB */ Ditto. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de No man knows what true happiness is until he gets married. By then, of course, its too late. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
