Re: [PATCH v3 0/3] drm: describe display bus format

2014-11-30 Thread Boris Brezillon
Hi Laurent,

On Sat, 29 Nov 2014 00:29:10 +0200
Laurent Pinchart laurent.pinch...@ideasonboard.com wrote:

 Hi Boris,
 
 On Thursday 27 November 2014 14:37:50 Boris Brezillon wrote:
  On Tue, 18 Nov 2014 14:46:17 +0100 Boris Brezillon wrote:
   Hello,
   
   This series makes use of the MEDIA_BUS_FMT definition to describe how
   the data are transmitted to the display.
   
   This will allow drivers to configure their output display bus according
   to the display capabilities.
   For example some display controllers support DPI (or raw RGB) connectors
   and need to specify which format will be transmitted on the DPI bus
   (RGB444, RGB565, RGB888, ...).
   
   This series also adds a field to the panel_desc struct so that one
   can specify which format is natevely supported by a panel.
  
  Thierry, Laurent, Dave, can you take a look at this patch series: this
  is the last missing dependency to get the atmel-hlcdc DRM driver
  mainlined, and I was expecting to get this driver in 3.19...
 
 I've reviewed the series, it looks globally fine to me. I just had two small 
 comments on patch 1/3.
 

Thanks for the review, I'll address your comments in the next version.

Regards,

Boris

-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 0/3] drm: describe display bus format

2014-11-28 Thread Laurent Pinchart
Hi Boris,

On Thursday 27 November 2014 14:37:50 Boris Brezillon wrote:
 On Tue, 18 Nov 2014 14:46:17 +0100 Boris Brezillon wrote:
  Hello,
  
  This series makes use of the MEDIA_BUS_FMT definition to describe how
  the data are transmitted to the display.
  
  This will allow drivers to configure their output display bus according
  to the display capabilities.
  For example some display controllers support DPI (or raw RGB) connectors
  and need to specify which format will be transmitted on the DPI bus
  (RGB444, RGB565, RGB888, ...).
  
  This series also adds a field to the panel_desc struct so that one
  can specify which format is natevely supported by a panel.
 
 Thierry, Laurent, Dave, can you take a look at this patch series: this
 is the last missing dependency to get the atmel-hlcdc DRM driver
 mainlined, and I was expecting to get this driver in 3.19...

I've reviewed the series, it looks globally fine to me. I just had two small 
comments on patch 1/3.

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 0/3] drm: describe display bus format

2014-11-27 Thread Boris Brezillon
Hi,

On Tue, 18 Nov 2014 14:46:17 +0100
Boris Brezillon boris.brezil...@free-electrons.com wrote:

 Hello,
 
 This series makes use of the MEDIA_BUS_FMT definition to describe how
 the data are transmitted to the display.
 
 This will allow drivers to configure their output display bus according
 to the display capabilities.
 For example some display controllers support DPI (or raw RGB) connectors
 and need to specify which format will be transmitted on the DPI bus
 (RGB444, RGB565, RGB888, ...).
 
 This series also adds a field to the panel_desc struct so that one
 can specify which format is natevely supported by a panel.

Thierry, Laurent, Dave, can you take a look at this patch series: this
is the last missing dependency to get the atmel-hlcdc DRM driver
mainlined, and I was expecting to get this driver in 3.19...

Best Regards,

Boris

-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v3 0/3] drm: describe display bus format

2014-11-18 Thread Boris Brezillon
Hello,

This series makes use of the MEDIA_BUS_FMT definition to describe how
the data are transmitted to the display.

This will allow drivers to configure their output display bus according
to the display capabilities.
For example some display controllers support DPI (or raw RGB) connectors
and need to specify which format will be transmitted on the DPI bus
(RGB444, RGB565, RGB888, ...).

This series also adds a field to the panel_desc struct so that one
can specify which format is natevely supported by a panel.

Regards,

Boris

Changes since v2:
 - use the MEDIA_BUS_FMT macros

Changes since v1:
 - rename nformats into num_formats
 - declare num_formats as an unsigned int

Boris Brezillon (3):
  drm: add bus_formats and nbus_formats fields to drm_display_info
  drm: panel: simple-panel: add support for bus_format retrieval
  drm: panel: simple-panel: add bus format information for foxlink panel

 drivers/gpu/drm/drm_crtc.c   | 30 ++
 drivers/gpu/drm/panel/panel-simple.c |  6 ++
 include/drm/drm_crtc.h   |  7 +++
 3 files changed, 43 insertions(+)

-- 
1.9.1

--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html