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 DF684EB64D9 for ; Mon, 19 Jun 2023 20:19:03 +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: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc: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=iCnye02UblnO7+k5k+viz17Eo6aq5Ko1iuieUlX5b1o=; b=grczFDY+6mNGM6tOwXZgGGuBBF WET3s08jxrL2rm18EsTbsm3RXjFpjbjHjkjlI+tBlAgDZ8vm23FGG4VRKw+RrOwQFHKSAXKS0el0U kM6k/gQt0pYNEq9zUFu5LKI933CS/BphuyjM/wEPkBbLK562GGl4pa/PypgSi9BbMPr6Ai1/39RnE BGCYYn+XhYAcwWDuuyigBEA4jIovntSKJowTM+7orv6TgQZ2pN2R8eTvxnJ0VjSo+bDddmBs3wssK mnG/HIsoBon73wDWe8ZqomCLVDuSGvExtGdbUYuDjX9EJgmCYcXHEqVU00Sr2cup0YQyARlQGFV2f EGzJpycw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qBLKj-009PRJ-0Y; Mon, 19 Jun 2023 20:18:41 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qBLKb-009PN1-13 for linux-arm-kernel@lists.infradead.org; Mon, 19 Jun 2023 20:18:34 +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 C7C8260E99; Mon, 19 Jun 2023 20:18:31 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id BDFB9C433C8; Mon, 19 Jun 2023 20:18:27 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1687205911; bh=HhghJ0yctP0ql16cZYZqxzheR0DW80MolXe2adxh4PI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=JFJ28e8C/9xbrFv+nFHoy99cXRAzDYp/c1J1kHsrsr0fK0UZGoFs1GeQTFLH5GFBe LGYTdHv/yg8aDWvwuBCREEXjxpv/EyT7i08ksdgAFpWcP+8l9Rjx++O88/5udK+kZJ Hknr4Rkn9Go5AsgmBzjjfpsqtuoKiT57cVyAxVeR/jJESFWk5NwbXME3/XHfa4JmV+ TgcKts6kxfDCyi67QVuFaEJi8tD2hJ2EIiTYe6AyAyDLvpsB5BUcgVp3PX5VS4R0Ej 6/oKCF9bnEFeuC2anUCBG7v60rG1tJuj+gGt39p8EDCWvnjMrQDoUW975N2KrNHxla o3kP7IsfJdEPw== Date: Mon, 19 Jun 2023 21:18:25 +0100 From: Conor Dooley To: Dario Binacchi Cc: linux-kernel@vger.kernel.org, Philippe Cornu , Raphael Gallais-Pou , Amarula patchwork , michael@amarulasolutions.com, Alexandre Torgue , Conor Dooley , Daniel Vetter , David Airlie , Krzysztof Kozlowski , Maxime Coquelin , Rob Herring , Yannick Fertre , devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com Subject: Re: [PATCH v4 4/6] dt-bindings: display: stm32-ltdc: add optional st,fb-bpp property Message-ID: <20230619-ion-decree-c63d2eb11e83@spud> References: <20230619165525.1035243-1-dario.binacchi@amarulasolutions.com> <20230619165525.1035243-5-dario.binacchi@amarulasolutions.com> MIME-Version: 1.0 In-Reply-To: <20230619165525.1035243-5-dario.binacchi@amarulasolutions.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230619_131833_444517_11BEAD98 X-CRM114-Status: GOOD ( 19.03 ) 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: , Content-Type: multipart/mixed; boundary="===============2027044828183274896==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============2027044828183274896== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="wB3prgNFdxLAzRiP" Content-Disposition: inline --wB3prgNFdxLAzRiP Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hey, On Mon, Jun 19, 2023 at 06:55:23PM +0200, Dario Binacchi wrote: > Boards that use the STM32F{4,7} series have limited amounts of RAM. The > added property allows to size, within certain limits, the memory footprint > required by the framebuffer. Hmm, this sounds quite a lot like "software policy", since the actual display doesn't have these limitations. Rob, Krzysztof? >=20 > Signed-off-by: Dario Binacchi > --- >=20 > (no changes since v1) Really? https://lore.kernel.org/all/?q=3Ddfn:st,stm32-ltdc.yaml%20 You sure this shouldn't be "new in v4"? > .../devicetree/bindings/display/st,stm32-ltdc.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) >=20 > diff --git a/Documentation/devicetree/bindings/display/st,stm32-ltdc.yaml= b/Documentation/devicetree/bindings/display/st,stm32-ltdc.yaml > index d6ea4d62a2cf..1c3a3653579f 100644 > --- a/Documentation/devicetree/bindings/display/st,stm32-ltdc.yaml > +++ b/Documentation/devicetree/bindings/display/st,stm32-ltdc.yaml > @@ -42,6 +42,12 @@ properties: > - for internal dpi input of the MIPI DSI host controller. > Note: These 2 endpoints cannot be activated simultaneously. > =20 > + st,fb-bpp: Is there not a more understandable property name than this? Maybe I just had to think about it because fbdev stuff aint something I've worked with... > + $ref: /schemas/types.yaml#/definitions/uint32 > + description: | > + bit depth of framebuffer (8, 16 or 32) > + maxItems: 1 Why not make it an enum, since there are only 4 values? Cheers, Conor. > + > required: > - compatible > - reg > --=20 > 2.32.0 >=20 --wB3prgNFdxLAzRiP Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZJC4EQAKCRB4tDGHoIJi 0oPpAP92h4xspKGtI1yHs7XjwoYCJXDpWoGzANgb0IX7QDSxWQEA+mwt+9KLghOY jMdSLkmbkdR8P3BWLJOmAnP6AsLtKQQ= =IYJt -----END PGP SIGNATURE----- --wB3prgNFdxLAzRiP-- --===============2027044828183274896== 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 --===============2027044828183274896==--