| Message ID | 20251103164752.533aee39@endymion |
|---|---|
| State | New |
| Headers | show |
| Series | pwm: argon-fan-hat: Add hardware dependency | expand |
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.
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,
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.
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,
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.
--- 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.
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(-)