Linux MIPS Architecture development
 help / color / mirror / Atom feed
* [PATCH 0/2] MIPS: Make reset_control_ops const
@ 2016-02-25  9:45 Philipp Zabel
  2016-02-25  9:45 ` [PATCH 1/2] MIPS: lantiq: " Philipp Zabel
  2016-02-25  9:45 ` [PATCH 2/2] MIPS: ralink: " Philipp Zabel
  0 siblings, 2 replies; 5+ messages in thread
From: Philipp Zabel @ 2016-02-25  9:45 UTC (permalink / raw)
  To: Ralf Baechle; +Cc: John Crispin, linux-mips, kernel, Philipp Zabel

Since commit 203d4f347d86 ("reset: Make reset_control_ops const") marked
the ops pointer in struct reset_controller_dev as const, these structs
can be made const, too. The commit is currently sitting in the
arm-soc/for-next branch.

regards
Philipp

Philipp Zabel (2):
  MIPS: lantiq: Make reset_control_ops const
  MIPS: ralink: Make reset_control_ops const

 arch/mips/lantiq/xway/reset.c | 2 +-
 arch/mips/ralink/reset.c      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.7.0

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

* [PATCH 1/2] MIPS: lantiq: Make reset_control_ops const
  2016-02-25  9:45 [PATCH 0/2] MIPS: Make reset_control_ops const Philipp Zabel
@ 2016-02-25  9:45 ` Philipp Zabel
  2016-02-25  9:51   ` John Crispin
  2016-02-25  9:45 ` [PATCH 2/2] MIPS: ralink: " Philipp Zabel
  1 sibling, 1 reply; 5+ messages in thread
From: Philipp Zabel @ 2016-02-25  9:45 UTC (permalink / raw)
  To: Ralf Baechle; +Cc: John Crispin, linux-mips, kernel, Philipp Zabel

The reset_ops structure is never modified. Make it const.

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 arch/mips/lantiq/xway/reset.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/lantiq/xway/reset.c b/arch/mips/lantiq/xway/reset.c
index bc29bb3..1f34608 100644
--- a/arch/mips/lantiq/xway/reset.c
+++ b/arch/mips/lantiq/xway/reset.c
@@ -258,7 +258,7 @@ static int ltq_reset_device(struct reset_controller_dev *rcdev,
 	return ltq_deassert_device(rcdev, id);
 }
 
-static struct reset_control_ops reset_ops = {
+static const struct reset_control_ops reset_ops = {
 	.reset = ltq_reset_device,
 	.assert = ltq_assert_device,
 	.deassert = ltq_deassert_device,
-- 
2.7.0

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

* [PATCH 2/2] MIPS: ralink: Make reset_control_ops const
  2016-02-25  9:45 [PATCH 0/2] MIPS: Make reset_control_ops const Philipp Zabel
  2016-02-25  9:45 ` [PATCH 1/2] MIPS: lantiq: " Philipp Zabel
@ 2016-02-25  9:45 ` Philipp Zabel
  2016-02-25  9:51   ` John Crispin
  1 sibling, 1 reply; 5+ messages in thread
From: Philipp Zabel @ 2016-02-25  9:45 UTC (permalink / raw)
  To: Ralf Baechle; +Cc: John Crispin, linux-mips, kernel, Philipp Zabel

The reset_ops structure is never modified. Make it const.

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
---
 arch/mips/ralink/reset.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/mips/ralink/reset.c b/arch/mips/ralink/reset.c
index ee117c4..09ccdd1 100644
--- a/arch/mips/ralink/reset.c
+++ b/arch/mips/ralink/reset.c
@@ -61,7 +61,7 @@ static int ralink_reset_device(struct reset_controller_dev *rcdev,
 	return ralink_deassert_device(rcdev, id);
 }
 
-static struct reset_control_ops reset_ops = {
+static const struct reset_control_ops reset_ops = {
 	.reset = ralink_reset_device,
 	.assert = ralink_assert_device,
 	.deassert = ralink_deassert_device,
-- 
2.7.0

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

* Re: [PATCH 1/2] MIPS: lantiq: Make reset_control_ops const
  2016-02-25  9:45 ` [PATCH 1/2] MIPS: lantiq: " Philipp Zabel
@ 2016-02-25  9:51   ` John Crispin
  0 siblings, 0 replies; 5+ messages in thread
From: John Crispin @ 2016-02-25  9:51 UTC (permalink / raw)
  To: Philipp Zabel, Ralf Baechle; +Cc: linux-mips, kernel



On 25/02/2016 10:45, Philipp Zabel wrote:
> The reset_ops structure is never modified. Make it const.
> 
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>

Acked-by: John Crispin <blogic@openwrt.org>

> ---
>  arch/mips/lantiq/xway/reset.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/mips/lantiq/xway/reset.c b/arch/mips/lantiq/xway/reset.c
> index bc29bb3..1f34608 100644
> --- a/arch/mips/lantiq/xway/reset.c
> +++ b/arch/mips/lantiq/xway/reset.c
> @@ -258,7 +258,7 @@ static int ltq_reset_device(struct reset_controller_dev *rcdev,
>  	return ltq_deassert_device(rcdev, id);
>  }
>  
> -static struct reset_control_ops reset_ops = {
> +static const struct reset_control_ops reset_ops = {
>  	.reset = ltq_reset_device,
>  	.assert = ltq_assert_device,
>  	.deassert = ltq_deassert_device,
> 

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

* Re: [PATCH 2/2] MIPS: ralink: Make reset_control_ops const
  2016-02-25  9:45 ` [PATCH 2/2] MIPS: ralink: " Philipp Zabel
@ 2016-02-25  9:51   ` John Crispin
  0 siblings, 0 replies; 5+ messages in thread
From: John Crispin @ 2016-02-25  9:51 UTC (permalink / raw)
  To: Philipp Zabel, Ralf Baechle; +Cc: linux-mips, kernel



On 25/02/2016 10:45, Philipp Zabel wrote:
> The reset_ops structure is never modified. Make it const.
> 
> Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>

Acked-by: John Crispin <blogic@openwrt.org>

> ---
>  arch/mips/ralink/reset.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/mips/ralink/reset.c b/arch/mips/ralink/reset.c
> index ee117c4..09ccdd1 100644
> --- a/arch/mips/ralink/reset.c
> +++ b/arch/mips/ralink/reset.c
> @@ -61,7 +61,7 @@ static int ralink_reset_device(struct reset_controller_dev *rcdev,
>  	return ralink_deassert_device(rcdev, id);
>  }
>  
> -static struct reset_control_ops reset_ops = {
> +static const struct reset_control_ops reset_ops = {
>  	.reset = ralink_reset_device,
>  	.assert = ralink_assert_device,
>  	.deassert = ralink_deassert_device,
> 

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

end of thread, other threads:[~2016-02-25  9:51 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-25  9:45 [PATCH 0/2] MIPS: Make reset_control_ops const Philipp Zabel
2016-02-25  9:45 ` [PATCH 1/2] MIPS: lantiq: " Philipp Zabel
2016-02-25  9:51   ` John Crispin
2016-02-25  9:45 ` [PATCH 2/2] MIPS: ralink: " Philipp Zabel
2016-02-25  9:51   ` John Crispin

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox