Re: [PATCH v4 3/3] coccinelle: Add script to check for platform_get_irq() excessive prints

2019-07-24 Thread Stephen Boyd
Quoting Markus Elfring (2019-07-24 06:18:35) > > +@script:python depends on org@ > > +p1 << r.p1; > > +@@ > > + > > +cocci.print_main(p1) > > Will an additional message be helpful at this place? > > > Will further software development considerations become more interesting > also around a

Re: [PATCH v4 3/3] coccinelle: Add script to check for platform_get_irq() excessive prints

2019-07-24 Thread Stephen Boyd
Quoting Markus Elfring (2019-07-24 02:30:16) > I would prefer to concentrate the usage of SmPL disjunctions on changing > implementation details so that the specification of duplicate code > can be avoided. > > > > +( > > +platform_get_irq(E, ...) > > +| > > +platform_get_irq_byname(E, ...) > >

Re: [Cocci] [PATCH v4 3/3] coccinelle: Add script to check for platform_get_irq() excessive prints

2019-07-24 Thread Markus Elfring
> +@script:python depends on org@ > +p1 << r.p1; > +@@ > + > +cocci.print_main(p1) Will an additional message be helpful at this place? Will further software development considerations become more interesting also around a contribution like “Coccinelle: Add a SmPL script for the reconsideration

Re: [PATCH v4 3/3] coccinelle: Add script to check for platform_get_irq() excessive prints

2019-07-24 Thread Markus Elfring
I would prefer to concentrate the usage of SmPL disjunctions on changing implementation details so that the specification of duplicate code can be avoided. > +( > +platform_get_irq(E, ...) > +| > +platform_get_irq_byname(E, ...) > +); Function names: +(platform_get_irq

[PATCH v4 3/3] coccinelle: Add script to check for platform_get_irq() excessive prints

2019-07-23 Thread Stephen Boyd
Add a coccinelle script to check for the usage of dev_err() after a call to platform_get_irq{,_byname}() as it's redundant now that the function already prints an error when it fails. Cc: Greg Kroah-Hartman Cc: Rob Herring Cc: Bartlomiej Zolnierkiewicz Cc: Javier Martinez Canillas Cc: Andrzej