Hi Javier, On Tuesday 16 September 2014 14:44:02 Javier Martinez Canillas wrote: > [adding Laurent Pinchart to cc who had concerns with a previous > version of this patch-set] Thank you, I've indeed missed v7. > On Wed, Aug 27, 2014 at 4:29 PM, Ajay Kumar wrote: > > This series is based on master branch of Linus tree at: > > git://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git > > > > I have tested this after adding few DT changes for exynos5250-snow and > > exynos5420-peach-pit boards. > > > > The V4 series of this particular patchset was also tested by: > > Rahul Sharma <rahul.sharma@xxxxxxxxxxx> > > Javier Martinez Canillas <javier@xxxxxxxxxxxx> > > > > Changes since V2: > > -- Address comments from Jingoo Han for ps8622 driver > > -- Address comments from Daniel, Rob and Thierry regarding > > bridge chaining > > -- Address comments from Thierry regarding the names for > > new drm_panel functions > > > > Changes since V3: > > -- Remove hotplug based initialization of exynos_dp > > -- Make exynos_dp work directly with drm_panel, remove > > dependency on panel_binder > > -- Minor cleanups in panel_binder and panel_lvds driver > > > > Changes since V4: > > -- Use gpiod interface for panel-lvds and ps8622 drivers. > > -- Address comments from Javier. > > -- Fix compilation issues when PANEL_BINDER is selected as module. > > -- Split Documentation patches from driver patches. > > -- Rebase on top of the tree. > > > > Changes since V5: > > -- Modify bridge drivers to support driver model. > > -- Drop the concept of bridge chain(sincle there are no 2 real > > bridges) > > Hence drop bridge-panel_binder layer. > > -- Drop panel-lvds driver and accomodate the required changes in > > panel-simple driver. > > -- Use gpiod interface in ptn3460 driver. > > -- Address all comments by Thierry Reding for V5 series. > > -- Address comments from Sean Paul for exynos_dp_commit issue. > > > > Changes since V6: > > -- Panel patches were seperated and they are merged already. > > -- Fix few issues with ptn3460, before modifying the bridge core. > > -- Modify drm_bridge as per Thierry's comments for V6 series. > > -- Add drm_bridge changes minimally without breaking existing > > code. > > -- Add new features for ptn3460, step-by-step. > > -- Address comments from Thierry and Andreas for ptn3460 and > > ps8622. > > -- Split documentation patches from driver patches. > > I've tested your series on an Exynos5420 Peach Pit and an Exynos5250 > Snow Chromebooks and display worked for me on both machines. > > I also needed "[PATCH] drm/panel: simple: Add AUO B116XW03 panel > support" [0] which does not apply cleanly on linux-next so you may > want to do a re-spin for that patch. > > For Snow I also had to disable CONFIG_FB_SIMPLE, otherwise I just saw > a blink on boot and only the backlight remained turned on (no display > output). I don't know if that is expected since IIUC it should be > possible to do a transition from simplefb to a DRM/KMS driver. I don't > have a serial console hooked on this machine so I couldn't debug it > further, sorry. > > Also, I saw that Laurent mentioned some concerns today about the > previous version (v6) of your series [1]. I guess he missed this v7 > although AFAIU there was no fundamental change on this one so his > concerns remains? I was really hoping that this could make it to 3.18 > since display support is one of the last major functionalities that is > missing on these machines. My main concern as far as merging this patch set goes is why we can't use the component framework instead of introducing yet another new subsystem-specific component registration framework. My other concerns could be addressed as follow-up patches (but I'd like to start discussing them now). > [0]: http://patchwork.ozlabs.org/patch/384744/ > [1]: http://www.spinics.net/lists/linux-samsung-soc/msg36791.html -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |