[PATCH] drm/panel: Remove the get_timings() function.

Eric Anholt eric at anholt.net
Tue Jul 12 18:26:29 UTC 2016


Thierry Reding <thierry.reding at gmail.com> writes:

> On Wed, Jun 01, 2016 at 12:18:01PM -0700, Eric Anholt wrote:
>> It appears to have no callers.
>> 
>> Signed-off-by: Eric Anholt <eric at anholt.net>
>> ---
>>  drivers/gpu/drm/panel/panel-simple.c | 18 ------------------
>>  include/drm/drm_panel.h              |  4 ----
>>  2 files changed, 22 deletions(-)
>
> Looks like I never replied to this, though I remember at least making up
> the reply in my head.
>
> The reason why I'd like to keep this is that it's technically the right
> interface for display drivers to use. It was introduced in order to fix
> some of the short-comings of ->get_modes(), though it seems like there
> simply hasn't been a need so far for drivers to do this.

OK.  That makes some sense then.  I was writing a new panel driver, and
it was confusing to have this unused code in panel-simple that I was
deriving from.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20160712/a6d9712e/attachment.sig>


More information about the dri-devel mailing list