Re: [PATCH] staging: octeon: type change from uint<bits>_t to u<bits>

From: Greg KH

Date: Mon Feb 09 2026 - 23:58:53 EST


On Tue, Feb 10, 2026 at 12:36:31AM -0400, Yoelvis Oliveros wrote:
> Runing the ckeckpatch.pl on the staging/octeon driver they where using
> uint<8/16/32/64>_T as type declaration and the checkpatch.pl was
> putting a [CHECK] flag on those and that they should be change to
> u<8/16/32/64>
>
> Signed-off-by: Yoelvis Oliveros <yoelvisoliveros@xxxxxxxxx>
> ---
> drivers/staging/octeon/octeon-stubs.h | 1922 +++++++++++++------------
> 1 file changed, 972 insertions(+), 950 deletions(-)
>
> diff --git a/drivers/staging/octeon/octeon-stubs.h b/drivers/staging/octeon/octeon-stubs.h
> index 35b5078ba51e..aea3dad37cfe 100644
> --- a/drivers/staging/octeon/octeon-stubs.h
> +++ b/drivers/staging/octeon/octeon-stubs.h
> @@ -1,188 +1,190 @@
> /* SPDX-License-Identifier: GPL-2.0 */
> -#define CONFIG_CAVIUM_OCTEON_CVMSEG_SIZE 512
> +#define CONFIG_CAVIUM_OCTEON_CVMSEG_SIZE 512
>
> #ifndef XKPHYS_TO_PHYS
> -# define XKPHYS_TO_PHYS(p) (p)
> +#define XKPHYS_TO_PHYS(p) (p)

<snip>

Why change all of these #define formats at the same time? Please do not
do that.

thanks,

greg k-h