= Patchset changelog =

This updated version of the patchset fixes a comment style issue in patch 2/2.
Patch 1/2, which was acked by Jes Sorensen <Jes.Sorensen at redhat.com>, is
resubmitted unchanged.


= Patchset description =

Apparently, there is some error in struct rt_8723a_firmware_hdr. Indeed,
according to the comments in the code, the first 6 fields should be confined
in the first 8 bytes, but they actually occupy 9 bytes. Moreover, the total
size of the struct is 33 bytes, while function rtl8723a_FirmwareDownload() in
hal/rtl8723a_hal_init.c assumes a 32-byte-long header.

The "guilty" field is probably Rsvd1, which in similar drivers (e.g.,
rtl8723ae), has type u8, while in this case has type u16. Therefore, the first
patch in this patchset assumes that the type of Rsvd is wrong and changes it
to u8.

The second patch also changes struct rt_8723a_firmware_hdr by making its other
fields use the __leXX types in order to fix some sparse warnings.


Daniele Alessandrelli (2):
  staging: rtl8723au: include/rtl8723a_hal.h: fix Rsvd1 size in fw
    header struct
  staging: rtl8723au: include/rtl8723a_hal.h: use __leXX types in fw
    header struct

 drivers/staging/rtl8723au/include/rtl8723a_hal.h | 19 ++++++++++---------
 1 file changed, 10 insertions(+), 9 deletions(-)

-- 
1.8.3.2

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to