Re: [PATCH 1/8] brcmfmac: reset two D11 cores if chip has two D11 cores

From: Kalle Valo
Date: Tue Dec 10 2019 - 05:20:08 EST


Soeren Moch <smoch@xxxxxx> writes:

> On 10.12.19 10:08, Kalle Valo wrote:
>> Soeren Moch <smoch@xxxxxx> writes:
>>
>>> From: Wright Feng <wright.feng@xxxxxxxxxxx>
>>>
>>> There are two D11 cores in RSDB chips like 4359. We have to reset two
>>> D11 cores simutaneously before firmware download, or the firmware may
>>> not be initialized correctly and cause "fw initialized failed" error.
>>>
>>> Signed-off-by: Wright Feng <wright.feng@xxxxxxxxxxx>
>> Soeren's s-o-b missing at least in patches 1, 6 and 7. Please read:
>>
>> https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#signed-off-by_missing
>>
>
> OK, also for unmodified patches another s-o-b is required.

Yes, every patch you submit needs to have your s-o-b to mark that you
agree with Developer's Certificate of Origin.

https://www.kernel.org/doc/html/latest/process/submitting-patches.html#sign-your-work-the-developer-s-certificate-of-origin

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