[SRU][Xenial][PATCH v2] Add 0cf3:e009 to btusb

Stefan Bader stefan.bader at canonical.com
Tue Nov 15 10:01:56 UTC 2016


On 15.11.2016 10:41, Phidias Chiang wrote:
> On Tue, Nov 15, 2016 at 5:15 PM, Stefan Bader
> <stefan.bader at canonical.com> wrote:
>> On 15.11.2016 10:13, Stefan Bader wrote:
>>> On 15.11.2016 09:27, Phidias Chiang wrote:
>>>
>>> Which of the three versions (two of those are v2) is valid and what is the
>>> difference between v1 and v2?
>>>
>> Actually 4. I missed a second version of v1...
>>
> 
> One is for Xenial and one is for Yakkety, because the patch cannot be
> used on different releases. And I messed up the title for the patch so
> I sent a v2 with the title fixed and that's the only difference...
> 
> 
Ah ok. I missed that detail when reading those patches this morning. Maybe,
though this is not a requirement, it would help if you mentioned the additional
version in one of the cover emails. Or for single patch submissions you may
re-arrange things to use a single cover followed by the patches. Like:

- [SRU Xenial/Yakkety] <generic description>
  +- [PATCH Xenial] ...
  +- [PATCH Yakkety] ...

Again, those are just suggestions to help someone (like me) reviewing things in
the morning. ;)

-Stefan


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20161115/349b7cc8/attachment.sig>


More information about the kernel-team mailing list