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

From: Dan Carpenter

Date: Tue Feb 10 2026 - 01:50:46 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)
> #endif

This isn't related to the commit message. I'm not sure that it's
really an improvement, either. There is honestly a lot in this
patch which is not necessarily more readable afterwards.

regards,
dan carpenter