Re: [PATCH] staging: r8188eu: clean up endianness issues

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

 



On 8/20/21 5:26 AM, David Laight wrote:
From: Aakash Hemadri
Sent: 18 August 2021 16:53

Fix these sparse warnings:

Did you test this code before and after the changes?

I think you've changed the behaviour on LE systems which
are probably the ones it was actually tested on.

Don't blindly change code to fix sparse warnings.

I'm late getting into this conversation. Gmail thought the original patches were spam.

First of all, we need to change the behavior as the original code is wrong; however, the code involves PPPoE, which I have never used, and never tested. I still need to check the correct endian values for one of the packet types. I hate reading IEEE official documents!!!!

I should have a review of the patches later today.

Larry





[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux