Re: [PATCH v3 4/4] staging: greybus: firmware: Convert sscanf calls to strtoul

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Mar 08, 2017 at 12:28:05AM -0700, Michael Sartain wrote:
> Also convert the fw_update_type and fw_timeout variables to
> unsigned and update the printf specifier to %u.
> 
> The FW_MGMT_IOC_SET_TIMEOUT_MS ioctl takes an unsigned int
> and checkpatch was complaining about not checking the sscanf
> return values.
> 
> Signed-off-by: Michael Sartain <mikesart@xxxxxxxxxxxx>
> Acked-by: Viresh Kumar <viresh.kumar at linaro.org>
> ---
> V2:
> Add "greybus: firmware:" prefix to subject lines
> V3:
> Add "staging:" prefix to subject lines
> 
>  drivers/staging/greybus/Documentation/firmware/firmware.c | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git drivers/staging/greybus/Documentation/firmware/firmware.c drivers/staging/greybus/Documentation/firmware/firmware.c
> index 70b7fcbba5ad..c73dee9d13c1 100644
> --- drivers/staging/greybus/Documentation/firmware/firmware.c
> +++ drivers/staging/greybus/Documentation/firmware/firmware.c

These patches do not apply at all, did you use git to create them?

They are one level off :(

Please do a "normal" git format-patch to create them and you should be
fine.

thanks,

greg k-h
_______________________________________________
greybus-dev mailing list
greybus-dev@xxxxxxxxxxxxxxxx
https://lists.linaro.org/mailman/listinfo/greybus-dev




[Index of Archives]     [Asterisk App Development]     [PJ SIP]     [Gnu Gatekeeper]     [IETF Sipping]     [Info Cyrus]     [ALSA User]     [Fedora Linux Users]     [Linux SCTP]     [DCCP]     [Gimp]     [Yosemite News]     [Deep Creek Hot Springs]     [Yosemite Campsites]     [ISDN Cause Codes]     [Asterisk Books]

  Powered by Linux