V2AS问路
[V5] ARM: dts: Change i2s compatible string on exynos5250【转】
阅读原文时间：2023年07月14日阅读：2

本文转载自:https://patchwork.kernel.org/patch/2845464/

Padmavathi VennaAug. 16, 2013, 4:26 a.m. UTC

This patch removes quirks from i2s node and change the i2s
compatible names.

Signed-off-by: Padmavathi Venna padma.v@samsung.com

Changes since V4:
- Mark Brown reverted the below two patches due to below build failure with
exynos_defconfig.
"ARM: dts: exynos5250: move common i2s properties to exynos5 dtsi"
"ARM: dts: Change i2s compatible string on exynos5250"

  build error:  
  DTC     arch/arm/boot/dts/exynos5420-smdk5420.dtb  
  ERROR (phandle\_references): Reference to non-existent node or label "pdma1"  
  ERROR (phandle\_references): Reference to non-existent node or label "pdma1"  
  ERROR (phandle\_references): Reference to non-existent node or label "pdma0"  
  ERROR (phandle\_references): Reference to non-existent node or label "pdma0"

But with out "ARM: dts: Change i2s compatible string on exynos5250" will break the i2s
driver.

So posting this patch now and will take care of posting other patch later after dependent
patches got merged.

arch/arm/boot/dts/exynos5250.dtsi | 9 +++------
1 files changed, 3 insertions(+), 6 deletions(-)

Olof JohanssonAug. 16, 2013, 4:48 a.m. UTC | #1

On Thu, Aug 15, 2013 at 9:26 PM, Padmavathi Venna padma.v@samsung.com wrote:

This patch removes quirks from i2s node and change the i2s
compatible names.

Signed-off-by: Padmavathi Venna padma.v@samsung.com

Changes since V4:
- Mark Brown reverted the below two patches due to below build failure with
exynos_defconfig.
"ARM: dts: exynos5250: move common i2s properties to exynos5 dtsi"
"ARM: dts: Change i2s compatible string on exynos5250" 

      build error:  
      DTC     arch/arm/boot/dts/exynos5420-smdk5420.dtb  
      ERROR (phandle\_references): Reference to non-existent node or label "pdma1"  
      ERROR (phandle\_references): Reference to non-existent node or label "pdma1"  
      ERROR (phandle\_references): Reference to non-existent node or label "pdma0"  
      ERROR (phandle\_references): Reference to non-existent node or label "pdma0"

But with out "ARM: dts: Change i2s compatible string on exynos5250" will break the i2s
driver.

So posting this patch now and will take care of posting other patch later after dependent
patches got merged.

arch/arm/boot/dts/exynos5250.dtsi | 9 +++------
1 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
index ef57277..376090f 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -405,7 +405,7 @@
}; 

    i2s0: i2s@03830000 {
  • compatible = "samsung,i2s-v5";
  • compatible = "samsung,s5pv210-i2s";

Device tree reviewers, this is something to look out for in the
future. Some samsung platforms/drivers use "samsung,-",
others "samsung,-". I don't personally care much one way or
another, but it really should be consistent.

-Olof

To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

kgene@kernel.orgAug. 16, 2013, 5:43 a.m. UTC | #2

Olof Johansson wrote:

On Thu, Aug 15, 2013 at 9:26 PM, Padmavathi Venna padma.v@samsung.com
wrote:

This patch removes quirks from i2s node and change the i2s
compatible names.

Signed-off-by: Padmavathi Venna padma.v@samsung.com

Changes since V4:
- Mark Brown reverted the below two patches due to below build
failure with
exynos_defconfig.
"ARM: dts: exynos5250: move common i2s properties to
exynos5
dtsi"
"ARM: dts: Change i2s compatible string on exynos5250" 

      build error:  
      DTC     arch/arm/boot/dts/exynos5420-smdk5420.dtb  
      ERROR (phandle\_references): Reference to non-existent node or

label "pdma1"
ERROR (phandle_references): Reference to non-existent node or
label "pdma1"
ERROR (phandle_references): Reference to non-existent node or
label "pdma0"
ERROR (phandle_references): Reference to non-existent node or
label "pdma0"

But with out "ARM: dts: Change i2s compatible string on exynos5250" will
break the i2s
driver.

So posting this patch now and will take care of posting other patch
later after dependent
patches got merged.

arch/arm/boot/dts/exynos5250.dtsi | 9 +++------
1 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5250.dtsi
b/arch/arm/boot/dts/exynos5250.dtsi
index ef57277..376090f 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -405,7 +405,7 @@
}; 

    i2s0: i2s@03830000 {
  • compatible = "samsung,i2s-v5";
  • compatible = "samsung,s5pv210-i2s";

Device tree reviewers, this is something to look out for in the
future. Some samsung platforms/drivers use "samsung,-",
others "samsung,-". I don't personally care much one way or
another, but it really should be consistent.

Hmm…I think, if "samsung,-" is possible, it would be
nice. I remember there are no versions in datasheet for some IPs but
something have like i2s and mfc. So "samsung,-" is used for
only i2s and mfc. But actually there are versions for Samsung IPs, no
comments for that in datasheet. So I think, if Samsung can provide the
specific version of Samsung IPs, we can use that like other platforms. I
will prepare some table for that after meeting with Samsung hardware IP team
so that samsung platform use one format "samsung,-".

Thanks,
Kukjin

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

Tomasz FigaAug. 16, 2013, 7:48 a.m. UTC | #3

Hi Olof,

On Thursday 15 of August 2013 21:48:32 Olof Johansson wrote:

On Thu, Aug 15, 2013 at 9:26 PM, Padmavathi Venna padma.v@samsung.com
wrote:

This patch removes quirks from i2s node and change the i2s
compatible names.

Signed-off-by: Padmavathi Venna padma.v@samsung.com

Changes since V4:
- Mark Brown reverted the below two patches due to below build
failure with>
exynos_defconfig. 

            "ARM: dts: exynos5250: move common i2s properties to  
            exynos5 dtsi"  
            "ARM: dts: Change i2s compatible string on exynos5250"  

      build error:  
      DTC     arch/arm/boot/dts/exynos5420-smdk5420.dtb  
      ERROR (phandle\_references): Reference to non-existent node  
      or label "pdma1" ERROR (phandle\_references): Reference to  
      non-existent node or label "pdma1" ERROR  
      (phandle\_references): Reference to non-existent node or  
      label "pdma0" ERROR (phandle\_references): Reference to  
      non-existent node or label "pdma0">

But with out "ARM: dts: Change i2s compatible string on exynos5250"
will break the i2s driver.

So posting this patch now and will take care of posting other patch
later after dependent patches got merged.

arch/arm/boot/dts/exynos5250.dtsi | 9 +++------
1 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5250.dtsi
b/arch/arm/boot/dts/exynos5250.dtsi index ef57277..376090f 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -405,7 +405,7 @@ 

    };  

    i2s0: i2s@03830000 {
  • compatible = "samsung,i2s-v5";
  • compatible = "samsung,s5pv210-i2s";

Device tree reviewers, this is something to look out for in the
future. Some samsung platforms/drivers use "samsung,-",
others "samsung,-". I don't personally care much one way or
another, but it really should be consistent.

Hmm, I just did a quick git grep over arch/arm/boot/dts and could not
really find the "samsung,-" pattern.

Except MFC (and without Padma's patch also I2S), which used "samsung,-
, all the compatible values I can see are based on
"samsung,-" pattern.

I believe we discussed this issue several times already and finally agreed
to use "samsung,-" for any new compatible values.

Best regards,
Tomasz

Tomasz FigaAug. 16, 2013, 7:53 a.m. UTC | #4

Hi Kukjin,

On Friday 16 of August 2013 14:43:17 Kukjin Kim wrote:

Olof Johansson wrote:

On Thu, Aug 15, 2013 at 9:26 PM, Padmavathi Venna
padma.v@samsung.com

wrote:

This patch removes quirks from i2s node and change the i2s
compatible names.

Signed-off-by: Padmavathi Venna padma.v@samsung.com

Changes since V4:
- Mark Brown reverted the below two patches due to below
build

failure with 

      exynos\_defconfig.  

            "ARM: dts: exynos5250: move common i2s properties to

exynos5

dtsi" 

            "ARM: dts: Change i2s compatible string on  
            exynos5250"  

      build error:  
      DTC     arch/arm/boot/dts/exynos5420-smdk5420.dtb  
      ERROR (phandle\_references): Reference to non-existent node  
      or

label "pdma1" 

      ERROR (phandle\_references): Reference to non-existent node  
      or

label "pdma1" 

      ERROR (phandle\_references): Reference to non-existent node  
      or

label "pdma0" 

      ERROR (phandle\_references): Reference to non-existent node  
      or

label "pdma0"

But with out "ARM: dts: Change i2s compatible string on exynos5250"
will>
break the i2s

driver.

So posting this patch now and will take care of posting other patch

later after dependent

patches got merged.

arch/arm/boot/dts/exynos5250.dtsi | 9 +++------
1 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5250.dtsi

b/arch/arm/boot/dts/exynos5250.dtsi

index ef57277..376090f 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -405,7 +405,7 @@ 

    };  

    i2s0: i2s@03830000 {
  • compatible = "samsung,i2s-v5";
  • compatible = "samsung,s5pv210-i2s";

Device tree reviewers, this is something to look out for in the
future. Some samsung platforms/drivers use "samsung,-",
others "samsung,-". I don't personally care much one way or
another, but it really should be consistent.

Hmm…I think, if "samsung,-" is possible, it would
be nice. I remember there are no versions in datasheet for some IPs but
something have like i2s and mfc. So "samsung,-" is used
for only i2s and mfc. But actually there are versions for Samsung IPs,
no comments for that in datasheet. So I think, if Samsung can provide
the specific version of Samsung IPs, we can use that like other
platforms. I will prepare some table for that after meeting with
Samsung hardware IP team so that samsung platform use one format
"samsung,-".

Sorry, I don't think this is a good idea, unless you can force the IP team
to release a version table containing version of _every_ IP for _every_
released SoC, including those historical ones, like S3C24xx and S3C64xx.
Of course such tables should be available publicly.

In addition, there might be other funny things going on with IPs and their
surroundings, that could make a need to create several separate compatible
values for the same IP revision, but on different SoCs, because it was
integrated in a slightly different way.

I believe we have choses the "samsung,-" scheme to avoid being
dependent upon data that is not always publicly available, which is more
future- (and past-) proof and also solves the integration problem.

Best regards,
Tomasz

Mark BrownAug. 16, 2013, 9:59 a.m. UTC | #5

On Fri, Aug 16, 2013 at 09:53:23AM +0200, Tomasz Figa wrote:

Sorry, I don't think this is a good idea, unless you can force the IP team
to release a version table containing version of _every_ IP for _every_
released SoC, including those historical ones, like S3C24xx and S3C64xx.
Of course such tables should be available publicly.

I don't think public availability is much of an issue here, the
datasheets are all NDAed anyway.

Mark BrownAug. 16, 2013, 10:06 a.m. UTC | #6

On Fri, Aug 16, 2013 at 09:56:18AM +0530, Padmavathi Venna wrote:

This patch removes quirks from i2s node and change the i2s
compatible names.

Applied, thanks.

Kim KukjinAug. 18, 2013, 6:05 p.m. UTC | #7

On 08/16/13 16:53, Tomasz Figa wrote:

Hi Kukjin,

[…]

  • compatible = "samsung,i2s-v5";
  • compatible = "samsung,s5pv210-i2s";

Device tree reviewers, this is something to look out for in the
future. Some samsung platforms/drivers use "samsung,-",
others "samsung,-". I don't personally care much one way or
another, but it really should be consistent.

Hmm…I think, if "samsung,-" is possible, it would
be nice. I remember there are no versions in datasheet for some IPs but
something have like i2s and mfc. So "samsung,-" is used
for only i2s and mfc. But actually there are versions for Samsung IPs,
no comments for that in datasheet. So I think, if Samsung can provide
the specific version of Samsung IPs, we can use that like other
platforms. I will prepare some table for that after meeting with
Samsung hardware IP team so that samsung platform use one format
"samsung,-".

Sorry, I don't think this is a good idea, unless you can force the IP team
to release a version table containing version of _every_ IP for _every_
released SoC, including those historical ones, like S3C24xx and S3C64xx.
Of course such tables should be available publicly.

It's different issue and I agree with Mark Brown's comments.

In addition, there might be other funny things going on with IPs and their
surroundings, that could make a need to create several separate compatible
values for the same IP revision, but on different SoCs, because it was
integrated in a slightly different way.

I believe we have choses the "samsung,-" scheme to avoid being
dependent upon data that is not always publicly available, which is more
future- (and past-) proof and also solves the integration problem.

Well, I don't think so, because Samsung is no more just SoC vendor.
Actually Samsung is providing just IP to customers, I think, they don't
like to use the name, exynos**** for their SoC or Chip, because exynos,
s3c or s5p whatever is Samsung's SoC brand name. If so, I think, my
suggestion is more reasonable. See the example of Synopsys and ARM
primecell…that's why I will try to do it. But I know I need to get
agreement from DT guys and maybe I need more time than I expect ;-)

Thanks,

Kukjin

To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html

2845464diffmbox

diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi
index ef57277..376090f 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -405,7 +405,7 @@
};

 i2s0: i2s@03830000 {

  • compatible = "samsung,i2s-v5";

  • compatible = "samsung,s5pv210-i2s";
    reg = <0x03830000 0x100>;
    dmas = <&pdma0 10 &pdma0 9 @@ -415,16 +415,13 @@ <&clock_audss EXYNOS_I2S_BUS>,
    <&clock_audss EXYNOS_SCLK_I2S>;
    clock-names = "iis", "i2s_opclk0", "i2s_opclk1";

  • samsung,supports-6ch;

  • samsung,supports-rstclr;

  • samsung,supports-secdai;
    samsung,idma-addr = <0x03000000>;
    pinctrl-names = "default";
    pinctrl-0 = <&i2s0_bus>;
    };

    i2s1: i2s@12D60000 {

  • compatible = "samsung,i2s-v5";

  • compatible = "samsung,s3c6410-i2s";
    reg = <0x12D60000 0x100>;
    dmas = <&pdma1 12 &pdma1 11>;
    @@ -436,7 +433,7 @@
    };

    i2s2: i2s@12D70000 {

  • compatible = "samsung,i2s-v5";

  • compatible = "samsung,s3c6410-i2s";
    reg = <0x12D70000 0x100>;
    dmas = <&pdma0 12 &pdma0 11>;

手机扫一扫

移动阅读更方便

阿里云服务器
腾讯云服务器
七牛云服务器

你可能感兴趣的文章