[PATCH v3 00/10] Use media bus formats in imx-drm and add drm panel support
Hi Mauro, Am Donnerstag, den 12.03.2015, 10:58 +0100 schrieb Philipp Zabel: > Currently the imx-drm driver misuses the V4L2_PIX_FMT constants to describe > the > pixel format on the parallel bus between display controllers and encoders. Now > that MEDIA_BUS_FMT is available, use that instead. > > I'd like to get the V4L2 maintainers' acks for the four necessary media > bus format patches to be merged through the drm tree, if that still is the > preferred way for the media format patches to go in together with the driver > changes using them. > > The two drm/imx core patches replace V4L2_PIX_FMT with MEDIA_BUS_FMT where > appropriate and consolidate the variable names for clarification. > > The three LDB patches depend on the of-graph helper series: > http://permalink.gmane.org/gmane.linux.kernel/1898485 > They allow to optionally use LVDS panels with drm_panel drivers, connected to > the LDB encoder in the device tree via of-graph endpoint links. > > Changes since v2: > - Added explanation for component-wise padded format names (CPADHI/LO) > - Improved documentation (wording, spelling and syntax fixes) > - Also rename pixel_fmt parameter to ipu_dc_init_sync to bus_format > (- Added linux-media to Cc for the media format patches) > > regards > Philipp > > Boris Brezillion (1): > Add RGB444_1X12 and RGB565_1X16 media bus formats > > Philipp Zabel (9): > Add LVDS RGB media bus formats > Add BGR888_1X24 and GBR888_1X24 media bus formats > Add YUV8_1X24 media bus format > Add RGB666_1X24_CPADHI media bus format I'd like to send a pull request for these if you could give your ack for the first five patches above. regards Philipp
[PATCH v3 00/10] Use media bus formats in imx-drm and add drm panel support
Am Donnerstag, den 12.03.2015, 18:53 +0100 schrieb Emil Renner Berthing: > Hi Philipp > > On 12 March 2015 at 10:58, Philipp Zabel wrote: > > Currently the imx-drm driver misuses the V4L2_PIX_FMT constants to describe > > the > > pixel format on the parallel bus between display controllers and encoders. > > Now > > that MEDIA_BUS_FMT is available, use that instead. > > I've tested this series on the Hercules eCAFE Slim HD, which uses the > RGB666_1X24_CPADHI format, and it still boots up with screen output. > You can add > > Tested-by: Emil Renner Berthing Thanks for testing! I'll add that to the relevant patches (5-7). regards Philipp
[PATCH v3 00/10] Use media bus formats in imx-drm and add drm panel support
Hi Philipp On 12 March 2015 at 10:58, Philipp Zabel wrote: > Currently the imx-drm driver misuses the V4L2_PIX_FMT constants to describe > the > pixel format on the parallel bus between display controllers and encoders. Now > that MEDIA_BUS_FMT is available, use that instead. I've tested this series on the Hercules eCAFE Slim HD, which uses the RGB666_1X24_CPADHI format, and it still boots up with screen output. You can add Tested-by: Emil Renner Berthing -- /Emil
[PATCH v3 00/10] Use media bus formats in imx-drm and add drm panel support
Hi Philipp, On 03/12/15 10:58, Philipp Zabel wrote: > Currently the imx-drm driver misuses the V4L2_PIX_FMT constants to describe > the > pixel format on the parallel bus between display controllers and encoders. Now > that MEDIA_BUS_FMT is available, use that instead. > > I'd like to get the V4L2 maintainers' acks for the four necessary media > bus format patches to be merged through the drm tree, if that still is the > preferred way for the media format patches to go in together with the driver > changes using them. Yes, that's the preferred way. For patches 1-5: Acked-by: Hans Verkuil Regards, Hans > > The two drm/imx core patches replace V4L2_PIX_FMT with MEDIA_BUS_FMT where > appropriate and consolidate the variable names for clarification. > > The three LDB patches depend on the of-graph helper series: > http://permalink.gmane.org/gmane.linux.kernel/1898485 > They allow to optionally use LVDS panels with drm_panel drivers, connected to > the LDB encoder in the device tree via of-graph endpoint links. > > Changes since v2: > - Added explanation for component-wise padded format names (CPADHI/LO) > - Improved documentation (wording, spelling and syntax fixes) > - Also rename pixel_fmt parameter to ipu_dc_init_sync to bus_format > (- Added linux-media to Cc for the media format patches) > > regards > Philipp > > Boris Brezillion (1): > Add RGB444_1X12 and RGB565_1X16 media bus formats > > Philipp Zabel (9): > Add LVDS RGB media bus formats > Add BGR888_1X24 and GBR888_1X24 media bus formats > Add YUV8_1X24 media bus format > Add RGB666_1X24_CPADHI media bus format > drm/imx: switch to use media bus formats > drm/imx: consolidate bus format variable names > drm/imx: imx-ldb: add drm_panel support > drm/imx: imx-ldb: reset display clock input when disabling LVDS > drm/imx: imx-ldb: allow to determine bus format from the connected > panel > > Documentation/DocBook/media/v4l/subdev-formats.xml | 426 > - > Documentation/devicetree/bindings/drm/imx/ldb.txt | 62 ++- > drivers/gpu/drm/imx/Kconfig| 1 + > drivers/gpu/drm/imx/dw_hdmi-imx.c | 2 +- > drivers/gpu/drm/imx/imx-drm-core.c | 14 +- > drivers/gpu/drm/imx/imx-drm.h | 10 +- > drivers/gpu/drm/imx/imx-ldb.c | 196 +++--- > drivers/gpu/drm/imx/imx-tve.c | 6 +- > drivers/gpu/drm/imx/ipuv3-crtc.c | 13 +- > drivers/gpu/drm/imx/parallel-display.c | 13 +- > drivers/gpu/ipu-v3/ipu-dc.c| 18 +- > include/uapi/linux/media-bus-format.h | 13 +- > include/video/imx-ipu-v3.h | 2 +- > 13 files changed, 655 insertions(+), 121 deletions(-) >
[PATCH v3 00/10] Use media bus formats in imx-drm and add drm panel support
Currently the imx-drm driver misuses the V4L2_PIX_FMT constants to describe the pixel format on the parallel bus between display controllers and encoders. Now that MEDIA_BUS_FMT is available, use that instead. I'd like to get the V4L2 maintainers' acks for the four necessary media bus format patches to be merged through the drm tree, if that still is the preferred way for the media format patches to go in together with the driver changes using them. The two drm/imx core patches replace V4L2_PIX_FMT with MEDIA_BUS_FMT where appropriate and consolidate the variable names for clarification. The three LDB patches depend on the of-graph helper series: http://permalink.gmane.org/gmane.linux.kernel/1898485 They allow to optionally use LVDS panels with drm_panel drivers, connected to the LDB encoder in the device tree via of-graph endpoint links. Changes since v2: - Added explanation for component-wise padded format names (CPADHI/LO) - Improved documentation (wording, spelling and syntax fixes) - Also rename pixel_fmt parameter to ipu_dc_init_sync to bus_format (- Added linux-media to Cc for the media format patches) regards Philipp Boris Brezillion (1): Add RGB444_1X12 and RGB565_1X16 media bus formats Philipp Zabel (9): Add LVDS RGB media bus formats Add BGR888_1X24 and GBR888_1X24 media bus formats Add YUV8_1X24 media bus format Add RGB666_1X24_CPADHI media bus format drm/imx: switch to use media bus formats drm/imx: consolidate bus format variable names drm/imx: imx-ldb: add drm_panel support drm/imx: imx-ldb: reset display clock input when disabling LVDS drm/imx: imx-ldb: allow to determine bus format from the connected panel Documentation/DocBook/media/v4l/subdev-formats.xml | 426 - Documentation/devicetree/bindings/drm/imx/ldb.txt | 62 ++- drivers/gpu/drm/imx/Kconfig| 1 + drivers/gpu/drm/imx/dw_hdmi-imx.c | 2 +- drivers/gpu/drm/imx/imx-drm-core.c | 14 +- drivers/gpu/drm/imx/imx-drm.h | 10 +- drivers/gpu/drm/imx/imx-ldb.c | 196 +++--- drivers/gpu/drm/imx/imx-tve.c | 6 +- drivers/gpu/drm/imx/ipuv3-crtc.c | 13 +- drivers/gpu/drm/imx/parallel-display.c | 13 +- drivers/gpu/ipu-v3/ipu-dc.c| 18 +- include/uapi/linux/media-bus-format.h | 13 +- include/video/imx-ipu-v3.h | 2 +- 13 files changed, 655 insertions(+), 121 deletions(-) -- 2.1.4
