Re: [PATCH 10/13] drivers: Introduce variants of class_find_device()

From: Suzuki K Poulose
Date: Wed Jun 05 2019 - 14:14:49 EST

Hi Greg,

On 06/05/2019 05:14 PM, Greg KH wrote:
On Wed, Jun 05, 2019 at 04:13:47PM +0100, Suzuki K Poulose wrote:
+ * class_find_device_by_devt : device iterator for locating a particular device
+ * matching the device type.
+ * @class: class type
+ * @start: device to start search from
+ * @devt: device type of the device to match.
+ */
+static inline struct device *class_find_device_by_devt(struct class *class,
+ struct device *start,
+ dev_t devt)
+ return class_find_device(class, start, &devt, device_match_devt);

Still has the start parameter, despite the changelog saying it would not

Well, I kept the start parameter just for the _devt variants, as
mentioned in the changelog:

" The new wrappers except the lookup by devt, drops the "start" device pointer as none of the existing users need it and the attributes are usually unique. The idea is to stop the proliferation of custom match.."

Somehow I thought the dev_t may be duplicate for devices, which is why
I didn't change that alone. But that was silly of me to ignore the minor
number part. I will respin it fixing that.