Re: [PATCH] usb: storage: make ndecimals unsigned in parse_delay_str()

From: Greg KH

Date: Fri Jul 17 2026 - 00:44:52 EST


On Thu, Jul 16, 2026 at 03:04:00PM -0400, David Cusimano wrote:
> >From 73039aaf92449baf12dd4f774164cab4b9987a5c Mon Sep 17 00:00:00 2001
> From: David Cusimano <david@xxxxxxxxxxxx>
> Date: Thu, 16 Jul 2026 14:04:29 -0400
> Subject: [PATCH] usb: storage: make ndecimals unsigned in parse_delay_str()

Something went really wrong with your email system, this shouldn't be
here, and your patch is corrupted :(

>
> The ndecimals parameter controls how many decimal places are added when the
> expected suffix is missing. It is never used as a signed quantity, so make
> it
> unsigned to better reflect its intended semantics.

While nice, is this needed? Are you going to do this for all usages in
this file of the same thing?

We use "int" a lot in the kernel, so be aware that this is going to be
an interesting task, if not kind of pointless?

thanks,

greg k-h