On Tue, Jan 23, 2018 at 11:24:07AM +0100, Greg Kroah-Hartman wrote: > Instead of "open coding" a DEVICE_ATTR() define, use the > DEVICE_ATTR_WO() macro instead, which does everything properly instead. > > This does require a few static functions to be renamed to work properly, > but thanks to a script from Joe Perches, this was easily done. > > Reported-by: Joe Perches <joe@xxxxxxxxxxx> > Cc: Peter Chen <Peter.Chen@xxxxxxx> > Cc: Felipe Balbi <balbi@xxxxxxxxxx> > Cc: Johan Hovold <johan@xxxxxxxxxx> > Cc: Valentina Manea <valentina.manea.m@xxxxxxxxx> > Cc: Shuah Khan <shuah@xxxxxxxxxx> > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> > --- > drivers/usb/chipidea/otg_fsm.c | 4 ++-- > drivers/usb/gadget/udc/core.c | 8 ++++---- > drivers/usb/phy/phy-mv-usb.c | 4 ++-- > drivers/usb/serial/ftdi_sio.c | 4 ++-- For usb-serial: Acked-by: Johan Hovold <johan@xxxxxxxxxx> -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html