* [PATCH 1/1] i2c: designware-pci: use IRQF_COND_SUSPEND flag
@ 2015-07-08 10:15 Andy Shevchenko
[not found] ` <1436350536-99102-1-git-send-email-andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
0 siblings, 1 reply; 5+ messages in thread
From: Andy Shevchenko @ 2015-07-08 10:15 UTC (permalink / raw)
To: Wolfram Sang, linux-i2c-u79uwXL29TY76Z2rM5mHXA; +Cc: Andy Shevchenko
The mentioned flag fixes a warning on Intel Edison board since one of the I2C
controller shares IRQ line with watchdog timer.
Signed-off-by: Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
---
drivers/i2c/busses/i2c-designware-pcidrv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/i2c/busses/i2c-designware-pcidrv.c b/drivers/i2c/busses/i2c-designware-pcidrv.c
index 6643d2d..df23e8c 100644
--- a/drivers/i2c/busses/i2c-designware-pcidrv.c
+++ b/drivers/i2c/busses/i2c-designware-pcidrv.c
@@ -260,8 +260,8 @@ static int i2c_dw_pci_probe(struct pci_dev *pdev,
snprintf(adap->name, sizeof(adap->name), "i2c-designware-pci");
- r = devm_request_irq(&pdev->dev, pdev->irq, i2c_dw_isr, IRQF_SHARED,
- adap->name, dev);
+ r = devm_request_irq(&pdev->dev, pdev->irq, i2c_dw_isr,
+ IRQF_SHARED | IRQF_COND_SUSPEND, adap->name, dev);
if (r) {
dev_err(&pdev->dev, "failure requesting irq %i\n", dev->irq);
return r;
--
2.1.4
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] i2c: designware-pci: use IRQF_COND_SUSPEND flag
[not found] ` <1436350536-99102-1-git-send-email-andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
@ 2015-07-27 11:37 ` Andy Shevchenko
[not found] ` <1437997043.29746.86.camel-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2015-08-01 10:13 ` Wolfram Sang
1 sibling, 1 reply; 5+ messages in thread
From: Andy Shevchenko @ 2015-07-27 11:37 UTC (permalink / raw)
To: Wolfram Sang, linux-i2c-u79uwXL29TY76Z2rM5mHXA
On Wed, 2015-07-08 at 13:15 +0300, Andy Shevchenko wrote:
> The mentioned flag fixes a warning on Intel Edison board since one of
> the I2C
> controller shares IRQ line with watchdog timer.
>
Wolfram, do you have any comments on this?
> Signed-off-by: Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> ---
> drivers/i2c/busses/i2c-designware-pcidrv.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/i2c/busses/i2c-designware-pcidrv.c
> b/drivers/i2c/busses/i2c-designware-pcidrv.c
> index 6643d2d..df23e8c 100644
> --- a/drivers/i2c/busses/i2c-designware-pcidrv.c
> +++ b/drivers/i2c/busses/i2c-designware-pcidrv.c
> @@ -260,8 +260,8 @@ static int i2c_dw_pci_probe(struct pci_dev *pdev,
>
> snprintf(adap->name, sizeof(adap->name), "i2c-designware
> -pci");
>
> - r = devm_request_irq(&pdev->dev, pdev->irq, i2c_dw_isr,
> IRQF_SHARED,
> - adap->name, dev);
> + r = devm_request_irq(&pdev->dev, pdev->irq, i2c_dw_isr,
> + IRQF_SHARED | IRQF_COND_SUSPEND, adap->name,
> dev);
> if (r) {
> dev_err(&pdev->dev, "failure requesting irq %i\n",
> dev->irq);
> return r;
--
Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Intel Finland Oy
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] i2c: designware-pci: use IRQF_COND_SUSPEND flag
[not found] ` <1437997043.29746.86.camel-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
@ 2015-07-31 10:07 ` Wolfram Sang
2015-07-31 11:35 ` Mika Westerberg
0 siblings, 1 reply; 5+ messages in thread
From: Wolfram Sang @ 2015-07-31 10:07 UTC (permalink / raw)
To: Andy Shevchenko, Mika Westerberg; +Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 1500 bytes --]
On Mon, Jul 27, 2015 at 02:37:23PM +0300, Andy Shevchenko wrote:
> On Wed, 2015-07-08 at 13:15 +0300, Andy Shevchenko wrote:
> > The mentioned flag fixes a warning on Intel Edison board since one of
> > the I2C
> > controller shares IRQ line with watchdog timer.
> >
>
> Wolfram, do you have any comments on this?
Adding Mika to cc, he is more experienced with these platforms.
>
> > Signed-off-by: Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> > ---
> > drivers/i2c/busses/i2c-designware-pcidrv.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/i2c/busses/i2c-designware-pcidrv.c
> > b/drivers/i2c/busses/i2c-designware-pcidrv.c
> > index 6643d2d..df23e8c 100644
> > --- a/drivers/i2c/busses/i2c-designware-pcidrv.c
> > +++ b/drivers/i2c/busses/i2c-designware-pcidrv.c
> > @@ -260,8 +260,8 @@ static int i2c_dw_pci_probe(struct pci_dev *pdev,
> >
> > snprintf(adap->name, sizeof(adap->name), "i2c-designware
> > -pci");
> >
> > - r = devm_request_irq(&pdev->dev, pdev->irq, i2c_dw_isr,
> > IRQF_SHARED,
> > - adap->name, dev);
> > + r = devm_request_irq(&pdev->dev, pdev->irq, i2c_dw_isr,
> > + IRQF_SHARED | IRQF_COND_SUSPEND, adap->name,
> > dev);
> > if (r) {
> > dev_err(&pdev->dev, "failure requesting irq %i\n",
> > dev->irq);
> > return r;
>
> --
> Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> Intel Finland Oy
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] i2c: designware-pci: use IRQF_COND_SUSPEND flag
2015-07-31 10:07 ` Wolfram Sang
@ 2015-07-31 11:35 ` Mika Westerberg
0 siblings, 0 replies; 5+ messages in thread
From: Mika Westerberg @ 2015-07-31 11:35 UTC (permalink / raw)
To: Wolfram Sang; +Cc: Andy Shevchenko, linux-i2c-u79uwXL29TY76Z2rM5mHXA
On Fri, Jul 31, 2015 at 12:07:23PM +0200, Wolfram Sang wrote:
> On Mon, Jul 27, 2015 at 02:37:23PM +0300, Andy Shevchenko wrote:
> > On Wed, 2015-07-08 at 13:15 +0300, Andy Shevchenko wrote:
> > > The mentioned flag fixes a warning on Intel Edison board since one of
> > > the I2C
> > > controller shares IRQ line with watchdog timer.
> > >
> >
> > Wolfram, do you have any comments on this?
>
> Adding Mika to cc, he is more experienced with these platforms.
intel_mid_wdt.c requests the same IRQ with IRQF_NO_SUSPEND so this patch
makes sense to me,
Acked-by: Mika Westerberg <mika.westerberg-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/1] i2c: designware-pci: use IRQF_COND_SUSPEND flag
[not found] ` <1436350536-99102-1-git-send-email-andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2015-07-27 11:37 ` Andy Shevchenko
@ 2015-08-01 10:13 ` Wolfram Sang
1 sibling, 0 replies; 5+ messages in thread
From: Wolfram Sang @ 2015-08-01 10:13 UTC (permalink / raw)
To: Andy Shevchenko; +Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 330 bytes --]
On Wed, Jul 08, 2015 at 01:15:36PM +0300, Andy Shevchenko wrote:
> The mentioned flag fixes a warning on Intel Edison board since one of the I2C
> controller shares IRQ line with watchdog timer.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
Applied to for-next, thanks!
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-08-01 10:13 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-07-08 10:15 [PATCH 1/1] i2c: designware-pci: use IRQF_COND_SUSPEND flag Andy Shevchenko
[not found] ` <1436350536-99102-1-git-send-email-andriy.shevchenko-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2015-07-27 11:37 ` Andy Shevchenko
[not found] ` <1437997043.29746.86.camel-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2015-07-31 10:07 ` Wolfram Sang
2015-07-31 11:35 ` Mika Westerberg
2015-08-01 10:13 ` Wolfram Sang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.