Re: [PATCH v3 3/5] drm/i915: Extend private i915_param_named macro with description

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux