[PATCH] pwm: argon-fan-hat: Add hardware dependency

[PATCH] pwm: argon-fan-hat: Add hardware dependency

[Jean Delvare]

The Argon fan hat is designed to be connected to Raspberry Pi 3+
systems, so only offer this option when building a kernel which
supports this architecture (unless build-testing). Also extend the
description to reflect that.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
---
 drivers/pwm/Kconfig |    4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

--- linux-6.17.orig/drivers/pwm/Kconfig
+++ linux-6.17/drivers/pwm/Kconfig
@@ -68,9 +68,11 @@ config PWM_APPLE
 
 config PWM_ARGON_FAN_HAT
 	tristate "Argon40 Fan HAT support"
+	depends on ARCH_BCM2835 || COMPILE_TEST
 	depends on I2C && OF
 	help
-	  Generic PWM framework driver for Argon40 Fan HAT.
+	  Generic PWM framework driver for Argon40 Fan HAT, found in
+	  some Raspberry Pi 3+ cases.
 
 	  To compile this driver as a module, choose M here: the module
 	  will be called pwm-argon-fan-hat.


-- 
Jean Delvare
SUSE L3 Support

Re: [PATCH] pwm: argon-fan-hat: Add hardware dependency

[Marek Vasut]

On 11/3/25 4:47 PM, Jean Delvare wrote:
> The Argon fan hat is designed to be connected to Raspberry Pi 3+
> systems, so only offer this option when building a kernel which
> supports this architecture (unless build-testing). Also extend the
> description to reflect that.
> 
> Signed-off-by: Jean Delvare <jdelvare@suse.de>
> ---
>   drivers/pwm/Kconfig |    4 +++-
>   1 file changed, 3 insertions(+), 1 deletion(-)
> 
> --- linux-6.17.orig/drivers/pwm/Kconfig
> +++ linux-6.17/drivers/pwm/Kconfig
> @@ -68,9 +68,11 @@ config PWM_APPLE
>   
>   config PWM_ARGON_FAN_HAT
>   	tristate "Argon40 Fan HAT support"
> +	depends on ARCH_BCM2835 || COMPILE_TEST

NAK, this is wrong.

This fan plugs into any arbitrary board with compatible 2x20 connector, 
not only RPi. That connector is present on all kinds of fruitboards with 
different non-BCM2835 SoCs.

Re: [PATCH] pwm: argon-fan-hat: Add hardware dependency

[Jean Delvare]

Hi Marek,

On Mon, 2025-11-03 at 16:55 +0100, Marek Vasut wrote:
> On 11/3/25 4:47 PM, Jean Delvare wrote:
> > The Argon fan hat is designed to be connected to Raspberry Pi 3+
> > systems, so only offer this option when building a kernel which
> > supports this architecture (unless build-testing). Also extend the
> > description to reflect that.
> > 
> > Signed-off-by: Jean Delvare <jdelvare@suse.de>
> > ---
> >   drivers/pwm/Kconfig |    4 +++-
> >   1 file changed, 3 insertions(+), 1 deletion(-)
> > 
> > --- linux-6.17.orig/drivers/pwm/Kconfig
> > +++ linux-6.17/drivers/pwm/Kconfig
> > @@ -68,9 +68,11 @@ config PWM_APPLE
> >   
> >   config PWM_ARGON_FAN_HAT
> >   	tristate "Argon40 Fan HAT support"
> > +	depends on ARCH_BCM2835 || COMPILE_TEST
> 
> NAK, this is wrong.
> 
> This fan plugs into any arbitrary board with compatible 2x20 connector, 
> not only RPi. That connector is present on all kinds of fruitboards with 
> different non-BCM2835 SoCs.

Yet the company who builds and sells this fan device, markets it as
compatible with Raspberry Pi 3 and Raspberry Pi 4 only:

https://argon40.com/products/argon-fan-hat

Which other "fruitboards" would be supported, and what SoCs are they
using?

Thanks,
-- 
Jean Delvare
SUSE L3 Support

Re: [PATCH] pwm: argon-fan-hat: Add hardware dependency

[Marek Vasut]

On 11/3/25 5:21 PM, Jean Delvare wrote:

Hello Jean,

> On Mon, 2025-11-03 at 16:55 +0100, Marek Vasut wrote:
>> On 11/3/25 4:47 PM, Jean Delvare wrote:
>>> The Argon fan hat is designed to be connected to Raspberry Pi 3+
>>> systems, so only offer this option when building a kernel which
>>> supports this architecture (unless build-testing). Also extend the
>>> description to reflect that.
>>>
>>> Signed-off-by: Jean Delvare <jdelvare@suse.de>
>>> ---
>>>    drivers/pwm/Kconfig |    4 +++-
>>>    1 file changed, 3 insertions(+), 1 deletion(-)
>>>
>>> --- linux-6.17.orig/drivers/pwm/Kconfig
>>> +++ linux-6.17/drivers/pwm/Kconfig
>>> @@ -68,9 +68,11 @@ config PWM_APPLE
>>>    
>>>    config PWM_ARGON_FAN_HAT
>>>    	tristate "Argon40 Fan HAT support"
>>> +	depends on ARCH_BCM2835 || COMPILE_TEST
>>
>> NAK, this is wrong.
>>
>> This fan plugs into any arbitrary board with compatible 2x20 connector,
>> not only RPi. That connector is present on all kinds of fruitboards with
>> different non-BCM2835 SoCs.
> 
> Yet the company who builds and sells this fan device, markets it as
> compatible with Raspberry Pi 3 and Raspberry Pi 4 only:
> 
> https://argon40.com/products/argon-fan-hat

