[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c120e1c4-ac5c-afd5-8dd1-b4b51e0dcca9@suse.de>
Date: Mon, 2 May 2022 12:35:29 +0200
From: Thomas Zimmermann <tzimmermann@...e.de>
To: Javier Martinez Canillas <javierm@...hat.com>,
linux-kernel@...r.kernel.org
Cc: Alain Volmat <alain.volmat@...s.st.com>,
Alex Deucher <alexander.deucher@....com>,
Alexandre Belloni <alexandre.belloni@...tlin.com>,
Alexandre Torgue <alexandre.torgue@...s.st.com>,
Alexey Brodkin <abrodkin@...opsys.com>,
Alison Wang <alison.wang@....com>,
Andrew Jeffery <andrew@...id.au>,
Andrey Grodzovsky <andrey.grodzovsky@....com>,
Boris Brezillon <bbrezillon@...nel.org>,
Brian Starkey <brian.starkey@....com>,
Chen Feng <puck.chen@...ilicon.com>,
Chen-Yu Tsai <wens@...e.org>, Chia-I Wu <olvaffe@...il.com>,
Christian König <christian.koenig@....com>,
Chun-Kuang Hu <chunkuang.hu@...nel.org>,
Claudiu Beznea <claudiu.beznea@...rochip.com>,
Daniel Vetter <daniel@...ll.ch>,
Dave Airlie <airlied@...hat.com>,
David Airlie <airlied@...ux.ie>, Emma Anholt <emma@...olt.net>,
Evan Quan <evan.quan@....com>,
Fabio Estevam <festevam@...il.com>,
Gerd Hoffmann <kraxel@...hat.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Gurchetan Singh <gurchetansingh@...omium.org>,
Hans de Goede <hdegoede@...hat.com>,
Hao Fang <fanghao11@...wei.com>,
Hyun Kwon <hyun.kwon@...inx.com>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Jerome Brunet <jbrunet@...libre.com>,
Joel Stanley <joel@....id.au>,
John Stultz <john.stultz@...aro.org>,
Jyri Sarha <jyri.sarha@....fi>,
Kevin Hilman <khilman@...libre.com>,
Kieran Bingham <kieran.bingham+renesas@...asonboard.com>,
Laurent Pinchart <laurent.pinchart@...asonboard.com>,
Laurentiu Palcu <laurentiu.palcu@....nxp.com>,
Linus Walleij <linus.walleij@...aro.org>,
Liviu Dudau <liviu.dudau@....com>,
Lucas De Marchi <lucas.demarchi@...el.com>,
Lucas Stach <l.stach@...gutronix.de>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Marek Vasut <marex@...x.de>,
Mario Limonciello <mario.limonciello@....com>,
Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
Matthias Brugger <matthias.bgg@...il.com>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
Maxime Ripard <mripard@...nel.org>,
Michal Simek <michal.simek@...inx.com>,
NXP Linux Team <linux-imx@....com>,
Neil Armstrong <narmstrong@...libre.com>,
Nicolas Ferre <nicolas.ferre@...rochip.com>,
Nirmoy Das <nirmoy.das@....com>,
Noralf Trønnes <noralf@...nnes.org>,
"Pan, Xinhui" <Xinhui.Pan@....com>,
Paul Cercueil <paul@...pouillou.net>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Philipp Zabel <p.zabel@...gutronix.de>,
Philippe Cornu <philippe.cornu@...s.st.com>,
Raphael Gallais-Pou <raphael.gallais-pou@...s.st.com>,
Sam Ravnborg <sam@...nborg.org>,
Samuel Holland <samuel@...lland.org>,
Sascha Hauer <s.hauer@...gutronix.de>,
Shawn Guo <shawnguo@...nel.org>,
Solomon Chiu <solomon.chiu@....com>,
Stefan Agner <stefan@...er.ch>,
Tian Tao <tiantao6@...ilicon.com>,
Tomi Valkeinen <tomba@...nel.org>,
Xinliang Liu <xinliang.liu@...aro.org>,
Xinwei Kong <kong.kongxinwei@...ilicon.com>,
Yannick Fertre <yannick.fertre@...s.st.com>,
Yong Wu <yong.wu@...iatek.com>, amd-gfx@...ts.freedesktop.org,
dri-devel@...ts.freedesktop.org, linux-amlogic@...ts.infradead.org,
linux-arm-kernel@...ts.infradead.org,
linux-aspeed@...ts.ozlabs.org, linux-mediatek@...ts.infradead.org,
linux-mips@...r.kernel.org, linux-renesas-soc@...r.kernel.org,
linux-stm32@...md-mailman.stormreply.com,
linux-sunxi@...ts.linux.dev, spice-devel@...ts.freedesktop.org,
virtualization@...ts.linux-foundation.org
Subject: Re: [PATCH 0/3] drm: Allow simpledrm to setup its emulated FB as
firmware provided
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
> "[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@...e.de>
Thanks a lot!
>
> [0]: https://lore.kernel.org/lkml/20220429084253.1085911-1-javierm@redhat.com/
>
>
> Javier Martinez Canillas (3):
> drm: Remove superfluous arg when calling to drm_fbdev_generic_setup()
> drm/fb-helper: Rename preferred_bpp drm_fbdev_generic_setup()
> parameter
> drm: Allow simpledrm to setup its emulated FB as firmware provided
>
> drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 6 +++--
> drivers/gpu/drm/arm/hdlcd_drv.c | 2 +-
> drivers/gpu/drm/arm/malidp_drv.c | 2 +-
> drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 2 +-
> drivers/gpu/drm/ast/ast_drv.c | 2 +-
> drivers/gpu/drm/atmel-hlcdc/atmel_hlcdc_dc.c | 2 +-
> drivers/gpu/drm/drm_drv.c | 2 +-
> drivers/gpu/drm/drm_fb_helper.c | 25 ++++++++++++++++---
> drivers/gpu/drm/fsl-dcu/fsl_dcu_drm_drv.c | 2 +-
> .../gpu/drm/hisilicon/hibmc/hibmc_drm_drv.c | 2 +-
> .../gpu/drm/hisilicon/kirin/kirin_drm_drv.c | 2 +-
> drivers/gpu/drm/imx/dcss/dcss-kms.c | 2 +-
> drivers/gpu/drm/imx/imx-drm-core.c | 2 +-
> drivers/gpu/drm/ingenic/ingenic-drm-drv.c | 2 +-
> drivers/gpu/drm/mcde/mcde_drv.c | 2 +-
> drivers/gpu/drm/mediatek/mtk_drm_drv.c | 2 +-
> drivers/gpu/drm/meson/meson_drv.c | 2 +-
> drivers/gpu/drm/mxsfb/mxsfb_drv.c | 2 +-
> drivers/gpu/drm/pl111/pl111_drv.c | 2 +-
> drivers/gpu/drm/qxl/qxl_drv.c | 2 +-
> drivers/gpu/drm/rcar-du/rcar_du_drv.c | 2 +-
> drivers/gpu/drm/sti/sti_drv.c | 2 +-
> drivers/gpu/drm/stm/drv.c | 2 +-
> drivers/gpu/drm/sun4i/sun4i_drv.c | 2 +-
> drivers/gpu/drm/tidss/tidss_drv.c | 2 +-
> drivers/gpu/drm/tilcdc/tilcdc_drv.c | 2 +-
> drivers/gpu/drm/tiny/arcpgu.c | 2 +-
> drivers/gpu/drm/tiny/bochs.c | 2 +-
> drivers/gpu/drm/tiny/cirrus.c | 2 +-
> drivers/gpu/drm/tiny/simpledrm.c | 2 +-
> drivers/gpu/drm/tve200/tve200_drv.c | 2 +-
> drivers/gpu/drm/vboxvideo/vbox_drv.c | 2 +-
> drivers/gpu/drm/vc4/vc4_drv.c | 2 +-
> drivers/gpu/drm/virtio/virtgpu_drv.c | 2 +-
> drivers/gpu/drm/xlnx/zynqmp_dpsub.c | 2 +-
> include/drm/drm_fb_helper.h | 22 ++++++++++++++++
> 36 files changed, 80 insertions(+), 39 deletions(-)
>
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Maxfeldstr. 5, 90409 Nürnberg, Germany
(HRB 36809, AG Nürnberg)
Geschäftsführer: Ivo Totev
Download attachment "OpenPGP_signature" of type "application/pgp-signature" (841 bytes)
Powered by blists - more mailing lists