(dropping the very long cc list just cc'ing LKML and devicetree)
On Fri, 2019-06-14 at 18:53 +0100, Suzuki K Poulose wrote:
> We have device iterators to find a particular device matching a criteria
> for a given bus/class/driver. i.e, {bus,class,driver}_find_device() APIs.
> The matching criteria is a function pointer for the APIs. Often the lookup
> is based on a generic property of a device (e.g, name, fwnode, of node pointer
> or device type) rather than a driver specific information. However, each
> driver
> writes up its own "match" function, spilling the similar match functions all
> over the driver subsystems.
>
> Additionally the prototype for the "match" functions accepted by the above
> APIs
> have a minute difference which prevents us otherwise sharing the match
> functions.
> i.e,
> int (*match)(struct device *dev, void *data) for
> {bus/driver}_find_device()
> vs
> int (*match)(struct device *dev, const void *) for class_find_device()
>
As you are doing treewide conversions, perhaps using
bool (*match)(...)
is a more sensible api.
> Changes since v1:
> - Drop start parameter for *_find_device_by_devt().
> - Fix build warnings for s390
> - Add *_find_device_by_acpi_dev() wrappers.
> - Group wrappers and the consumers into single patch, reducing
> the total patches to 28 from 57. (Rafael).
> - Better description for acpi cleanup patch.
> - Added tags from v1.
Below this is a _very_ long list of cc:'s.
If the list is generated using scripts/get_maintainer.pl
perhaps it is more sensible to add --nogit --nogit-fallback
to its arguments to cc actual maintainers and avoid people
that have submitted cleanup style patches to various files.
> Cc: Alan Tull <[email protected]>
> Cc: Alessandro Zummo <[email protected]>
> Cc: Alexander Aring <[email protected]>
> Cc: Alexander Shishkin <[email protected]>
> Cc: Alexandre Belloni <[email protected]>
> Cc: Andreas Noever <[email protected]>
> Cc: Andrew Lunn <[email protected]>
> Cc: Arnd Bergmann <[email protected]>
> Cc: Bjorn Helgaas <[email protected]>
> Cc: Corey Minyard <[email protected]>
> Cc: Daniel Vetter <[email protected]>
> Cc: Dan Murphy <[email protected]>
> Cc: David Airlie <[email protected]>
> Cc: David Kershner <[email protected]>
> Cc: "David S. Miller" <[email protected]>
> Cc: Doug Ledford <[email protected]>
> Cc: [email protected]
> Cc: Elie Morisse <[email protected]>
> Cc: Eric Anholt <[email protected]>
> Cc: Felipe Balbi <[email protected]>
> Cc: Florian Fainelli <[email protected]>
> Cc: Frank Rowand <[email protected]>
> Cc: Grant Likely <[email protected]>
> Cc: Greg Kroah-Hartman <[email protected]>
> Cc: Grygorii Strashko <[email protected]>
> Cc: Harald Freudenberger <[email protected]>
> Cc: Hartmut Knaack <[email protected]>
> Cc: Heikki Krogerus <[email protected]>
> Cc: Heiko Carstens <[email protected]>
> Cc: "Heiko Stübner" <[email protected]>
> Cc: Heiko Stuebner <[email protected]>
> Cc: Heiner Kallweit <[email protected]>
> Cc: Inki Dae <[email protected]>
> Cc: Jacek Anaszewski <[email protected]>
> Cc: "James E.J. Bottomley" <[email protected]>
> Cc: Jason Gunthorpe <[email protected]>
> Cc: Jiri Slaby <[email protected]>
> Cc: Joe Perches <[email protected]>
> Cc: Joerg Roedel <[email protected]>
> Cc: Jonathan Cameron <[email protected]>
> Cc: Jonathan Hunter <[email protected]>
> Cc: Lee Jones <[email protected]>
> Cc: Len Brown <[email protected]
> Cc: Liam Girdwood <[email protected]>
> Cc: Maarten Lankhorst <[email protected]>
> Cc: Mark Brown <[email protected]>
> Cc: "Martin K. Petersen" <[email protected]>
> Cc: Mathieu Poirier <[email protected]>
> Cc: Maxime Coquelin <[email protected]>
> Cc: Maxime Ripard <[email protected]>
> Cc: Michael Jamet <[email protected]>
> Cc: Mika Westerberg <[email protected]>
> Cc: Moritz Fischer <[email protected]>
> Cc: Nehal Shah <[email protected]>
> Cc: Oliver Neukum <[email protected]>
> Cc: Pavel Machek <[email protected]>
> Cc: Peter Oberparleiter <[email protected]>
> Cc: Peter Rosin <[email protected]>
> Cc: "Rafael J. Wysocki" <[email protected]>
> Cc: "Rafael J. Wysocki" <[email protected]>
> Cc: Rafael J. Wysocki" <[email protected]>
> Cc: Rob Herring <[email protected]>
> Cc: Robin Murphy <[email protected]>
> Cc: Russell King <[email protected]>
> Cc: Sandy Huang <[email protected]>
> Cc: Sebastian Andrzej Siewior <[email protected]>
> Cc: Sebastian Ott <[email protected]>
> Cc: Seung-Woo Kim <[email protected]>
> Cc: Shyam Sundar S K <[email protected]>
> Cc: Srinivas Kandagatla <[email protected]>
> Cc: Stefan Schmidt <[email protected]>
> Cc: Takashi Iwai <[email protected]>
> Cc: Thierry Reding <[email protected]>
> Cc: Thor Thayer <[email protected]>
> Cc: Tomas Winkler <[email protected]>
> Cc: Ulf Hansson <[email protected]>
> Cc: Will Deacon <[email protected]>
> Cc: Wolfram Sang <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]
> Cc: [email protected]