[U-Boot] [PATCH] powerpc/fm: remove the TBIPA setting on platform code

Zang Roy-R61911 r61911 at freescale.com
Mon Oct 24 04:42:01 CEST 2011



> -----Original Message-----
> From: Wolfgang Denk [mailto:wd at denx.de]
> Sent: Monday, October 24, 2011 3:37 AM
> To: Zang Roy-R61911
> Cc: u-boot at lists.denx.de; Fleming Andy-AFLEMING; Kumar Gala
> Subject: Re: [U-Boot] [PATCH] powerpc/fm: remove the TBIPA setting on platform
> code
> 
> Dear Roy Zang,
> 
> In message <1319178713-12472-1-git-send-email-tie-fei.zang at freescale.com> you
> wrote:
> > TBIPA has been set in dtsec_init_phy () funciton in drivers/net/fm/eth.c
> >
> > So remove the duplicate code on platform Ethernet code.
> >
> > Signed-off-by: Roy Zang <tie-fei.zang at freescale.com>
> > Cc: Andy Fleming <afleming at freescale.com>
> > Cc: Kumar Gala <galak at kernel.crashing.org>
> 
> Please change the Subject: so everybody understands what you are
> doing. "powerpc/fm" is not exactly clear to everybody, and neither is
> TBIPA.
> 
> Nor is clear which processors / processor families / boards are
> affected.
Per my understand, subject is a summary of the patch. poweper/fm and TBIPA should almost be OK for the subject. I can point out that the code is about the network code in subject.
for example, Subject:
powerpc/fm: remove the TBIPA setting on platform network related code

Then I add more in the patch description to explain fm, TBIPA, processors, processor families/boards affected.

Thanks.
Roy



More information about the U-Boot mailing list