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

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: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Mon, 2 May 2022 12:49:50 +0200
From:   Javier Martinez Canillas <javierm@...hat.com>
To:     Thomas Zimmermann <tzimmermann@...e.de>,
        linux-kernel@...r.kernel.org
Cc:     Alexandre Belloni <alexandre.belloni@...tlin.com>,
        Emma Anholt <emma@...olt.net>,
        Chun-Kuang Hu <chunkuang.hu@...nel.org>,
        David Airlie <airlied@...ux.ie>,
        spice-devel@...ts.freedesktop.org,
        Laurentiu Palcu <laurentiu.palcu@....nxp.com>,
        Liviu Dudau <liviu.dudau@....com>,
        Alexandre Torgue <alexandre.torgue@...s.st.com>,
        Gurchetan Singh <gurchetansingh@...omium.org>,
        Paul Cercueil <paul@...pouillou.net>,
        Nirmoy Das <nirmoy.das@....com>,
        Gerd Hoffmann <kraxel@...hat.com>,
        Shawn Guo <shawnguo@...nel.org>,
        linux-stm32@...md-mailman.stormreply.com,
        Jerome Brunet <jbrunet@...libre.com>,
        Marek Vasut <marex@...x.de>, Hao Fang <fanghao11@...wei.com>,
        linux-aspeed@...ts.ozlabs.org,
        Samuel Holland <samuel@...lland.org>,
        Kevin Hilman <khilman@...libre.com>,
        Neil Armstrong <narmstrong@...libre.com>,
        Alexey Brodkin <abrodkin@...opsys.com>,
        Michal Simek <michal.simek@...inx.com>,
        Jernej Skrabec <jernej.skrabec@...il.com>,
        Xinliang Liu <xinliang.liu@...aro.org>,
        Chen-Yu Tsai <wens@...e.org>,
        Noralf Trønnes <noralf@...nnes.org>,
        Joel Stanley <joel@....id.au>,
        Dave Airlie <airlied@...hat.com>,
        Xinwei Kong <kong.kongxinwei@...ilicon.com>,
        virtualization@...ts.linux-foundation.org,
        NXP Linux Team <linux-imx@....com>,
        Maxime Coquelin <mcoquelin.stm32@...il.com>,
        Raphael Gallais-Pou <raphael.gallais-pou@...s.st.com>,
        Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
        Chen Feng <puck.chen@...ilicon.com>,
        Sascha Hauer <s.hauer@...gutronix.de>,
        Alison Wang <alison.wang@....com>,
        Lucas De Marchi <lucas.demarchi@...el.com>,
        linux-mips@...r.kernel.org, Sam Ravnborg <sam@...nborg.org>,
        Hans de Goede <hdegoede@...hat.com>,
        linux-mediatek@...ts.infradead.org,
        dri-devel@...ts.freedesktop.org,
        Alain Volmat <alain.volmat@...s.st.com>,
        Matthias Brugger <matthias.bgg@...il.com>,
        linux-amlogic@...ts.infradead.org,
        Mario Limonciello <mario.limonciello@....com>,
        Evan Quan <evan.quan@....com>,
        Pengutronix Kernel Team <kernel@...gutronix.de>,
        Yong Wu <yong.wu@...iatek.com>,
        linux-arm-kernel@...ts.infradead.org, linux-sunxi@...ts.linux.dev,
        amd-gfx@...ts.freedesktop.org, Tomi Valkeinen <tomba@...nel.org>,
        Hyun Kwon <hyun.kwon@...inx.com>,
        Boris Brezillon <bbrezillon@...nel.org>,
        Andrew Jeffery <andrew@...id.au>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Yannick Fertre <yannick.fertre@...s.st.com>,
        "Pan, Xinhui" <Xinhui.Pan@....com>,
        Nicolas Ferre <nicolas.ferre@...rochip.com>,
        Claudiu Beznea <claudiu.beznea@...rochip.com>,
        linux-renesas-soc@...r.kernel.org,
        Solomon Chiu <solomon.chiu@....com>,
        Kieran Bingham <kieran.bingham+renesas@...asonboard.com>,
        Philippe Cornu <philippe.cornu@...s.st.com>,
        Alex Deucher <alexander.deucher@....com>,
        Tian Tao <tiantao6@...ilicon.com>,
        Jyri Sarha <jyri.sarha@....fi>,
        Christian König <christian.koenig@....com>,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>
Subject: Re: [PATCH 0/3] drm: Allow simpledrm to setup its emulated FB as
 firmware provided

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@...e.de>
> 
> Thanks a lot!
> 

Thanks for the prompt review!

-- 
Best regards,

Javier Martinez Canillas
Linux Engineering
Red Hat