]> git.proxmox.com Git - mirror_ubuntu-jammy-kernel.git/commit
spi: spi-nxp-fspi: don't depend on a specific node name erratum workaround
authorMichael Walle <michael@walle.cc>
Fri, 1 Oct 2021 21:27:26 +0000 (23:27 +0200)
committerMark Brown <broonie@kernel.org>
Sat, 2 Oct 2021 00:31:49 +0000 (01:31 +0100)
commit67a12ae52599c9f2f24ef14adb43fc3b164792b5
treec82d0ab9dc87881fef000573fbed69a0cd6532b9
parent3672bb820f3292b6f035469f863c020317a3dd28
spi: spi-nxp-fspi: don't depend on a specific node name erratum workaround

In commit 7e71b85473f8 ("arm64: dts: ls1028a: fix node name for the
sysclk") the sysclk node name was renamed and broke the erratum
workaround because it tries to fetch a device tree node by its name,
which is very fragile in general. We don't even need the sysclk node
because the only possible sysclk frequency input is 100MHz. In fact, the
erratum says it applies if SYS_PLL_RAT is 3, not that the platform clock
is 300 MHz. Make the workaround more reliable and just drop the unneeded
sysclk lookup.

For reference, the error during the bootup is the following:
[    4.898400] nxp-fspi 20c0000.spi: Errata cannot be executed. Read via IP bus may not work

Fixes: 82ce7d0e74b6 ("spi: spi-nxp-fspi: Implement errata workaround for LS1028A")
Cc: Vladimir Oltean <vladimir.oltean@nxp.com>
Signed-off-by: Michael Walle <michael@walle.cc>
Link: https://lore.kernel.org/r/20211001212726.159437-1-michael@walle.cc
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-nxp-fspi.c