From mboxrd@z Thu Jan 1 00:00:00 1970 From: Krzysztof Kozlowski Subject: Re: [PATCH 4/5] ASoC: samsung: Document compatible strings for Odroid X/X2, Odroid XU3 Date: Mon, 19 Mar 2018 11:57:40 +0100 Message-ID: References: <20180314164242.22927-1-s.nawrocki@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by alsa0.perex.cz (Postfix) with ESMTP id 958AA2672C0 for ; Mon, 19 Mar 2018 11:57:43 +0100 (CET) Received: from mail-wm0-f49.google.com (mail-wm0-f49.google.com [74.125.82.49]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 35807217D9 for ; Mon, 19 Mar 2018 10:57:42 +0000 (UTC) Received: by mail-wm0-f49.google.com with SMTP id x82so1893440wmg.1 for ; Mon, 19 Mar 2018 03:57:42 -0700 (PDT) In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Sylwester Nawrocki Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, linux-samsung-soc@vger.kernel.org, =?UTF-8?B?QmFydMWCb21pZWogxbtvxYJuaWVya2lld2ljeg==?= , sbkim73@samsung.com, broonie@kernel.org, lgirdwood@gmail.com, robh+dt@kernel.org, Marek Szyprowski List-Id: alsa-devel@alsa-project.org On Mon, Mar 19, 2018 at 11:17 AM, Sylwester Nawrocki wrote: > On 03/16/2018 08:37 AM, Krzysztof Kozlowski wrote: >>> diff --git a/Documentation/devicetree/bindings/sound/samsung,odroid.txt b/Documentation/devicetree/bindings/sound/samsung,odroid.txt >>> index f35a2694eb04..a50a0c6a354b 100644 >>> --- a/Documentation/devicetree/bindings/sound/samsung,odroid.txt >>> +++ b/Documentation/devicetree/bindings/sound/samsung,odroid.txt >>> @@ -1,8 +1,10 @@ >>> -Samsung Exynos Odroid XU3/XU4 audio complex with MAX98090 codec >>> +Hardkernel Odroid X/X2/U3/XU3/XU4 audio complex with MAX98090 codec >>> >>> Required properties: >>> >>> - - compatible - "hardkernel,odroid-xu3-audio" - for Odroid XU3 board, >>> + - compatible - "hardkernel,odroid-u3-audio" - for Odroid U3 board, >>> + "hardkernel,odroid-x-audio" - for Odroid X, Odroid X2 boards, >>> + "hardkernel,odroid-xu3-audio" - for Odroid XU3 board, >>> "hardkernel,odroid-xu4-audio" - for Odroid XU4 board >> >> These are the same so I think they should not get separate >> compatibles. By definition [2] the compatible expresses the >> compatibility with family of devices. In this case - these belong to >> the same family. >> >> The same applies to patch 5/5 (implementation of this). > > Given that everything except XU4 is already obsoleted, are we going to use > "hardkernel,odroid-xu3-audio" for Odroid X and Odroid U3? > > We could also make it: > "hardkernel,odroid-xu3-audio", "hardkernel,odroid-x-audio" - for Odroid X, Odroid X2 > "hardkernel,odroid-xu3-audio", "hardkernel,odroid-u3-audio" - for Odroid U3 > > And in the driver wee would only have "hardkernel,odroid-xu3-audio", while dts > would contain detailed compatible. Add only these compatibles which are different. If the binding is the same, then there is no point of adding new compatible. If such difference in binding appears, then it is not a problem to add new compatible. It is however more difficult to remove or change existing binding... Best regards, Krzysztof