Messages in this thread | | | Subject | Re: [PATCH v2] i2c: designware: Suppress error message if platform_get_irq() < 0 | From | christian.ruppert@alitech ... | Date | Mon, 9 Mar 2015 17:34:51 +0100 |
| |
Alexey Brodkin <Alexey.Brodkin@synopsys.com> wrote on 03/09/2015 10:03:12 AM:
> From: Alexey Brodkin <Alexey.Brodkin@synopsys.com> > To: linux-i2c@vger.kernel.org, > Cc: linux-kernel@vger.kernel.org, Alexey Brodkin > <Alexey.Brodkin@synopsys.com>, Vineet Gupta > <Vineet.Gupta1@synopsys.com>, Christian Ruppert > <christian.ruppert@abilis.com>, Mika Westerberg > <mika.westerberg@linux.intel.com>, Wolfram Sang <wsa@the-dreams.de>, > Andy Shevchenko <andriy.shevchenko@linux.intel.com> > Date: 03/09/2015 10:03 AM > > As discussed here https://lkml.org/lkml/2015/3/3/568 and here > https://lkml.org/lkml/2015/3/3/453 we're looking forward for > implementing warnings and errors outputs right in platform_get_irq() > instead of having all kind of different outputs in each and every driver > that uses platform_get_irq(). > > Signed-off-by: Alexey Brodkin <abrodkin@synopsys.com> > Cc: Vineet Gupta <vgupta@synopsys.com> > Cc: Christian Ruppert <christian.ruppert@abilis.com> Acked-by: Christian Ruppert <christian.ruppert@alitech.com>
Please note that my email address has changed and use the alitech one in future. The abilis address might be switched off soon.
> Cc: Mika Westerberg <mika.westerberg@linux.intel.com> > Cc: Wolfram Sang <wsa@the-dreams.de> > Cc: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > drivers/i2c/busses/i2c-designware-platdrv.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/ > i2c/busses/i2c-designware-platdrv.c > index c270f5f..fa4e2b5 100644 > --- a/drivers/i2c/busses/i2c-designware-platdrv.c > +++ b/drivers/i2c/busses/i2c-designware-platdrv.c > @@ -143,10 +143,8 @@ static int dw_i2c_probe(struct platform_device *pdev) > u32 clk_freq, ht = 0; > > irq = platform_get_irq(pdev, 0); > - if (irq < 0) { > - dev_err(&pdev->dev, "no irq resource?\n"); > - return irq; /* -ENXIO */ > - } > + if (irq < 0) > + return irq; > > dev = devm_kzalloc(&pdev->dev, sizeof(struct dw_i2c_dev), GFP_KERNEL); > if (!dev) > -- > 2.1.0 >
| |