[U-Boot] [U-BOOT][PATCH] drivers: ahci: write upper 32 bits for clb and fis registers

If 64-bit capability is supported, commandlistbase and fis base should be split as lower32 and upper32. upper32 should be written to PORT_(LST/FIS)_ADDR_HI.
Signed-off-by: Suneel Garapati suneelglinux@gmail.com --- drivers/ata/ahci.c | 11 +++++++++-- include/ahci.h | 1 + 2 files changed, 10 insertions(+), 2 deletions(-)
diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c index 5e4df19..df50c82 100644 --- a/drivers/ata/ahci.c +++ b/drivers/ata/ahci.c @@ -607,10 +607,17 @@ static int ahci_port_start(struct ahci_uc_priv *uc_priv, u8 port) pp->cmd_tbl_sg = (struct ahci_sg *)(uintptr_t)virt_to_phys((void *)mem);
- writel_with_flush((unsigned long)pp->cmd_slot, + if (uc_priv->cap & HOST_CAP_64) + writel_with_flush(cpu_to_le32(UPPER32(pp->cmd_slot)), + port_mmio + PORT_LST_ADDR_HI); + writel_with_flush(cpu_to_le32(LOWER32(pp->cmd_slot)), port_mmio + PORT_LST_ADDR);
- writel_with_flush(pp->rx_fis, port_mmio + PORT_FIS_ADDR); + if (uc_priv->cap & HOST_CAP_64) + writel_with_flush(cpu_to_le32(UPPER32(pp->rx_fis)), + port_mmio + PORT_FIS_ADDR_HI); + writel_with_flush(cpu_to_le32(LOWER32(pp->rx_fis)), + port_mmio + PORT_FIS_ADDR);
#ifdef CONFIG_SUNXI_AHCI sunxi_dma_init(port_mmio); diff --git a/include/ahci.h b/include/ahci.h index 33171b7..80e7f13 100644 --- a/include/ahci.h +++ b/include/ahci.h @@ -40,6 +40,7 @@ #define HOST_RESET (1 << 0) /* reset controller; self-clear */ #define HOST_IRQ_EN (1 << 1) /* global IRQ enable */ #define HOST_AHCI_EN (1 << 31) /* AHCI enabled */ +#define HOST_CAP_64 (1 << 31) /* 64bit addressing capability */
/* Registers for each SATA port */ #define PORT_LST_ADDR 0x00 /* command list DMA addr */

On Thu, Aug 24, 2017 at 2:02 PM, Suneel Garapati suneelglinux@gmail.com wrote:
If 64-bit capability is supported, commandlistbase and fis base should be split as lower32 and upper32. upper32 should be written to PORT_(LST/FIS)_ADDR_HI.
Signed-off-by: Suneel Garapati suneelglinux@gmail.com
drivers/ata/ahci.c | 11 +++++++++-- include/ahci.h | 1 + 2 files changed, 10 insertions(+), 2 deletions(-)
Reviewed-by: Bin Meng bmeng.cn@gmail.com

Hi,
I need to send v1 with LOWER32 and UPPER32 macro definitions, so bad to miss them.
Regards, Suneel
On Thu, Aug 24, 2017 at 12:54 AM, Bin Meng bmeng.cn@gmail.com wrote:
On Thu, Aug 24, 2017 at 2:02 PM, Suneel Garapati suneelglinux@gmail.com wrote:
If 64-bit capability is supported, commandlistbase and fis base should be split as lower32 and upper32. upper32 should be written to PORT_(LST/FIS)_ADDR_HI.
Signed-off-by: Suneel Garapati suneelglinux@gmail.com
drivers/ata/ahci.c | 11 +++++++++-- include/ahci.h | 1 + 2 files changed, 10 insertions(+), 2 deletions(-)
Reviewed-by: Bin Meng bmeng.cn@gmail.com
participants (2)
-
Bin Meng
-
Suneel Garapati