Search Linux Wireless

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
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux