On Tue, Jan 23, 2018 at 12:26:38PM +0200, Felipe Balbi wrote: > > Hi, > > Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> writes: > > 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 drivers/usb/phy: > > Acked-by: Felipe Balbi <felipe.balbi@xxxxxxxxxxxxxxx> Great, thanks for the quick review of these. greg k-h -- 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
