mbox series

[v2,0/4] i2c: Replace lists of special clients with flagging of such clients

Message ID 3982b2a6-975e-40d2-bf02-2155e5c36c14@gmail.com
Headers show
Series i2c: Replace lists of special clients with flagging of such clients | expand

Message

Heiner Kallweit Aug. 22, 2024, 7:45 p.m. UTC 
So far lists are used to track special clients, i.e. auto-detected and
userspace-created clients. The same functionality can be achieved much
simpler by flagging such clients.

v2:
- The i2c_driver.clients list is core-internal, however there's an ALSA
  driver using it. So add patch 1 to address this first.

Heiner Kallweit (4):
  ALSA: ppc: Remove i2c client removal hack
  i2c: Replace list-based mechanism for handling auto-detected clients
  i2c: Replace list-based mechanism for handling userspace-created
    clients
  i2c: core: Remove obsolete members of i2c_adapter and i2c_client

 drivers/i2c/i2c-core-base.c | 108 +++++++++++-------------------------
 include/linux/i2c.h         |  10 +---
 sound/ppc/keywest.c         |   7 +--
 3 files changed, 36 insertions(+), 89 deletions(-)

Comments

Wolfram Sang Sept. 1, 2024, 9:30 a.m. UTC | #1 
On Thu, Aug 22, 2024 at 09:45:37PM +0200, Heiner Kallweit wrote:
> So far lists are used to track special clients, i.e. auto-detected and
> userspace-created clients. The same functionality can be achieved much
> simpler by flagging such clients.

This looks promising and I like the idea from a high-level perspective.
Need to dive into the details. However, I think this is 6.13 material. I
want to let it cook in linux-next for a full cycle.
Heiner Kallweit Sept. 1, 2024, 8:08 p.m. UTC | #2 
On 01.09.2024 11:30, Wolfram Sang wrote:
> On Thu, Aug 22, 2024 at 09:45:37PM +0200, Heiner Kallweit wrote:
>> So far lists are used to track special clients, i.e. auto-detected and
>> userspace-created clients. The same functionality can be achieved much
>> simpler by flagging such clients.
> 
> This looks promising and I like the idea from a high-level perspective.
> Need to dive into the details. However, I think this is 6.13 material. I
> want to let it cook in linux-next for a full cycle.
> 
Fine with me.