Re: [PATCH v2 v4.4.y-cip] ARM: dts: iwg20d-q7-common: Add LCD support


Fabrizio Castro
 

Hi Pavel,

thank you for your feedback!

From: cip-dev@lists.cip-project.org <cip-dev@lists.cip-project.org> On Behalf
Of Pavel Machek via lists.cip-project.org
Sent: 05 June 2020 12:45
To: Fabrizio Castro <fabrizio.castro.jz@renesas.com>
Cc: Pavel Machek <pavel@denx.de>; cip-dev@lists.cip-project.org; Chris
Paterson <Chris.Paterson2@renesas.com>
Subject: Re: [cip-dev] [PATCH v2 v4.4.y-cip] ARM: dts: iwg20d-q7-common: Add
LCD support

Hi!

SIMPLE_PANEL is not enabled in
4.4.y-cip/arm/renesas_shmobile_defconfig , and it seems to be driver
relevant to this panel.
This seems to be a problem. DRM_PANEL_SIMPLE depends on DRM_PANEL,
but DRM_PANEL is not user selectable, it needs to be selected by another
option.
In upstream, commit:
https://github.com/torvalds/linux/commit/bf7149f34

modified the LVDS driver to use the panel API to delegate panel handling,
rather
than parsing the DT manually, and of course in the same context Kconfig is
modified to select DRM_PANEL.

I think we should have another look at this, do you guys think it's possible to
drop
this patch?
It is possible to drop the patch and force push before new commits are
added on top. It is also possible to queue a revert at any later time.

OTOH if the patch is not wrong, just needs another patch to be useful,
I'd keep it.

Adding prompt to Kconfig so that it can be enabled manually should not
be too hard to do.
Thank you for laying down all of the options for me. Since you are open
to a revert patch, I think we'll have another look at this at a later stage,
fixes will follow.

Thanks,
Fab


Best regards,
Pavel
--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Renesas Electronics Europe GmbH, Geschaeftsfuehrer/President: Carsten Jauch, Sitz der Gesellschaft/Registered office: Duesseldorf, Arcadiastrasse 10, 40472 Duesseldorf, Germany, Handelsregister/Commercial Register: Duesseldorf, HRB 3708 USt-IDNr./Tax identification no.: DE 119353406 WEEE-Reg.-Nr./WEEE reg. no.: DE 14978647

Join cip-dev@lists.cip-project.org to automatically receive all group messages.