Re: [PATCH v3 09/10] gpio: aggregator: add possibility to attach data to the forwarder

From: Andy Shevchenko
Date: Thu Apr 17 2025 - 13:43:32 EST

Next message: Jonathan Cameron: "Re: [PATCH 6/8] iio: imu: adis16550: align buffers for timestamp"
Previous message: Michael Grzeschik: "[PATCH] usb: dwc2: also exit clock_gating when stopping udc while suspended"
In reply to: Thomas Richard: "[PATCH v3 09/10] gpio: aggregator: add possibility to attach data to the forwarder"
Next in thread: Thomas Richard: "[PATCH v3 07/10] gpio: aggregator: export symbols of the GPIO forwarder library"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Wed, Apr 16, 2025 at 04:08:17PM +0200, Thomas Richard wrote:
> Add a data pointer to store private data in the forwarder.

...

> -int gpio_fwd_register(struct gpiochip_fwd *fwd)
> +int gpio_fwd_register(struct gpiochip_fwd *fwd, void *data)

Since it comes from outside there is no need to have a getter at all.
No? Currently I don't understand how this change is used.

--
With Best Regards,
Andy Shevchenko

Next message: Jonathan Cameron: "Re: [PATCH 6/8] iio: imu: adis16550: align buffers for timestamp"
Previous message: Michael Grzeschik: "[PATCH] usb: dwc2: also exit clock_gating when stopping udc while suspended"
In reply to: Thomas Richard: "[PATCH v3 09/10] gpio: aggregator: add possibility to attach data to the forwarder"
Next in thread: Thomas Richard: "[PATCH v3 07/10] gpio: aggregator: export symbols of the GPIO forwarder library"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]