Messages in this thread | | | From | "Rafael J. Wysocki" <> | Subject | Re: [PATCH 4/6] ACPI / device_sysfs: make use of new strtolower() function | Date | Fri, 01 Jul 2016 22:44:33 +0200 |
| |
On Thursday, June 30, 2016 04:50:10 PM Markus Mayer wrote: > Call strtolower() rather than walking the string explicitly to convert > it to lowercase. > > Signed-off-by: Markus Mayer <mmayer@broadcom.com>
ACK
> --- > drivers/acpi/device_sysfs.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/acpi/device_sysfs.c b/drivers/acpi/device_sysfs.c > index 7b2c48f..1db38c7 100644 > --- a/drivers/acpi/device_sysfs.c > +++ b/drivers/acpi/device_sysfs.c > @@ -200,12 +200,10 @@ static int create_of_modalias(struct acpi_device *acpi_dev, char *modalias, > const union acpi_object *of_compatible, *obj; > int len, count; > int i, nval; > - char *c; > > acpi_get_name(acpi_dev->handle, ACPI_SINGLE_NAME, &buf); > /* DT strings are all in lower case */ > - for (c = buf.pointer; *c != '\0'; c++) > - *c = tolower(*c); > + strtolower(buf.pointer); > > len = snprintf(modalias, size, "of:N%sT", (char *)buf.pointer); > ACPI_FREE(buf.pointer); >
| |