Re: [PATCH 2/2] arm64: defconfig: Enable Lontium LT8713sx driver

[Vishnu Saini]

On Tue, Mar 17, 2026 at 07:51:36PM +0200, Dmitry Baryshkov wrote:
> On Tue, Mar 17, 2026 at 08:49:10PM +0530, Vishnu Saini wrote:
> > On Tue, Mar 17, 2026 at 03:40:54PM +0100, Krzysztof Kozlowski wrote:
> > > On 17/03/2026 15:23, Vishnu Saini wrote:
> > > > On Tue, Mar 17, 2026 at 02:49:56PM +0100, Krzysztof Kozlowski wrote:
> > > >> On 17/03/2026 14:38, Vishnu Saini wrote:
> > > >>> On Tue, Mar 17, 2026 at 12:44:05PM +0100, Krzysztof Kozlowski wrote:
> > > >>>> On 17/03/2026 07:03, Vishnu Saini wrote:
> > > >>>>> Lontium LT8713sx DP bridge hub can be found on a Qualcomm
> > > >>>>> Monaco EVK board for converting 1 DP to 3 DP outputs.
> > > >>>>>
> > > >>>>> Signed-off-by: Vishnu Saini <vishnu.saini@xxxxxxxxxxxxxxxx>
> > > >>>>> ---
> > > >>>>>  arch/arm64/configs/defconfig | 1 +
> > > >>>>>  1 file changed, 1 insertion(+)
> > > >>>>>
> > > >>>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> > > >>>>> index 1d967a81b82a..c8f89c87672b 100644
> > > >>>>> --- a/arch/arm64/configs/defconfig
> > > >>>>> +++ b/arch/arm64/configs/defconfig
> > > >>>>> @@ -1002,6 +1002,7 @@ CONFIG_DRM_PANEL_VISIONOX_VTDR6130=m
> > > >>>>>  CONFIG_DRM_DISPLAY_CONNECTOR=m
> > > >>>>>  CONFIG_DRM_FSL_LDB=m
> > > >>>>>  CONFIG_DRM_ITE_IT6263=m
> > > >>>>> +CONFIG_DRM_LONTIUM_LT8713SX=m
> > > >>>>
> > > >>>> Wrongly placed, at least at next I am checking now. Look at Kconfig or
> > > >>>> just run savedefconfig.
> > > >>>
> > > >>> Cross checked this few rescent changes were added configs in same file.
> > > >>> verified added config is present in generated .config file and lt8713sx driver is compiled with it.
> > > >>> Let me know if need to add this config is another file or at different place.
> > > >>
> > > >> So did you run `make savedefconfig`? If you do not understand review
> > > >> then at least follow what I asked for...
> > > > Yes, 'make savedefconfig' executed as below, sorry i did not mention about this earlier.
> > > > make savedefconfig
> > > 
> > > And? What is the point of savedefconfig?
> > This generate a minimal defconfig containing only non‑default options.
> > A defconfig file is generated in my root dir, CONFIG_DRM_LONTIUM_LT8713SX is part of this generated defconfig.
> > shall i update arch/arm64/configs/defconfig with generated defconfig and verify ?
> 
> In which place is the config entry present in defconfig and is it the
> same as the one in your patch? That was the question that Krzysztof
> asked in the first email, which you seem to ignore.

Thank you for the clarity, I thought lexicographical order is followed here.
savedefconfig generated in below order, i will follow same in v4 patch.
Sorry Krzysztof i completely misunderstood the question.

CONFIG_DRM_LONTIUM_LT8912B=m
CONFIG_DRM_LONTIUM_LT9611=m
CONFIG_DRM_LONTIUM_LT9611UXC=m
CONFIG_DRM_LONTIUM_LT8713SX=m
CONFIG_DRM_ITE_IT66121=m
CONFIG_DRM_NWL_MIPI_DSI=m
 
> -- 
> With best wishes
> Dmitry