diff mbox

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

Message ID 1503554561-18257-1-git-send-email-suneelglinux@gmail.com
State Superseded
Delegated to: Tom Rini
Headers show

Commit Message

Suneel Garapati Aug. 24, 2017, 6:02 a.m. UTC
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(-)

Comments

Bin Meng Aug. 24, 2017, 7:54 a.m. UTC | #1
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>
Suneel Garapati Sept. 1, 2017, 7:46 a.m. UTC | #2
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>
diff mbox

Patch

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 */