Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751343AbdFENNc (ORCPT ); Mon, 5 Jun 2017 09:13:32 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:36493 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751280AbdFENNb (ORCPT ); Mon, 5 Jun 2017 09:13:31 -0400 Date: Mon, 5 Jun 2017 15:13:19 +0200 From: Alexandre Belloni To: Greg KH Cc: Arvind Yadav , nicolas.ferre@microchip.com, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2] misc: atmel-ssc: Handle return value of clk_prepare_enable and clk_prepare Message-ID: <20170605131319.l3a7aw4xvgwls2ry@piout.net> References: <3a6a9ccd68fe298ab33009eaaace85c5af73e1ea.1496381884.git.arvind.yadav.cs@gmail.com> <20170603221753.lfiqthuc7dxy3dmc@piout.net> <20170605093652.4dpeb2ovjehfsxwl@piout.net> <20170605120515.GB6867@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170605120515.GB6867@kroah.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1779 Lines: 52 On 05/06/2017 at 14:05:15 +0200, Greg KH wrote: > On Mon, Jun 05, 2017 at 11:36:52AM +0200, Alexandre Belloni wrote: > > On 05/06/2017 at 14:53:30 +0530, Arvind Yadav wrote: > > > Hi, > > > > > > Yes, Patch v1 was wrong that's why i have push v2. > > > clk_prepare and clk_prepare_enable can fail. There > > > > No this is not true, they will never fail for the SSC. > > How is anyone supposed to know this? Just check the functions correctly > and move on, if they can never fail, wonderful, that's a code path that > is not going to be run. > Anyone able (and taking the time) to read code can know that. I'm really not okay with having useless code paths. Many people are already complaining (rightfully) that the kernel is bloated, let's try to not make the situation worse and keep in mind that Linux can run on tiny SoCs. > > > is not harm to check it's return value. It'll not impact present > > > functionality. > > > > > > > It does impact boot time, this patch adds 112 bytes to a compressed > > kernel for no reason. > > It solves the issue that if someone copy/paste from this code, they will > implement something incorrectly. You can spare the 112 bytes (which > really, seems very large for just 2 error cases, are you sure that's > right? Just drop the string if you care about size here. > I'm sure about the size, I've measured it on top of v4.12-rc1 before replying. I don't think anyone will copy/paste from this code but if that happen and is missed by the maintainer, I'm pretty sure someone will be quick to run his preferred static analysis tool and send a patch (hopefully, testing it really compiles first). > thanks, > > greg k-h -- Alexandre Belloni, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com