Re: [PATCH v1 00/15] i2c-multi-instantiate: Adapt for INT3515 and alike

[Hans de Goede]

Hi,

On 21-11-18 12:27, Hans de Goede wrote:
> Hi,
>
> On 20-11-18 16:59, Andy Shevchenko wrote:
> > Currently i2c-multi-instantiate driver does not support the case
> > of INT3515 USB PD device, where:
> >
> > - Interrupt() is used instead of GpioInt()
> > - All slaves may be the same from IP point of view
> > - There can be variadic amount of slaves
> >
> > This series is addressing all above.
> >
> > Notes:
> > - series has been smoke tested (Heikki would do BAT or more) on Coffee Lake system
> > - patch for BOSC0200 might be considered as RFC since the modalias potential issue
>
> Yeah the BOSC0200 patch cannot go upstream as is, that will break the
> orientation quirks in hwdb on:
>
> [hans@shalem systemd]$ grep BOSC0200 hwdb/60-sensor.hwdb | wc -l
> 17
> [hans@shalem systemd]$
>
> At least 17 different models laptops / 2-in-1s
>
> > The idea is to push this either through PDx86 tree (needs Rafael's ACKs) or ACPI.
> > In any case it needs tags from Heikki, Hans, Mika, Wolfram and Jonathan.
>
> Patches 1-5, 7-13 and 15 are: >
> Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>

Correction, I also have some small remarks to patch 9:

[PATCH v1 09/15] platform/x86: i2c-multi-instantiate: Count I2cSerialBus() resources

Which will need to be addressed before you can add my Reviewed-by.

Regards,

Hans




> I will send separate replies to:
>
> [PATCH v1 06/15] i2c: acpi: Assign fwnode for devices created via i2c_acpi_new_device()
> [PATCH v1 14/15] ACPI / scan: Create platform device for BOSC0200 ACPI nodes
>
> Which are both a NACK from me (in their current state) because they both will
> cause regressions.
>
> Regards,
>
> Hans
>
>
>
>
>
>
> > Testing and comments are warmly welcome.
> >
> > Andy Shevchenko (15):
> > ÂÂ platform/x86: intel_cht_int33fe: Remove duplicate NULL check
> > ÂÂ platform/x86: intel_cht_int33fe: Accept errors of
> > ÂÂÂÂ i2c_acpi_new_device()
> > ÂÂ platform/x86: i2c-multi-instantiate: Accept errors of
> > ÂÂÂÂ i2c_acpi_new_device()
> > ÂÂ platform/x86: i2c-mutli-instantiate: Defer probe when no adapter found
> > ÂÂ i2c: acpi: Return error pointers from i2c_acpi_new_device()
> > ÂÂ i2c: acpi: Assign fwnode for devices created via i2c_acpi_new_device()
> > ÂÂ i2c: acpi: Use ACPI_FAILURE instead of !ACPI_SUCCESS
> > ÂÂ i2c: acpi: Introduce i2c_acpi_get_i2c_resource() helper
> > ÂÂ platform/x86: i2c-multi-instantiate: Count I2cSerialBus() resources
> > ÂÂ platform/x86: i2c-multi-instantiate: Distinguish IRQ resource type
> > ÂÂ platform/x86: i2c-multi-instantiate: Introduce IOAPIC IRQ support
> > ÂÂ platform/x86: i2c-multi-instantiate: Allow to have same slaves
> > ÂÂ ACPI / scan: Create platform device for INT3515 ACPI nodes
> > ÂÂ ACPI / scan: Create platform device for BOSC0200 ACPI nodes
> > ÂÂ iio: inv_mpu6050: Use i2c_acpi_get_i2c_resource() helper
> >
> > Â drivers/acpi/scan.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |Â 2 +
> > Â drivers/i2c/i2c-core-acpi.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ | 43 +++++----
> > Â drivers/iio/accel/bmc150-accel-i2c.cÂÂÂÂÂÂÂÂ |Â 1 -
> > Â drivers/iio/imu/inv_mpu6050/inv_mpu_acpi.cÂÂ | 16 ++--
> > Â drivers/platform/x86/i2c-multi-instantiate.c | 99 ++++++++++++++++----
> > Â drivers/platform/x86/intel_cht_int33fe.cÂÂÂÂ | 34 +++++--
> > Â drivers/usb/typec/tps6598x.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |Â 8 +-
> > Â include/linux/acpi.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ | 11 +++
> > Â 8 files changed, 158 insertions(+), 56 deletions(-)