Re: [PATCH] ata: ahci_mvebu: Fix wrongly set base address for the MBus window setting

From: Tejun Heo
Date: Tue May 26 2015 - 14:47:38 EST


On Tue, May 26, 2015 at 06:47:23PM +0200, Gregory CLEMENT wrote:
> From: Nadav Haklai <nadavh@xxxxxxxxxxx>
>
> According to the Armada 38x datasheet, the window base address
> registers value is set in bits [31:4] of the register and corresponds
> to the transaction address bits [47:20].
>
> Therefore, the 32bit base address value should be shifted right by
> 20bits and left by 4bits, resulting in 16 bit shift right.
>
> The bug as not been noticed yet because if the memory available on
> the platform is less than 2GB, then the base address is zero.
>
> [gregory.clement@xxxxxxxxxxxxxxxxxx: add extra-explanation]
>
> Fixes: a3464ed2f14 (ata: ahci_mvebu: new driver for Marvell Armada 380
> AHCI interfaces)
> Signed-off-by: Nadav Haklai <nadavh@xxxxxxxxxxx>
> Reviewed-by: Omri Itach <omrii@xxxxxxxxxxx>
> Signed-off-by: Gregory CLEMENT <gregory.clement@xxxxxxxxxxxxxxxxxx>
> Cc: <stable@xxxxxxxxxxxxxxx>

Applied to libata/for-4.1-fixes.

Thank you.

--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/