Re: [PATCH v4 11/11] PCI: imx6: Use usleep_range() in imx6_pcie_enable_ref_clk()

2019-04-15 Thread Lucas Stach
Am Sonntag, den 14.04.2019, 17:46 -0700 schrieb Andrey Smirnov:
> imx6_pcie_enable_ref_clk() is never called in atomic context, so
> there's no need to use udelay(). Replace it with usleep_range().
> 
> Signed-off-by: Andrey Smirnov 
> Cc: Lorenzo Pieralisi 
> Cc: Bjorn Helgaas 
> Cc: Chris Healy 
> Cc: Lucas Stach 
> Cc: linux-kernel@vger.kernel.org
> Cc: linux-...@vger.kernel.org

Reviewed-by: Lucas Stach 

> ---
>  drivers/pci/controller/dwc/pci-imx6.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/pci/controller/dwc/pci-imx6.c 
> b/drivers/pci/controller/dwc/pci-imx6.c
> index 3e45f49b8a4f..c6d6bde4d860 100644
> --- a/drivers/pci/controller/dwc/pci-imx6.c
> +++ b/drivers/pci/controller/dwc/pci-imx6.c
> @@ -449,7 +449,7 @@ static int imx6_pcie_enable_ref_clk(struct imx6_pcie 
> *imx6_pcie)
> >      * reset time is too short, cannot meet the requirement.
> >      * add one ~10us delay here.
> >      */
> > -   udelay(10);
> > +   usleep_range(10, 100);
> >     regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1,
> >        IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16);
> >     break;


[PATCH v4 11/11] PCI: imx6: Use usleep_range() in imx6_pcie_enable_ref_clk()

2019-04-14 Thread Andrey Smirnov
imx6_pcie_enable_ref_clk() is never called in atomic context, so
there's no need to use udelay(). Replace it with usleep_range().

Signed-off-by: Andrey Smirnov 
Cc: Lorenzo Pieralisi 
Cc: Bjorn Helgaas 
Cc: Chris Healy 
Cc: Lucas Stach 
Cc: linux-kernel@vger.kernel.org
Cc: linux-...@vger.kernel.org
---
 drivers/pci/controller/dwc/pci-imx6.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pci/controller/dwc/pci-imx6.c 
b/drivers/pci/controller/dwc/pci-imx6.c
index 3e45f49b8a4f..c6d6bde4d860 100644
--- a/drivers/pci/controller/dwc/pci-imx6.c
+++ b/drivers/pci/controller/dwc/pci-imx6.c
@@ -449,7 +449,7 @@ static int imx6_pcie_enable_ref_clk(struct imx6_pcie 
*imx6_pcie)
 * reset time is too short, cannot meet the requirement.
 * add one ~10us delay here.
 */
-   udelay(10);
+   usleep_range(10, 100);
regmap_update_bits(imx6_pcie->iomuxc_gpr, IOMUXC_GPR1,
   IMX6Q_GPR1_PCIE_REF_CLK_EN, 1 << 16);
break;
-- 
2.20.1