On 2019-06-14 19:54, Suzuki K Poulose wrote: > Add a wrapper to class_find_device() to search for a device > by the of_node pointer, reusing the generic match function. > Also convert the existing users to make use of the new helper. > > Cc: Alan Tull <[email protected]> > Cc: Moritz Fischer <[email protected]> > Cc: [email protected] > Cc: Peter Rosin <[email protected]> > Cc: Mark Brown <[email protected]> > Cc: Florian Fainelli <[email protected]> > Cc: Heiner Kallweit <[email protected]> > Cc: "David S. Miller" <[email protected]> > Cc: Andrew Lunn <[email protected]> > Cc: Liam Girdwood <[email protected]> > Cc: Greg Kroah-Hartman <[email protected]> > Cc: "Rafael J. Wysocki" <[email protected]> > Cc: Jiri Slaby <[email protected]> > Acked-by: Mark Brown <[email protected]> > Reviewed-by: Andrew Lunn <[email protected]> > Reviewed-by: Peter Rosin <[email protected]>
Whoooa! I reviewed only the drivers/mux/core.c changes when this was done in a series of much smaller patches. This tag makes it seem as if I have reviewed the whole thing, which I had not done when you added this tag out of the blue. Now, this stuff is trivial and by now I have looked at the other files and it all seems simple enough. So, you can keep the tag, but it is NOT ok to handle tags like you have done here. Cheers, Peter > Signed-off-by: Suzuki K Poulose <[email protected]> > --- > drivers/fpga/fpga-bridge.c | 8 +------- > drivers/fpga/fpga-mgr.c | 8 +------- > drivers/mux/core.c | 7 +------ > drivers/net/phy/mdio_bus.c | 9 +-------- > drivers/regulator/of_regulator.c | 7 +------ > drivers/spi/spi.c | 11 ++--------- > include/linux/device.h | 12 ++++++++++++ > 7 files changed, 19 insertions(+), 43 deletions(-)
