Re: [PATCHv2 00/82] staging: brcm80211: resubmit previously posted patches

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

 



On Mon, Aug 08, 2011 at 04:29:49PM +0200, Sedat Dilek wrote:
>
> 1. Drop "staging" from subject-lines? Saves some chars.

The Staging is required for staging code.

> 2. Using "staging/brcm80211" would also save some chars (and is more
> common to me).

No one would complain if the prefix were "staging/brcm80211" but
actually the way Arend has it is prefered.

git log --pretty=oneline --author=gregkh drivers/staging/

> 3. "PATCH v2" should be with a space?

git am doesn't care.  It strips everything out correctly.

> 4. Something like "for-3.1" would clearly indicate for which
> kernel-release this mega-series is for (also "staging" could be
> dropped as it is known where the driver is managed)
> 
> $ git format-patch --signoff --cover-letter --subject-prefix="PATCH v2
> for-3.1" -82 (or whatever $FIRST..$LAST)

Everything was explained in the cover letter, so I think it's ok.

regards,
dan carpenter
_______________________________________________
devel mailing list
devel@xxxxxxxxxxxxxxxxxxxxxx
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel


[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux