Re: [PATCH V5] ARM: dts: Change i2s compatible string on exynos5250

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Aug 15, 2013 at 9:26 PM, Padmavathi Venna <padma.v@xxxxxxxxxxx> wrote:
> This patch removes quirks from i2s node and change the i2s
> compatible names.
>
> Signed-off-by: Padmavathi Venna <padma.v@xxxxxxxxxxx>
> ---
>
> 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,<chip>-<ip>",
others "samsung,<ip>-<chip>". 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux