spi: sophgo: Fix incorrect use of bus width value macros
Subsystems (2)
PATCH 1/1 spi: sophgo: Fix incorrect use of bus width value macros
The previous code initialized the 'reg' value with specific bus-width values (BUS_WIDTH_2_BIT and BUS_WIDTH_4_BIT), which introduces ambiguity. Replace them with BUS_WIDTH_MASK to express the intention clearly.
--- drivers/spi/spi-sg2044-nor.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/spi/spi-sg2044-nor.c +2 -2
@@ -42,6 +42,7 @@#define SPIFMC_TRAN_CSR_TRAN_MODE_RX BIT(0)#define SPIFMC_TRAN_CSR_TRAN_MODE_TX BIT(1)#define SPIFMC_TRAN_CSR_FAST_MODE BIT(3) +#define SPIFMC_TRAN_CSR_BUS_WIDTH_MASK GENMASK(5, 4)#define SPIFMC_TRAN_CSR_BUS_WIDTH_1_BIT (0x00 << 4)#define SPIFMC_TRAN_CSR_BUS_WIDTH_2_BIT (0x01 << 4)#define SPIFMC_TRAN_CSR_BUS_WIDTH_4_BIT (0x02 << 4)
@@ -122,8 +123,7 @@ static u32 sg2044_spifmc_init_reg(struct sg2044_spifmc *spifmc) reg =readl (spifmc->io_base +SPIFMC_TRAN_CSR ); reg &= ~(SPIFMC_TRAN_CSR_TRAN_MODE_MASK | SPIFMC_TRAN_CSR_FAST_MODE | - SPIFMC_TRAN_CSR_BUS_WIDTH_2_BIT | - SPIFMC_TRAN_CSR_BUS_WIDTH_4_BIT | + SPIFMC_TRAN_CSR_BUS_WIDTH_MASK | SPIFMC_TRAN_CSR_DMA_EN | SPIFMC_TRAN_CSR_ADDR_BYTES_MASK | SPIFMC_TRAN_CSR_WITH_CMD |
--
2.51.2
Mark Brown
maintainerQuoted 5 lines · Longbin Li
The previous code initialized the 'reg' value with specific bus-width values (
BUS_WIDTH_2_BITandBUS_WIDTH_4_BIT), which introduces ambiguity. Replace them withBUS_WIDTH_MASKto express the intention clearly.
Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/broonie/spi.git for-next
Thanks!
[1/1] spi: sophgo: Fix incorrect use of bus width value macros
commit: d9813cd23d5a7b254cc1b1c1ea042634d8da62e6
All being well this means that it will be integrated into the linux-next tree (usually sometime in the next 24 hours) and sent to Linus during the next merge window (or sooner if it is a bug fix), however if problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing and review of the tree, please engage with people reporting problems and send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they should be sent as incremental updates against current git, existing patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying to this mail.
Thanks,
Mark