Re: [PATCH] hwmon/occ: OCC sensors aren't arch-specific

[Guenter Roeck <linux@roeck-us.net>]

On Mon, Oct 24, 2022 at 04:15:27PM +0800, Jeremy Kerr wrote:
> 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@codeconstruct.com.au>

Should the original patch be reverted instead, and should the revert
be backported ? In other words, is this a bug fix which needs to be
applied to v6.1, or is it needed for v6.2+ only ?

Thanks,
Guenter

> ---
>  drivers/hwmon/occ/Kconfig | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/hwmon/occ/Kconfig b/drivers/hwmon/occ/Kconfig
> index 35a7070db827..348c21100a37 100644
> --- a/drivers/hwmon/occ/Kconfig
> +++ b/drivers/hwmon/occ/Kconfig
> @@ -6,7 +6,6 @@
>  config SENSORS_OCC_P8_I2C
>  	tristate "POWER8 OCC through I2C"
>  	depends on I2C
> -	depends on ARM || ARM64 || COMPILE_TEST
>  	select SENSORS_OCC
>  	help
>  	 This option enables support for monitoring sensors provided by the
> @@ -21,7 +20,6 @@ config SENSORS_OCC_P8_I2C
>  config SENSORS_OCC_P9_SBE
>  	tristate "POWER9 OCC through SBE"
>  	depends on FSI_OCC
> -	depends on ARM || ARM64 || COMPILE_TEST
>  	select SENSORS_OCC
>  	help
>  	 This option enables support for monitoring sensors provided by the