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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxxxx>

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

Reviewed-by: Simon Horman <horms+renesas@xxxxxxxxxxxx>

> ---
>  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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux