Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 7:50 PM, Archit Taneja wrote: > On 11/23/2016 01:16 AM, John Stultz wrote: >> On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart >> wrote: >>> On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: On Tue, Nov 22,

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 7:50 PM, Archit Taneja wrote: > On 11/23/2016 01:16 AM, John Stultz wrote: >> On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart >> wrote: >>> On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > On Monday

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On 11/23/2016 01:16 AM, John Stultz wrote: On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart wrote: Hi John, On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: On Monday 21 Nov 2016 16:37:30 John

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On 11/23/2016 01:16 AM, John Stultz wrote: On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart wrote: Hi John, On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: On Monday 21 Nov 2016 16:37:30 John Stultz wrote: @@ -545,24 +554,13

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart wrote: > Hi John, > > On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: >> On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: >> > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: >> @@ -545,24 +554,13

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart wrote: > Hi John, > > On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: >> On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: >> > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: >> @@ -545,24 +554,13 @@ static int

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart wrote: > Hi John, > > On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: >> On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: >> > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: >> @@ -545,24 +554,13

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 9:38 AM, Laurent Pinchart wrote: > Hi John, > > On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: >> On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: >> > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: >> @@ -545,24 +554,13 @@ static int

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

Hi John, On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: > On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: > @@ -545,24 +554,13 @@ static int adv7511_get_modes(struct adv7511 *adv7511, > >> unsigned int count; > >> >

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

Hi John, On Tuesday 22 Nov 2016 09:25:22 John Stultz wrote: > On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: > @@ -545,24 +554,13 @@ static int adv7511_get_modes(struct adv7511 *adv7511, > >> unsigned int count; > >> >

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: @@ -545,24 +554,13 @@ static int adv7511_get_modes(struct adv7511 *adv7511, >> unsigned int count; >> >> /* Reading the EDID only works if

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: @@ -545,24 +554,13 @@ static int adv7511_get_modes(struct adv7511 *adv7511, >> unsigned int count; >> >> /* Reading the EDID only works if the device is powered */ >> -

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

Hi John, On Tuesday 22 Nov 2016 00:16:55 John Stultz wrote: > On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: > >> In chasing down issues with EDID probing, I found some > >> duplicated but incomplete logic used to power the chip on

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

Hi John, On Tuesday 22 Nov 2016 00:16:55 John Stultz wrote: > On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: > >> In chasing down issues with EDID probing, I found some > >> duplicated but incomplete logic used to power the chip on

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > Hi John, > > Thank you for the patch. > > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: >> In chasing down issues with EDID probing, I found some >> duplicated but incomplete logic used to power the

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

On Tue, Nov 22, 2016 at 12:14 AM, Laurent Pinchart wrote: > Hi John, > > Thank you for the patch. > > On Monday 21 Nov 2016 16:37:30 John Stultz wrote: >> In chasing down issues with EDID probing, I found some >> duplicated but incomplete logic used to power the chip on and >> off. >> >> This

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

Hi John, Thank you for the patch. On Monday 21 Nov 2016 16:37:30 John Stultz wrote: > In chasing down issues with EDID probing, I found some > duplicated but incomplete logic used to power the chip on and > off. > > This patch refactors the adv7511_power_on/off functions, so > they can be used

Re: [RFC][PATCH 1/3] drm/bridge: adv7511: Rework adv7511_power_on/off() so they can be reused internally

Hi John, Thank you for the patch. On Monday 21 Nov 2016 16:37:30 John Stultz wrote: > In chasing down issues with EDID probing, I found some > duplicated but incomplete logic used to power the chip on and > off. > > This patch refactors the adv7511_power_on/off functions, so > they can be used