Re: [PATCH v2 04/28] bus_find_device: Unify the match callback with class_find_device
From: Wolfram Sang
Date: Fri Jun 14 2019 - 16:35:45 EST
On Fri, Jun 14, 2019 at 06:53:59PM +0100, Suzuki K Poulose wrote:
> There is an arbitrary difference between the prototypes of
> bus_find_device() and class_find_device() preventing their callers
> from passing the same pair of data and match() arguments to both of
> them, which is the const qualifier used in the prototype of
> class_find_device(). If that qualifier is also used in the
> bus_find_device() prototype, it will be possible to pass the same
> match() callback function to both bus_find_device() and
> class_find_device(), which will allow some optimizations to be made in
> order to avoid code duplication going forward. Also with that, constify
> the "data" parameter as it is passed as a const to the match function.
>
> For this reason, change the prototype of bus_find_device() to match
> the prototype of class_find_device() and adjust its callers to use the
> const qualifier in accordance with the new prototype of it.
Acked-by: Wolfram Sang <wsa@xxxxxxxxxxxxx> # for the I2C parts
Attachment:
signature.asc
Description: PGP signature