spi: spi-fsl-dspi: Fix casting warnings

hardevsinh-1 · broonie · commit 66c1c4175bbd · 2024-10-08T18:17:32.000+01:00
Sparse warnings: drivers/spi/spi-fsl-dspi.c:283:17: warning: incorrect type in assignment (different base types) drivers/spi/spi-fsl-dspi.c:283:17: expected unsigned int [usertype] drivers/spi/spi-fsl-dspi.c:283:17: got restricted __be32 [usertype] drivers/spi/spi-fsl-dspi.c:289:28: warning: cast to restricted __be32 drivers/spi/spi-fsl-dspi.c:289:28: warning: cast to restricted __be32 drivers/spi/spi-fsl-dspi.c:289:28: warning: cast to restricted __be32 drivers/spi/spi-fsl-dspi.c:289:28: warning: cast to restricted __be32 drivers/spi/spi-fsl-dspi.c:289:28: warning: cast to restricted __be32 drivers/spi/spi-fsl-dspi.c:289:28: warning: cast to restricted __be32 drivers/spi/spi-fsl-dspi.c:295:17: warning: incorrect type in assignment (different base types) drivers/spi/spi-fsl-dspi.c:295:17: expected unsigned int [usertype] drivers/spi/spi-fsl-dspi.c:295:17: got restricted __be16 [usertype] drivers/spi/spi-fsl-dspi.c:301:28: warning: cast to restricted __be16 drivers/spi/spi-fsl-dspi.c:301:28: warning: cast to restricted __be16 drivers/spi/spi-fsl-dspi.c:301:28: warning: cast to restricted __be16 drivers/spi/spi-fsl-dspi.c:301:28: warning: cast to restricted __be16 Signed-off-by: Hardevsinh Palaniya <hardevsinh.palaniya@siliconsignals.io> Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com> Tested-by: Vladimir Oltean <vladimir.oltean@nxp.com> Link: https://patch.msgid.link/20241008055644.4900-2-hardevsinh.palaniya@siliconsignals.io Signed-off-by: Mark Brown <broonie@kernel.org>
diff --git a/drivers/spi/spi-fsl-dspi.c b/drivers/spi/spi-fsl-dspi.c
@@ -280,25 +280,25 @@ static void dspi_native_dev_to_host(struct fsl_dspi *dspi, u32 rxdata)
 
 static void dspi_8on32_host_to_dev(struct fsl_dspi *dspi, u32 *txdata)
 {
-	*txdata = cpu_to_be32(*(u32 *)dspi->tx);
+	*txdata = (__force u32)cpu_to_be32(*(u32 *)dspi->tx);
 	dspi->tx += sizeof(u32);
 }
 
 static void dspi_8on32_dev_to_host(struct fsl_dspi *dspi, u32 rxdata)
 {
-	*(u32 *)dspi->rx = be32_to_cpu(rxdata);
+	*(u32 *)dspi->rx = be32_to_cpu((__force __be32)rxdata);
 	dspi->rx += sizeof(u32);
 }
 
 static void dspi_8on16_host_to_dev(struct fsl_dspi *dspi, u32 *txdata)
 {
-	*txdata = cpu_to_be16(*(u16 *)dspi->tx);
+	*txdata = (__force u32)cpu_to_be16(*(u16 *)dspi->tx);
 	dspi->tx += sizeof(u16);
 }
 
 static void dspi_8on16_dev_to_host(struct fsl_dspi *dspi, u32 rxdata)
 {
-	*(u16 *)dspi->rx = be16_to_cpu(rxdata);
+	*(u16 *)dspi->rx = be16_to_cpu((__force __be16)rxdata);
 	dspi->rx += sizeof(u16);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -280,25 +280,25 @@ static void dspi_native_dev_to_host(struct fsl_dspi *dspi, u32 rxdata)`
`280`	`280`
`281`	`281`	`static void dspi_8on32_host_to_dev(struct fsl_dspi dspi, u32 txdata)`
`282`	`282`	`{`
`283`		`- txdata = cpu_to_be32((u32 *)dspi->tx);`
	`283`	`+ txdata = (__force u32)cpu_to_be32((u32 *)dspi->tx);`
`284`	`284`	`dspi->tx += sizeof(u32);`
`285`	`285`	`}`
`286`	`286`
`287`	`287`	`static void dspi_8on32_dev_to_host(struct fsl_dspi *dspi, u32 rxdata)`
`288`	`288`	`{`
`289`		`- (u32 )dspi->rx = be32_to_cpu(rxdata);`
	`289`	`+ (u32 )dspi->rx = be32_to_cpu((__force __be32)rxdata);`
`290`	`290`	`dspi->rx += sizeof(u32);`
`291`	`291`	`}`
`292`	`292`
`293`	`293`	`static void dspi_8on16_host_to_dev(struct fsl_dspi dspi, u32 txdata)`
`294`	`294`	`{`
`295`		`- txdata = cpu_to_be16((u16 *)dspi->tx);`
	`295`	`+ txdata = (__force u32)cpu_to_be16((u16 *)dspi->tx);`
`296`	`296`	`dspi->tx += sizeof(u16);`
`297`	`297`	`}`
`298`	`298`
`299`	`299`	`static void dspi_8on16_dev_to_host(struct fsl_dspi *dspi, u32 rxdata)`
`300`	`300`	`{`
`301`		`- (u16 )dspi->rx = be16_to_cpu(rxdata);`
	`301`	`+ (u16 )dspi->rx = be16_to_cpu((__force __be16)rxdata);`
`302`	`302`	`dspi->rx += sizeof(u16);`
`303`	`303`	`}`
`304`	`304`