Thierry Reding <thierry.reding@xxxxxxxxx> writes: Hello Thierry, > From: Thierry Reding <treding@xxxxxxxxxx> > > The drm_num_crtcs() helper determines the number of CRTCs by iterating > over the list of CRTCs that have been registered with the mode config. > However, we already keep track of that number in the mode config's > num_crtcs field, so we can simply retrieve the value from that and > remove the extra helper function. > > Signed-off-by: Thierry Reding <treding@xxxxxxxxxx> > --- > drivers/gpu/drm/drm_crtc.c | 15 +-------------- > 1 file changed, 1 insertion(+), 14 deletions(-) > Indeed. I don't see why this helper would be needed. Your patch makes sense to me. Reviewed-by: Javier Martinez Canillas <javierm@xxxxxxxxxx> -- Best regards, Javier Martinez Canillas Core Platforms Red Hat