diff mbox series

[v2] usb: ehci-orion: Extend DMA mask to 64 bit for AC5 platform

Message ID 20221018113401.32229-1-vadym.kochan@plvision.eu
State New
Headers
Series [v2] usb: ehci-orion: Extend DMA mask to 64 bit for AC5 platform |

Commit Message

Vadym Kochan Oct. 18, 2022, 11:34 a.m. UTC 
  From: Elad Nachman <enachman@marvell.com>

AC5 is a 64-bit platform so extend the dma mask accordingly.

Checked this mask on armv7 a38x SoC (which has this
USB controller) platform with simple fs ops on the storage device
but on older 4.14 Linux version.

Signed-off-by: Elad Nachman <enachman@marvell.com>
Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
---
v2:
   Add missing description.

 drivers/usb/host/ehci-orion.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Greg KH Oct. 18, 2022, 11:57 a.m. UTC | #1 
On Tue, Oct 18, 2022 at 02:34:00PM +0300, Vadym Kochan wrote:
> From: Elad Nachman <enachman@marvell.com>
> 
> AC5 is a 64-bit platform so extend the dma mask accordingly.
> 
> Checked this mask on armv7 a38x SoC (which has this
> USB controller) platform with simple fs ops on the storage device
> but on older 4.14 Linux version.

Odd line-wrapping.

That aside, I have no idea what this changelog text means at all.  What
is 4.14 here for?  Does this fix a specific commit id?  What is being
changed?  Why is it being changed?  What is happening?

confused,

greg k-h
Alan Stern Oct. 18, 2022, 2:35 p.m. UTC | #2 
On Tue, Oct 18, 2022 at 02:34:00PM +0300, Vadym Kochan wrote:
> From: Elad Nachman <enachman@marvell.com>
> 
> AC5 is a 64-bit platform so extend the dma mask accordingly.
> 
> Checked this mask on armv7 a38x SoC (which has this
> USB controller) platform with simple fs ops on the storage device
> but on older 4.14 Linux version.
> 
> Signed-off-by: Elad Nachman <enachman@marvell.com>
> Signed-off-by: Vadym Kochan <vadym.kochan@plvision.eu>
> ---
> v2:
>    Add missing description.
> 
>  drivers/usb/host/ehci-orion.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/host/ehci-orion.c b/drivers/usb/host/ehci-orion.c
> index a3454a3ea4e0..c6205abebbdf 100644
> --- a/drivers/usb/host/ehci-orion.c
> +++ b/drivers/usb/host/ehci-orion.c
> @@ -230,7 +230,7 @@ static int ehci_orion_drv_probe(struct platform_device *pdev)
>  	 * set. Since shared usb code relies on it, set it here for
>  	 * now. Once we have dma capability bindings this can go away.
>  	 */
> -	err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
> +	err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64));
>  	if (err)
>  		goto err;
>  

NAK.  The description says this changes the DMA mask on AC5 platforms to 
64 bits, but that isn't what the patch actually does.  It changes the 
DMA mask on _all_ platforms using the orion hardware.

Alan Stern
diff mbox series

Patch

diff --git a/drivers/usb/host/ehci-orion.c b/drivers/usb/host/ehci-orion.c
index a3454a3ea4e0..c6205abebbdf 100644
--- a/drivers/usb/host/ehci-orion.c
+++ b/drivers/usb/host/ehci-orion.c
@@ -230,7 +230,7 @@  static int ehci_orion_drv_probe(struct platform_device *pdev)
 	 * set. Since shared usb code relies on it, set it here for
 	 * now. Once we have dma capability bindings this can go away.
 	 */
-	err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
+	err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(64));
 	if (err)
 		goto err;