All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const
@ 2024-05-02 23:34 ` Stephen Boyd
  0 siblings, 0 replies; 5+ messages in thread
From: Stephen Boyd @ 2024-05-02 23:34 UTC (permalink / raw)
  To: Daniel Lezcano, Thomas Gleixner
  Cc: linux-kernel, patches, linux-arm-kernel, dann frazier,
	Hanjun Guo, Marc Zyngier, Mark Rutland

This isn't modified at runtime. Mark it const so it can move to
read-only data.

Cc: dann frazier <dann.frazier@canonical.com>
Cc: Hanjun Guo <hanjun.guo@linaro.org>
Cc: Marc Zyngier <maz@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Signed-off-by: Stephen Boyd <swboyd@chromium.org>
---
 drivers/clocksource/arm_arch_timer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c
index 8d4a52056684..5bb43cc1a8df 100644
--- a/drivers/clocksource/arm_arch_timer.c
+++ b/drivers/clocksource/arm_arch_timer.c
@@ -331,7 +331,7 @@ static u64 notrace hisi_161010101_read_cntvct_el0(void)
 	return __hisi_161010101_read_reg(cntvct_el0);
 }
 
-static struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
+static const struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
 	/*
 	 * Note that trailing spaces are required to properly match
 	 * the OEM table information.

base-commit: 4cece764965020c22cff7665b18a012006359095
-- 
https://chromeos.dev


^ permalink raw reply related	[flat|nested] 5+ messages in thread
* [PATCH] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const
@ 2024-05-02 23:34 ` Stephen Boyd
  0 siblings, 0 replies; 5+ messages in thread
From: Stephen Boyd @ 2024-05-02 23:34 UTC (permalink / raw)
  To: Daniel Lezcano, Thomas Gleixner
  Cc: linux-kernel, patches, linux-arm-kernel, dann frazier,
	Hanjun Guo, Marc Zyngier, Mark Rutland

This isn't modified at runtime. Mark it const so it can move to
read-only data.

Cc: dann frazier <dann.frazier@canonical.com>
Cc: Hanjun Guo <hanjun.guo@linaro.org>
Cc: Marc Zyngier <maz@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Signed-off-by: Stephen Boyd <swboyd@chromium.org>
---
 drivers/clocksource/arm_arch_timer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c
index 8d4a52056684..5bb43cc1a8df 100644
--- a/drivers/clocksource/arm_arch_timer.c
+++ b/drivers/clocksource/arm_arch_timer.c
@@ -331,7 +331,7 @@ static u64 notrace hisi_161010101_read_cntvct_el0(void)
 	return __hisi_161010101_read_reg(cntvct_el0);
 }
 
