On Thu, Aug 27, 2015 at 02:33:06PM -0300, Henrique de Moraes Holschuh wrote:
> On Wed, 26 Aug 2015, Joe Perches wrote:
> > vdbg_printk when not using CONFIG_THINKPAD_ACPI_DEBUG uses
> > no_printk which produces no logging output but always
> > evaluates arguments.
> >
> > Change the macro to surround the no_printk call with
> > do { if (0) no_printk(...); } while (0)
> > to avoid the unnecessary argument evaluations.
> >
> > $ size drivers/platform/x86/thinkpad_acpi.o*
> > text data bss dec hex filename
> > 60918 6184 824 67926 10956
> > drivers/platform/x86/thinkpad_acpi.o.new
> > 60927 6184 824 67935 1095f
> > drivers/platform/x86/thinkpad_acpi.o.old
> >
> > Signed-off-by: Joe Perches <[email protected]>
>
> Acked-by: Henrique de Moraes Holschuh <[email protected]>
Queued, thanks.
--
Darren Hart
Intel Open Source Technology Center
------------------------------------------------------------------------------
_______________________________________________
ibm-acpi-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ibm-acpi-devel