From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jonathan Cameron Date: Sun, 04 Feb 2018 11:23:46 +0000 Subject: Re: [PATCH] hid-sensor-accel-3d: Delete an error message for a failed memory allocation in hid_accel Message-Id: <20180204112346.0977e938@archlinux> List-Id: References: <0406765c-bdd1-1a82-cf66-1c248063ae4f@users.sourceforge.net> In-Reply-To: <0406765c-bdd1-1a82-cf66-1c248063ae4f@users.sourceforge.net> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: SF Markus Elfring Cc: linux-iio@vger.kernel.org, linux-input@vger.kernel.org, Hartmut Knaack , Jiri Kosina , Lars-Peter Clausen , Peter Meerwald-Stadler , Srinivas Pandruvada , LKML , kernel-janitors@vger.kernel.org On Wed, 31 Jan 2018 22:26:14 +0100 SF Markus Elfring wrote: > From: Markus Elfring > Date: Wed, 31 Jan 2018 22:20:56 +0100 > > Omit an extra message for a memory allocation failure in this function. > > This issue was detected by using the Coccinelle software. > > Signed-off-by: Markus Elfring Marcus, If making changes like this I would suggest only sending one until you have have a response from the relevant maintainer. It would save you time as often these sorts of changes are a matter of personal taste and weighing up of costs vs gains - hence it is not obvious that they will be accepted. Jonathan > --- > drivers/iio/accel/hid-sensor-accel-3d.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/drivers/iio/accel/hid-sensor-accel-3d.c b/drivers/iio/accel/hid-sensor-accel-3d.c > index c066a3bdbff7..3d0acde40285 100644 > --- a/drivers/iio/accel/hid-sensor-accel-3d.c > +++ b/drivers/iio/accel/hid-sensor-accel-3d.c > @@ -383,11 +383,9 @@ static int hid_accel_3d_probe(struct platform_device *pdev) > return ret; > } > indio_dev->channels = kmemdup(channel_spec, channel_size, GFP_KERNEL); > - > - if (!indio_dev->channels) { > - dev_err(&pdev->dev, "failed to duplicate channels\n"); > + if (!indio_dev->channels) > return -ENOMEM; > - } > + > ret = accel_3d_parse_report(pdev, hsdev, > (struct iio_chan_spec *)indio_dev->channels, > hsdev->usage, accel_state);