I don't think 5V , GND , and 3V3 I2C signals are in any way special to 
RPi , so that statement seems to be only marketing .

> Which other "fruitboards" would be supported, and what SoCs are they
> using?

I know of at least one Retronix R-Car Sparrow Hawk board (currently 
upstream including DTO for this fan, uses Renesas R-Car V4H SoC), ST 
STM32MP13xx DHCOM DHSBC board also works with this fan (tested locally).

The 2x20 RPi-like connector is increasingly present on those evaluation 
boards, with matching pinout, to be compatible with all those expansion 
modules, like this fan.

-- 
Best regards,
Marek Vasut

Re: [PATCH] pwm: argon-fan-hat: Add hardware dependency

[Jean Delvare]

On Mon, 2025-11-03 at 18:05 +0100, Marek Vasut wrote:
> On 11/3/25 5:21 PM, Jean Delvare wrote:
> > On Mon, 2025-11-03 at 16:55 +0100, Marek Vasut wrote:
> > > This fan plugs into any arbitrary board with compatible 2x20 connector,
> > > not only RPi. That connector is present on all kinds of fruitboards with
> > > different non-BCM2835 SoCs.
> > 
> > Yet the company who builds and sells this fan device, markets it as
> > compatible with Raspberry Pi 3 and Raspberry Pi 4 only:
> > 
> > https://argon40.com/products/argon-fan-hat
> 
> I don't think 5V , GND , and 3V3 I2C signals are in any way special to 
> RPi , so that statement seems to be only marketing .

Might be because they also sell compatible casings for Raspberry Pi 3+
where this fan fits nicely.

> > Which other "fruitboards" would be supported, and what SoCs are they
> > using?
> 
> I know of at least one Retronix R-Car Sparrow Hawk board (currently 
> upstream including DTO for this fan, uses Renesas R-Car V4H SoC), ST 
> STM32MP13xx DHCOM DHSBC board also works with this fan (tested locally).
> 
> The 2x20 RPi-like connector is increasingly present on those evaluation 
> boards, with matching pinout, to be compatible with all those expansion 
> modules, like this fan.

Does this connector have a name? If so, it could be included in the
description. And even if not, maybe we can extend the description to
make it clear that more boards might use this fan hat?

Thanks,
-- 
Jean Delvare
SUSE L3 Support

Re: [PATCH] pwm: argon-fan-hat: Add hardware dependency

[Marek Vasut]

On 11/3/25 10:06 PM, Jean Delvare wrote:
> On Mon, 2025-11-03 at 18:05 +0100, Marek Vasut wrote:
>> On 11/3/25 5:21 PM, Jean Delvare wrote:
>>> On Mon, 2025-11-03 at 16:55 +0100, Marek Vasut wrote:
>>>> This fan plugs into any arbitrary board with compatible 2x20 connector,
>>>> not only RPi. That connector is present on all kinds of fruitboards with
>>>> different non-BCM2835 SoCs.
>>>
>>> Yet the company who builds and sells this fan device, markets it as
>>> compatible with Raspberry Pi 3 and Raspberry Pi 4 only:
>>>
>>> https://argon40.com/products/argon-fan-hat
>>
>> I don't think 5V , GND , and 3V3 I2C signals are in any way special to
>> RPi , so that statement seems to be only marketing .
> 
> Might be because they also sell compatible casings for Raspberry Pi 3+
> where this fan fits nicely.

I _think_ that the Argon fan/case combo has different firmware than the 
fan, and different I2C commands are used to control the case. It does 
not seem to be very consistent:

https://github.com/Argon40Tech/Argon-ONE-i2c-Codes/issues/1

>>> Which other "fruitboards" would be supported, and what SoCs are they
>>> using?
>>
>> I know of at least one Retronix R-Car Sparrow Hawk board (currently
>> upstream including DTO for this fan, uses Renesas R-Car V4H SoC), ST
>> STM32MP13xx DHCOM DHSBC board also works with this fan (tested locally).
>>
>> The 2x20 RPi-like connector is increasingly present on those evaluation
>> boards, with matching pinout, to be compatible with all those expansion
>> modules, like this fan.
> 
> Does this connector have a name? If so, it could be included in the
> description. And even if not, maybe we can extend the description to
> make it clear that more boards might use this fan hat?
I don't know, various pages call it differently, Raspberry Pi 2x20 pin 
header or some such. I don't think it is in any way "industry standard" 
, but various fruitboard vendors implement it to be compatible with the 
various "hat"s that are on the market, and in the process avoid calling 
it Raspberry anything, because that is I think a trademark or some such 
thing.

Re: [PATCH] pwm: argon-fan-hat: Add hardware dependency

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 205 bytes --]

Hello,

it seems we all agree that this patch isn't suitable to go in as is. So
I mark it as "Changes-Requested" in patchwork and discard this thread
from my inbox.

Thanks and happy holidays,
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH] pwm: argon-fan-hat: Add hardware dependency

[Marek Vasut]

On 12/24/25 6:57 AM, Uwe Kleine-König wrote:
> Hello,

Hello Uwe,

> it seems we all agree that this patch isn't suitable to go in as is. So
> I mark it as "Changes-Requested" in patchwork and discard this thread
> from my inbox.

Indeed, the fan can be used on any arbitrary hardware with the right 
"rpi" connector.

Please drop this patch.

> Thanks and happy holidays,
Likewise. Thanks