LKML Archive mirror
 help / color / mirror / Atom feed
* [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
  2024-05-10 11:10 ` [tip: timers/core] " tip-bot2 for Stephen Boyd
  0 siblings, 2 replies; 3+ 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] 3+ messages in thread

* Re: [PATCH] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const
  2024-05-02 23:34 [PATCH] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const Stephen Boyd
@ 2024-05-06  1:03 ` Hanjun Guo
  2024-05-10 11:10 ` [tip: timers/core] " tip-bot2 for Stephen Boyd
  1 sibling, 0 replies; 3+ 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] 3+ messages in thread

* [tip: timers/core] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const
  2024-05-02 23:34 [PATCH] clocksource/drivers/arm_arch_timer: Mark hisi_161010101_oem_info const Stephen Boyd
  2024-05-06  1:03 ` Hanjun Guo
@ 2024-05-10 11:10 ` tip-bot2 for Stephen Boyd
  1 sibling, 0 replies; 3+ 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] 3+ messages in thread

end of thread, other threads:[~2024-05-10 11:10 UTC | newest]

Thread overview: 3+ 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-06  1:03 ` Hanjun Guo
2024-05-10 11:10 ` [tip: timers/core] " tip-bot2 for Stephen Boyd

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).