[PATCH] irqchip: mmp: make mmp_intc_conf const

[PATCH] irqchip: mmp: make mmp_intc_conf const

[Bhumika Goyal]

Make these const as they are only used during a copy operation. Done
using Coccinelle.

@match disable optional_qualifier@
identifier s;
@@
static struct mmp_intc_conf s = {...};

@ref@
position p;
identifier match.s;
@@
s@p

@good1@
position ref.p;
identifier match.s,f,c;
expression e;
@@
(
e = s@p
|
e = s@p.f
|
c(...,s@p.f,...)
|
c(...,s@p,...)
)

@bad depends on  !good1@
position ref.p;
identifier match.s;
@@
s@p

@depends on forall !bad disable optional_qualifier@
identifier match.s;
@@
static
+ const
struct mmp_intc_conf s;

Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>
---
 drivers/irqchip/irq-mmp.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/irqchip/irq-mmp.c b/drivers/irqchip/irq-mmp.c
index 013fc96..25f32e1 100644
--- a/drivers/irqchip/irq-mmp.c
+++ b/drivers/irqchip/irq-mmp.c
@@ -181,13 +181,13 @@ static int mmp_irq_domain_xlate(struct irq_domain *d, struct device_node *node,
 	.xlate		= mmp_irq_domain_xlate,
 };
 
-static struct mmp_intc_conf mmp_conf = {
+static const struct mmp_intc_conf mmp_conf = {
 	.conf_enable	= 0x51,
 	.conf_disable	= 0x0,
 	.conf_mask	= 0x7f,
 };
 
-static struct mmp_intc_conf mmp2_conf = {
+static const struct mmp_intc_conf mmp2_conf = {
 	.conf_enable	= 0x20,
 	.conf_disable	= 0x0,
 	.conf_mask	= 0x7f,
-- 
1.9.1

Re: [PATCH] irqchip: mmp: make mmp_intc_conf const

[Marc Zyngier]

On Thu, Aug 24 2017 at  3:56:21 pm BST, Bhumika Goyal <bhumirks@gmail.com> wrote:
> Make these const as they are only used during a copy operation. Done
> using Coccinelle.
>
> @match disable optional_qualifier@
> identifier s;
> @@
> static struct mmp_intc_conf s = {...};
>
> @ref@
> position p;
> identifier match.s;
> @@
> s@p
>
> @good1@
> position ref.p;
> identifier match.s,f,c;
> expression e;
> @@
> (
> e = s@p
> |
> e = s@p.f
> |
> c(...,s@p.f,...)
> |
> c(...,s@p,...)
> )
>
> @bad depends on  !good1@
> position ref.p;
> identifier match.s;
> @@
> s@p
>
> @depends on forall !bad disable optional_qualifier@
> identifier match.s;
> @@
> static
> + const
> struct mmp_intc_conf s;
>
> Signed-off-by: Bhumika Goyal <bhumirks@gmail.com>

Can you please post these fixes as a series instead of individual
patches? It would make the maintainers' life much easier. I'll apply
this patch anyway, but please keep this in mind for the future.

Thanks,

	M.
-- 
Jazz is not dead, it just smell funny.