Re: [PATCH v2 3/3] mfd: mt6397: check the EPROBE_DEFER from platform_get_irq.
On Mon, 2016-04-11 at 09:50 +0100, Lee Jones wrote: > On Fri, 08 Apr 2016, Henry Chen wrote: > > > If get the -EPROBE_DEFER from platform_get_irq, it should return back > > directly. > > > > Signed-off-by: Henry Chen> > --- > > drivers/mfd/mt6397-core.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/mfd/mt6397-core.c b/drivers/mfd/mt6397-core.c > > index 15050cb..b9b1e28 100644 > > --- a/drivers/mfd/mt6397-core.c > > +++ b/drivers/mfd/mt6397-core.c > > @@ -271,6 +271,8 @@ static int mt6397_probe(struct platform_device *pdev) > > } > > > > pmic->irq = platform_get_irq(pdev, 0); > > + if (pmic->irq == -EPROBE_DEFER) > > + return -EPROBE_DEFER; > > if (ret < 0) >return ret; > > ... is more common, as it covers more bases, etc. Okay, the interrupt should be a required properties, if get failed just return back. I will also change the binding doc. Thanks. > > > switch (id & 0xff) { > > case MT6323_CID_CODE: >
Re: [PATCH v2 3/3] mfd: mt6397: check the EPROBE_DEFER from platform_get_irq.
On Mon, 2016-04-11 at 09:50 +0100, Lee Jones wrote: > On Fri, 08 Apr 2016, Henry Chen wrote: > > > If get the -EPROBE_DEFER from platform_get_irq, it should return back > > directly. > > > > Signed-off-by: Henry Chen > > --- > > drivers/mfd/mt6397-core.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/mfd/mt6397-core.c b/drivers/mfd/mt6397-core.c > > index 15050cb..b9b1e28 100644 > > --- a/drivers/mfd/mt6397-core.c > > +++ b/drivers/mfd/mt6397-core.c > > @@ -271,6 +271,8 @@ static int mt6397_probe(struct platform_device *pdev) > > } > > > > pmic->irq = platform_get_irq(pdev, 0); > > + if (pmic->irq == -EPROBE_DEFER) > > + return -EPROBE_DEFER; > > if (ret < 0) >return ret; > > ... is more common, as it covers more bases, etc. Okay, the interrupt should be a required properties, if get failed just return back. I will also change the binding doc. Thanks. > > > switch (id & 0xff) { > > case MT6323_CID_CODE: >
Re: [PATCH v2 3/3] mfd: mt6397: check the EPROBE_DEFER from platform_get_irq.
On Fri, 08 Apr 2016, Henry Chen wrote: > If get the -EPROBE_DEFER from platform_get_irq, it should return back > directly. > > Signed-off-by: Henry Chen> --- > drivers/mfd/mt6397-core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/mfd/mt6397-core.c b/drivers/mfd/mt6397-core.c > index 15050cb..b9b1e28 100644 > --- a/drivers/mfd/mt6397-core.c > +++ b/drivers/mfd/mt6397-core.c > @@ -271,6 +271,8 @@ static int mt6397_probe(struct platform_device *pdev) > } > > pmic->irq = platform_get_irq(pdev, 0); > + if (pmic->irq == -EPROBE_DEFER) > + return -EPROBE_DEFER; if (ret < 0) return ret; ... is more common, as it covers more bases, etc. > switch (id & 0xff) { > case MT6323_CID_CODE: -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
Re: [PATCH v2 3/3] mfd: mt6397: check the EPROBE_DEFER from platform_get_irq.
On Fri, 08 Apr 2016, Henry Chen wrote: > If get the -EPROBE_DEFER from platform_get_irq, it should return back > directly. > > Signed-off-by: Henry Chen > --- > drivers/mfd/mt6397-core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/mfd/mt6397-core.c b/drivers/mfd/mt6397-core.c > index 15050cb..b9b1e28 100644 > --- a/drivers/mfd/mt6397-core.c > +++ b/drivers/mfd/mt6397-core.c > @@ -271,6 +271,8 @@ static int mt6397_probe(struct platform_device *pdev) > } > > pmic->irq = platform_get_irq(pdev, 0); > + if (pmic->irq == -EPROBE_DEFER) > + return -EPROBE_DEFER; if (ret < 0) return ret; ... is more common, as it covers more bases, etc. > switch (id & 0xff) { > case MT6323_CID_CODE: -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog
[PATCH v2 3/3] mfd: mt6397: check the EPROBE_DEFER from platform_get_irq.
If get the -EPROBE_DEFER from platform_get_irq, it should return back directly. Signed-off-by: Henry Chen--- drivers/mfd/mt6397-core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/mfd/mt6397-core.c b/drivers/mfd/mt6397-core.c index 15050cb..b9b1e28 100644 --- a/drivers/mfd/mt6397-core.c +++ b/drivers/mfd/mt6397-core.c @@ -271,6 +271,8 @@ static int mt6397_probe(struct platform_device *pdev) } pmic->irq = platform_get_irq(pdev, 0); + if (pmic->irq == -EPROBE_DEFER) + return -EPROBE_DEFER; switch (id & 0xff) { case MT6323_CID_CODE: -- 1.8.1.1.dirty
[PATCH v2 3/3] mfd: mt6397: check the EPROBE_DEFER from platform_get_irq.
If get the -EPROBE_DEFER from platform_get_irq, it should return back directly. Signed-off-by: Henry Chen --- drivers/mfd/mt6397-core.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/mfd/mt6397-core.c b/drivers/mfd/mt6397-core.c index 15050cb..b9b1e28 100644 --- a/drivers/mfd/mt6397-core.c +++ b/drivers/mfd/mt6397-core.c @@ -271,6 +271,8 @@ static int mt6397_probe(struct platform_device *pdev) } pmic->irq = platform_get_irq(pdev, 0); + if (pmic->irq == -EPROBE_DEFER) + return -EPROBE_DEFER; switch (id & 0xff) { case MT6323_CID_CODE: -- 1.8.1.1.dirty