[PATCH] drm: edid: add support for E-DDC

2012-08-30 Thread y...@samsung.com
From: Shirish S The current logic for probing ddc is limited to 2 blocks (256 bytes), this patch adds support for the 4 block (512) data. To do this, a single 8-bit segment index is passed to the display via the I2C address 30h. Data from the selected segment is then immediately read via the reg

[PATCH] drm: edid: add support for E-DDC

2012-08-29 Thread y
From: Shirish S The current logic for probing ddc is limited to 2 blocks (256 bytes), this patch adds support for the 4 block (512) data. To do this, a single 8-bit segment index is passed to the display via the I2C address 30h. Data from the selected segment is then immediately read via the reg

[PATCH] drm: edid: add support for E-DDC

2012-08-24 Thread Daniel Vetter
On Thu, Aug 23, 2012 at 07:06:53AM -0700, Shirish S wrote: > Hello Daniel, > > On Thu, Aug 23, 2012 at 1:54 AM, Daniel Vetter wrote: > > > On Wed, Aug 22, 2012 at 10:52:26AM +0300, Ville Syrj?l? wrote: > > > On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > > > > On Tue, Aug 21, 2012

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Shirish S
Small clarification: On Thu, Aug 23, 2012 at 4:23 PM, Daniel Vetter wrote: > On Thu, Aug 23, 2012 at 07:06:53AM -0700, Shirish S wrote: > > Hello Daniel, > > > > On Thu, Aug 23, 2012 at 1:54 AM, Daniel Vetter wrote: > > > > > On Wed, Aug 22, 2012 at 10:52:26AM +0300, Ville Syrjälä wrote: > > >

[PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Shirish S
Small clarification: On Thu, Aug 23, 2012 at 4:23 PM, Daniel Vetter wrote: > On Thu, Aug 23, 2012 at 07:06:53AM -0700, Shirish S wrote: > > Hello Daniel, > > > > On Thu, Aug 23, 2012 at 1:54 AM, Daniel Vetter wrote: > > > > > On Wed, Aug 22, 2012 at 10:52:26AM +0300, Ville Syrj?l? wrote: > > >

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Daniel Vetter
On Thu, Aug 23, 2012 at 07:06:53AM -0700, Shirish S wrote: > Hello Daniel, > > On Thu, Aug 23, 2012 at 1:54 AM, Daniel Vetter wrote: > > > On Wed, Aug 22, 2012 at 10:52:26AM +0300, Ville Syrjälä wrote: > > > On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > > > > On Tue, Aug 21, 2012

[PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Daniel Vetter
On Wed, Aug 22, 2012 at 10:52:26AM +0300, Ville Syrj?l? wrote: > On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > > On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrj?l? < > > ville.syrjala at linux.intel.com> wrote: > > > > > On Tue, Aug 21, 2012 at 06:55:53AM -0700, Shirish S wrote: > > Her

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Shirish S
Hello Daniel, On Thu, Aug 23, 2012 at 1:54 AM, Daniel Vetter wrote: > On Wed, Aug 22, 2012 at 10:52:26AM +0300, Ville Syrjälä wrote: > > On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > > > On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrjälä < > > > ville.syrj...@linux.intel.com> wrote: >

[PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Shirish S
Hello Daniel, On Thu, Aug 23, 2012 at 1:54 AM, Daniel Vetter wrote: > On Wed, Aug 22, 2012 at 10:52:26AM +0300, Ville Syrj?l? wrote: > > On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > > > On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrj?l? < > > > ville.syrjala at linux.intel.com> wrote

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Shirish S
Hello Ville, On Wed, Aug 22, 2012 at 12:52 AM, Ville Syrjälä < ville.syrj...@linux.intel.com> wrote: > On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > > On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrjälä < > > ville.syrj...@linux.intel.com> wrote: > > > > > On Tue, Aug 21, 2012 at 06:55:

[PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Shirish S
Hello Ville, On Wed, Aug 22, 2012 at 12:52 AM, Ville Syrj?l? < ville.syrjala at linux.intel.com> wrote: > On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > > On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrj?l? < > > ville.syrjala at linux.intel.com> wrote: > > > > > On Tue, Aug 21, 2012 at

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-23 Thread Daniel Vetter
On Wed, Aug 22, 2012 at 10:52:26AM +0300, Ville Syrjälä wrote: > On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > > On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrjälä < > > ville.syrj...@linux.intel.com> wrote: > > > > > On Tue, Aug 21, 2012 at 06:55:53AM -0700, Shirish S wrote: > > Here a

[PATCH] drm: edid: add support for E-DDC

2012-08-22 Thread Ville Syrjälä
On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrj?l? < > ville.syrjala at linux.intel.com> wrote: > > > On Tue, Aug 21, 2012 at 06:55:53AM -0700, Shirish S wrote: > Here are my earlier comments on Jean's patch: > > http://lists.freedesktop.org

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-22 Thread Ville Syrjälä
On Tue, Aug 21, 2012 at 04:28:20PM -0700, Shirish S wrote: > On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrjälä < > ville.syrj...@linux.intel.com> wrote: > > > On Tue, Aug 21, 2012 at 06:55:53AM -0700, Shirish S wrote: > Here are my earlier comments on Jean's patch: > > http://lists.freedesktop.org/ar

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Ville Syrjälä
On Tue, Aug 21, 2012 at 06:55:53AM -0700, Shirish S wrote: > Hello Ville Syrj?l?, > > On Tue, Aug 21, 2012 at 4:26 AM, Ville Syrj?l? < > ville.syrjala at linux.intel.com> wrote: > > > On Tue, Aug 21, 2012 at 12:40:48PM +0530, Shirish S wrote: > > > The current logic for probing ddc is limited to

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrjälä < ville.syrj...@linux.intel.com> wrote: > On Tue, Aug 21, 2012 at 06:55:53AM -0700, Shirish S wrote: > > Hello Ville Syrjälä, > > > > On Tue, Aug 21, 2012 at 4:26 AM, Ville Syrjälä < > > ville.syrj...@linux.intel.com> wrote: > > > > > On Tue, Aug 21,

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
On Tue, Aug 21, 2012 at 7:56 AM, Ville Syrj?l? < ville.syrjala at linux.intel.com> wrote: > On Tue, Aug 21, 2012 at 06:55:53AM -0700, Shirish S wrote: > > Hello Ville Syrj?l?, > > > > On Tue, Aug 21, 2012 at 4:26 AM, Ville Syrj?l? < > > ville.syrjala at linux.intel.com> wrote: > > > > > On Tue, Au

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Ville Syrjälä
On Tue, Aug 21, 2012 at 12:40:48PM +0530, Shirish S wrote: > The current logic for probing ddc is limited to > 2 blocks (256 bytes), this patch adds support > for the 4 block (512) data. A patch for this was already sent a long time ago: http://lists.freedesktop.org/archives/dri-devel/2011-Decembe

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Daniel Vetter
On Tue, Aug 21, 2012 at 9:10 AM, Shirish S wrote: > The current logic for probing ddc is limited to > 2 blocks (256 bytes), this patch adds support > for the 4 block (512) data. > > To do this, a single 8-bit segment index is > passed to the display via the I2C address 30h. > Data from the selecte

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
The current logic for probing ddc is limited to 2 blocks (256 bytes), this patch adds support for the 4 block (512) data. To do this, a single 8-bit segment index is passed to the display via the I2C address 30h. Data from the selected segment is then immediately read via the regular DDC2 address

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
This patch adds support in probing 4 block edid data, for E-DDC. This is the first test case in CTS, for HDMI compliance. Based on drm-next branch Shirish S (1): drm: edid: add support for E-DDC drivers/gpu/drm/drm_edid.c | 17 - 1 files changed, 12 insertions(+), 5 deletion

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Paul Menzel
Am Dienstag, den 21.08.2012, 12:40 +0530 schrieb Shirish S: > The current logic for probing ddc is limited to > 2 blocks (256 bytes), this patch adds support > for the 4 block (512) data. > > To do this, a single 8-bit segment index is > passed to the display via the I2C address 30h. > Data from t

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Ville Syrjälä
On Tue, Aug 21, 2012 at 06:55:53AM -0700, Shirish S wrote: > Hello Ville Syrjälä, > > On Tue, Aug 21, 2012 at 4:26 AM, Ville Syrjälä < > ville.syrj...@linux.intel.com> wrote: > > > On Tue, Aug 21, 2012 at 12:40:48PM +0530, Shirish S wrote: > > > The current logic for probing ddc is limited to > >

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
Hello Ville Syrjälä, On Tue, Aug 21, 2012 at 4:26 AM, Ville Syrjälä < ville.syrj...@linux.intel.com> wrote: > On Tue, Aug 21, 2012 at 12:40:48PM +0530, Shirish S wrote: > > The current logic for probing ddc is limited to > > 2 blocks (256 bytes), this patch adds support > > for the 4 block (512)

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
Hello Ville Syrj?l?, On Tue, Aug 21, 2012 at 4:26 AM, Ville Syrj?l? < ville.syrjala at linux.intel.com> wrote: > On Tue, Aug 21, 2012 at 12:40:48PM +0530, Shirish S wrote: > > The current logic for probing ddc is limited to > > 2 blocks (256 bytes), this patch adds support > > for the 4 block (51

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
Hello Daniel, On Tue, Aug 21, 2012 at 4:18 AM, Daniel Vetter wrote: > On Tue, Aug 21, 2012 at 9:10 AM, Shirish S wrote: > > The current logic for probing ddc is limited to > > 2 blocks (256 bytes), this patch adds support > > for the 4 block (512) data. > > > > To do this, a single 8-bit segmen

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
Hello Daniel, On Tue, Aug 21, 2012 at 4:18 AM, Daniel Vetter wrote: > On Tue, Aug 21, 2012 at 9:10 AM, Shirish S wrote: > > The current logic for probing ddc is limited to > > 2 blocks (256 bytes), this patch adds support > > for the 4 block (512) data. > > > > To do this, a single 8-bit segmen

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Ville Syrjälä
On Tue, Aug 21, 2012 at 12:40:48PM +0530, Shirish S wrote: > The current logic for probing ddc is limited to > 2 blocks (256 bytes), this patch adds support > for the 4 block (512) data. A patch for this was already sent a long time ago: http://lists.freedesktop.org/archives/dri-devel/2011-Decembe

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Daniel Vetter
On Tue, Aug 21, 2012 at 9:10 AM, Shirish S wrote: > The current logic for probing ddc is limited to > 2 blocks (256 bytes), this patch adds support > for the 4 block (512) data. > > To do this, a single 8-bit segment index is > passed to the display via the I2C address 30h. > Data from the selecte

Re: [PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Paul Menzel
Am Dienstag, den 21.08.2012, 12:40 +0530 schrieb Shirish S: > The current logic for probing ddc is limited to > 2 blocks (256 bytes), this patch adds support > for the 4 block (512) data. > > To do this, a single 8-bit segment index is > passed to the display via the I2C address 30h. > Data from t

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
This patch adds support in probing 4 block edid data, for E-DDC. This is the first test case in CTS, for HDMI compliance. Based on drm-next branch Shirish S (1): drm: edid: add support for E-DDC drivers/gpu/drm/drm_edid.c | 17 - 1 files changed, 12 insertions(+), 5 deletion

[PATCH] drm: edid: add support for E-DDC

2012-08-21 Thread Shirish S
The current logic for probing ddc is limited to 2 blocks (256 bytes), this patch adds support for the 4 block (512) data. To do this, a single 8-bit segment index is passed to the display via the I2C address 30h. Data from the selected segment is then immediately read via the regular DDC2 address