linux-toradex.git/drivers/i2c/muxes, branch v6.6-rc3

i2c: mux: gpio: Add missing fwnode_handle_put()

2023-09-21T08:48:03+00:00

In i2c_mux_gpio_probe_fw(), we should add fwnode_handle_put()
when break out of the iteration device_for_each_child_node()
as it will automatically increase and decrease the refcounter.

Fixes: 98b2b712bc85 ("i2c: i2c-mux-gpio: Enable this driver in ACPI land")
Signed-off-by: Liang He 
Signed-off-by: Wolfram Sang

i2c: mux: demux-pinctrl: check the return value of devm_kstrdup()

2023-09-20T21:26:37+00:00

devm_kstrdup() returns pointer to allocated string on success,
NULL on failure. So it is better to check the return value of it.

Fixes: e35478eac030 ("i2c: mux: demux-pinctrl: run properly with multiple instances")
Signed-off-by: Xiaoke Wang 
Signed-off-by: Wolfram Sang

i2c: muxes: pca954x: Add regulator support

2023-08-23T21:09:12+00:00

Add a vdd regulator and enable it for boards that have the
mux powered off by default.

Signed-off-by: Patrick Rudolph 
Reviewed-by: Andi Shyti 
Acked-by: Peter Rosin 
Signed-off-by: Wolfram Sang

i2c: muxes: pca954x: Add MAX735x/MAX736x support

2023-08-23T21:09:03+00:00

Add support for the following Maxim chips using the existing PCA954x
driver:
- MAX7356
- MAX7357
- MAX7358
- MAX7367
- MAX7368
- MAX7369

All added Maxim chips behave like the PCA954x, where a single SMBUS byte
write selects up to 8 channels to be bridged to the primary bus.

While the MAX7357/MAX7358 have interrupt support, they don't act as
interrupt controller like the PCA9545 does. Thus don't enable IRQ support
and handle them like the PCA9548.

Tested using the MAX7357.

Signed-off-by: Patrick Rudolph 
Reviewed-by: Andi Shyti 
Acked-by: Peter Rosin 
Signed-off-by: Wolfram Sang

I2C: Explicitly include correct DT includes

2023-08-14T15:47:21+00:00

The DT of_device.h and of_platform.h date back to the separate
of_platform_bus_type before it as merged into the regular platform bus.
As part of that merge prepping Arm DT support 13 years ago, they
"temporarily" include each other. They also include platform_device.h
and of.h. As a result, there's a pretty much random mix of those include
files used throughout the tree. In order to detangle these headers and
replace the implicit includes with struct declarations, users need to
explicitly include the correct includes.

Signed-off-by: Rob Herring 
Reviewed-by: Andi Shyti 
Signed-off-by: Wolfram Sang

i2c: mux: ltc4306: Remove an unnecessary ternary operator

2023-08-08T13:36:58+00:00

The true or false judgement of the ternary operator is unnecessary
in C language semantics. So remove it to clean Code.

Signed-off-by: Ruan Jinjie 
Acked-by: Andi Shyti 
Link: https://lore.kernel.org/r/20230801025328.3380963-1-ruanjinjie@huawei.com
Signed-off-by: Andi Shyti

i2c: mux: pca954x: Remove #ifdef guards for PM related functions

2023-08-08T13:36:53+00:00

Use the new PM macros for the suspend and resume functions to be
automatically dropped by the compiler when CONFIG_PM or
CONFIG_PM_SLEEP are disabled, without having to use #ifdef guards.

This has the advantage of always compiling these functions in,
independently of any Kconfig option. Thanks to that, bugs and other
regressions are subsequently easier to catch.

Signed-off-by: Paul Cercueil 
Reviewed-by: Jonathan Cameron 
Acked-by: Peter Rosin 
Link: https://lore.kernel.org/r/20230722115310.27681-6-paul@crapouillou.net
Signed-off-by: Andi Shyti

i2c: Convert to platform remove callback returning void

2023-06-05T07:47:37+00:00

The .remove() callback for a platform driver returns an int which makes
many driver authors wrongly assume it's possible to do error handling by
returning an error code. However the value returned is (mostly) ignored
and this typically results in resource leaks. To improve here there is a
quest to make the remove callback return void. In the first step of this
quest all drivers are converted to .remove_new() which already returns
void.

Signed-off-by: Uwe Kleine-König 
Acked-by: Alain Volmat 
Acked-by: Ard Biesheuvel 
Acked-by: Baruch Siach 
Acked-by: Florian Fainelli 
Acked-by: Heiko Stuebner 
Acked-by: Jarkko Nikula 
Acked-by: Jernej Skrabec 
Acked-by: Jochen Friedrich 
Acked-by: Peter Rosin 
Acked-by: Vadim Pasternak 
Reviewed-by: Asmaa Mnebhi 
Reviewed-by: Bartosz Golaszewski 
Reviewed-by: Chris Packham 
Reviewed-by: Chris Pringle 
Reviewed-by: Claudiu Beznea 
Reviewed-by: Conor Dooley 
Reviewed-by: Geert Uytterhoeven 
Reviewed-by: Hans de Goede 
Reviewed-by: Jean Delvare 
Reviewed-by: Konrad Dybcio 
Reviewed-by: Krzysztof Kozlowski 
Reviewed-by: Linus Walleij 
Reviewed-by: Martin Blumenstingl 
Reviewed-by: Matthias Brugger 
Reviewed-by: Patrice Chotard 
Reviewed-by: Tali Perry 
Reviewed-by: Vignesh Raghavendra 
Signed-off-by: Wolfram Sang

i2c: mux: Convert all drivers to new .probe() callback

2023-03-09T20:59:18+00:00

Now that .probe() was changed not to get the id parameter, drivers can
be converted back to that with the eventual goal to drop .probe_new().

Implement that for the i2c mux drivers.

Acked-by: Guenter Roeck 
Acked-by: Peter Rosin 
Signed-off-by: Uwe Kleine-König 
Signed-off-by: Wolfram Sang

i2c: mux: reg: check return value after calling platform_get_resource()

2022-12-07T20:19:32+00:00

It will cause null-ptr-deref in resource_size(), if platform_get_resource()
returns NULL, move calling resource_size() after devm_ioremap_resource() that
will check 'res' to avoid null-ptr-deref.
And use devm_platform_get_and_ioremap_resource() to simplify code.

Fixes: b3fdd32799d8 ("i2c: mux: Add register-based mux i2c-mux-reg")
Signed-off-by: Yang Yingliang 
Signed-off-by: Wolfram Sang