| Message ID | 1446767000-31398-1-git-send-email-aduggan@synaptics.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
On Fri, Nov 6, 2015 at 12:43 AM, Andrew Duggan <aduggan@synaptics.com> wrote: > Currently, the rmi device has the name sensorXX. Sensor is very generic > and in cases when there is little context it is hard to tell what the > device is (ie in /sys/devices or /proc/interrupts). This patch changes > the name to rmiXX which at least indicates that it is an rmi device. > > Signed-off-by: Andrew Duggan <aduggan@synaptics.com> > --- > Linus Walleij suggested that we use the TM number as the device name. > But, we need to set the device name when we add the device before we > read the TM number. Changing the name to rmiXX is a compromise since > it's not as unique as the TM, but not as generic as sensor. OK OK I give up. No big deal anyway, the device works. Squash into core patch? Acked-by: Linus Walleij <linus.walleij@linaro.org> Yours, Linus Walleij -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/input/rmi4/rmi_bus.c b/drivers/input/rmi4/rmi_bus.c index a04468d..998b4e1 100644 --- a/drivers/input/rmi4/rmi_bus.c +++ b/drivers/input/rmi4/rmi_bus.c @@ -66,7 +66,7 @@ int rmi_register_transport_device(struct rmi_transport_dev *xport) rmi_dev->xport = xport; rmi_dev->number = atomic_inc_return(&transport_device_count) - 1; - dev_set_name(&rmi_dev->dev, "sensor%02d", rmi_dev->number); + dev_set_name(&rmi_dev->dev, "rmi%02d", rmi_dev->number); rmi_dev->dev.bus = &rmi_bus_type; rmi_dev->dev.type = &rmi_device_type;
Currently, the rmi device has the name sensorXX. Sensor is very generic and in cases when there is little context it is hard to tell what the device is (ie in /sys/devices or /proc/interrupts). This patch changes the name to rmiXX which at least indicates that it is an rmi device. Signed-off-by: Andrew Duggan <aduggan@synaptics.com> --- Linus Walleij suggested that we use the TM number as the device name. But, we need to set the device name when we add the device before we read the TM number. Changing the name to rmiXX is a compromise since it's not as unique as the TM, but not as generic as sensor. drivers/input/rmi4/rmi_bus.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)