Re: [PATCH] mmc: dw_mmc: fix 32bit little-endian access of des1 field

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

 



On 08/06/16 10:55, Shawn Lin wrote:
> Hi Ben,
> 
> On 2016-6-7 21:37, Ben Dooks wrote:
>> The IDMAC_SET_BUFFER1_SIZE() macro modifies des1, but does
>> not check if the value being passed is big or little endian
>> desptire the des1 field being marked as __le32.
> 
> Looks good.
> 
> Reviewed-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx>
> 
> Test on Rockchip platform, so
> 
> Tested-by: Shawn Lin <shawn.lin@xxxxxxxxxxxxxx>

Thanks, I'lkl prod matt to add his tested line.


-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux