Re: [PATCH 9/9] drm: rcar-du: Drop local encoder variable

2020-12-14 Thread Kieran Bingham
Hi Laurent, On 04/12/2020 22:01, Laurent Pinchart wrote: > The local encoder variable is an alias for &renc->base, and is only use > twice. It doesn't help much, drop it, along with the > rcar_encoder_to_drm_encoder() macro that is then unused. It does seem overkill to have a macro for only a sin

Re: [PATCH 9/9] drm: rcar-du: Drop local encoder variable

2020-12-14 Thread Jacopo Mondi
Hi Laurent, On Sat, Dec 05, 2020 at 12:01:39AM +0200, Laurent Pinchart wrote: > The local encoder variable is an alias for &renc->base, and is only use > twice. It doesn't help much, drop it, along with the > rcar_encoder_to_drm_encoder() macro that is then unused. > > Signed-off-by: Laurent Pinch

[PATCH 9/9] drm: rcar-du: Drop local encoder variable

2020-12-04 Thread Laurent Pinchart
The local encoder variable is an alias for &renc->base, and is only use twice. It doesn't help much, drop it, along with the rcar_encoder_to_drm_encoder() macro that is then unused. Signed-off-by: Laurent Pinchart --- drivers/gpu/drm/rcar-du/rcar_du_encoder.c | 6 ++ drivers/gpu/drm/rcar-du/