From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id AB91BEB64D7 for ; Mon, 26 Jun 2023 17:17:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=AU+5/w02u5ysEn+A8kiKPPkSOPPVpdg+fqXEbofybPI=; b=ljl0wXtx/9USZolwAUHE4bxKhR QtwuKHPXOjJW68Ynu0ywGxCOgZTxuw5eWRXJQT70bG5fGY73tpd9rSEbr7aSjef7LJmcKj4zxiCGD KQOZ63UHyfGlqMxjU0SYxpoD+/aYM+VRVytIicMTK2nnFb5jPTN3otGgGmcs5us4UrHgX1Z0yWhRA oTgh8O+7nu2rUPXJbnKsbvYnXa+WWRzgeMttHvcMEXzW1lRrz2vhtsb6XTc1SiLXkdzidiHV5WBNx l+p1QH+E/QiFzFr/vQG80CRbRXvKh9seBDGlmsKtazaQ00BnBrRNlVsQ48sj6fQe/mJ+DaEaRt+q2 +WCpcNPA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qDppa-00AjPW-0h; Mon, 26 Jun 2023 17:16:50 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qDppW-00AjOb-2q for linux-arm-kernel@lists.infradead.org; Mon, 26 Jun 2023 17:16:48 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 4377E60F0F; Mon, 26 Jun 2023 17:16:46 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 126D4C433C8; Mon, 26 Jun 2023 17:16:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1687799805; bh=8059eygYXUGESJrlTDcfhmLR43khYpXnsgBg/+/Am1Y=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WYi5nd0oIT09XyUPi7Ii818TZbuL+v152d/cmWR431S0REOjyDEaJ14zKrbDjm5gp AVJtkXWllIMRTKqN5f+WTlqcBXfSM/N61vPRC2XCjClr0CDD8Kl7FCCdqzs1P0UMzK 2hYVOvIO+7OoKVv3CgLWsJvuYtsdaLQ/lBCKHL48xg3dVBJCIN2jL9+rjlwxhVth+F uYjsrJLHRT2aODj+F7uS087TNgaHkRCdIaTXFmhN1ZAUnI8yNoV+C2YJHB68Xu4Tpc KBq9n98Y6oGusx7eLPaHn/81li34Hlo84KooOcFaV6ZAIvLQ+/b3tABL33CKm09+iX Ms+MJxroLigbA== Date: Mon, 26 Jun 2023 18:16:38 +0100 From: Conor Dooley To: Manikandan.M@microchip.com Subject: Re: [PATCH 1/9] dt-bindings: mfd: Add bindings for SAM9X7 LCD controller Message-ID: <20230626-coping-criteria-e5aa8b2ae212@spud> References: <20230613070426.467389-1-manikandan.m@microchip.com> <20230613070426.467389-2-manikandan.m@microchip.com> <20230613-slider-coherent-d508d67afc91@spud> <423eef3c-54f0-5c88-9bc9-82c0198b6da6@microchip.com> <3a3f4463-981a-e8d8-8ec4-06f2abfa3b4d@microchip.com> <1252fa40-dd8e-73f3-18b7-7010a7f8085e@microchip.com> MIME-Version: 1.0 In-Reply-To: <1252fa40-dd8e-73f3-18b7-7010a7f8085e@microchip.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230626_101646_994431_14AB118D X-CRM114-Status: GOOD ( 29.34 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: alexandre.belloni@bootlin.com, devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, Hari.PrasathGE@microchip.com, krzysztof.kozlowski+dt@linaro.org, Varshini.Rajendran@microchip.com, airlied@gmail.com, sam@ravnborg.org, Balamanikandan.Gunasundar@microchip.com, lee@kernel.org, Dharma.B@microchip.com, Nayabbasha.Sayed@microchip.com, conor+dt@kernel.org, robh+dt@kernel.org, Durai.ManickamKR@microchip.com, linux-arm-kernel@lists.infradead.org, Balakrishnan.S@microchip.com, bbrezillon@kernel.org, krzysztof.kozlowski@linaro.org, daniel@ffwll.ch, Claudiu.Beznea@microchip.com Content-Type: multipart/mixed; boundary="===============4983557486723612744==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============4983557486723612744== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="1j/2jjguTlX+Xhnw" Content-Disposition: inline --1j/2jjguTlX+Xhnw Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Jun 26, 2023 at 05:31:59AM +0000, Manikandan.M@microchip.com wrote: > On 21/06/23 13:17, Nicolas Ferre wrote: > > On 16/06/2023 at 08:44, Manikandan M - I67131 wrote: > >> On 14/06/23 20:10, Nicolas Ferre wrote: > >>> On 13/06/2023 at 20:21, Conor Dooley wrote: > >>>> On Tue, Jun 13, 2023 at 08:17:13PM +0200, Krzysztof Kozlowski wrote: > >>>>> On 13/06/2023 09:04, Manikandan Muralidharan wrote: > >>>>>> Add new compatible string for the XLCD controller on SAM9X7 SoC. > >>>>>> > >>>>>> Signed-off-by: Manikandan Muralidharan > >>>>>> --- > >>>>>> =A0=A0 Documentation/devicetree/bindings/mfd/atmel-hlcdc.txt | 1 + > >>>>>> =A0=A0 1 file changed, 1 insertion(+) > >>>>>> > >>>>>> diff --git a/Documentation/devicetree/bindings/mfd/atmel-hlcdc.txt > >>>>>> b/Documentation/devicetree/bindings/mfd/atmel-hlcdc.txt > >>>>>> index 5f8880cc757e..7c77b6bf4adb 100644 > >>>>>> --- a/Documentation/devicetree/bindings/mfd/atmel-hlcdc.txt > >>>>>> +++ b/Documentation/devicetree/bindings/mfd/atmel-hlcdc.txt > >>>>>> @@ -8,6 +8,7 @@ Required properties: > >>>>>> =A0=A0=A0=A0=A0 "atmel,sama5d3-hlcdc" > >>>>>> =A0=A0=A0=A0=A0 "atmel,sama5d4-hlcdc" > >>>>>> =A0=A0=A0=A0=A0 "microchip,sam9x60-hlcdc" > >>>>>> +=A0=A0 "microchip,sam9x7-xlcdc" > >>>>> Google says sam9x7 is a series, not a SoC. Please add compatibles f= or > >>>>> specific SoCs, not for series. > >>>> We had this one a few weeks ago, see > >>>> https://lore.kernel.org/all/add5e49e-8416-ba9f-819a-da944938c05f@mic= rochip.com/ > >>>> and its parents. Outcome of that seemed to be that using "sam9x7" was > >>>> fine. > >>> > >>> And it's where it begins to be funny, as the LCD is precisely one asp= ect > >>> where we differentiate between sam9x75, sam9x72 and sam9x70... > >>> So please Manikandan sort this out if difference between these chips > >>> will be better handled with different compatibility string, in > >>> particular about //, LVDS and MIPI-DSI variants! > >> Yes Sure, I will replace the compatible as s/sam9x7/sam9x75/g to handle > >> the different variants of sam9x7 better. > >=20 > > Moving to sam9x75 is probably good. But what is your plan for=20 > > differentiating parallel and LVDS (on sam9x72) and precisely this=20 > > sam9x75 variant which in addition has MIPI-DSI? > In sam9x75 with support for LVDS and MIPI, Parallel interfacing=20 > peripherals, the additions performed on the LCD controller driver is the= =20 > same.Considering the LCDC IP used in sam9x75, there are no registers=20 > sets that needs modification per connecting peripheral variants, only=20 > the clock and DRM_ENCODER_MODE_XXX (set by connecting peripheral driver)= =20 > differs, which can be handled in DT, atmel-lcdc MFD driver and=20 > peripheral driver. >=20 > In future, sam9x72 with XLCD controller can be differentiated with=20 > sam9x72 compatible string and with a IP version flag(is_xlcdc_v2) in its= =20 > driver data if an upgraded XLCD IP is used with difference in bits or=20 > register set exist compared to current IP. Trying to covert that into what the binding will look like... sam9x72 & sam9x75 each get their own compatibles for the lcd controller. =46rom there, we permit `compatible =3D "microchip,sam9x75-foo"` in isolation. It *sounds* like the basic featureset of the sam9x75 is compatible with the sam9x72, so for that we permit `compatible =3D "microchip,sam9x72-foo", "microchip,sam9x75-foo"`. Although, if the hardware for the sam9x72 isn't set in stone yet, it might be best to hold off on documenting it until things settle down, and only add the sam9x75 for now. Cheers, Conor. --1j/2jjguTlX+Xhnw Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZJnH9gAKCRB4tDGHoIJi 0vJeAQCKTNo8DxOHcmMtCltpTrHcnzIsZYChyQOAwNVVzXJhJwEAuZ+yPxkQmRc/ 3vViIUv3oFaK8ps5UbhrKrBYzz5BQA4= =3Eob -----END PGP SIGNATURE----- --1j/2jjguTlX+Xhnw-- --===============4983557486723612744== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============4983557486723612744==--