2014-04-18 20:35:16

by Nicolas Ferre

[permalink] [raw]
Subject: Re: [PATCH 0/8] ARM: at91: sama5d3: enable sound support

On 17/03/2014 10:45, Bo Shen :
> This patch add sound support on sama5d3xek board.
>
> This patch series based on the following patch set:
> clk: at91: better support for the PCKs
> http://comments.gmane.org/gmane.linux.kernel/1664477
>
>
> Bo Shen (5):
> ARM: at91: sama5d3: add DMA property for SSC devices
> ARM: at91: sama5d3: disable sound by default
> ARM: at91: sama5d3: correct the sound compatible string
> ARM: at91: sama5d3: add the missing property
> ARM: at91: sama5d3: clock for ssc from rk pin
>
> Boris BREZILLON (3):
> ASoC: atmel: support CCF based clks
> ASoC: atmel: document clock properties of the wm8904 driver
> ARM: at91/dt: add clock properties to the wm8904 codec node

Hi Voice and Boris,

For this series I am a little bit puzzled: should I take the ASoC part
of it? I see no acknowledgement from Mark and I also read some messages
still discussing about CCF and wm8904 afterwards.

So, I take the 5 patches by Voice Shen and tell me what is the proper
sequence with this wm8904 work.

I will be happy to stack more patches about this topic during the 3.16
development phase.


Otherwise, the 5 patches by Voice go into at91-3.16-dt branch.

Thanks, bye.


> .../devicetree/bindings/sound/atmel-wm8904.txt | 2 ++
> arch/arm/boot/dts/sama5d3.dtsi | 6 ++++++
> arch/arm/boot/dts/sama5d3xmb.dtsi | 11 +++++++++-
> sound/soc/atmel/atmel_wm8904.c | 25 +++++++++++-----------
> 4 files changed, 31 insertions(+), 13 deletions(-)
>


--
Nicolas Ferre


2014-04-18 20:39:28

by Mark Brown

[permalink] [raw]
Subject: Re: [PATCH 0/8] ARM: at91: sama5d3: enable sound support

On Fri, Apr 18, 2014 at 10:35:02PM +0200, Nicolas Ferre wrote:

> For this series I am a little bit puzzled: should I take the ASoC part
> of it? I see no acknowledgement from Mark and I also read some messages
> still discussing about CCF and wm8904 afterwards.

I've not seen these patches, judging by where this message went they
weren't sent to the address in MAINTAINERS ([email protected]) but to
my work address instead.


Attachments:
(No filename) (429.00 B)
signature.asc (836.00 B)
Digital signature
Download all attachments

2014-04-18 22:06:31

by Boris BREZILLON

[permalink] [raw]
Subject: Re: [PATCH 0/8] ARM: at91: sama5d3: enable sound support


On 18/04/2014 22:35, Nicolas Ferre wrote:
> On 17/03/2014 10:45, Bo Shen :
>> This patch add sound support on sama5d3xek board.
>>
>> This patch series based on the following patch set:
>> clk: at91: better support for the PCKs
>> http://comments.gmane.org/gmane.linux.kernel/1664477
>>
>>
>> Bo Shen (5):
>> ARM: at91: sama5d3: add DMA property for SSC devices
>> ARM: at91: sama5d3: disable sound by default
>> ARM: at91: sama5d3: correct the sound compatible string
>> ARM: at91: sama5d3: add the missing property
>> ARM: at91: sama5d3: clock for ssc from rk pin
>>
>> Boris BREZILLON (3):
>> ASoC: atmel: support CCF based clks
>> ASoC: atmel: document clock properties of the wm8904 driver
>> ARM: at91/dt: add clock properties to the wm8904 codec node
> Hi Voice and Boris,
>
> For this series I am a little bit puzzled: should I take the ASoC part
> of it? I see no acknowledgement from Mark and I also read some messages
> still discussing about CCF and wm8904 afterwards.
>
> So, I take the 5 patches by Voice Shen and tell me what is the proper
> sequence with this wm8904 work.

Bo was waiting for the "move sam9n12 SoC to the CCF" (which has been
submitted earlier this week) series before sending a new version of his
"wm8904: add CCF support" series.

Best Regards,

Boris

> I will be happy to stack more patches about this topic during the 3.16
> development phase.
>
>
> Otherwise, the 5 patches by Voice go into at91-3.16-dt branch.
>
> Thanks, bye.
>
>
>> .../devicetree/bindings/sound/atmel-wm8904.txt | 2 ++
>> arch/arm/boot/dts/sama5d3.dtsi | 6 ++++++
>> arch/arm/boot/dts/sama5d3xmb.dtsi | 11 +++++++++-
>> sound/soc/atmel/atmel_wm8904.c | 25 +++++++++++-----------
>> 4 files changed, 31 insertions(+), 13 deletions(-)
>>
>

2014-04-18 22:11:51

by Nicolas Ferre

[permalink] [raw]
Subject: Re: [PATCH 0/8] ARM: at91: sama5d3: enable sound support

On 19/04/2014 00:06, Boris BREZILLON :
>
> On 18/04/2014 22:35, Nicolas Ferre wrote:
>> On 17/03/2014 10:45, Bo Shen :
>>> This patch add sound support on sama5d3xek board.
>>>
>>> This patch series based on the following patch set:
>>> clk: at91: better support for the PCKs
>>> http://comments.gmane.org/gmane.linux.kernel/1664477
>>>
>>>
>>> Bo Shen (5):
>>> ARM: at91: sama5d3: add DMA property for SSC devices
>>> ARM: at91: sama5d3: disable sound by default
>>> ARM: at91: sama5d3: correct the sound compatible string
>>> ARM: at91: sama5d3: add the missing property
>>> ARM: at91: sama5d3: clock for ssc from rk pin
>>>
>>> Boris BREZILLON (3):
>>> ASoC: atmel: support CCF based clks
>>> ASoC: atmel: document clock properties of the wm8904 driver
>>> ARM: at91/dt: add clock properties to the wm8904 codec node
>> Hi Voice and Boris,
>>
>> For this series I am a little bit puzzled: should I take the ASoC part
>> of it? I see no acknowledgement from Mark and I also read some messages
>> still discussing about CCF and wm8904 afterwards.
>>
>> So, I take the 5 patches by Voice Shen and tell me what is the proper
>> sequence with this wm8904 work.
>
> Bo was waiting for the "move sam9n12 SoC to the CCF" (which has been
> submitted earlier this week) series before sending a new version of his
> "wm8904: add CCF support" series.

Ok, that makes perfect sense.
So I have the whole dependency graph in mind now: we will have fun
stacking everything in the proper order ;-)

Thanks for the clarification, Bye.


>> I will be happy to stack more patches about this topic during the 3.16
>> development phase.
>>
>>
>> Otherwise, the 5 patches by Voice go into at91-3.16-dt branch.
>>
>> Thanks, bye.
>>
>>
>>> .../devicetree/bindings/sound/atmel-wm8904.txt | 2 ++
>>> arch/arm/boot/dts/sama5d3.dtsi | 6 ++++++
>>> arch/arm/boot/dts/sama5d3xmb.dtsi | 11 +++++++++-
>>> sound/soc/atmel/atmel_wm8904.c | 25 +++++++++++-----------
>>> 4 files changed, 31 insertions(+), 13 deletions(-)
>>>
>>
>
>
>


--
Nicolas Ferre