On Fri, 15 Sep 2017, Chris Wilson <[email protected]> wrote: > 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 <[email protected]> >> Signed-off-by: Michal Wajdeczko <[email protected]> >> Cc: Jani Nikula <[email protected]> >> Cc: Chris Wilson <[email protected]> > Reviewed-by: Chris Wilson <[email protected]>
Pushed patches 1-3, undecided for the rest. Thanks for the patches and review. BR, Jani. -- Jani Nikula, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
