On Fri, 22 Sep 2017 21:05:11 +0200, Jani Nikula <[email protected]>
wrote:
On Fri, 22 Sep 2017, Chris Wilson <[email protected]> wrote:
Quoting Michal Wajdeczko (2017-09-22 15:27:25)
By combining default value into helper macro we can initialize
modparams struct in the same automatic way as it was declared.
This will initialize members in the same order as declared
and additionally will disallow declaring new member without
proper default value for it.
Signed-off-by: Michal Wajdeczko <[email protected]>
Cc: Chris Wilson <[email protected]>
Cc: Jani Nikula <[email protected]>
Cc: Joonas Lahtinen <[email protected]>
Overall, I think this is a positive change. I'm not completely happy
that the param() macro is more readable than the struct assignment, but
that is offset by the reduction in duplication.
I'm also not completely happy that the default values get moved away
from the param descriptions. (Hmm, what next, putting the permissions
and descriptions in I915_PARAMS_FOR_EACH too?! :o)
In fact, I was already trying that ;)
https://patchwork.freedesktop.org/patch/162037/
There's also the benefit of being able to highlight the changed values
and displaying the defaults in debugfs if desired.
And such highlight was also there
https://patchwork.freedesktop.org/patch/162038/
Michal
On the series,
Acked-by: Jani Nikula <[email protected]>
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
