From mboxrd@z Thu Jan 1 00:00:00 1970 From: Darren Hart Date: Wed, 03 Jan 2018 01:13:52 +0000 Subject: Re: platform/x86/thinkpad_acpi: Adjustments for four function implementations Message-Id: <20180103011352.GE24794@fury> List-Id: References: <81459d11-693a-eb51-9173-9c189677f422@users.sourceforge.net> <20171223014033.jx7fzu7uzjfbzyca@khazad-dum.debian.net> <80ae8328-5851-01b0-cb31-474f7baf1686@users.sourceforge.net> <20180103001034.GC24794@fury> <1514940583.31497.4.camel@perches.com> In-Reply-To: <1514940583.31497.4.camel@perches.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Joe Perches Cc: SF Markus Elfring , Henrique de Moraes Holschuh , ibm-acpi-devel@lists.sourceforge.net, platform-driver-x86@vger.kernel.org, Andy Shevchenko , Andy Shevchenko , Henrique de Moraes Holschuh , LKML , kernel-janitors@vger.kernel.org On Tue, Jan 02, 2018 at 04:49:43PM -0800, Joe Perches wrote: > On Tue, 2018-01-02 at 16:10 -0800, Darren Hart wrote: > > > > Leave those pr_ messages alone, please, > [] > > Andy and Henrique raised a few reasons why these patches should not be > > accepted: > > > > 1. This is init code (so any space savings is short lived) > > Not exactly true. > > The object code itself is short lived, but the > string constant of the format is not as it is > placed in const and not discarded. Thanks for the clarification, appreciate it. I hadn't looked into it in enough detail previously. -- Darren Hart VMware Open Source Technology Center