mbox series

[v2,0/6] platform/x86: int3472: discrete: Various improvements

Message ID 20230616172132.37859-1-hdegoede@redhat.com (mailing list archive)
Headers show
Series platform/x86: int3472: discrete: Various improvements | expand

Message

Hans de Goede June 16, 2023, 5:21 p.m. UTC
Hi All,

Here is a v2 of my previous 2 int3472 patch series merged into
1 bigger series.

Changes in v2:
- Use static_assert() to check that GPIO_REGULATOR_SUPPLY_MAP_COUNT
  and ARRAY_SIZE(skl_int3472_regulator_map_supplies) match

and this also collects all the various Tested-by-s + Reviewed-by-s.

Regards,

Hans


Hans de Goede (6):
  platform/x86: int3472: discrete: Drop GPIO remapping support
  platform/x86: int3472: discrete: Remove sensor_config-s
  platform/x86: int3472: discrete: Add support for 1 GPIO regulator
    shared between 2 sensors
  platform/x86: int3472: discrete: Add alternative "AVDD" regulator
    supply name
  platform/x86: int3472: discrete: Use FIELD_GET() on the GPIO _DSM
    return value
  platform/x86: int3472: discrete: Log a warning if the pin-numbers
    don't match

 .../x86/intel/int3472/clk_and_regulator.c     | 72 +++++++++++---
 drivers/platform/x86/intel/int3472/common.h   | 14 +--
 drivers/platform/x86/intel/int3472/discrete.c | 94 ++++---------------
 3 files changed, 79 insertions(+), 101 deletions(-)

Comments

Hans de Goede June 20, 2023, 10:44 a.m. UTC | #1
Hi all,

On 6/16/23 19:21, Hans de Goede wrote:
> Hi All,
> 
> Here is a v2 of my previous 2 int3472 patch series merged into
> 1 bigger series.
> 
> Changes in v2:
> - Use static_assert() to check that GPIO_REGULATOR_SUPPLY_MAP_COUNT
>   and ARRAY_SIZE(skl_int3472_regulator_map_supplies) match
> 
> and this also collects all the various Tested-by-s + Reviewed-by-s.

I've added this (v2) to my review-hans (soon to be for-next) branch now (again).

Regards,

Hans




> Hans de Goede (6):
>   platform/x86: int3472: discrete: Drop GPIO remapping support
>   platform/x86: int3472: discrete: Remove sensor_config-s
>   platform/x86: int3472: discrete: Add support for 1 GPIO regulator
>     shared between 2 sensors
>   platform/x86: int3472: discrete: Add alternative "AVDD" regulator
>     supply name
>   platform/x86: int3472: discrete: Use FIELD_GET() on the GPIO _DSM
>     return value
>   platform/x86: int3472: discrete: Log a warning if the pin-numbers
>     don't match
> 
>  .../x86/intel/int3472/clk_and_regulator.c     | 72 +++++++++++---
>  drivers/platform/x86/intel/int3472/common.h   | 14 +--
>  drivers/platform/x86/intel/int3472/discrete.c | 94 ++++---------------
>  3 files changed, 79 insertions(+), 101 deletions(-)
>