[U-Boot] [PATCH] usb: sunxi: ohci: make ohci_t the first member in private data

Marek Vasut marex at denx.de
Mon Jun 18 01:14:58 UTC 2018


On 06/17/2018 06:13 PM, Vasily Khoruzhick wrote:
> ohci-hcd casts priv_data pointer to (ohci_t *), thus it must be
> the first member in private data struct.
> 
> Fixes 831cc98b1 ("usb: sunxi: Simplify ccm reg base code")
> 
> Signed-off-by: Vasily Khoruzhick <anarsoul at gmail.com>

Sigh, I really wonder how (or if at all!) the original series was
tested. And then I get flak for scrutinizing patches, right ...

Thanks for the fix, applied.

> ---
>  drivers/usb/host/ohci-sunxi.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/usb/host/ohci-sunxi.c b/drivers/usb/host/ohci-sunxi.c
> index e13f6ec9a4..db6f438275 100644
> --- a/drivers/usb/host/ohci-sunxi.c
> +++ b/drivers/usb/host/ohci-sunxi.c
> @@ -33,9 +33,9 @@ struct ohci_sunxi_cfg {
>  };
>  
>  struct ohci_sunxi_priv {
> +	ohci_t ohci;
>  	struct sunxi_ccm_reg *ccm;
>  	u32 *reset0_cfg;
> -	ohci_t ohci;
>  	int ahb_gate_mask; /* Mask of ahb_gate0 clk gate bits for this hcd */
>  	int usb_gate_mask; /* Mask of usb_clk_cfg clk gate bits for this hcd */
>  	struct phy phy;
> 


-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list