Fwd: [PATCH] drm: panel-orientation-quirks: Get rid of superfluous (void *) casting

Jani Nikula jani.nikula at linux.intel.com
Thu Feb 14 10:55:57 UTC 2019


On Wed, 13 Feb 2019, Thierry Reding <thierry.reding at gmail.com> wrote:
> On Wed, Feb 13, 2019 at 11:13:23AM +0100, David SantamarĂ­a Rogado wrote:
>> Sorry previous one wasn't signed off by.
>
> And now you've got this comment in the commit message that's going to
> confuse people looking at the log.
>
> It's usually to put this kind of comment below the --- line...
>
>> The (void *) casting in the driver_data variable assignment is superfluous.
>> Spotted by Jani Nikula.

Please either make that

Suggested-by: Jani Nikula <jani.nikula at intel.com>

or leave out completely.


>> 
>> Signed-off-by: David SantamarĂ­a Rogado <howl.nsp at gmail.com>
>> ---
>
> ... right here.
>
>>  drivers/gpu/drm/drm_panel_orientation_quirks.c | 18 +++++++++---------
>>  1 file changed, 9 insertions(+), 9 deletions(-)
>
> Otherwise, looks good to me:
>
> Acked-by: Thierry Reding <treding at nvidia.com>

With Thierry's suggestion fixed,

Reviewed-by: Jani Nikula <jani.nikula at intel.com>



-- 
Jani Nikula, Intel Open Source Graphics Center


More information about the dri-devel mailing list