Search Linux Wireless

Re: [PATCH v2 01/18] mt76: mt76u: check tx_status_data pointer in mt76u_tx_tasklet

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

 



Lorenzo Bianconi <lorenzo@xxxxxxxxxx> writes:

> On Jan 14, Kalle Valo wrote:
>> Lorenzo Bianconi <lorenzo@xxxxxxxxxx> writes:
>> 
>> > New devices (e.g. mt7663u) do not rely on stats workqueue to load tx
>> > statistics but will be reported by the firmware. Check tx_status_data
>> > pointer in mt76u_tx_tasklet in order to reuse tx tasklet for new devices
>> >
>> > Signed-off-by: Sean Wang <sean.wang@xxxxxxxxxxxx>
>> > Signed-off-by: Lorenzo Bianconi <lorenzo@xxxxxxxxxx>
>> 
>> Why is Sean's s-o-b first? Lorenzo is marked as the author so his s-o-b
>> should be first.
>
> Hi Kalle,
>
> my bad, I did not pay attention on it, I should put my SoB first.
>
>> 
>> Also I recommend taking a look at the new Co-Developed-by tag.
>
> I think Co-Developed-by fits better.

Do note that you still need s-o-b as well:

https://www.kernel.org/doc/html/latest/process/submitting-patches.html#when-to-use-acked-by-cc-and-co-developed-by

> @Felix and Kalle: do you prefer to resubmit?

I prefer to resubmit, but not a strong requirement and don't know if
Felix already applied this.

-- 
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches



[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Wireless Personal Area Network]     [Linux Bluetooth]     [Wireless Regulations]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite Hiking]     [MIPS Linux]     [ARM Linux]     [Linux RAID]

  Powered by Linux