mtd: rawnand: realtek: Make rtl_ecc_engine_ops const

Li Qiang · miquelraynal · commit 963135088592 · 2025-10-22T11:54:42.000+02:00
The rtl_ecc_engine_ops structure is only used to provide a set of
callback functions and is never modified after initialization.
Mark it as const so it can be placed in the read-only section, which
improves safety and allows better compiler optimization.

Signed-off-by: Li Qiang &lt;liqiang01@kylinos.cn&gt;
Signed-off-by: Miquel Raynal &lt;miquel.raynal@bootlin.com&gt;
diff --git a/drivers/mtd/nand/ecc-realtek.c b/drivers/mtd/nand/ecc-realtek.c
@@ -380,7 +380,7 @@ static void rtl_ecc_cleanup_ctx(struct nand_device *nand)
 		nand_ecc_cleanup_req_tweaking(&ctx->req_ctx);
 }
 
-static struct nand_ecc_engine_ops rtl_ecc_engine_ops = {
+static const struct nand_ecc_engine_ops rtl_ecc_engine_ops = {
 	.init_ctx = rtl_ecc_init_ctx,
 	.cleanup_ctx = rtl_ecc_cleanup_ctx,
 	.prepare_io_req = rtl_ecc_prepare_io_req,

Original file line number	Diff line number	Diff line change
`@@ -380,7 +380,7 @@ static void rtl_ecc_cleanup_ctx(struct nand_device *nand)`
`380`	`380`	`nand_ecc_cleanup_req_tweaking(&ctx->req_ctx);`
`381`	`381`	`}`
`382`	`382`
`383`		`-static struct nand_ecc_engine_ops rtl_ecc_engine_ops = {`
	`383`	`+static const struct nand_ecc_engine_ops rtl_ecc_engine_ops = {`
`384`	`384`	`.init_ctx = rtl_ecc_init_ctx,`
`385`	`385`	`.cleanup_ctx = rtl_ecc_cleanup_ctx,`
`386`	`386`	`.prepare_io_req = rtl_ecc_prepare_io_req,`