Re: [PATCH 0/2] Cygnus Audio Driver

2015-04-10 Thread Lori Hikichi
On 15-04-08 11:54 AM, Mark Brown wrote: > On Tue, Apr 07, 2015 at 07:28:40PM -0700, Lori Hikichi wrote: >> On 15-04-06 02:58 AM, Mark Brown wrote: > >>> OK, then it's going to need to be a clock provider at some point - the >>> clock will be going into exter

Re: [PATCH 2/2] ASoC: add core audio driver for Broadcom Cygnus SOC.

2015-04-07 Thread Lori Hikichi
On 15-04-06 09:19 AM, Mark Brown wrote: On Thu, Apr 02, 2015 at 11:47:18AM -0700, Lori Hikichi wrote: On 15-03-30 11:42 PM, Mark Brown wrote: +config SND_SOC_CYGNUS + tristate "SoC platform audio for Broadcom Cygnus chips" + depends on ARCH_BCM_CYGNUS || CO

Re: [PATCH 0/2] Cygnus Audio Driver

2015-04-07 Thread Lori Hikichi
On 15-04-06 02:58 AM, Mark Brown wrote: On Fri, Apr 03, 2015 at 12:33:12PM -0700, Scott Branden wrote: On 15-03-30 11:43 PM, Mark Brown wrote: On Mon, Mar 30, 2015 at 08:16:22PM -0700, Scott Branden wrote: The audio PLL is embedded in the audio block and only used by the audio block. The a

Re: [alsa-devel] [PATCH 1/2] ASoC: cygnus-audio: adding device tree bindings

2015-04-02 Thread Lori Hikichi
On 15-03-31 12:26 AM, Lars-Peter Clausen wrote: On 03/31/2015 05:16 AM, Scott Branden wrote: [...] +- ssp-port-id: The ssp port interface to use +Valid value are 0, 1, 2, or 3 (for spdif) How about using 'reg' as the property name here. It is the standard property name for identifying or

Re: [PATCH 2/2] ASoC: add core audio driver for Broadcom Cygnus SOC.

2015-04-02 Thread Lori Hikichi
On 15-03-30 11:42 PM, Mark Brown wrote: On Mon, Mar 30, 2015 at 08:16:24PM -0700, Scott Branden wrote: The CC list for this patch is pretty wide - please look at who you're sending this to and try to send to only relevant people (for example I'm not sure the Raspberry Pi people need to review

Re: [PATCH 1/2] ASoC: cygnus-audio: adding device tree bindings

2015-04-02 Thread Lori Hikichi
On 15-03-30 10:58 PM, Mark Brown wrote: On Mon, Mar 30, 2015 at 08:16:23PM -0700, Scott Branden wrote: +SSP Subnode properties: +- dai-name: The name of the DAI registered with ASOC ASoC. Okay. Why is this in the DT - it sounds like this is just an internal implementation detail for Lin