| From: |
| Hans de Goede <hansg-AT-kernel.org> |
| To: |
| Israel Cepeda <israel.a.cepeda.lopez-AT-intel.com>, Sakari Ailus <sakari.ailus-AT-linux.intel.com>, Wolfram Sang <wsa-AT-kernel.org>, Andi Shyti <andi.shyti-AT-kernel.org>, Greg Kroah-Hartman <gregkh-AT-linuxfoundation.org>, Bartosz Golaszewski <brgl-AT-bgdev.pl>, Linus Walleij <linus.walleij-AT-linaro.org> |
| Subject: |
| [PATCH v4 0/3] usb/gpio/i2c: Add Intel USBIO USB IO-expander drivers |
| Date: |
| Wed, 10 Sep 2025 15:39:55 +0200 |
| Message-ID: |
| <20250910133958.224921-1-hansg@kernel.org> |
| Cc: |
| Hans de Goede <hansg-AT-kernel.org>, Richard Hughes <rhughes-AT-redhat.com>, linux-i2c-AT-vger.kernel.org, linux-usb-AT-vger.kernel.org, linux-gpio-AT-vger.kernel.org |
| Archive-link: |
| Article |
Hi All,
Here is v4 of the patch series to add support for the Intel USBIO USB
IO-expander used by the MIPI cameras on various new (Meteor Lake and later)
Intel laptops.
Changes in v4:
- GPIO: Drop include <linux/dev_printk.h>, unneeded auxiliary_set_drvdata()
Changes in v3:
- Drop (offset >= gc->ngpio) check and make usbio_gpio_get_bank_and_pin()
return void
- Propagate usbio_gpio_set() ret val in usbio_gpio_direction_output()
- Use devm_gpiochip_add_data() and drop auxiliary_driver remove() callback
Changes in v2:
- Split usbio-bridge mutex into ctrl_mutex and bulk_mutex
- Drop SPI support since this is not used on devices in the field
- Rework disconnect handling to be more robust
- Several different revisions need special casing add a quirks mechanism
for this
- Stop using stdint.h (uintX_t) types
- Use __le16, __le32 type + cpu_to_le16() and friends for on wire words
- Properly check auxiliary_device_add() return value
- Add a mutex to the GPIO driver to protect usbio_gpio_update_config()
calls, which read-modify-write banks[x].config, racing with each other
- Adjust usbio_gpio_get() to have an int return value and propagate the
usbio_control_msg() return value
- Various (small) style fixes from Sakari's review of all 3 patches
The first patch adds an USB bridge driver which registers auxbus children
for the GPIO and I2C functions of the USBIO chip.
The second and third patch add a GPIO resp. an I2C driver for the
auxbus children using the IO functions exported by the USB bridge driver.
The second and third patch depend on the IO functions exported by
the first patch. So to merge this we will need either an immutable tag on
the USB tree, or all 3 patches can be merged through the USB tree with
acks from the GPIO and I2C subsystem maintainers.
Regards,
Hans
Israel Cepeda (3):
usb: misc: Add Intel USBIO bridge driver
gpio: Add Intel USBIO GPIO driver
i2c: Add Intel USBIO I2C driver
MAINTAINERS | 10 +
drivers/gpio/Kconfig | 11 +
drivers/gpio/Makefile | 1 +
drivers/gpio/gpio-usbio.c | 247 +++++++++++
drivers/i2c/busses/Kconfig | 11 +
drivers/i2c/busses/Makefile | 1 +
drivers/i2c/busses/i2c-usbio.c | 325 ++++++++++++++
drivers/usb/misc/Kconfig | 14 +
drivers/usb/misc/Makefile | 1 +
drivers/usb/misc/usbio.c | 749 +++++++++++++++++++++++++++++++++
include/linux/usb/usbio.h | 177 ++++++++
11 files changed, 1547 insertions(+)
create mode 100644 drivers/gpio/gpio-usbio.c
create mode 100644 drivers/i2c/busses/i2c-usbio.c
create mode 100644 drivers/usb/misc/usbio.c
create mode 100644 include/linux/usb/usbio.h
--
2.51.0
