mbox series

[0/6] Fix sparse warnings in zynq platform

Message ID 20230609092202.14259-1-ashok.reddy.soma@amd.com
Headers show
Series Fix sparse warnings in zynq platform | expand

Message

Ashok Reddy Soma June 9, 2023, 9:21 a.m. UTC 
Fix below sparse warnings
 - Add missing prototype for zynqmp_mmio_write
 - Add missing prototype for zynq_qspi_mem_exec_op
 - Change datatype of status and ecc_status from u32 to int
 - Pass the missing argument type in function definition
 - Add the missing function prototypes
 - Add missing prototype for xilinx_qspi_mem_exec_op



Algapally Santosh Sagar (6):
  spi: xilinx_spi: Add missing prototype for xilinx_qspi_mem_exec_op
  xilinx: zynq: Add missing prototype for zynqmp_mmio_write
  spi: zynq_qspi: Add missing prototype for zynq_qspi_mem_exec_op
  mtd: nand: zynq_nand: Change datatype of status and ecc_status to int
  arm: zynq: Pass the missing argument type in function definition
  xilinx: zynq: Add the missing function prototypes

 arch/arm/mach-zynq/include/mach/sys_proto.h | 1 +
 board/xilinx/zynq/board.c                   | 2 ++
 board/xilinx/zynq/zynq-zc706/ps7_init_gpl.c | 4 ++--
 drivers/mtd/nand/raw/zynq_nand.c            | 4 ++--
 drivers/spi/xilinx_spi.c                    | 4 ++--
 drivers/spi/zynq_qspi.c                     | 4 ++--
 6 files changed, 11 insertions(+), 8 deletions(-)

Comments

Michal Simek June 12, 2023, 6:42 a.m. UTC | #1 
On 6/9/23 11:21, Ashok Reddy Soma wrote:
> Fix below sparse warnings
>   - Add missing prototype for zynqmp_mmio_write
>   - Add missing prototype for zynq_qspi_mem_exec_op
>   - Change datatype of status and ecc_status from u32 to int
>   - Pass the missing argument type in function definition
>   - Add the missing function prototypes
>   - Add missing prototype for xilinx_qspi_mem_exec_op
> 
> 
> 
> Algapally Santosh Sagar (6):
>    spi: xilinx_spi: Add missing prototype for xilinx_qspi_mem_exec_op
>    xilinx: zynq: Add missing prototype for zynqmp_mmio_write
>    spi: zynq_qspi: Add missing prototype for zynq_qspi_mem_exec_op
>    mtd: nand: zynq_nand: Change datatype of status and ecc_status to int
>    arm: zynq: Pass the missing argument type in function definition
>    xilinx: zynq: Add the missing function prototypes
> 
>   arch/arm/mach-zynq/include/mach/sys_proto.h | 1 +
>   board/xilinx/zynq/board.c                   | 2 ++
>   board/xilinx/zynq/zynq-zc706/ps7_init_gpl.c | 4 ++--
>   drivers/mtd/nand/raw/zynq_nand.c            | 4 ++--
>   drivers/spi/xilinx_spi.c                    | 4 ++--
>   drivers/spi/zynq_qspi.c                     | 4 ++--
>   6 files changed, 11 insertions(+), 8 deletions(-)
> 

The rest looks good to me.

Thanks,
Michal