[PATCH v5 0/8] Add tsd,mule-i2c-mux support
From: Farouk Bouabid
Date: Mon Jul 08 2024 - 12:13:45 EST
Theobroma Systems Mule is an MCU that emulates a set of I2C devices which
are reachable through an I2C-mux.
The devices on the mux can be selected by writing the appropriate
device number to an I2C config register (0xff) that is not used by
amc6821 logic. This required us to add a new compatible to the amc6821
driver, from which, the new platform device "tsd,mule-i2c-mux" is probed.
The selected device on the mux can be accessed for reading and writing
at I2C address 0x6f.
+--------+----------------+------------------------------+
| Mule |
0x18 | +------------------+ |
--------+----->| amc6821 | |
| | +------------------+ |
| +----->| tsd,mule-i2c-mux |---+ |
| +------------------+ | |
| V__ +---------+ |
| | \-------->| isl1208 | |
| | | +---------+ |
0x6f | | M |-------->| dev #1 | |
------------------------------------>| U | +---------+ |
| | X |-------->| dev #2 | |
| | | +---------+ |
| | /-------->| dev #3 | |
| |__/ +---------+ |
+--------------------------------------------------------+
This patch-series adds support for the tsd,mule-i2c multiplexer
as part of rk3399-puma, px30-ringneck, rk3588-tiger and rk3588-jaguar
boards.
this patch-series depends on https://lore.kernel.org/lkml/20240705213547.1155690-1-linux@xxxxxxxxxxxx/
to add regmap support to amc6821.
Signed-off-by: Farouk Bouabid <farouk.bouabid@xxxxxxxxx>
Changes in v5:
- Drop the mfd implementation of v4
- Add more dev_probe_err callbacks to tsd,mule-i2c-mux
- Instantiate tsd,mule-i2c-mux as a platform device from amc6821 driver
- add "Theobroma Systems" when describing mule.
- Link to v4: https://lore.kernel.org/lkml/20240618-dev-mule-i2c-mux-v4-0-5462d28354c8@xxxxxxxxx/
Changes in v4:
- Drop the previously added i2c adapter quirks
- Add platform driver probe to amc6821.
- Change mule-i2c-mux driver to a platform driver
- Add dev_probe_err in mule-i2c-mux driver
- Add support for tsd,mule in simple-mfd-i2c
- Add tsd,mule mfd to supported dts
- Link to v3: https://lore.kernel.org/r/20240611-dev-mule-i2c-mux-v3-0-08d26a28e001@xxxxxxxxx
Changes in v3:
- Change "i2c" in comments/commit-logs to "I2C"
- Fix long line-length
- Warn when "share_addr_with_children" is set and the Mux is not an I2C device
- Fix/stop propagating "I2C_AQ_SKIP_ADDR_CHECK" flag if "share_addr_with_children"
is not set.
- Fix "old_fw" variable is used to indicate the reversed meaning.
- Link to v2: https://lore.kernel.org/r/20240506-dev-mule-i2c-mux-v2-0-a91c954f65d7@xxxxxxxxx
Changes in v2:
- Add i2c-adapter quirks to skip checking for conflict between the mux core
and a child device address.
- Rename dt-binding to "tsd,mule-i2c-mux.yaml"
- Add Mule description to kconfig
- Fix indentation
- Move device table after probe
- Link to v1: https://lore.kernel.org/r/20240426-dev-mule-i2c-mux-v1-0-045a482f6ffb@xxxxxxxxxxxxxxxxxxxxx
---
Farouk Bouabid (8):
dt-bindings: i2c: add support for tsd,mule-i2c-mux
i2c: muxes: add support for tsd,mule-i2c multiplexer
dt-bindings: i2c: add support for tsd,mule
hwmon: (amc6821) add support for tsd,mule
arm64: dts: rockchip: add tsd,mule-i2c-mux on rk3588-jaguar
arm64: dts: rockchip: add tsd,mule-i2c-mux on rk3399-puma
arm64: dts: rockchip: add tsd,mule-i2c-mux on rk3588-tiger
arm64: dts: rockchip: add tsd,mule-i2c-mux on px30-ringneck
.../devicetree/bindings/i2c/tsd,mule-i2c-mux.yaml | 69 +++++++++
.../devicetree/bindings/i2c/tsd,mule.yaml | 63 +++++++++
arch/arm64/boot/dts/rockchip/px30-ringneck.dtsi | 24 +++-
arch/arm64/boot/dts/rockchip/rk3399-puma.dtsi | 24 +++-
arch/arm64/boot/dts/rockchip/rk3588-jaguar.dts | 25 +++-
arch/arm64/boot/dts/rockchip/rk3588-tiger.dtsi | 23 ++-
drivers/hwmon/amc6821.c | 25 +++-
drivers/i2c/muxes/Kconfig | 16 +++
drivers/i2c/muxes/Makefile | 1 +
drivers/i2c/muxes/i2c-mux-mule.c | 155 +++++++++++++++++++++
10 files changed, 401 insertions(+), 24 deletions(-)
---
base-commit: 97c57cb030b79def7b664c253e0c4f0702b9e758
change-id: 20240404-dev-mule-i2c-mux-9103cde07021
Best regards,
--
Farouk Bouabid <farouk.bouabid@xxxxxxxxx>