Quoting Michal Wajdeczko (2017-09-14 16:08:03) > We're always specifying description of each module param in > separate macro. Let's combine description into our main macro. > Started with Coccinelle, followed by minor cleanup. > > @match1@ > declarer name MODULE_PARM_DESC; > identifier n; > constant c; > @@ > ( > - MODULE_PARM_DESC(n, c); > ) > > @fix1 depends on match1@ > declarer name i915_param_named; > declarer name i915_param_named_unsafe; > identifier match1.n; > constant match1.c; > @@ > ( > i915_param_named(n, ... > + , c > ); > | > i915_param_named_unsafe(n, ... > + , c > ); > ) > > Suggested-by: Jani Nikula <jani.nikula@xxxxxxxxx> > Signed-off-by: Michal Wajdeczko <michal.wajdeczko@xxxxxxxxx> > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx