Re: [PATCH v6 12/14] drm/bridge: imx: Add LDB support for i.MX8qxp

2021-03-31 Thread Liu Ying
On Tue, 2021-03-30 at 11:54 +0200, Robert Foss wrote: > Hey Liu, > > checkpatch --strict lists some nit and a warning. With those fixed > feel free to add my r-b. > > On Wed, 17 Mar 2021 at 04:57, Liu Ying wrote: > > This patch adds a drm bridge driver for i.MX8qxp LVDS display bridge(LDB) > >

Re: [PATCH v6 12/14] drm/bridge: imx: Add LDB support for i.MX8qxp

2021-03-31 Thread Liu Ying
Hi Robert, On Tue, 2021-03-30 at 11:59 +0200, Robert Foss wrote: > Hey Liu, > > checkpatch --strict lists some nits and a warning. I think the kconfig > warning can be ignored. With the rest fixed, feel free to add my r-b. > > Reviewed-by: Robert Foss Thanks for your review. Will fix those

Re: [PATCH v6 12/14] drm/bridge: imx: Add LDB support for i.MX8qxp

2021-03-30 Thread Robert Foss
Hey Liu, checkpatch --strict lists some nits and a warning. I think the kconfig warning can be ignored. With the rest fixed, feel free to add my r-b. Reviewed-by: Robert Foss On Wed, 17 Mar 2021 at 04:57, Liu Ying wrote: > > This patch adds a drm bridge driver for i.MX8qxp LVDS display

Re: [PATCH v6 12/14] drm/bridge: imx: Add LDB support for i.MX8qxp

2021-03-30 Thread Robert Foss
Hey Liu, checkpatch --strict lists some nit and a warning. With those fixed feel free to add my r-b. On Wed, 17 Mar 2021 at 04:57, Liu Ying wrote: > > This patch adds a drm bridge driver for i.MX8qxp LVDS display bridge(LDB) > which is officially named as pixel mapper. The LDB has two

Re: [PATCH v6 12/14] drm/bridge: imx: Add LDB support for i.MX8qxp

2021-03-19 Thread kernel test robot
Hi Liu, Thank you for the patch! Yet something to improve: [auto build test ERROR on shawnguo/for-next] [also build test ERROR on robh/for-next drm-intel/for-linux-next drm-tip/drm-tip drm-exynos/exynos-drm-next tegra-drm/drm/tegra/for-next linus/master v5.12-rc3 next-20210319] [cannot apply

[PATCH v6 12/14] drm/bridge: imx: Add LDB support for i.MX8qxp

2021-03-16 Thread Liu Ying
This patch adds a drm bridge driver for i.MX8qxp LVDS display bridge(LDB) which is officially named as pixel mapper. The LDB has two channels. Each of them supports up to 24bpp parallel input color format and can map the input to VESA or JEIDA standards. The two channels cannot be used