-static struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
+static const struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
 	/*
 	 * Note that trailing spaces are required to properly match
 	 * the OEM table information.

base-commit: 4cece764965020c22cff7665b18a012006359095
-- 
https://chromeos.dev


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 5+ messages in thread
* Re: [PATCH] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const
  2024-05-02 23:34 ` Stephen Boyd
@ 2024-05-06  1:03   ` Hanjun Guo
  -1 siblings, 0 replies; 5+ messages in thread
From: Hanjun Guo @ 2024-05-06  1:03 UTC (permalink / raw)
  To: Stephen Boyd, Daniel Lezcano, Thomas Gleixner
  Cc: linux-kernel, patches, linux-arm-kernel, dann frazier,
	Hanjun Guo, Marc Zyngier, Mark Rutland

On 2024/5/3 7:34, Stephen Boyd wrote:
> This isn't modified at runtime. Mark it const so it can move to
> read-only data.
> 
> Cc: dann frazier <dann.frazier@canonical.com>
> Cc: Hanjun Guo <hanjun.guo@linaro.org>
> Cc: Marc Zyngier <maz@kernel.org>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Signed-off-by: Stephen Boyd <swboyd@chromium.org>
> ---
>   drivers/clocksource/arm_arch_timer.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c
> index 8d4a52056684..5bb43cc1a8df 100644
> --- a/drivers/clocksource/arm_arch_timer.c
> +++ b/drivers/clocksource/arm_arch_timer.c
> @@ -331,7 +331,7 @@ static u64 notrace hisi_161010101_read_cntvct_el0(void)
>   	return __hisi_161010101_read_reg(cntvct_el0);
>   }
>   
> -static struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
> +static const struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
>   	/*
>   	 * Note that trailing spaces are required to properly match
>   	 * the OEM table information.

Looks good to me,

Reviewed-by: Hanjun Guo <guohanjun@huawei.com>

^ permalink raw reply	[flat|nested] 5+ messages in thread
* Re: [PATCH] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const
@ 2024-05-06  1:03   ` Hanjun Guo
  0 siblings, 0 replies; 5+ messages in thread
From: Hanjun Guo @ 2024-05-06  1:03 UTC (permalink / raw)
  To: Stephen Boyd, Daniel Lezcano, Thomas Gleixner
  Cc: linux-kernel, patches, linux-arm-kernel, dann frazier,
	Hanjun Guo, Marc Zyngier, Mark Rutland

On 2024/5/3 7:34, Stephen Boyd wrote:
> This isn't modified at runtime. Mark it const so it can move to
> read-only data.
> 
> Cc: dann frazier <dann.frazier@canonical.com>
> Cc: Hanjun Guo <hanjun.guo@linaro.org>
> Cc: Marc Zyngier <maz@kernel.org>
> Cc: Mark Rutland <mark.rutland@arm.com>
> Signed-off-by: Stephen Boyd <swboyd@chromium.org>
> ---
>   drivers/clocksource/arm_arch_timer.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c
> index 8d4a52056684..5bb43cc1a8df 100644
> --- a/drivers/clocksource/arm_arch_timer.c
> +++ b/drivers/clocksource/arm_arch_timer.c
> @@ -331,7 +331,7 @@ static u64 notrace hisi_161010101_read_cntvct_el0(void)
>   	return __hisi_161010101_read_reg(cntvct_el0);
>   }
>   
> -static struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
> +static const struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
>   	/*
>   	 * Note that trailing spaces are required to properly match
>   	 * the OEM table information.

Looks good to me,

Reviewed-by: Hanjun Guo <guohanjun@huawei.com>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 5+ messages in thread
* [tip: timers/core] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const
  2024-05-02 23:34 ` Stephen Boyd
  (?)
  (?)
@ 2024-05-10 11:10 ` tip-bot2 for Stephen Boyd
  -1 siblings, 0 replies; 5+ messages in thread
From: tip-bot2 for Stephen Boyd @ 2024-05-10 11:10 UTC (permalink / raw)
  To: linux-tip-commits
  Cc: dann frazier, Hanjun Guo, Marc Zyngier, Mark Rutland,
	Stephen Boyd, Hanjun Guo, Daniel Lezcano, x86, linux-kernel

The following commit has been merged into the timers/core branch of tip:

Commit-ID:     2030a7e11f161b4067bd4eadd984cdb36446fcca
Gitweb:        https://git.kernel.org/tip/2030a7e11f161b4067bd4eadd984cdb36446fcca
Author:        Stephen Boyd <swboyd@chromium.org>
AuthorDate:    Thu, 02 May 2024 16:34:46 -07:00
Committer:     Daniel Lezcano <daniel.lezcano@linaro.org>
CommitterDate: Fri, 10 May 2024 10:43:21 +02:00

clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const

This isn't modified at runtime. Mark it const so it can move to
read-only data.

Cc: dann frazier <dann.frazier@canonical.com>
Cc: Hanjun Guo <hanjun.guo@linaro.org>
Cc: Marc Zyngier <maz@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Signed-off-by: Stephen Boyd <swboyd@chromium.org>
Reviewed-by: Hanjun Guo <guohanjun@huawei.com>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/20240502233447.420888-1-swboyd@chromium.org
---
 drivers/clocksource/arm_arch_timer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/clocksource/arm_arch_timer.c b/drivers/clocksource/arm_arch_timer.c
index 8d4a520..5bb43cc 100644
--- a/drivers/clocksource/arm_arch_timer.c
+++ b/drivers/clocksource/arm_arch_timer.c
@@ -331,7 +331,7 @@ static u64 notrace hisi_161010101_read_cntvct_el0(void)
 	return __hisi_161010101_read_reg(cntvct_el0);
 }
 
-static struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
+static const struct ate_acpi_oem_info hisi_161010101_oem_info[] = {
 	/*
 	 * Note that trailing spaces are required to properly match
 	 * the OEM table information.

^ permalink raw reply related	[flat|nested] 5+ messages in thread
end of thread, other threads:[~2024-05-10 11:10 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-02 23:34 [PATCH] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const Stephen Boyd
2024-05-02 23:34 ` Stephen Boyd
2024-05-06  1:03 ` Hanjun Guo
2024-05-06  1:03   ` Hanjun Guo
2024-05-10 11:10 ` [tip: timers/core] " tip-bot2 for Stephen Boyd

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.