[PATCH 2/6] powerpc/xmon: fix sparse warning "Using plain integer as NULL pointer"
Madhavan Srinivasan
maddy at linux.ibm.com
Sun May 18 13:51:27 AEST 2025
On 5/2/25 3:38 PM, Christophe Leroy wrote:
>
>
> Le 22/04/2025 à 15:10, Madhavan Srinivasan a écrit :
>> Fix passing of argument 0 to NULL to avoid sparse warning
>
> I thought we wanted to keep this file in sync with opcodes/ppc-opc.c in the binutils repository.
>
> Are you sure you want to do such manual change ? Shouldn't the change first be done in binutils then backported in linux kernel ?
yep, my bad, dont know what i was thinking :) . But nice catch. Will drop this.
Thanks
>
> Christophe
>
>>
>> Cleans up sparse warning:
>> arch/powerpc/xmon/ppc-opc.c:797:15: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:797:18: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:801:15: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:801:18: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:805:14: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:805:17: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:809:16: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:809:19: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:863:15: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:863:18: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:868:15: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:868:18: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:872:15: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:872:18: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:875:15: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:875:18: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:878:15: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:878:18: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:881:14: warning: Using plain integer as NULL pointer
>> arch/powerpc/xmon/ppc-opc.c:881:17: warning: Using plain integer as NULL pointer
>>
>> Signed-off-by: Madhavan Srinivasan <maddy at linux.ibm.com>
>> ---
>> arch/powerpc/xmon/ppc-opc.c | 20 ++++++++++----------
>> 1 file changed, 10 insertions(+), 10 deletions(-)
>>
>> diff --git a/arch/powerpc/xmon/ppc-opc.c b/arch/powerpc/xmon/ppc-opc.c
>> index 0774d711453e..b4ca7eb8d765 100644
>> --- a/arch/powerpc/xmon/ppc-opc.c
>> +++ b/arch/powerpc/xmon/ppc-opc.c
>> @@ -794,19 +794,19 @@ const struct powerpc_operand powerpc_operands[] =
>> #define PSWM WS + 1
>> /* The BO16 field in a BD8 form instruction. */
>> #define BO16 PSWM
>> - { 0x1, 10, 0, 0, 0 },
>> + { 0x1, 10, NULL, NULL, 0 },
>> /* IDX bits for quantization in the pair singles instructions. */
>> #define PSQ PSWM + 1
>> - { 0x7, 12, 0, 0, 0 },
>> + { 0x7, 12, NULL, NULL, 0 },
>> /* IDX bits for quantization in the pair singles x-type instructions. */
>> #define PSQM PSQ + 1
>> - { 0x7, 7, 0, 0, 0 },
>> + { 0x7, 7, NULL, NULL, 0 },
>> /* Smaller D field for quantization in the pair singles instructions. */
>> #define PSD PSQM + 1
>> - { 0xfff, 0, 0, 0, PPC_OPERAND_PARENS | PPC_OPERAND_SIGNED },
>> + { 0xfff, 0, NULL, NULL, PPC_OPERAND_PARENS | PPC_OPERAND_SIGNED },
>> /* The L field in an mtmsrd or A form instruction or R or W in an X form. */
>> #define A_L PSD + 1
>> @@ -860,25 +860,25 @@ const struct powerpc_operand powerpc_operands[] =
>> /* Xilinx APU related masks and macros */
>> #define FCRT XFL_L + 1
>> #define FCRT_MASK (0x1f << 21)
>> - { 0x1f, 21, 0, 0, PPC_OPERAND_FCR },
>> + { 0x1f, 21, NULL, NULL, PPC_OPERAND_FCR },
>> /* Xilinx FSL related masks and macros */
>> #define FSL FCRT + 1
>> #define FSL_MASK (0x1f << 11)
>> - { 0x1f, 11, 0, 0, PPC_OPERAND_FSL },
>> + { 0x1f, 11, NULL, NULL, PPC_OPERAND_FSL },
>> /* Xilinx UDI related masks and macros */
>> #define URT FSL + 1
>> - { 0x1f, 21, 0, 0, PPC_OPERAND_UDI },
>> + { 0x1f, 21, NULL, NULL, PPC_OPERAND_UDI },
>> #define URA URT + 1
>> - { 0x1f, 16, 0, 0, PPC_OPERAND_UDI },
>> + { 0x1f, 16, NULL, NULL, PPC_OPERAND_UDI },
>> #define URB URA + 1
>> - { 0x1f, 11, 0, 0, PPC_OPERAND_UDI },
>> + { 0x1f, 11, NULL, NULL, PPC_OPERAND_UDI },
>> #define URC URB + 1
>> - { 0x1f, 6, 0, 0, PPC_OPERAND_UDI },
>> + { 0x1f, 6, NULL, NULL, PPC_OPERAND_UDI },
>> /* The VLESIMM field in a D form instruction. */
>> #define VLESIMM URC + 1
>
More information about the Linuxppc-dev
mailing list