lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite for Android: free password hash cracker in your pocket
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