On 24/11/2020 18:38, Sam Ravnborg wrote: >>> IMO you should get all the patches at least up including this patch applied. >>> They are all reviewed/acked. And then you have a much smaller stack of >>> patches to spam us with. >> >> Yes, I think that makes sense. I did not want to merge them earlier, as with the v3, I could not get >> videomode panels work at all (while cmd mode panels did work). So I was not sure if something is >> totally silly and broken in the series. >> >> Now that I can get video mode panels work with some hacks on top, I'm fine with merging these. >> >> But it's too late for 5.11, as we need testing and work on the video mode panels. So targeting 5.12. > Obviously your call, but I see no reason to wait for working videomode > panles if what you have now do not introduce any (known) regressions. > > ofc I assume videomode panels is something new and not something that worked > before. It gets a bit muddy here. The omap dsi host driver has had videomode support for a long time, but there has been no upstream videomode panel drivers (omapdrm specific drivers, as omapdrm had its own panel framework) and no board dts files using it. I have a board with a custom made DSI videomode panel setup, but it's broken (cable, I think) and works only randomly. I have an old 4.14 based branch with a hacky panel driver and dts file which get the panel working. I don't know if videomode works on current upstream, or has it been broken between 4.14 and current upstream, as the 4.14 panel driver doesn't work without modifications on current upstream. In this series we convert the omap dsi host driver to be a proper DRM citizen, removing support for omapdrm specific panels, so new DRM panel drivers are needed to replace the omapdrm specific ones. With this series applied, and adding a new panel driver and dts changes, videomode works (Nikolaus confirmed that his panel works. Mine doesn't, as afaics it needs more finetuned initialization which may not be possible with the current DRM bridge/panel callbacks. But mine works with some hacks). But I'm sure in the middle of this series videomode won't work. So, I think one can argue that this causes regressions in the middle of the series to non-upstream panel drivers, but at the end of the series, they probably work, presuming you have a new DRM panel driver for it. Tomi -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
