Hi Joerg, Dave, Am Mittwoch, 26. November 2014, 09:12:56 schrieb Dave Airlie: > On 26 November 2014 at 02:38, Heiko Stübner <heiko@xxxxxxxxx> wrote: > > Mark, > > > > Am Donnerstag, 20. November 2014, 09:46:34 schrieb Mark Yao: > >> This a series of patches is a DRM Driver for Rockchip Socs, add support > >> for vop devices. Future patches will add additional encoders/connectors, > >> such as eDP, HDMI. > >> > >> The basic "crtc" for rockchip is a "VOP" - Video Output Processor. > >> the vop devices found on Rockchip rk3288 Soc, rk3288 soc have two similar > >> Vop devices. Vop devices support iommu mapping, we use dma-mapping API > >> with > >> ARM_DMA_USE_IOMMU. > > > > it looks like everybody is more or less happy with this version - in past > > versions responses voicing concerns where quite swift. > > > > As David requested a pull request the last time, it might be time to do > > so, so that we maybe still reach 3.19. > > I need the iommu changes the drm driver depends on in a stable > maintainer git tree, > I'm not pulling in code I can't build. Joerg, is your arm/rockchip branch [0] considered stable? [0] https://git.kernel.org/cgit/linux/kernel/git/joro/iommu.git/log/?h=arm/rockchip > I'm happy if the rockchip drm pull stays static and I can pull it in > once the iommu bits > are done. While testing this I noticed a quite big kconfig warning (on both 3.18-rc1 and 3.18-rc6): drivers/video/fbdev/Kconfig:5:error: recursive dependency detected! drivers/video/fbdev/Kconfig:5: symbol FB is selected by DRM_KMS_FB_HELPER drivers/gpu/drm/Kconfig:34: symbol DRM_KMS_FB_HELPER is selected by DRM_ROCKCHIP drivers/gpu/drm/rockchip/Kconfig:1: symbol DRM_ROCKCHIP depends on ARM_DMA_USE_IOMMU arch/arm/Kconfig:95: symbol ARM_DMA_USE_IOMMU is selected by VIDEO_OMAP3 drivers/media/platform/Kconfig:96: symbol VIDEO_OMAP3 depends on VIDEO_V4L2 drivers/media/v4l2-core/Kconfig:6: symbol VIDEO_V4L2 depends on I2C drivers/i2c/Kconfig:7: symbol I2C is selected by FB_DDC drivers/video/fbdev/Kconfig:59: symbol FB_DDC is selected by FB_CYBER2000_DDC drivers/video/fbdev/Kconfig:374: symbol FB_CYBER2000_DDC depends on FB_CYBER2000 drivers/video/fbdev/Kconfig:362: symbol FB_CYBER2000 depends on FB It looks like the VIDEO_OMAP3 is selecting ARM_DMA_USE_IOMMU and OMAP_IOMMU instead of depending on it. I'm not sure which driver should change but looking at the exynos drm driver it is also depends on the iommu parts instead of selecting them [like the rockchip drm currently does] Heiko -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html