Re: [PATCH] drm/imx: imx-tve: Make the 'dac' regulator optional

2017-02-08 Thread Fabio Estevam
On Wed, Feb 8, 2017 at 7:52 AM, Philipp Zabel wrote: > Good point, I suppose what the driver should really do is warn if the > voltage not set correctly? Yes, I can do as suggested. Will prepare a patch. Thanks ___ dri-devel

Re: [PATCH] drm/imx: imx-tve: Make the 'dac' regulator optional

2017-02-08 Thread Philipp Zabel
On Tue, 2017-02-07 at 14:52 -0200, Fabio Estevam wrote: > Hi Philipp, > > On Tue, Feb 7, 2017 at 2:45 PM, Philipp Zabel wrote: > > > I've applied this to the fixes branch, since the current device trees > > don't have the regulator set. > > I have sent a patch providing

Re: [PATCH] drm/imx: imx-tve: Make the 'dac' regulator optional

2017-02-08 Thread Philipp Zabel
On Tue, 2017-02-07 at 18:09 +0100, Lucas Stach wrote: > Am Dienstag, den 07.02.2017, 17:45 +0100 schrieb Philipp Zabel: > > On Fri, 2017-02-03 at 10:52 -0200, Fabio Estevam wrote: > > > Hi Philipp, > > > > > > On Tue, Jan 3, 2017 at 5:11 PM, Fabio Estevam wrote: > > > > From:

Re: [PATCH] drm/imx: imx-tve: Make the 'dac' regulator optional

2017-02-07 Thread Lucas Stach
Am Dienstag, den 07.02.2017, 17:45 +0100 schrieb Philipp Zabel: > On Fri, 2017-02-03 at 10:52 -0200, Fabio Estevam wrote: > > Hi Philipp, > > > > On Tue, Jan 3, 2017 at 5:11 PM, Fabio Estevam wrote: > > > From: Fabio Estevam > > > > > > Commit

Re: [PATCH] drm/imx: imx-tve: Make the 'dac' regulator optional

2017-02-07 Thread Fabio Estevam
Hi Philipp, On Tue, Feb 7, 2017 at 2:45 PM, Philipp Zabel wrote: > I've applied this to the fixes branch, since the current device trees > don't have the regulator set. I have sent a patch providing the dac-supply regulator for imx53-qsb:

Re: [PATCH] drm/imx: imx-tve: Make the 'dac' regulator optional

2017-02-07 Thread Philipp Zabel
On Fri, 2017-02-03 at 10:52 -0200, Fabio Estevam wrote: > Hi Philipp, > > On Tue, Jan 3, 2017 at 5:11 PM, Fabio Estevam wrote: > > From: Fabio Estevam > > > > Commit deb65870b5d9d ("drm/imx: imx-tve: check the value returned by > >

Re: [PATCH] drm/imx: imx-tve: Make the 'dac' regulator optional

2017-02-03 Thread Fabio Estevam
Hi Philipp, On Tue, Jan 3, 2017 at 5:11 PM, Fabio Estevam wrote: > From: Fabio Estevam > > Commit deb65870b5d9d ("drm/imx: imx-tve: check the value returned by > regulator_set_voltage()") exposes the following probe issue: > > 63ff.tve supply dac

[PATCH] drm/imx: imx-tve: Make the 'dac' regulator optional

2017-01-03 Thread Fabio Estevam
From: Fabio Estevam Commit deb65870b5d9d ("drm/imx: imx-tve: check the value returned by regulator_set_voltage()") exposes the following probe issue: 63ff.tve supply dac not found, using dummy regulator imx-drm display-subsystem: failed to bind 63ff.tve (ops