diff mbox

[2/3] usb: host: xhci-rcar: Simplify getting the firmware name for R-Car Gen3

Message ID 1530614818-7634-3-git-send-email-yoshihiro.shimoda.uh@renesas.com (mailing list archive)
State New, archived
Headers show

Commit Message

Yoshihiro Shimoda July 3, 2018, 10:46 a.m. UTC
This patch simplifies getting the firmware name for R-Car Gen3.
Almost all R-Car Gen3 USB3.0 Host controllers use "V3". But,
r8a7795 ES1.x SoCs only should use "V2". Since the xhci-plat already
has the firmware_name of R-Car Gen3 as "V3", the xhci-rcar doesn't
need some members of rcar_quirks_match.

Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
---
 drivers/usb/host/xhci-rcar.c | 17 ++---------------
 1 file changed, 2 insertions(+), 15 deletions(-)

Comments

Simon Horman July 13, 2018, 7:25 a.m. UTC | #1
On Tue, Jul 03, 2018 at 07:46:57PM +0900, Yoshihiro Shimoda wrote:
> This patch simplifies getting the firmware name for R-Car Gen3.
> Almost all R-Car Gen3 USB3.0 Host controllers use "V3". But,
> r8a7795 ES1.x SoCs only should use "V2". Since the xhci-plat already
> has the firmware_name of R-Car Gen3 as "V3", the xhci-rcar doesn't
> need some members of rcar_quirks_match.
> 
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

I would slightly prefer if this patch was squashed into 1/3.
But nonetheless:

Reviewed-by: Simon Horman <horms+renesas@verge.net.au>

> ---
>  drivers/usb/host/xhci-rcar.c | 17 ++---------------
>  1 file changed, 2 insertions(+), 15 deletions(-)
> 
> diff --git a/drivers/usb/host/xhci-rcar.c b/drivers/usb/host/xhci-rcar.c
> index f33ffc2..4a22f25 100644
> --- a/drivers/usb/host/xhci-rcar.c
> +++ b/drivers/usb/host/xhci-rcar.c
> @@ -17,9 +17,8 @@
>  #include "xhci-rcar.h"
>  
>  /*
> -* - The V3 firmware is for r8a7796 (with good performance) and r8a7795 es2.0
> -*   or later.
> -* - The V2 firmware can be used on both r8a7795 (es1.x) and r8a7796.
> +* - The V3 firmware is for almost all R-Car Gen3 (except r8a7795 ES1.x)
> +* - The V2 firmware is for r8a7795 ES1.x.
>  * - The V2 firmware is possible to use on R-Car Gen2. However, the V2 causes
>  *   performance degradation. So, this driver continues to use the V1 if R-Car
>  *   Gen2.
> @@ -75,18 +74,6 @@
>  		.soc_id = "r8a7795", .revision = "ES1.*",
>  		.data = (void *)RCAR_XHCI_FIRMWARE_V2,
>  	},
> -	{
> -		.soc_id = "r8a7795",
> -		.data = (void *)RCAR_XHCI_FIRMWARE_V3,
> -	},
> -	{
> -		.soc_id = "r8a7796",
> -		.data = (void *)RCAR_XHCI_FIRMWARE_V3,
> -	},
> -	{
> -		.soc_id = "r8a77965",
> -		.data = (void *)RCAR_XHCI_FIRMWARE_V3,
> -	},
>  	{ /* sentinel */ },
>  };
>  
> -- 
> 1.9.1
> 
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/usb/host/xhci-rcar.c b/drivers/usb/host/xhci-rcar.c
index f33ffc2..4a22f25 100644
--- a/drivers/usb/host/xhci-rcar.c
+++ b/drivers/usb/host/xhci-rcar.c
@@ -17,9 +17,8 @@ 
 #include "xhci-rcar.h"
 
 /*
-* - The V3 firmware is for r8a7796 (with good performance) and r8a7795 es2.0
-*   or later.
-* - The V2 firmware can be used on both r8a7795 (es1.x) and r8a7796.
+* - The V3 firmware is for almost all R-Car Gen3 (except r8a7795 ES1.x)
+* - The V2 firmware is for r8a7795 ES1.x.
 * - The V2 firmware is possible to use on R-Car Gen2. However, the V2 causes
 *   performance degradation. So, this driver continues to use the V1 if R-Car
 *   Gen2.
@@ -75,18 +74,6 @@ 
 		.soc_id = "r8a7795", .revision = "ES1.*",
 		.data = (void *)RCAR_XHCI_FIRMWARE_V2,
 	},
-	{
-		.soc_id = "r8a7795",
-		.data = (void *)RCAR_XHCI_FIRMWARE_V3,
-	},
-	{
-		.soc_id = "r8a7796",
-		.data = (void *)RCAR_XHCI_FIRMWARE_V3,
-	},
-	{
-		.soc_id = "r8a77965",
-		.data = (void *)RCAR_XHCI_FIRMWARE_V3,
-	},
 	{ /* sentinel */ },
 };