On Sat, Mar 28, 2026 at 05:30:53PM +0000, cristian_ci wrote: > On Friday, March 27th, 2026 at 23:57, Dmitry Baryshkov > <[email protected]> wrote: > > > On Fri, Mar 27, 2026 at 03:30:49PM +0100, Cristian Cozzolino via B4 Relay > > wrote: > > > From: Cristian Cozzolino <[email protected]> > > > > > > Add the description for the display panel found on this phone. > > > And with this done we can also enable the GPU and set the zap shader > > > firmware path. > > > > > > Signed-off-by: Cristian Cozzolino <[email protected]> > > > --- > > > .../arm64/boot/dts/qcom/msm8953-flipkart-rimob.dts | 73 > > > ++++++++++++++++++++++ > > > 1 file changed, 73 insertions(+) > > > > > > > Reviewed-by: Dmitry Baryshkov <[email protected]> > > I wonder if I should, instead, edit the compatible property by adding a > second string (for the fallback), like this: > > compatible = "flipkart,rimob-panel-nt35532-cs", "novatek,nt35532"; > > and, therefore, add "novatek,nt35532" string also to (patch 1/6)'s > bindings example. Let me know what you think.
What would it mean? I think we usually don't include the IC into the compat list for the panel, but feel free to prove me wrong. -- With best wishes Dmitry
