On Wed, Dec 26, 2018 at 09:53:41AM +0900, Kuninori Morimoto wrote:
>
> From: Kuninori Morimoto <[email protected]>
>
> KingFisher has pcm3168 sound codec. This patch enables it.
> Because pcm3168 can't handle symmetric channel on playback/
> capture, we need to handle it as different DAI.
Hi Morimoto-san,
as per my response to other patches in this series I have found,
hopefully, minor problems, when applying this patch.
When compiling I see:
# make W=1 dtbs
...
DTC arch/arm64/boot/dts/renesas/r8a7795-h3ulcb.dtb
DTC arch/arm64/boot/dts/renesas/r8a7795-h3ulcb-kf.dtb
arch/arm64/boot/dts/renesas/r8a7795-h3ulcb-kf.dtb: Warning
(unit_address_vs_reg):
/soc/i2c@e6510000/i2c-switch@71/i2c@7/audio-codec@44/ports/port@0: node has a
unit name, but no reg property
arch/arm64/boot/dts/renesas/r8a7795-h3ulcb-kf.dtb: Warning
(unit_address_vs_reg):
/soc/i2c@e6510000/i2c-switch@71/i2c@7/audio-codec@44/ports/port@1: node has a
unit name, but no reg property
arch/arm64/boot/dts/renesas/r8a7795-h3ulcb-kf.dtb: Warning
(unit_address_vs_reg): /soc/sound@ec500000/ports/port@2: node has a unit name,
but no reg property
arch/arm64/boot/dts/renesas/r8a7795-h3ulcb-kf.dtb: Warning
(unit_address_vs_reg): /soc/sound@ec500000/ports/port@3: node has a unit name,
but no reg property
DTC arch/arm64/boot/dts/renesas/r8a7795-es1-h3ulcb.dtb
DTC arch/arm64/boot/dts/renesas/r8a7795-es1-h3ulcb-kf.dtb
arch/arm64/boot/dts/renesas/r8a7795-es1-h3ulcb-kf.dtb: Warning
(unit_address_vs_reg):
/soc/i2c@e6510000/i2c-switch@71/i2c@7/audio-codec@44/ports/port@0: node has a
unit name, but no reg property
arch/arm64/boot/dts/renesas/r8a7795-es1-h3ulcb-kf.dtb: Warning
(unit_address_vs_reg):
/soc/i2c@e6510000/i2c-switch@71/i2c@7/audio-codec@44/ports/port@1: node has a
unit name, but no reg property
arch/arm64/boot/dts/renesas/r8a7795-es1-h3ulcb-kf.dtb: Warning
(unit_address_vs_reg): /soc/sound@ec500000/ports/port@2: node has a unit name,
but no reg property
arch/arm64/boot/dts/renesas/r8a7795-es1-h3ulcb-kf.dtb: Warning
(unit_address_vs_reg): /soc/sound@ec500000/ports/port@3: node has a unit name,
but no reg property
DTC arch/arm64/boot/dts/renesas/r8a7796-m3ulcb.dtb
DTC arch/arm64/boot/dts/renesas/r8a7796-m3ulcb-kf.dtb
arch/arm64/boot/dts/renesas/r8a7796-m3ulcb-kf.dtb: Warning
(unit_address_vs_reg):
/soc/i2c@e6510000/i2c-switch@71/i2c@7/audio-codec@44/ports/port@0: node has a
unit name, but no reg property
arch/arm64/boot/dts/renesas/r8a7796-m3ulcb-kf.dtb: Warning
(unit_address_vs_reg):
/soc/i2c@e6510000/i2c-switch@71/i2c@7/audio-codec@44/ports/port@1: node has a
unit name, but no reg property
arch/arm64/boot/dts/renesas/r8a7796-m3ulcb-kf.dtb: Warning
(unit_address_vs_reg): /soc/sound@ec500000/ports/port@2: node has a unit name,
but no reg property
arch/arm64/boot/dts/renesas/r8a7796-m3ulcb-kf.dtb: Warning
(unit_address_vs_reg): /soc/sound@ec500000/ports/port@3: node has a unit name,
but no reg property
DTC arch/arm64/boot/dts/renesas/r8a77965-m3nulcb.dtb
DTC arch/arm64/boot/dts/renesas/r8a77965-m3nulcb-kf.dtb
arch/arm64/boot/dts/renesas/r8a77965-m3nulcb-kf.dtb: Warning
(unit_address_vs_reg):
/soc/i2c@e6510000/i2c-switch@71/i2c@7/audio-codec@44/ports/port@0: node has a
unit name, but no reg property
arch/arm64/boot/dts/renesas/r8a77965-m3nulcb-kf.dtb: Warning
(unit_address_vs_reg):
/soc/i2c@e6510000/i2c-switch@71/i2c@7/audio-codec@44/ports/port@1: node has a
unit name, but no reg property
arch/arm64/boot/dts/renesas/r8a77965-m3nulcb-kf.dtb: Warning
(unit_address_vs_reg): /soc/sound@ec500000/ports/port@2: node has a unit name,
but no reg property
arch/arm64/boot/dts/renesas/r8a77965-m3nulcb-kf.dtb: Warning
(unit_address_vs_reg): /soc/sound@ec500000/ports/port@3: node has a unit name,
but no reg property
Locally the following seems to resolve this problem.
Please consider rolling this incremental change or another fix
into a v2 posting of this patchset.
diff --git a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
index 401c166599a2..5cfa193fa8a9 100644
--- a/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
+++ b/arch/arm64/boot/dts/renesas/ulcb-kf.dtsi
@@ -163,14 +163,22 @@
VCCDA2-supply = <&snd_vcc5v>;
ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
mclk-fs = <512>;
+
port@0 {
+ reg = <0>;
+
pcm3168a_endpoint_p: endpoint {
remote-endpoint =
<&rsnd_for_pcm3168a_play>;
clocks = <&clksndsel>;
};
};
+
port@1 {
+ reg = <1>;
+
pcm3168a_endpoint_c: endpoint {
remote-endpoint =
<&rsnd_for_pcm3168a_capture>;
clocks = <&clksndsel>;
@@ -294,8 +302,13 @@
&sound_pcm_pins>;
ports {
+ #address-cells = <1>;
+ #size-cells = <0>;
+
/* rsnd_port0/1 are on salvator-common */
rsnd_port2: port@2 {
+ reg = <2>;
+
rsnd_for_pcm3168a_play: endpoint {
remote-endpoint = <&pcm3168a_endpoint_p>;
@@ -307,7 +320,10 @@
playback = <&ssi3>;
};
};
+
rsnd_port3: port@3 {
+ reg = <3>;
+
rsnd_for_pcm3168a_capture: endpoint {
remote-endpoint = <&pcm3168a_endpoint_c>;
