Re: [PATCH 0/3] drm: Allow simpledrm to setup its emulated FB as firmware provided

From: Javier Martinez Canillas <javierm@redhat.com>
To: Thomas Zimmermann <tzimmermann@suse.de>, linux-kernel@vger.kernel.org
Cc: "Alexandre Belloni" <alexandre.belloni@bootlin.com>,
	"Emma Anholt" <emma@anholt.net>,
	"Chun-Kuang Hu" <chunkuang.hu@kernel.org>,
	"David Airlie" <airlied@linux.ie>,
	spice-devel@lists.freedesktop.org,
	"Laurentiu Palcu" <laurentiu.palcu@oss.nxp.com>,
	"Liviu Dudau" <liviu.dudau@arm.com>,
	"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
	"Gurchetan Singh" <gurchetansingh@chromium.org>,
	"Paul Cercueil" <paul@crapouillou.net>,
	"Nirmoy Das" <nirmoy.das@amd.com>,
	"Gerd Hoffmann" <kraxel@redhat.com>,
	"Shawn Guo" <shawnguo@kernel.org>,
	linux-stm32@st-md-mailman.stormreply.com,
	"Jerome Brunet" <jbrunet@baylibre.com>,
	"Marek Vasut" <marex@denx.de>, "Hao Fang" <fanghao11@huawei.com>,
	linux-aspeed@lists.ozlabs.org,
	"Samuel Holland" <samuel@sholland.org>,
	"Kevin Hilman" <khilman@baylibre.com>,
	"Neil Armstrong" <narmstrong@baylibre.com>,
	"Alexey Brodkin" <abrodkin@synopsys.com>,
	"Michal Simek" <michal.simek@xilinx.com>,
	"Jernej Skrabec" <jernej.skrabec@gmail.com>,
	"Xinliang Liu" <xinliang.liu@linaro.org>,
	"Chen-Yu Tsai" <wens@csie.org>,
	"Noralf Trønnes" <noralf@tronnes.org>,
	"Joel Stanley" <joel@jms.id.au>,
	"Dave Airlie" <airlied@redhat.com>,
	"Xinwei Kong" <kong.kongxinwei@hisilicon.com>,
	virtualization@lists.linux-foundation.org,
	"NXP Linux Team" <linux-imx@nxp.com>,
	"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
	"Raphael Gallais-Pou" <raphael.gallais-pou@foss.st.com>,
	"Martin Blumenstingl" <martin.blumenstingl@googlemail.com>,
	"Chen Feng" <puck.chen@hisilicon.com>,
	"Sascha Hauer" <s.hauer@pengutronix.de>,
	"Alison Wang" <alison.wang@nxp.com>,
	"Lucas De Marchi" <lucas.demarchi@intel.com>,
	linux-mips@vger.kernel.org, "Sam Ravnborg" <sam@ravnborg.org>,
	"Hans de Goede" <hdegoede@redhat.com>,
	linux-mediatek@lists.infradead.org,
	dri-devel@lists.freedesktop.org,
	"Alain Volmat" <alain.volmat@foss.st.com>,
	"Matthias Brugger" <matthias.bgg@gmail.com>,
	linux-amlogic@lists.infradead.org,
	"Mario Limonciello" <mario.limonciello@amd.com>,
	"Evan Quan" <evan.quan@amd.com>,
	"Pengutronix Kernel Team" <kernel@pengutronix.de>,
	"Yong Wu" <yong.wu@mediatek.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-sunxi@lists.linux.dev, amd-gfx@lists.freedesktop.org,
	"Tomi Valkeinen" <tomba@kernel.org>,
	"Hyun Kwon" <hyun.kwon@xilinx.com>,
	"Boris Brezillon" <bbrezillon@kernel.org>,
	"Andrew Jeffery" <andrew@aj.id.au>,
	"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
	"Yannick Fertre" <yannick.fertre@foss.st.com>,
	"Pan, Xinhui" <Xinhui.Pan@amd.com>,
	"Nicolas Ferre" <nicolas.ferre@microchip.com>,
	"Claudiu Beznea" <claudiu.beznea@microchip.com>,
	linux-renesas-soc@vger.kernel.org,
	"Solomon Chiu" <solomon.chiu@amd.com>,
	"Kieran Bingham" <kieran.bingham+renesas@ideasonboard.com>,
	"Philippe Cornu" <philippe.cornu@foss.st.com>,
	"Alex Deucher" <alexander.deucher@amd.com>,
	"Tian Tao" <tiantao6@hisilicon.com>,
	"Jyri Sarha" <jyri.sarha@iki.fi>,
	"Christian König" <christian.koenig@amd.com>,
	"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>
Subject: Re: [PATCH 0/3] drm: Allow simpledrm to setup its emulated FB as firmware provided
Date: Mon, 2 May 2022 12:49:50 +0200	[thread overview]
Message-ID: <1d2a03f7-d4f6-66ac-6e2e-adbd2eaf7d90@redhat.com> (raw)
In-Reply-To: <c120e1c4-ac5c-afd5-8dd1-b4b51e0dcca9@suse.de>

Hello Thomas,

On 5/2/22 12:35, Thomas Zimmermann wrote:
> Hi Javier
> 
> Am 02.05.22 um 10:48 schrieb Javier Martinez Canillas:
>> Hello,
>>
>> This series contain patches suggested by Thomas Zimmermannas a feedback for

Ups, I missed a space here. I meant to write "Zimmermann as a feedback..."

>> "[RFC PATCH v4 00/11] Fix some race between sysfb device registration and
>> drivers probe" [0].
>>
>> Since other changes in [0] were more controversial, I decided to just split
>> this part in a new patch-set and revisit the rest of the patches later.
>>
>> Patch #1 is just a cleanup since when working on this noticed that some DRM
>> drivers were passing as preferred bits per pixel to drm_fbdev_generic_setup()
>> the value that is the default anyways.
>>
>> Patch #2 renames the 'preferred_bpp' drm_fbdev_generic_setup() parameter to
>> 'options', and make this a multi field parameter so that it can be extended
>> later to pass other options as well.
>>
>> Patch #3 finally adds the new DRM_FB_FW option and makes simpledrm to use it
>> so that the registered framebuffer device is also marked as firmware provided.
> 
> For the whole patchset:
> 
> Reviewed-by: Thomas Zimmermann <tzimmermann@suse.de>
> 
> Thanks a lot!
> 

Thanks for the prompt review!

-- 
Best regards,

Javier Martinez Canillas
Linux Engineering
Red Hat