On 26-05-17 13:18, Hans de Goede wrote:
> Hi,
>
> On 26-05-17 13:15, Kalle Valo wrote:
>> Hans de Goede <hdegoede@xxxxxxxxxx> writes:
>>
>>> From: Arend Van Spriel <arend.vanspriel@xxxxxxxxxxxx>
>
> Ah I see I set the Author to Arend when I added this to my
> tree a while back, that is fine as he did all the work
> for this one. I was under the impression Arend would submit
> this himself, but since I did not see a submission yet
> I decided to go ahead and submit this.
I did not get to sending it last week. My work hours for open-source
have been reduced by 80% so time is limited. I have it queued.
>>> This fixes the following errors showing up in dmesg:
>>>
>>> [ 32.528271] brcmfmac: brcmf_sdio_rxglom: sublen 524 not multiple of 8
>>> [ 32.528296] brcmfmac: brcmf_sdio_rxglom: sublen 84 not multiple of 8
>>> [ 33.063241] brcmfmac: brcmf_sdio_rxglom: sublen 1492 not multiple
>>> of 8
>>>
>>> Fixes: 6e84ab604bde ("properly align buffers ... with 64 bit DMA")
>>> Suggested-by: Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx>
>>> Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx>
>>
>> I'll queue this to 4.12.
>
> Thank you, given that Arend is set as the Author you can add my:
>
> Tested-by: Hans de Goede <hdegoede@xxxxxxxxxx>
>
> And maybe drop the Suggested-by: Arend van Spriel ?
Seems to me you need my Signed-off-by: as I do not see that in this patch.
Regards,
Arend
