Re: [PATCH] USB: host: ehci: Use the defined variable to simplify code

From: Alan Stern
Date: Fri May 08 2020 - 09:56:10 EST


On Fri, 8 May 2020, Tang Bin wrote:

> Use the defined variable "dev" to make the code cleaner. And
> delete an extra blank line.

Again, the Subject: line should say "ehci-mxc".

> Signed-off-by: Zhang Shengju <zhangshengju@xxxxxxxxxxxxxxxxxxxx>
> Signed-off-by: Tang Bin <tangbin@xxxxxxxxxxxxxxxxxxxx>
> ---
> drivers/usb/host/ehci-mxc.c | 9 ++++-----
> 1 file changed, 4 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/usb/host/ehci-mxc.c b/drivers/usb/host/ehci-mxc.c
> index c9f91e6c7..a1eb5ee77 100644
> --- a/drivers/usb/host/ehci-mxc.c
> +++ b/drivers/usb/host/ehci-mxc.c
> @@ -56,7 +56,7 @@ static int ehci_mxc_drv_probe(struct platform_device *pdev)
> return -ENOMEM;
>
> res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> - hcd->regs = devm_ioremap_resource(&pdev->dev, res);
> + hcd->regs = devm_ioremap_resource(dev, res);

As long as you're making these changes, why not also move the
definition of dev up before the definition of pdata? Then you could
change the definition of pdata to:

struct mxc_usbh_platform_data *pdata = dev_get_platdata(dev);

Alan Stern

> if (IS_ERR(hcd->regs)) {
> ret = PTR_ERR(hcd->regs);
> goto err_alloc;
> @@ -69,14 +69,14 @@ static int ehci_mxc_drv_probe(struct platform_device *pdev)
> priv = (struct ehci_mxc_priv *) ehci->priv;
>
> /* enable clocks */
> - priv->usbclk = devm_clk_get(&pdev->dev, "ipg");
> + priv->usbclk = devm_clk_get(dev, "ipg");
> if (IS_ERR(priv->usbclk)) {
> ret = PTR_ERR(priv->usbclk);
> goto err_alloc;
> }
> clk_prepare_enable(priv->usbclk);
>
> - priv->ahbclk = devm_clk_get(&pdev->dev, "ahb");
> + priv->ahbclk = devm_clk_get(dev, "ahb");
> if (IS_ERR(priv->ahbclk)) {
> ret = PTR_ERR(priv->ahbclk);
> goto err_clk_ahb;
> @@ -84,13 +84,12 @@ static int ehci_mxc_drv_probe(struct platform_device *pdev)
> clk_prepare_enable(priv->ahbclk);
>
> /* "dr" device has its own clock on i.MX51 */
> - priv->phyclk = devm_clk_get(&pdev->dev, "phy");
> + priv->phyclk = devm_clk_get(dev, "phy");
> if (IS_ERR(priv->phyclk))
> priv->phyclk = NULL;
> if (priv->phyclk)
> clk_prepare_enable(priv->phyclk);
>
> -
> /* call platform specific init function */
> if (pdata->init) {
> ret = pdata->init(pdev);
>