mbox series

[v1,0/9] i2c: designware: code consolidation & cleanups

Message ID 20230725143023.86325-1-andriy.shevchenko@linux.intel.com
Headers show
Series i2c: designware: code consolidation & cleanups | expand

Message

Andy Shevchenko July 25, 2023, 2:30 p.m. UTC
Mainly this is about firmware parsing and configuring code
consolidation. Besides that some cleanups here and there.

Andy Shevchenko (9):
  i2c: designware: Move has_acpi_companion() to common code
  i2c: designware: Change i2c_dw_acpi_configure() prototype
  i2c: designware: Align dw_i2c_of_configure() with
    i2c_dw_acpi_configure()
  i2c: designware: Propagate firmware node
  i2c: designware: Always provide ID tables
  i2c: designware: Consolidate firmware parsing and configure code
  i2c: desingware: Unify firmware type checks
  i2c: designware: Get rid of redundant 'else'
  i2c: designware: Fix spelling and other issues in the comments

 drivers/i2c/busses/i2c-designware-amdpsp.c  |  10 +-
 drivers/i2c/busses/i2c-designware-common.c  |  84 +++++++++++---
 drivers/i2c/busses/i2c-designware-core.h    |  25 ++---
 drivers/i2c/busses/i2c-designware-master.c  |  15 ++-
 drivers/i2c/busses/i2c-designware-pcidrv.c  |  13 +--
 drivers/i2c/busses/i2c-designware-platdrv.c | 117 ++++++--------------
 drivers/i2c/busses/i2c-designware-slave.c   |   6 +-
 7 files changed, 131 insertions(+), 139 deletions(-)

Comments

Mario Limonciello July 25, 2023, 3:22 p.m. UTC | #1
On 7/25/2023 09:30, Andy Shevchenko wrote:
> Mainly this is about firmware parsing and configuring code
> consolidation. Besides that some cleanups here and there.
> 
> Andy Shevchenko (9):
>    i2c: designware: Move has_acpi_companion() to common code
>    i2c: designware: Change i2c_dw_acpi_configure() prototype
>    i2c: designware: Align dw_i2c_of_configure() with
>      i2c_dw_acpi_configure()
>    i2c: designware: Propagate firmware node
>    i2c: designware: Always provide ID tables
>    i2c: designware: Consolidate firmware parsing and configure code
>    i2c: desingware: Unify firmware type checks
>    i2c: designware: Get rid of redundant 'else'
>    i2c: designware: Fix spelling and other issues in the comments
> 
>   drivers/i2c/busses/i2c-designware-amdpsp.c  |  10 +-
>   drivers/i2c/busses/i2c-designware-common.c  |  84 +++++++++++---
>   drivers/i2c/busses/i2c-designware-core.h    |  25 ++---
>   drivers/i2c/busses/i2c-designware-master.c  |  15 ++-
>   drivers/i2c/busses/i2c-designware-pcidrv.c  |  13 +--
>   drivers/i2c/busses/i2c-designware-platdrv.c | 117 ++++++--------------
>   drivers/i2c/busses/i2c-designware-slave.c   |   6 +-
>   7 files changed, 131 insertions(+), 139 deletions(-)
> 

Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>