From: Dhaval Shah <dhaval.shah@softnautics.com>
To: =?iso-8859-1?Q?Pali_Roh=E1r?= <pali.rohar@gmail.com>
CC: "dvhart@infradead.org" <dvhart@infradead.org>,
        "andy@infradead.org" <andy@infradead.org>,
        "platform-driver-x86@vger.kernel.org" 
        <platform-driver-x86@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] platform/x86: dell-smo8800: Possible unnecessary 'out of
 memory' message
Thread-Topic: [PATCH] platform/x86: dell-smo8800: Possible unnecessary 'out of
 memory' message
Thread-Index: AQHTc+u6asn84bwDOkaSHH5F3brCJ6NA8YmAgAAR1p0=
Date: Wed, 13 Dec 2017 09:40:14 +0000
Message-ID: <BM1PR0101MB14445D157BF6CF2BCC908BDBE6350@BM1PR0101MB1444.INDPRD01.PROD.OUTLOOK.COM>
References: <20171213082354.23273-1-dhaval.shah@softnautics.com>,<20171213083053.mywa4bgv56daxud6@pali>
In-Reply-To: <20171213083053.mywa4bgv56daxud6@pali>
Accept-Language: en-US
Content-Language: en-US
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="iso-8859-1"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: ceca9211-9ce7-425f-56e6-08d5420d8325
X-MS-Exchange-CrossTenant-originalarrivaltime: 13 Dec 2017 09:40:14.4126
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 17b76cff-50c6-4f7a-9198-dd4afc5f8bea
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BM1PR0101MB1443
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id vBD9eQs7019188
Content-Length: 1735
Lines: 44

Hi Pali Rohar,

Thanks for the review.

________________________________________
> From: Pali Roh?r <pali.rohar@gmail.com>
> Sent: Wednesday, December 13, 2017 2:00 PM
> To: Dhaval Shah
> Cc: dvhart@infradead.org; andy@infradead.org; platform-driver-x86@vger.kernel.org; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH] platform/x86: dell-smo8800: Possible unnecessary 'out of memory' message
> 
> On Wednesday 13 December 2017 13:53:54 Dhaval Shah wrote:
> > Removed Possible unnecessary 'out of memory' message checkpatch warnings.
> > Issue found by checkpatch.
> >
> > Signed-off-by: Dhaval Shah <dhaval.shah@softnautics.com>
> > ---
> >  drivers/platform/x86/dell-smo8800.c | 4 +---
> >  1 file changed, 1 insertion(+), 3 deletions(-)
> >
> > diff --git a/drivers/platform/x86/dell-smo8800.c b/drivers/platform/x86/dell-smo8800.c
> > index 1d87237bc731..9590d5e7c5ca 100644
> > --- a/drivers/platform/x86/dell-smo8800.c
> > +++ b/drivers/platform/x86/dell-smo8800.c
> > @@ -150,10 +150,8 @@ static int smo8800_add(struct acpi_device *device)
> >       struct smo8800_device *smo8800;
> >
> >       smo8800 = devm_kzalloc(&device->dev, sizeof(*smo8800), GFP_KERNEL);
> > -     if (!smo8800) {
> > -             dev_err(&device->dev, "failed to allocate device data\n");
> 
> Hi! Any particular reason for removing error message?
> 
When you give the -ENOMEM as error in return then this message info will be provided by the kernel subsystem. no need to give this printk and that's why checkpatch gives warning in this case.
> > +     if (!smo8800)
> >               return -ENOMEM;
> > -     }
> >
> >       smo8800->dev = &device->dev;
> >       smo8800->miscdev.minor = MISC_DYNAMIC_MINOR;

--
Pali Roh?r
pali.rohar@gmail.com