lkml.org 
[lkml]   [2016]   [Jan]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] ASoC: fsl: add imx-cs427x machine driver
From
Date
Hi Rob,

On 26/01/16 02:29, Rob Herring wrote:
> On Mon, Jan 25, 2016 at 05:53:23PM +0000, Felipe F. Tonello wrote:
>> This is the initial imx-cs427x device-tree-only machine driver working with
>> fsl_ssi driver. More features can be added on top of it later.
>>
>> Signed-off-by: Felipe F. Tonello <eu@felipetonello.com>
>> ---
>> .../devicetree/bindings/sound/imx-audio-cs427x.txt | 47 +++++
>> sound/soc/fsl/Kconfig | 12 ++
>> sound/soc/fsl/Makefile | 2 +
>> sound/soc/fsl/imx-cs427x.c | 218 +++++++++++++++++++++
>> 4 files changed, 279 insertions(+)
>> create mode 100644 Documentation/devicetree/bindings/sound/imx-audio-cs427x.txt
>> create mode 100644 sound/soc/fsl/imx-cs427x.c
>>
>> diff --git a/Documentation/devicetree/bindings/sound/imx-audio-cs427x.txt b/Documentation/devicetree/bindings/sound/imx-audio-cs427x.txt
>> new file mode 100644
>> index 000000000000..295f60b19418
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/sound/imx-audio-cs427x.txt
>> @@ -0,0 +1,47 @@
>> +Freescale i.MX audio complex with CS4271 or CS4272 codec
>
> Do all the i.MX audio bindings really vary more that the codec? Seems
> like a lot of duplication.

I agree, but that how it is today. These platform drivers for imx are
similar but not identical. Looking at them I would guess that they have
50 to 60% of duplicated code.

I believe we can add this driver and work on re-using the code on a
future series of patches.

>
>> +
>> +Required properties:
>> +
>> + - compatible : "fsl,imx-audio-cs427x"
>> +
>> + - model : The user-visible name of this sound complex
>> +
>> + - ssi-controller : The phandle of the i.MX SSI controller
>> +
>> + - audio-codec : The phandle of the CS4271 audio codec
>> +
>> + - audio-routing : A list of the connections between audio components.
>> + Each entry is a pair of strings, the first being the
>> + connection's sink, the second being the connection's
>> + source. Valid names could be power supplies, CS427x
>> + pins, and the jacks on the board:
>> +
>> + Board connectors:
>> + * Mic Jack
>> + * Headphone Jack
>
> This should be an exact list of possible strings and valid combinations.

This is the exact list for this version of this driver. At least that is
what I can verify on my hardware (custom build).

>
>> + - mux-int-port : The internal port of the i.MX audio muxer (AUDMUX)
>> +
>> + - mux-ext-port : The external port of the i.MX audio muxer
>> +
>> +Note: The AUDMUX port numbering should start at 1, which is consistent with
>> +hardware manual.
>> +
>> +Example:
>> +
>> +sound {
>> + compatible = "fsl,imx6-rex-cs427x",
>> + "fsl,imx-audio-cs427x";
>> + model = "audio-cs427x";
>> + ssi-controller = <&ssi1>;
>> + audio-codec = <&cs4271>;
>> + audio-routing =
>> + "Mic Jack", "AINA",
>> + "Mic Jack", "AINB",
>> + "Headphone Jack", "AOUTA+",
>> + "Headphone Jack", "AOUTA-",
>> + "Headphone Jack", "AOUTB+",
>> + "Headphone Jack", "AOUTB-";
>> + mux-int-port = <1>;
>> + mux-ext-port = <3>;
>> +};

Felipe
[unhandled content-type:application/pgp-keys]
\
 
 \ /
  Last update: 2016-01-26 11:21    [W:1.003 / U:0.212 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site