[PATCH 2/2] video: of: display_timing: remove broken of_display_timings_exist

Tomi Valkeinen tomi.valkeinen at ti.com
Thu Sep 26 02:37:19 PDT 2013

On 25/09/13 14:51, Andrzej Hajda wrote:
> of_display_timings_exist is implemented incorrectly.
> It tries to find property instead of node.
> The function is not used anyway so the patch removes it.
> Signed-off-by: Andrzej Hajda <a.hajda at samsung.com>
> Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
> ---
>  drivers/video/of_display_timing.c | 20 --------------------
>  include/video/of_display_timing.h |  1 -
>  2 files changed, 21 deletions(-)

Fix by removal? =).

Wouldn't it be better to fix the function? It's not used currently, but
the whole display-timings stuff is still quite new. One could use
of_get_display_timings() to check for the existence of timings, but that
function will print an error if no timings are found.

Then again, what would be the case where you want to check if the
timings exist... I'd presume that there either has to be timings or
there are no timings.

So, I don't know. Any else has opinions whether to remove or fix the


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 901 bytes
Desc: OpenPGP digital signature
URL: <http://lists.freedesktop.org/archives/dri-devel/attachments/20130926/94806670/attachment-0001.pgp>

More information about the dri-devel mailing list