All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/2] nvmem: fixes for 6.15
@ 2025-05-09 12:24 srini
  2025-05-09 12:24 ` [PATCH 1/2] nvmem: rmem: select CONFIG_CRC32 srini
  2025-05-09 12:24 ` [PATCH 2/2] nvmem: zynqmp_nvmem: unbreak driver after cleanup srini
  0 siblings, 2 replies; 4+ messages in thread
From: srini @ 2025-05-09 12:24 UTC (permalink / raw)
  To: gregkh; +Cc: linux-kernel, Srinivas Kandagatla

From: Srinivas Kandagatla <srini@kernel.org>

Hi Greg, 

Here are two fixes in nvmem for 6.15, Could you please queue
these for next possible rc.

fixes include
	- a fix in zynqmp_nvmem provider during cleanup.
	- fix a link failure due to newly added crc checking 


Thanks,
Srini

Arnd Bergmann (1):
  nvmem: rmem: select CONFIG_CRC32

Peter Korsgaard (1):
  nvmem: zynqmp_nvmem: unbreak driver after cleanup

 drivers/nvmem/Kconfig        | 1 +
 drivers/nvmem/zynqmp_nvmem.c | 1 +
 2 files changed, 2 insertions(+)

-- 
2.43.0


^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] nvmem: rmem: select CONFIG_CRC32
  2025-05-09 12:24 [PATCH 0/2] nvmem: fixes for 6.15 srini
@ 2025-05-09 12:24 ` srini
  2025-05-21 12:27   ` Greg KH
  2025-05-09 12:24 ` [PATCH 2/2] nvmem: zynqmp_nvmem: unbreak driver after cleanup srini
  1 sibling, 1 reply; 4+ messages in thread
From: srini @ 2025-05-09 12:24 UTC (permalink / raw)
  To: gregkh; +Cc: linux-kernel, Arnd Bergmann, Srinivas Kandagatla

From: Arnd Bergmann <arnd@arndb.de>

The newly added crc checking leads to a link failure if CRC32
itself is disabled:

x86_64-linux-ld: vmlinux.o: in function `rmem_eyeq5_checksum':
rmem.c:(.text+0x52341b): undefined reference to `crc32_le_arch'

Fixes: 7e606c311f70 ("nvmem: rmem: add CRC validation for Mobileye EyeQ5 NVMEM")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Srinivas Kandagatla <srini@kernel.org>
---
 drivers/nvmem/Kconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/nvmem/Kconfig b/drivers/nvmem/Kconfig
index 8671b7c974b9..eceb3cdb421f 100644
--- a/drivers/nvmem/Kconfig
+++ b/drivers/nvmem/Kconfig
@@ -260,6 +260,7 @@ config NVMEM_RCAR_EFUSE
 config NVMEM_RMEM
 	tristate "Reserved Memory Based Driver Support"
 	depends on HAS_IOMEM
+	select CRC32
 	help
 	  This driver maps reserved memory into an nvmem device. It might be
 	  useful to expose information left by firmware in memory.
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 2/2] nvmem: zynqmp_nvmem: unbreak driver after cleanup
  2025-05-09 12:24 [PATCH 0/2] nvmem: fixes for 6.15 srini
  2025-05-09 12:24 ` [PATCH 1/2] nvmem: rmem: select CONFIG_CRC32 srini
@ 2025-05-09 12:24 ` srini
  1 sibling, 0 replies; 4+ messages in thread
From: srini @ 2025-05-09 12:24 UTC (permalink / raw)
  To: gregkh
  Cc: linux-kernel, Peter Korsgaard, stable, Michal Simek,
	Srinivas Kandagatla

From: Peter Korsgaard <peter@korsgaard.com>

Commit 29be47fcd6a0 ("nvmem: zynqmp_nvmem: zynqmp_nvmem_probe cleanup")
changed the driver to expect the device pointer to be passed as the
"context", but in nvmem the context parameter comes from nvmem_config.priv
which is never set - Leading to null pointer exceptions when the device is
accessed.

Fixes: 29be47fcd6a0 ("nvmem: zynqmp_nvmem: zynqmp_nvmem_probe cleanup")
Cc: stable@vger.kernel.org
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
Reviewed-by: Michal Simek <michal.simek@amd.com>
Tested-by: Michal Simek <michal.simek@amd.com>
Signed-off-by: Srinivas Kandagatla <srini@kernel.org>
---
 drivers/nvmem/zynqmp_nvmem.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/nvmem/zynqmp_nvmem.c b/drivers/nvmem/zynqmp_nvmem.c
index 8682adaacd69..7da717d6c7fa 100644
--- a/drivers/nvmem/zynqmp_nvmem.c
+++ b/drivers/nvmem/zynqmp_nvmem.c
@@ -213,6 +213,7 @@ static int zynqmp_nvmem_probe(struct platform_device *pdev)
 	econfig.word_size = 1;
 	econfig.size = ZYNQMP_NVMEM_SIZE;
 	econfig.dev = dev;
+	econfig.priv = dev;
 	econfig.add_legacy_fixed_of_cells = true;
 	econfig.reg_read = zynqmp_nvmem_read;
 	econfig.reg_write = zynqmp_nvmem_write;
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH 1/2] nvmem: rmem: select CONFIG_CRC32
  2025-05-09 12:24 ` [PATCH 1/2] nvmem: rmem: select CONFIG_CRC32 srini
@ 2025-05-21 12:27   ` Greg KH
  0 siblings, 0 replies; 4+ messages in thread
From: Greg KH @ 2025-05-21 12:27 UTC (permalink / raw)
  To: srini; +Cc: linux-kernel, Arnd Bergmann

On Fri, May 09, 2025 at 01:24:06PM +0100, srini@kernel.org wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> The newly added crc checking leads to a link failure if CRC32
> itself is disabled:
> 
> x86_64-linux-ld: vmlinux.o: in function `rmem_eyeq5_checksum':
> rmem.c:(.text+0x52341b): undefined reference to `crc32_le_arch'
> 
> Fixes: 7e606c311f70 ("nvmem: rmem: add CRC validation for Mobileye EyeQ5 NVMEM")

You forgot to cc: stable :(


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-05-21 12:27 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-05-09 12:24 [PATCH 0/2] nvmem: fixes for 6.15 srini
2025-05-09 12:24 ` [PATCH 1/2] nvmem: rmem: select CONFIG_CRC32 srini
2025-05-21 12:27   ` Greg KH
2025-05-09 12:24 ` [PATCH 2/2] nvmem: zynqmp_nvmem: unbreak driver after cleanup srini

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.