On 26-07-17 22:25, Ian Molton wrote:
The 4 IO functions in this patch are incorrect as they use compiler types to determine how many bytes to send to the hardware.
I see no incorrectness here. It is validating the regsz value and those compiler types are pretty well defined I would say. Anyway, I find the use of sizeof() here pretty useless so thanks for the change. Reviewed-by: Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx>
Signed-off-by: Ian Molton <ian@xxxxxxxxxxxxxx> --- .../wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-)