From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Return-Path: Date: Thu, 25 Jul 2019 16:10:36 +0200 From: Greg KH Subject: Re: [PATCH v3 0/7] drivers: Add generic device lookup helpers Message-ID: <20190725141036.GA14950@kroah.com> References: <20190723221838.12024-1-suzuki.poulose@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190723221838.12024-1-suzuki.poulose@arm.com> To: Suzuki K Poulose Cc: linux-kernel@vger.kernel.org, rafael@kernel.org, linux-arm-kernel@lists.infradead.org, Alan Tull , Alessandro Zummo , Alexander Aring , Alexander Shishkin , Alexandre Belloni , Andrew Lunn , Arnd Bergmann , Bjorn Helgaas , Daniel Vetter , Dan Murphy , David Airlie , "David S. Miller" , devicetree@vger.kernel.org, Doug Ledford , dri-devel@lists.freedesktop.org, Elie Morisse , Eric Anholt , Florian Fainelli , Frank Rowand , Harald Freudenberger , Heikki Krogerus , Heiko Carstens , Heiko =?iso-8859-1?Q?St=FCbner?= , Heiner Kallweit , Inki Dae , Jacek Anaszewski , "James E.J. Bottomley" , Jason Gunthorpe , Jiri Slaby , Joe Perches , Joerg Roedel , Lee Jones , Len Brown , Liam Girdwood , linux-acpi@vger.kernel.org, linux-fpga@vger.kernel.org, linux-i2c@vger.kernel.org, linux-leds@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-rtc@vger.kernel.org, linux-spi@vger.kernel.org, linux-usb@vger.kernel.org, linux-wpan@vger.kernel.org, Maarten Lankhorst , Mark Brown , "Martin K. Petersen" , Mathieu Poirier , Maxime Coquelin , Maxime Ripard , Mika Westerberg , Moritz Fischer , Nehal Shah , Oliver Neukum , Pavel Machek , Peter Oberparleiter , Peter Rosin , "Rafael J. Wysocki" , Rob Herring , Robin Murphy , Sandy Huang , Sebastian Andrzej Siewior , Seung-Woo Kim , Shyam Sundar S K , Srinivas Kandagatla , Stefan Schmidt , Takashi Iwai , Thor Thayer , Tomas Winkler , Ulf Hansson , Will Deacon , Wolfram Sang List-ID: On Tue, Jul 23, 2019 at 11:18:31PM +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. > > This series adds device match helpers by generic device properties of a device. > Using these generic match functions, introduce wrappers to the existing > *_find_device() helpers and converts the existing users to use the new wrappers. > i.e, > {bus/class/driver}_find_device_by_name() > {bus/class/driver}_find_device_by_fwnode() > {bus/class/driver}_find_device_by_devt() > {bus/class/driver}_find_device_by_acpi_dev() > > Also adds a wrapper for finding a platform device by driver, to avoid > spilling the platform_bus internals in the individual drivers. Applies > on 5.3-rc1. > > [0] git://linux-arm.org/linux-skp.git driver-cleanup/v3 > RFC[1] https://marc.info/?i=1559577023-558-1-git-send-email-suzuki.poulose@arm.com > V1 [2] https://marc.info/?i=1559747630-28065-1-git-send-email-suzuki.poulose@arm.com > V2 [3] https://marc.info/?i=1560534863-15115-1-git-send-email-suzuki.poulose@arm.com > > Changes since v2: > - Merge the device match helper introduction patch with the variants > of the helpers, for better review. > - Use platform_find_device_by_driver for mcde drm driver. This looks good to me at first glance, thanks for doing this work. I'll wait a few days to collect acks from various subsystem maintainers before taking the series in my tree. thanks, greg k-h From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [PATCH v3 0/7] drivers: Add generic device lookup helpers Date: Thu, 25 Jul 2019 16:10:36 +0200 Message-ID: <20190725141036.GA14950@kroah.com> References: <20190723221838.12024-1-suzuki.poulose@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20190723221838.12024-1-suzuki.poulose@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Suzuki K Poulose Cc: Andrew Lunn , Alexander Aring , Heikki Krogerus , Jacek Anaszewski , rafael@kernel.org, Alexander Shishkin , linux-fpga@vger.kernel.org, Heiko Carstens , Alexandre Belloni , dri-devel@lists.freedesktop.org, Liam Girdwood , Elie Morisse , Srinivas Kandagatla , linux-i2c@vger.kernel.org, Pavel Machek , Shyam Sundar S K , Frank Rowand , linux-leds@vger.kernel.org, linux-rtc@vger.kernel.org, Maxime Ripard , Florian Fainelli , linux-acpi@vger.kernel.org, Jason Gunthorpe , Lee Jones List-Id: linux-i2c@vger.kernel.org T24gVHVlLCBKdWwgMjMsIDIwMTkgYXQgMTE6MTg6MzFQTSArMDEwMCwgU3V6dWtpIEsgUG91bG9z ZSB3cm90ZToKPiBXZSBoYXZlIGRldmljZSBpdGVyYXRvcnMgdG8gZmluZCBhIHBhcnRpY3VsYXIg ZGV2aWNlIG1hdGNoaW5nIGEgY3JpdGVyaWEKPiBmb3IgYSBnaXZlbiBidXMvY2xhc3MvZHJpdmVy LiBpLmUsIHtidXMsY2xhc3MsZHJpdmVyfV9maW5kX2RldmljZSgpCj4gQVBJcy4gVGhlIG1hdGNo aW5nIGNyaXRlcmlhIGlzIGEgZnVuY3Rpb24gcG9pbnRlciBmb3IgdGhlIEFQSXMuIE9mdGVuIHRo ZQo+IGxvb2t1cCBpcyBiYXNlZCBvbiBhIGdlbmVyaWMgcHJvcGVydHkgb2YgYSBkZXZpY2UgKGUu ZywgbmFtZSwgZndub2RlLCBvZiBub2RlCj4gcG9pbnRlciBvciBkZXZpY2UgdHlwZSkgcmF0aGVy IHRoYW4gYSBkcml2ZXIgc3BlY2lmaWMgaW5mb3JtYXRpb24uIEhvd2V2ZXIsIGVhY2gKPiBkcml2 ZXIgd3JpdGVzIHVwIGl0cyBvd24gIm1hdGNoIiBmdW5jdGlvbiwgc3BpbGxpbmcgdGhlIHNpbWls YXIgbWF0Y2ggZnVuY3Rpb25zCj4gYWxsIG92ZXIgdGhlIGRyaXZlciBzdWJzeXN0ZW1zLgo+IAo+ IFRoaXMgc2VyaWVzIGFkZHMgZGV2aWNlIG1hdGNoIGhlbHBlcnMgYnkgZ2VuZXJpYyBkZXZpY2Ug cHJvcGVydGllcyBvZiBhIGRldmljZS4KPiBVc2luZyB0aGVzZSBnZW5lcmljIG1hdGNoIGZ1bmN0 aW9ucywgaW50cm9kdWNlIHdyYXBwZXJzIHRvIHRoZSBleGlzdGluZwo+ICAqX2ZpbmRfZGV2aWNl KCkgaGVscGVycyBhbmQgY29udmVydHMgdGhlIGV4aXN0aW5nIHVzZXJzIHRvIHVzZSB0aGUgbmV3 IHdyYXBwZXJzLgo+ICBpLmUsCj4gCXtidXMvY2xhc3MvZHJpdmVyfV9maW5kX2RldmljZV9ieV9u YW1lKCkKPiAJe2J1cy9jbGFzcy9kcml2ZXJ9X2ZpbmRfZGV2aWNlX2J5X2Z3bm9kZSgpCj4gCXti dXMvY2xhc3MvZHJpdmVyfV9maW5kX2RldmljZV9ieV9kZXZ0KCkKPiAJe2J1cy9jbGFzcy9kcml2 ZXJ9X2ZpbmRfZGV2aWNlX2J5X2FjcGlfZGV2KCkKPiAKPiBBbHNvIGFkZHMgYSB3cmFwcGVyIGZv ciBmaW5kaW5nIGEgcGxhdGZvcm0gZGV2aWNlIGJ5IGRyaXZlciwgdG8gYXZvaWQKPiBzcGlsbGlu ZyB0aGUgcGxhdGZvcm1fYnVzIGludGVybmFscyBpbiB0aGUgaW5kaXZpZHVhbCBkcml2ZXJzLiBB cHBsaWVzCj4gb24gNS4zLXJjMS4KPiAKPiAgICBbMF0gZ2l0Oi8vbGludXgtYXJtLm9yZy9saW51 eC1za3AuZ2l0IGRyaXZlci1jbGVhbnVwL3YzCj4gUkZDWzFdIGh0dHBzOi8vbWFyYy5pbmZvLz9p PTE1NTk1NzcwMjMtNTU4LTEtZ2l0LXNlbmQtZW1haWwtc3V6dWtpLnBvdWxvc2VAYXJtLmNvbQo+ IFYxIFsyXSBodHRwczovL21hcmMuaW5mby8/aT0xNTU5NzQ3NjMwLTI4MDY1LTEtZ2l0LXNlbmQt ZW1haWwtc3V6dWtpLnBvdWxvc2VAYXJtLmNvbQo+IFYyIFszXSBodHRwczovL21hcmMuaW5mby8/ aT0xNTYwNTM0ODYzLTE1MTE1LTEtZ2l0LXNlbmQtZW1haWwtc3V6dWtpLnBvdWxvc2VAYXJtLmNv bQo+IAo+IENoYW5nZXMgc2luY2UgdjI6Cj4gIC0gTWVyZ2UgdGhlIGRldmljZSBtYXRjaCBoZWxw ZXIgaW50cm9kdWN0aW9uIHBhdGNoIHdpdGggdGhlIHZhcmlhbnRzCj4gICAgb2YgdGhlIGhlbHBl cnMsIGZvciBiZXR0ZXIgcmV2aWV3Lgo+ICAtIFVzZSBwbGF0Zm9ybV9maW5kX2RldmljZV9ieV9k cml2ZXIgZm9yIG1jZGUgZHJtIGRyaXZlci4KClRoaXMgbG9va3MgZ29vZCB0byBtZSBhdCBmaXJz dCBnbGFuY2UsIHRoYW5rcyBmb3IgZG9pbmcgdGhpcyB3b3JrLgoKSSdsbCB3YWl0IGEgZmV3IGRh eXMgdG8gY29sbGVjdCBhY2tzIGZyb20gdmFyaW91cyBzdWJzeXN0ZW0gbWFpbnRhaW5lcnMKYmVm b3JlIHRha2luZyB0aGUgc2VyaWVzIGluIG15IHRyZWUuCgp0aGFua3MsCgpncmVnIGstaApfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFp bGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWw= From mboxrd@z Thu Jan 1 00:00:00 1970 From: Greg KH Subject: Re: [PATCH v3 0/7] drivers: Add generic device lookup helpers Date: Thu, 25 Jul 2019 16:10:36 +0200 Message-ID: <20190725141036.GA14950@kroah.com> References: <20190723221838.12024-1-suzuki.poulose@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Cc: Andrew Lunn , Alexander Aring , Heikki Krogerus , Jacek Anaszewski , rafael@kernel.org, Alexander Shishkin , linux-fpga@vger.kernel.org, Heiko Carstens , Alexandre Belloni , dri-devel@lists.freedesktop.org, Liam Girdwood , Elie Morisse , Srinivas Kandagatla , linux-i2c@vger.kernel.org, Pavel Machek , Shyam Sundar S K , Frank Rowand , linux-leds@vger.kernel.org, linux-rtc@vger.kernel.org, Maxime Ripard , Florian Fainelli , linux-acpi@vger.kernel.org, Jason Gunthorpe , Lee Jones Return-path: Content-Disposition: inline In-Reply-To: <20190723221838.12024-1-suzuki.poulose@arm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" List-Id: linux-spi.vger.kernel.org T24gVHVlLCBKdWwgMjMsIDIwMTkgYXQgMTE6MTg6MzFQTSArMDEwMCwgU3V6dWtpIEsgUG91bG9z ZSB3cm90ZToKPiBXZSBoYXZlIGRldmljZSBpdGVyYXRvcnMgdG8gZmluZCBhIHBhcnRpY3VsYXIg ZGV2aWNlIG1hdGNoaW5nIGEgY3JpdGVyaWEKPiBmb3IgYSBnaXZlbiBidXMvY2xhc3MvZHJpdmVy LiBpLmUsIHtidXMsY2xhc3MsZHJpdmVyfV9maW5kX2RldmljZSgpCj4gQVBJcy4gVGhlIG1hdGNo aW5nIGNyaXRlcmlhIGlzIGEgZnVuY3Rpb24gcG9pbnRlciBmb3IgdGhlIEFQSXMuIE9mdGVuIHRo ZQo+IGxvb2t1cCBpcyBiYXNlZCBvbiBhIGdlbmVyaWMgcHJvcGVydHkgb2YgYSBkZXZpY2UgKGUu ZywgbmFtZSwgZndub2RlLCBvZiBub2RlCj4gcG9pbnRlciBvciBkZXZpY2UgdHlwZSkgcmF0aGVy IHRoYW4gYSBkcml2ZXIgc3BlY2lmaWMgaW5mb3JtYXRpb24uIEhvd2V2ZXIsIGVhY2gKPiBkcml2 ZXIgd3JpdGVzIHVwIGl0cyBvd24gIm1hdGNoIiBmdW5jdGlvbiwgc3BpbGxpbmcgdGhlIHNpbWls YXIgbWF0Y2ggZnVuY3Rpb25zCj4gYWxsIG92ZXIgdGhlIGRyaXZlciBzdWJzeXN0ZW1zLgo+IAo+ IFRoaXMgc2VyaWVzIGFkZHMgZGV2aWNlIG1hdGNoIGhlbHBlcnMgYnkgZ2VuZXJpYyBkZXZpY2Ug cHJvcGVydGllcyBvZiBhIGRldmljZS4KPiBVc2luZyB0aGVzZSBnZW5lcmljIG1hdGNoIGZ1bmN0 aW9ucywgaW50cm9kdWNlIHdyYXBwZXJzIHRvIHRoZSBleGlzdGluZwo+ICAqX2ZpbmRfZGV2aWNl KCkgaGVscGVycyBhbmQgY29udmVydHMgdGhlIGV4aXN0aW5nIHVzZXJzIHRvIHVzZSB0aGUgbmV3 IHdyYXBwZXJzLgo+ICBpLmUsCj4gCXtidXMvY2xhc3MvZHJpdmVyfV9maW5kX2RldmljZV9ieV9u YW1lKCkKPiAJe2J1cy9jbGFzcy9kcml2ZXJ9X2ZpbmRfZGV2aWNlX2J5X2Z3bm9kZSgpCj4gCXti dXMvY2xhc3MvZHJpdmVyfV9maW5kX2RldmljZV9ieV9kZXZ0KCkKPiAJe2J1cy9jbGFzcy9kcml2 ZXJ9X2ZpbmRfZGV2aWNlX2J5X2FjcGlfZGV2KCkKPiAKPiBBbHNvIGFkZHMgYSB3cmFwcGVyIGZv ciBmaW5kaW5nIGEgcGxhdGZvcm0gZGV2aWNlIGJ5IGRyaXZlciwgdG8gYXZvaWQKPiBzcGlsbGlu ZyB0aGUgcGxhdGZvcm1fYnVzIGludGVybmFscyBpbiB0aGUgaW5kaXZpZHVhbCBkcml2ZXJzLiBB cHBsaWVzCj4gb24gNS4zLXJjMS4KPiAKPiAgICBbMF0gZ2l0Oi8vbGludXgtYXJtLm9yZy9saW51 eC1za3AuZ2l0IGRyaXZlci1jbGVhbnVwL3YzCj4gUkZDWzFdIGh0dHBzOi8vbWFyYy5pbmZvLz9p PTE1NTk1NzcwMjMtNTU4LTEtZ2l0LXNlbmQtZW1haWwtc3V6dWtpLnBvdWxvc2VAYXJtLmNvbQo+ IFYxIFsyXSBodHRwczovL21hcmMuaW5mby8/aT0xNTU5NzQ3NjMwLTI4MDY1LTEtZ2l0LXNlbmQt ZW1haWwtc3V6dWtpLnBvdWxvc2VAYXJtLmNvbQo+IFYyIFszXSBodHRwczovL21hcmMuaW5mby8/ aT0xNTYwNTM0ODYzLTE1MTE1LTEtZ2l0LXNlbmQtZW1haWwtc3V6dWtpLnBvdWxvc2VAYXJtLmNv bQo+IAo+IENoYW5nZXMgc2luY2UgdjI6Cj4gIC0gTWVyZ2UgdGhlIGRldmljZSBtYXRjaCBoZWxw ZXIgaW50cm9kdWN0aW9uIHBhdGNoIHdpdGggdGhlIHZhcmlhbnRzCj4gICAgb2YgdGhlIGhlbHBl cnMsIGZvciBiZXR0ZXIgcmV2aWV3Lgo+ICAtIFVzZSBwbGF0Zm9ybV9maW5kX2RldmljZV9ieV9k cml2ZXIgZm9yIG1jZGUgZHJtIGRyaXZlci4KClRoaXMgbG9va3MgZ29vZCB0byBtZSBhdCBmaXJz dCBnbGFuY2UsIHRoYW5rcyBmb3IgZG9pbmcgdGhpcyB3b3JrLgoKSSdsbCB3YWl0IGEgZmV3IGRh eXMgdG8gY29sbGVjdCBhY2tzIGZyb20gdmFyaW91cyBzdWJzeXN0ZW0gbWFpbnRhaW5lcnMKYmVm b3JlIHRha2luZyB0aGUgc2VyaWVzIGluIG15IHRyZWUuCgp0aGFua3MsCgpncmVnIGstaApfX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fXwpkcmktZGV2ZWwgbWFp bGluZyBsaXN0CmRyaS1kZXZlbEBsaXN0cy5mcmVlZGVza3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5m cmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9kcmktZGV2ZWw=