On Fri, Jan 17, 2014 at 10:35:10AM +, Dimitris Papastamos wrote:
> > Charles (or someone else from Wolfson), you commented on previous
> > versions of this - are you still OK with it?
>
> Looks good to me.
>
> Privacy & Confidentiality Notice
>
On Fri, Jan 17, 2014 at 10:35:10AM +, Dimitris Papastamos wrote:
Charles (or someone else from Wolfson), you commented on previous
versions of this - are you still OK with it?
Looks good to me.
Privacy Confidentiality Notice
-
This
I'm sorry for this. Looks like I still had problems with some whitespaces. I
will resend the patch and hope, it will work now.
Daniel
Am 17.01.2014 um 13:22 schrieb Mark Brown :
> On Tue, Jan 14, 2014 at 08:34:10PM +0100, Daniel Matuschek wrote:
>
>> WM8804 can run with PLL frequencies of
I'm sorry for this. Looks like I still had problems with some whitespaces. I
will resend the patch and hope, it will work now.
Daniel
Am 17.01.2014 um 13:22 schrieb Mark Brown broo...@kernel.org:
On Tue, Jan 14, 2014 at 08:34:10PM +0100, Daniel Matuschek wrote:
WM8804 can run with PLL
On Fri, Jan 17, 2014 at 07:44:02PM +0100, Florian Meier wrote:
> On 01/17/2014 07:33 PM, Mark Brown wrote:
> > Setting it to false increases power consumption since the device is
> > kept more powered on when idle but reduces startup time from idle. For
> > digital only devices like the wm8804
On 01/17/2014 07:33 PM, Mark Brown wrote:
> On Fri, Jan 17, 2014 at 07:06:24PM +0100, Florian Meier wrote:
>>
> Intentionally off-list?
Oh no - I am sorry!
>> If I remember correctly the error was
>> "codec can not start from non-off bias with idle_bias_off==true"
>
>> I think the solution is
The idle_bias_off should not be part of this patch. I will check this again.
Am 17.01.2014 um 18:59 schrieb Mark Brown :
> On Fri, Jan 17, 2014 at 05:43:14PM +0100, Florian Meier wrote:
>> I have tested your patch.
>> There is a (non blocking) error message regarding .idle_bias_off, but I
>>
On Fri, Jan 17, 2014 at 05:43:14PM +0100, Florian Meier wrote:
> I have tested your patch.
> There is a (non blocking) error message regarding .idle_bias_off, but I
> assume that should not have something to do with your patch. Can we just
> set idle_bias_off to false here?
What is the error
I have tested your patch.
There is a (non blocking) error message regarding .idle_bias_off, but I
assume that should not have something to do with your patch. Can we just
set idle_bias_off to false here?
Otherwise, it looks good to me.
On 01/14/2014 08:34 PM, Daniel Matuschek wrote:
> WM8804 can
On Tue, Jan 14, 2014 at 08:34:10PM +0100, Daniel Matuschek wrote:
> WM8804 can run with PLL frequencies of 256xfs and 128xfs for
> most sample rates. At 192kHz only 128xfs is supported. The
> existing driver selects 128xfs automatically for some lower
This patch doesn't apply against current
> Charles (or someone else from Wolfson), you commented on previous
> versions of this - are you still OK with it?
Looks good to me.
Privacy & Confidentiality Notice
-
This message and any attachments contain privileged and confidential
On Tue, Jan 14, 2014 at 08:34:10PM +0100, Daniel Matuschek wrote:
> WM8804 can run with PLL frequencies of 256xfs and 128xfs for
> most sample rates. At 192kHz only 128xfs is supported. The
> existing driver selects 128xfs automatically for some lower
> samples rates. By using an additional
On Tue, Jan 14, 2014 at 08:34:10PM +0100, Daniel Matuschek wrote:
WM8804 can run with PLL frequencies of 256xfs and 128xfs for
most sample rates. At 192kHz only 128xfs is supported. The
existing driver selects 128xfs automatically for some lower
samples rates. By using an additional mclk_div
Charles (or someone else from Wolfson), you commented on previous
versions of this - are you still OK with it?
Looks good to me.
Privacy Confidentiality Notice
-
This message and any attachments contain privileged and confidential
information
On Tue, Jan 14, 2014 at 08:34:10PM +0100, Daniel Matuschek wrote:
WM8804 can run with PLL frequencies of 256xfs and 128xfs for
most sample rates. At 192kHz only 128xfs is supported. The
existing driver selects 128xfs automatically for some lower
This patch doesn't apply against current code.
I have tested your patch.
There is a (non blocking) error message regarding .idle_bias_off, but I
assume that should not have something to do with your patch. Can we just
set idle_bias_off to false here?
Otherwise, it looks good to me.
On 01/14/2014 08:34 PM, Daniel Matuschek wrote:
WM8804 can
On Fri, Jan 17, 2014 at 05:43:14PM +0100, Florian Meier wrote:
I have tested your patch.
There is a (non blocking) error message regarding .idle_bias_off, but I
assume that should not have something to do with your patch. Can we just
set idle_bias_off to false here?
What is the error message?
The idle_bias_off should not be part of this patch. I will check this again.
Am 17.01.2014 um 18:59 schrieb Mark Brown broo...@kernel.org:
On Fri, Jan 17, 2014 at 05:43:14PM +0100, Florian Meier wrote:
I have tested your patch.
There is a (non blocking) error message regarding .idle_bias_off,
On 01/17/2014 07:33 PM, Mark Brown wrote:
On Fri, Jan 17, 2014 at 07:06:24PM +0100, Florian Meier wrote:
Intentionally off-list?
Oh no - I am sorry!
If I remember correctly the error was
codec can not start from non-off bias with idle_bias_off==true
I think the solution is just to set
On Fri, Jan 17, 2014 at 07:44:02PM +0100, Florian Meier wrote:
On 01/17/2014 07:33 PM, Mark Brown wrote:
Setting it to false increases power consumption since the device is
kept more powered on when idle but reduces startup time from idle. For
digital only devices like the wm8804 there
On Tue, Jan 14, 2014 at 08:34:10PM +0100, Daniel Matuschek wrote:
> WM8804 can run with PLL frequencies of 256xfs and 128xfs for
> most sample rates. At 192kHz only 128xfs is supported. The
> existing driver selects 128xfs automatically for some lower
Charles (or someone else from Wolfson), you
On Tue, Jan 14, 2014 at 08:34:10PM +0100, Daniel Matuschek wrote:
WM8804 can run with PLL frequencies of 256xfs and 128xfs for
most sample rates. At 192kHz only 128xfs is supported. The
existing driver selects 128xfs automatically for some lower
Charles (or someone else from Wolfson), you
WM8804 can run with PLL frequencies of 256xfs and 128xfs for
most sample rates. At 192kHz only 128xfs is supported. The
existing driver selects 128xfs automatically for some lower
samples rates. By using an additional mclk_div divider, it
is now possible to control the behaviour. This allows
WM8804 can run with PLL frequencies of 256xfs and 128xfs for
most sample rates. At 192kHz only 128xfs is supported. The
existing driver selects 128xfs automatically for some lower
samples rates. By using an additional mclk_div divider, it
is now possible to control the behaviour. This allows
On Sun, Jan 12, 2014 at 10:11:25PM +0100, Daniel Matuschek wrote:
> Signed-off-by: Daniel Matuschek
>
> After some discussions of the patch last week, here is a new version.
> Simply reducing the post_table did not work, as for some frequencies
> both settings (MCLKDIV=0 and MCLKDIV=1) are
On 01/12/2014 10:11 PM, Daniel Matuschek wrote:
> Signed-off-by: Daniel Matuschek
>
> After some discussions of the patch last week, here is a new version.
> Simply reducing the post_table did not work, as for some frequencies
> both settings (MCLKDIV=0 and MCLKDIV=1) are needed (e.g. 96 and
On 01/12/2014 10:11 PM, Daniel Matuschek wrote:
Signed-off-by: Daniel Matuschek dan...@matuschek.net
After some discussions of the patch last week, here is a new version.
Simply reducing the post_table did not work, as for some frequencies
both settings (MCLKDIV=0 and MCLKDIV=1) are needed
On Sun, Jan 12, 2014 at 10:11:25PM +0100, Daniel Matuschek wrote:
Signed-off-by: Daniel Matuschek dan...@matuschek.net
After some discussions of the patch last week, here is a new version.
Simply reducing the post_table did not work, as for some frequencies
both settings (MCLKDIV=0 and
Signed-off-by: Daniel Matuschek
After some discussions of the patch last week, here is a new version.
Simply reducing the post_table did not work, as for some frequencies
both settings (MCLKDIV=0 and MCLKDIV=1) are needed (e.g. 96 and 192kHz)
WM8804 can run with PLL frequencies of 256xfs
Signed-off-by: Daniel Matuschek dan...@matuschek.net
After some discussions of the patch last week, here is a new version.
Simply reducing the post_table did not work, as for some frequencies
both settings (MCLKDIV=0 and MCLKDIV=1) are needed (e.g. 96 and 192kHz)
WM8804 can run with PLL
30 matches
Mail list logo