Re: [GIT PULL] RESEND: thermal for v5.9-rc1

From: Linus Torvalds
Date: Thu Aug 06 2020 - 22:44:23 EST


On Thu, Aug 6, 2020 at 1:19 PM Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> wrote:
>
>
> - Add generic netlink support for userspace notifications: events,
> temperature
> and discovery commands (Daniel Lezcano)

This is "default y".

Why?

The help text doesn't explain either.

Please explain, or remove the default y. We don't add new features and
then try to force people to use them by enabling them by default.

"default y" is mainly for when something unconditional gets split up
and becomes conditional (so now "default y" means that you don't break
peoples setups when they don't even know what it is).

Alternatively, "default y" is for things that are make peoples lives
immeasurably better somehow, and it would be a crime to not enable it
because it's _so_ wonderful.

So far, I'm not convinced we've ever hit that second case.

Convince me that the thermal layer is so magical that it really
warrants it. Tell me why and how my life is improved by enabling it.

Linus