[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <8eadc1cb-3648-b0b8-0250-9b745bb8b71d@roeck-us.net>
Date: Sun, 4 Dec 2022 13:32:50 -0800
From: Guenter Roeck <linux@...ck-us.net>
To: Borislav Petkov <bp@...en8.de>
Cc: Eddie James <eajames@...ux.vnet.ibm.com>,
linux-hwmon@...r.kernel.org, lkml <linux-kernel@...r.kernel.org>
Subject: Re: CONFIG_SENSORS_OCC_P8_I2C
On 12/4/22 11:13, Borislav Petkov wrote:
> On Sun, Dec 04, 2022 at 10:37:09AM -0800, Guenter Roeck wrote:
>> I don't immediately see why this should depend on a P8 configuration.
>> The driver runs on a BMC, after all. My understanding is that a BMC can be
>> of any architecture. Is there a guarantee that there are and never will be
>> any x86 BMCs, and/or that BMCs are limited to a certain subset of
>> architectures ?
>
> Well, then it should depend on the architectures which are currently
> running BMCs for P8 machines. ARM maybe?
>
> At least on x86 the majority of BMCs are ARM-based AFAIK.
>
We are not typically that restrictive in the hwmon subsystem. That isn't done
for many of the other drivers which are typically only used on x86 either.
Why is that suddenly a problem ? Just don't enable it if you don't need it.
Anyway,
commit 58c0399db8bf6bf88c7580b742b0cb45e02f55d5
Author: Jeremy Kerr <jk@...econstruct.com.au>
AuthorDate: Mon Oct 24 16:15:27 2022 +0800
Commit: Guenter Roeck <linux@...ck-us.net>
CommitDate: Thu Dec 1 09:21:06 2022 -0800
hwmon: (occ) OCC sensors aren't arch-specific
Commit c112d75840fb ("hwmon: OCC drivers are ARM-only") made the OCC
sensor drivers not selectable on powerpc64:
These drivers are for a BMC inside PowerPC servers. The BMC runs on
ARM hardware, so only propose the drivers on this architecture, unless
build-testing.
... but we now have a powerpc64 BMC (still for a powerpc64 host), so
drop the `depends on` that excludes building for this platform.
Signed-off-by: Jeremy Kerr <jk@...econstruct.com.au>
Acked-by: Joel Stanley <joel@....id.au>
Link: https://lore.kernel.org/r/20221024081527.3842565-1-jk@codeconstruct.com.au
Signed-off-by: Guenter Roeck <linux@...ck-us.net>
I really don't want to end up in a situation where I constantly have
to deal with changing architecture support, so I would strongly prefer
to keep it that way unless it is a real problem.
Guenter
Powered by blists - more mailing lists