drivers: usb: Fix block comments whose trailing */ was not on a separate line
Subsystems (3)
PATCH 1/3 drivers: usb: Fix block comments whose trailing */ was not on a separate line
Fixed various instances where block comments trailing */ where not on a separate line and should be.
--- drivers/usb/atm/cxacru.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-)
drivers/usb/atm/cxacru.c +8 -4
@@ -55,7 +55,8 @@ static const char cxacru_driver_name[] = "cxacru";#define POLL_INTERVAL 1/* secs */ /* commands for interaction with the modem through the control channel before -* firmware is loaded */ +* firmware is loaded +*/ enum cxacru_fw_request {FW_CMD_ERR ,FW_GET_VER ,
@@ -67,7 +68,8 @@ enum cxacru_fw_request { };/* commands for interaction with the modem through the control channel once -* firmware is loaded */ +* firmware is loaded +*/ enum cxacru_cm_request {CM_REQUEST_UNDEFINED =0x80 ,CM_REQUEST_TEST ,
@@ -369,7 +371,8 @@ static ssize_t adsl_state_store(struct device *dev,/* Line status is only updated every second * and the device appears to only react to * START/STOP every second too. Wait 1.5s to -* be sure that restart will have an effect. */ +* be sure that restart will have an effect. +*/ if (!strcmp (str_cmd,"restart" ))msleep (1500 );
@@ -1237,7 +1240,8 @@ static void cxacru_unbind(struct usbatm_data *usbatm_instance,BUG_ON (instance->poll_state ==CXPOLL_SHUTDOWN );/* ensure that status polling continues unless -* it has already stopped */ +* it has already stopped +*/ if (instance->poll_state ==CXPOLL_STOPPED ) is_polling =0 ;
Signature · 1 line
2.34.1
Greg KH
maintainerQuoted 7 lines · Jonathan Bergh
Fixed various instances where block comments trailing */ where not on a separate line and should be.
Signed-off-by Jonathan Bergh <[email protected]>--- drivers/usb/atm/cxacru.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-)
Your subject line for these patches is not correct, please fix up.
And the patches are not properly threaded, what happened in your email client?
And finally, why make coding style changes outside of drivers/staging/? Did you get the maintainer's approval to do so before taking the time? Usually most maintainers do not want these so always ask before doing so, or just stick with drivers/staging/ to learn how the process works first please.
thanks,
greg k-h
Quoted 8 lines · Greg KH
Quoted 7 lines · Jonathan Bergh
Fixed various instances where block comments trailing */ where not on a separate line and should be.
Signed-off-by Jonathan Bergh <[email protected]>--- drivers/usb/atm/cxacru.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-)
Hi Greg, thanks a lot
Quoted 1 line
Your subject line for these patches is not correct, please fix up.
Thanks, will do.
Quoted 2 lines
And the patches are not properly threaded, what happened in your email client?
Ok thanks, I used the same process as last time but will check it out.
Quoted 5 lines
And finally, why make coding style changes outside of
drivers/staging/? Did you get the maintainer's approval to do so before taking the time? Usually most maintainers do not want these so always ask before doing so, or just stick withdrivers/staging/to learn how the process works first please.
Ok! Thanks! Would you like these patches? I'd be happy to resend them. Hopefully (like the guy you mentioned in your talk) unless they screw something up, are seen as making a (postive) contribution!
cheers!
jon
Quoted 3 lines
thanks,
greg k-h