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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 D6519C0015E for ; Mon, 19 Jun 2023 20:18:36 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id E2DB810E0BD; Mon, 19 Jun 2023 20:18:35 +0000 (UTC) Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by gabe.freedesktop.org (Postfix) with ESMTPS id 0C4C510E0BD for ; Mon, 19 Jun 2023 20:18:33 +0000 (UTC) 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 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 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="wB3prgNFdxLAzRiP" Content-Disposition: inline In-Reply-To: <20230619165525.1035243-5-dario.binacchi@amarulasolutions.com> X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Maxime Coquelin , Conor Dooley , Krzysztof Kozlowski , Raphael Gallais-Pou , Yannick Fertre , Alexandre Torgue , dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Rob Herring , Philippe Cornu , michael@amarulasolutions.com, Amarula patchwork , linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" --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--