On Tue, Jan 23, 2018 at 11:24:05AM +0100, Greg Kroah-Hartman wrote: > Instead of "open coding" a DEVICE_ATTR() define, use the > DEVICE_ATTR_RW() 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: Matthieu CASTET <castet.matthieu@xxxxxxx> > Cc: Stanislaw Gruszka <stf_xl@xxxxx> > Cc: Peter Chen <Peter.Chen@xxxxxxx> > Cc: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> > Cc: Mathias Nyman <mathias.nyman@xxxxxxxxx> > Cc: Bin Liu <b-liu@xxxxxx> > Cc: Felipe Balbi <balbi@xxxxxxxxxx> > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> For MUSB part in 1/6, and 5/6: Acked-by: Bin Liu <b-liu@xxxxxx> Regards, -Bin. -- 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