Messages in this thread | | | From | Andy Shevchenko <> | Date | Mon, 25 Nov 2019 21:06:06 +0200 | Subject | Re: [PATCH 3/8] platform/x86: asus-wmi: switch to use <linux/temperature.h> helpers |
| |
On Mon, Nov 25, 2019 at 4:32 PM Akinobu Mita <akinobu.mita@gmail.com> wrote: > 2019年11月25日(月) 4:54 Andy Shevchenko <andy.shevchenko@gmail.com>: > > On Sun, Nov 24, 2019 at 4:09 PM Akinobu Mita <akinobu.mita@gmail.com> wrote:
> > > - value = DECI_KELVIN_TO_CELSIUS((value & 0xFFFF)) * 1000; > > > - > > > - return sprintf(buf, "%d\n", value); > > > > > + return sprintf(buf, "%ld\n", > > > + deci_kelvin_to_millicelsius(value & 0xFFFF)); > > > > Leave it in one line. > > It causes this line over 80 characters.
And I don't care.
> Or do you prefer to save the > conversion result in 'value' temporarily as before?
No.
-- With Best Regards, Andy Shevchenko
| |