From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 642111CF2A3; Mon, 28 Oct 2024 10:50:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730112600; cv=none; b=BMJ44OHbzBy4BmUkYQMn/Uk/Yi7vuXjQ5ere5IFmCZMrjxufiFNHqabZmYb2o4qMGtPROSSzPCSjMFQ4r6XmmnwyKQepN0JFqBmrP7t0Ggp3H4GoDOSDq8tK+foM7mrlg3e8aEPr4Ijo5BFBIrsAtJpGTf2PjZINnS0IHV37ewU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730112600; c=relaxed/simple; bh=5gcR1znD1cGAtY6UrFbxboTgffk6OVdycQmErW5b3Qg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=N3KOWTatGrUSa18MG502us5umDmcQrteZ76yvL8uTBcfUP3lhNno/NFh/wdt1S1OTai0DyHixIc0X/hZaFY1OphklS2XwnAWU1ZZkrkENfSKFzOAo9slCm6LRtol6DxhONRy3QfBj27O33A0XnF48ekBY1yyzOTTgDTFI++vDKk= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=VGtDN4rC; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="VGtDN4rC" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6D388C4CEC3; Mon, 28 Oct 2024 10:49:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1730112599; bh=5gcR1znD1cGAtY6UrFbxboTgffk6OVdycQmErW5b3Qg=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=VGtDN4rCB75s5s0aUagy+xt8qdgqkWDO7bcJSQo6NCniEXIl6De1+xQqmvmZKTMNu 5pVmauCmMVuEgflzUrOepb/hIPUuq75qgoMLTlveFBuqPH96Btsbw0hhIqRvQW0cWi nkB+O7GV66nyRa7n0kUzql9jEPjVoB/Mk+JL/ToAybXnCb+Tm8TWc4Wmts02N7rcjw L3ns2Nn+j8o6kto65VgI862il+H/dP24X51QrlRMsGk3LHOLJcWcM7J3VT3M7a+9Ik bpBkonJL4OdxoMJMg03+WCJfi+QG5W3itXVJomovKxkqYjb+4CV8qhyJXWTjEJl7cX 7AoRu5HYBGSmw== Date: Mon, 28 Oct 2024 11:49:57 +0100 From: Maxime Ripard To: Dario Binacchi Cc: linux-kernel@vger.kernel.org, linux-amarula@amarulasolutions.com, Andrey Smirnov , Andrzej Hajda , Conor Dooley , David Airlie , Fabio Estevam , Inki Dae , Jagan Teki , Jernej Skrabec , Jessica Zhang , Jonas Karlman , Krzysztof Kozlowski , Krzysztof Kozlowski , Laurent Pinchart , Lucas Stach , Maarten Lankhorst , Marek Szyprowski , Marek Vasut , Michael Trimarchi , Neil Armstrong , Peng Fan , Pengutronix Kernel Team , Rob Herring , Robert Foss , Sascha Hauer , Shawn Guo , Shengjiu Wang , Simona Vetter , Stefan Agner , Thomas Zimmermann , Ulf Hansson , Uwe =?utf-8?Q?Kleine-K=C3=B6nig?= , devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org Subject: Re: [RFC PATCH 00/10] Support simple-framebuffer on imx8m Message-ID: <20241028-scrupulous-paper-condor-0742fb@houat> References: <20241028102559.1451383-1-dario.binacchi@amarulasolutions.com> Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha384; protocol="application/pgp-signature"; boundary="mpcp65bzqsipkaam" Content-Disposition: inline In-Reply-To: <20241028102559.1451383-1-dario.binacchi@amarulasolutions.com> --mpcp65bzqsipkaam Content-Type: text/plain; protected-headers=v1; charset=us-ascii Content-Disposition: inline Subject: Re: [RFC PATCH 00/10] Support simple-framebuffer on imx8m MIME-Version: 1.0 Hi, On Mon, Oct 28, 2024 at 11:25:23AM +0100, Dario Binacchi wrote: > This series is the Linux counterpart of what was sent to U-Boot [1] > for the support of the simple-framebuffer for the BSH SMM S2Pro board. I'm confused. simple-framebuffer is a mechanism for which the entire point is that the kernel doesn't need the driver for. Why do you need to have patches for bridges and panels for simple-framebuffer? They won't be used. > The need to avoid re-initializing the hardware (power domains, > controllers, bridges, display panels) that has already been initialized > and kept powered on by the bootloader has required updating more than > one YAML file, with the addition of boolean properties to inform the > driver that the corresponding hardware has been initialized and left > on by the bootloader. All these properties are added on the fly by the > bootloader to the various relevant nodes. So it's not about simple-framebuffer at all, but rather that you don't want to re-initialize the hardware that has already been setup? If so, this isn't how you should do it, but rather: https://lore.kernel.org/all/CAKMK7uHtqHy_oz4W7F+hmp9iqp7W5Ra8CxPvJ=9BwmvfU-O0gg@mail.gmail.com/ Maxime --mpcp65bzqsipkaam Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iJUEABMJAB0WIQTkHFbLp4ejekA/qfgnX84Zoj2+dgUCZx9sUAAKCRAnX84Zoj2+ dmtQAX9wVFF++uMLanShp1lqMmKbCAPRFsgHbuZ+e7vfogD2WBD/EF768xzbQaKx 8geIGicBgLRVa5ByCU/nhpBq9G9w4jzWcQ3tuYcJYH2sUP44PdFel9LS9HcmKc2G pooK6h9HIg== =7XGK -----END PGP SIGNATURE----- --mpcp65bzqsipkaam--