Re: [PATCH] acpi: remove unnecessary paranthesis from return

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

 



> checkpatch.pl is generating a warning when return value is
> enclosed in paranthesis. Remove the paranthesis to improve
…

              parenthesis?

You may occasionally put more than 58 characters into text lines
of such a change description.

Please avoid a typo also in the summary phrase.


…
> +++ b/drivers/acpi/acpica/dbconvert.c
> @@ -32,7 +32,7 @@ acpi_status acpi_db_hex_char_to_value(int hex_char, u8 *return_value)
>  	/* Digit must be ascii [0-9a-fA-F] */
>  
>  	if (!isxdigit(hex_char)) {
> -		return (AE_BAD_HEX_CONSTANT);
> +		return AE_BAD_HEX_CONSTANT;
>  	}
…

See also:
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/coding-style.rst?h=v6.16#n197

Regards,
Markus





[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]
  Powered by Linux