On 5/8/26 3:44 PM, Quentin Schulz wrote:
Hi Marek,

On 5/7/26 6:43 PM, Marek Vasut wrote:
Set the ops structure as static const. The structure is not accessible
from outside of this driver and is not going to be modified at runtime.

Signed-off-by: Marek Vasut <[email protected]>
---
Cc: Tom Rini <[email protected]>
Cc: [email protected]
---
  drivers/cpu/armv8_cpu.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/cpu/armv8_cpu.c b/drivers/cpu/armv8_cpu.c
index 4eedfe5e2c5..ed87841b723 100644
--- a/drivers/cpu/armv8_cpu.c
+++ b/drivers/cpu/armv8_cpu.c
@@ -124,7 +124,7 @@ int armv8_cpu_fill_madt(const struct udevice *dev, struct acpi_ctx *ctx)
      return 0;
  }
-struct acpi_ops armv8_cpu_acpi_ops = {
+static struct acpi_ops armv8_cpu_acpi_ops = {

Missing const?
Thanks. I just sent a V2, I had to fix up a few of the patches and didn't update the commit message.

Reply via email to