> -----Original Message-----
> From: Wolfgang Denk [mailto:w...@denx.de]
> Sent: Tuesday, October 25, 2011 3:05 AM
> To: Zang Roy-R61911
> Cc: u-boot@lists.denx.de; Fleming Andy-AFLEMING; Kumar Gala
> Subject: Re: [U-Boot] [PATCH] powerpc/fm: remove the TBIPA setting on platform
> code
> 
> Dear Zang Roy-R61911,
> 
> In message <2239AC579C7D3646A720227A37E02681200C29@039-SN1MPN1-
> 004.039d.mgd.msft.net> you wrote:
> >
> > How about this way:
> > Subject: net/frame manager: remove TBI PHY address register setting on plat=
> > form related code
> >
> > Then I add the other more detailed required information in the description =
> > body?
> 
> You are putting too much low level detail into the Subject: line,
> while leaving out important higher level information, like
> architecture, SoC, etc.
> 
> How about something like:
> 
>       QorIQ: fix network frame manager settings
This subject is too big. What kind of network frame manager setting? It will be 
hard for someone to find useful information in the subject.
Prefer:
QorIQ: fix network frame manager TBI PHY address settings

Roy

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to