Re: sparc-leon patches (to follow) round 2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Jun 20, 2009 at 01:50:12PM +0200, Sam Ravnborg wrote:
> On Fri, Jun 12, 2009 at 10:13:31AM +0200, Konrad Eisele wrote:
> > I'll shortly send the sparc-leon patches as message relies to this
> > thread. I applied the changes that Julian suggested. Thank you for the
> > comments. Waiting for more.
> 
> Hi Konrad.
> Some general comments.
> 
> 1) subject/changelog
> When writing your changelog keep the subject (the first line
> or the changelog) and the body apart with a single line.
> You failed to do so and the result is some over-long subject lines.
> 
> 2) sign-off your patches
> See Documentation/Submitting-patches
> 
> The syntax is simple:
> Signed-off-by: Sam Ravnborg <sam@xxxxxxxxxxxx>

The subject line needs fixing too.
It should read:

sparc,leon: CONFIG_SPARC_LEON option and leon specific files

When you use "git send-email" it will add [PATCH n/m],
so do not worry about that.

sparc - sibnals that this is sparc specific.
,leon - signals that this is leon specific within sparc.
So adding these small things to your subject allows us
to see where this belongs when we look at 500 new
patches applied to -linus.

	Sam
--
To unsubscribe from this list: send the line "unsubscribe sparclinux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Development]     [DCCP]     [Linux ARM Development]     [Linux]     [Photo]     [Yosemite Help]     [Linux ARM Kernel]     [Linux SCSI]     [Linux x86_64]     [Linux Hams]

  Powered by Linux