Re: [PATCH v2] spidev: Introduce "linux,spidev-name" property for device tree of spidev.

From: Conor Dooley
Date: Fri Jun 07 2024 - 12:08:52 EST


On Sun, Jun 02, 2024 at 05:31:10PM +0200, Szőke Benjamin wrote:

> spidev@0 {
> reg = <0>;
> compatible = "rohm,dh2228fv";
> spi-max-frequency = <1000000>;
>
> // via my kernel patch -> /dev/spidev-mysensor
> // linux,spidev-name = "mysensor";

Given you are describing a DAC as a "sensor", I doubt you actually even
have a dh2228fv*. You're looking to have a consistent name here, but you
can't even rely on Linux even continuing to bind the spidev driver
against this compatible if somebody comes along and writes an IIO driver
for this DAC!

That said, google seems to return no results for a dh2228fv, only for a
bh2228fv. Makes me wonder if this device actually even exists...

Attachment: signature.asc
Description: PGP signature