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

Ben Dooks ben.dooks at codethink.co.uk
Wed Jun 8 03:11:30 PDT 2016


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 at rock-chips.com>
> 
> Test on Rockchip platform, so
> 
> Tested-by: Shawn Lin <shawn.lin at rock-chips.com>

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


-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius



More information about the linux-arm-kernel mailing list