Re: [U-Boot] [PATCH] powerpc: clean up DIU macro definitions for the MPC8610HPCD and MPC5121ADS
Dear Timur Tabi, In message <20110215215557.8f4c2151...@gemini.denx.de> I wrote: > > In message <4d5af2c9.10...@freescale.com> you wrote: > > > > > And please put the respective arch custodians on Cc: To make myself more clear: Normally, you should put the respective board maintainer(s) on Cc:. Only in cases like here, where the boards are orphaned and without registered maintainers, the respective arch custodians should be Cc:ed. 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 Die ganzen Zahlen hat der liebe Gott geschaffen, alles andere ist Menschenwerk... Leopold Kronecker ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] powerpc: clean up DIU macro definitions for the MPC8610HPCD and MPC5121ADS
Dear Timur Tabi, In message <4d5af2c9.10...@freescale.com> you wrote: > > > And please put the respective arch custodians on Cc: > > I did CC: Kumar. He's the PowerPC arch custodian. No. There is no such thing as a "PowerPC" custodian. Kumar is responsible for 85xx/86xx. This patch also affects 5xxx. For details please see http://www.denx.de/wiki/U-Boot/Custodians 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 For every problem there is one solution which is simple, neat, and wrong.- H. L. Mencken ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] powerpc: clean up DIU macro definitions for the MPC8610HPCD and MPC5121ADS
Wolfgang Denk wrote: >> > -#undef CONFIG_VIDEO >> > +#undef CONFIG_FSL_DIU_FB > Please do not undef what is not defiend anyway. Would you be okay with this: /* video */ /* #define CONFIG_FSL_DIU_FB */ #ifdef CONFIG_FSL_DIU_FB > And please put the respective arch custodians on Cc: I did CC: Kumar. He's the PowerPC arch custodian. I don't consider this to be a patch for the video repository, so I didn't CC: Anatolij. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
Re: [U-Boot] [PATCH] powerpc: clean up DIU macro definitions for the MPC8610HPCD and MPC5121ADS
Dear Timur Tabi, In message <1297804966-21532-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 > --- > 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..d28e29b 100644 > --- a/include/configs/MPC8610HPCD.h > +++ b/include/configs/MPC8610HPCD.h > @@ -21,12 +21,13 @@ > > #define CONFIG_SYS_TEXT_BASE0xfff0 > > -#define CONFIG_FSL_DIU_FB1 /* FSL DIU */ > > /* video */ > -#undef CONFIG_VIDEO > +#undef CONFIG_FSL_DIU_FB Please do not undef what is not defiend anyway. ... > diff --git a/include/configs/mpc5121ads.h b/include/configs/mpc5121ads.h > index f966325..72c8e3f 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_FB1 /* FSL DIU */ > > #define CONFIG_SYS_TEXT_BASE0xFFF0 > > /* video */ > -#undef CONFIG_VIDEO > +#undef CONFIG_FSL_DIU_FB Ditto. And please put the respective arch custodians on Cc: 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 It is common sense to take a method and try it. If it fails, admit it frankly and try another. But above all, try something. - Franklin D. Roosevelt ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
[U-Boot] [PATCH] powerpc: clean up DIU macro definitions for the MPC8610HPCD and MPC5121ADS
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 --- 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..d28e29b 100644 --- a/include/configs/MPC8610HPCD.h +++ b/include/configs/MPC8610HPCD.h @@ -21,12 +21,13 @@ #defineCONFIG_SYS_TEXT_BASE0xfff0 -#define CONFIG_FSL_DIU_FB 1 /* FSL DIU */ /* video */ -#undef CONFIG_VIDEO +#undef CONFIG_FSL_DIU_FB -#ifdef CONFIG_VIDEO +#ifdef CONFIG_FSL_DIU_FB +#define CONFIG_SYS_DIU_ADDR(CONFIG_SYS_CCSRBAR + 0x2c000) +#define CONFIG_VIDEO #define CONFIG_CMD_BMP #define CONFIG_CFB_CONSOLE #define CONFIG_VGA_AS_SINGLE_DEVICE @@ -88,8 +89,6 @@ #define CONFIG_SYS_CCSRBAR_PHYS_HIGH 0x0 #define CONFIG_SYS_CCSRBAR_PHYSCONFIG_SYS_CCSRBAR_PHYS_LOW -#define CONFIG_SYS_DIU_ADDR(CONFIG_SYS_CCSRBAR+0x2c000) - /* DDR Setup */ #define CONFIG_FSL_DDR2 #undef CONFIG_FSL_DDR_INTERACTIVE @@ -494,9 +493,6 @@ #define CONFIG_WATCHDOG/* watchdog enabled */ #define CONFIG_SYS_WATCHDOG_FREQ 5000/* Feed interval, 5s */ -/*DIU Configuration*/ -#define DIU_CONNECT_TO_DVI /* DIU controller connects to DVI encoder*/ - /* * Miscellaneous configurable options */ diff --git a/include/configs/mpc5121ads.h b/include/configs/mpc5121ads.h index f966325..72c8e3f 100644 --- a/include/configs/mpc5121ads.h +++ b/include/configs/mpc5121ads.h @@ -46,14 +46,15 @@ */ #define CONFIG_E3001 /* E300 Family */ #define CONFIG_MPC512X 1 /* MPC512X family */ -#define CONFIG_FSL_DIU_FB 1 /* FSL DIU */ #defineCONFIG_SYS_TEXT_BASE0xFFF0 /* video */ -#undef CONFIG_VIDEO +#undef CONFIG_FSL_DIU_FB -#ifdef CONFIG_VIDEO +#ifdef CONFIG_FSL_DIU_FB +#define CONFIG_SYS_DIU_ADDR(CONFIG_SYS_IMMR + 0x2100) +#define CONFIG_VIDEO #define CONFIG_CMD_BMP #define CONFIG_CFB_CONSOLE #define CONFIG_VGA_AS_SINGLE_DEVICE @@ -74,7 +75,6 @@ #define CONFIG_MISC_INIT_R #define CONFIG_SYS_IMMR0x8000 -#define CONFIG_SYS_DIU_ADDR(CONFIG_SYS_IMMR+0x2100) #define CONFIG_SYS_MEMTEST_START 0x0020 /* memtest region */ #define CONFIG_SYS_MEMTEST_END 0x0040 -- 1.7.3.4 ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
