Hi Guenter,
On 28.06.24 02:05, Guenter Roeck wrote:
On 6/24/24 09:13, Farouk Bouabid wrote:
Hi Guenter,
On 19.06.24 15:31, Guenter Roeck wrote:
On 6/19/24 00:45, Farouk Bouabid wrote:
[...]
If we cannot proceed with that then we could add a compatible to the
amc6821 driver to add the mux device (Basically the "tsd,mule"
compatible in amc6821 compatible list would be a combo driver with
mux logic + amc6821). Do you think that is more appropriate ?
Implement the mux as part of the amc6821 driver ? No. We could discuss
instantiating the i2c mux driver from the amc6821 driver.
If I understand correctly we would have the amc6821 for "tsd,mule"
compatible probe the mule-i2c-mux as platform_device. Possible DT
representation would be something like:
i2c {
#address-cells = <1>;
#size-cells = <0>;
fan@18 {
compatible = "tsd,mule", "ti,amc6821";
reg = <0x18>;
i2c-mux {
compatible = "tsd,mule-i2c-mux";
#address-cells = <1>;
#size-cells = <0>;
i2c@0 {
reg = <0x0>;
#address-cells = <1>;
#size-cells = <0>;
rtc@6f {
compatible = "isil,isl1208";
reg = <0x6f>;
};
};
};
};
};
Cheers,
Farouk
