From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 295ECC433F5 for ; Mon, 2 May 2022 10:50:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1384683AbiEBKxd (ORCPT ); Mon, 2 May 2022 06:53:33 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43468 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1384662AbiEBKx1 (ORCPT ); Mon, 2 May 2022 06:53:27 -0400 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.133.124]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 9678312091 for ; Mon, 2 May 2022 03:49:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1651488597; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=3TKyHWDgC93OfXdwv3f/QQ+fWpsgCb5Bjp407TIVwrw=; b=W9aUkE1bnSxtGZDw6EH4QhshhIoxLy409jEaJmvf3QAeh5yIz3t6/ttF7ZT4jDEoOVHYVV F0UEm+bsoseXG7XhR9fEfgRUggywwGNJwaPuwfaveQ4oQ57lnGOd4+y7JrNtXzfVbBzrvY GybOlNCAWpcfYtoU1YBpnJkuPxU/VbM= Received: from mail-wm1-f71.google.com (mail-wm1-f71.google.com [209.85.128.71]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-202-kipQS7I7NieZSP6-b_WQ_A-1; Mon, 02 May 2022 06:49:56 -0400 X-MC-Unique: kipQS7I7NieZSP6-b_WQ_A-1 Received: by mail-wm1-f71.google.com with SMTP id 26-20020a05600c021a00b003940660c053so4721141wmi.2 for ; Mon, 02 May 2022 03:49:56 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=3TKyHWDgC93OfXdwv3f/QQ+fWpsgCb5Bjp407TIVwrw=; b=QBBGQkXwtK6m7aBYbUfjNuM6OSwAPzikifEa8B68LZXta7iycXq/cFk3I6KADWu+FF tyAWasuGHxcDtHws7PoLYqWOJT05WRAHIsJNsjtL0w8E6vLFZ9fVLkN2pya2TMr01h1p tKvLGpTq2jrGCv0yLTZKZfUXU9DyWSkanVbvpaj3ZnLL68XV8YPpWcrKXoc2OL79hPux YWpxIae+WAaoDM8QXm2Bi2pff2IX5OddZ2zMn55040Qgqd96EVvAbveWFCwDnfMh1JBj Fu+2xt9UG65xgcci/5YXrNP44eO3aTQg+l7XDtBA3fYxDnnySs77q55KBGG1NPu5qDqL C4zw== X-Gm-Message-State: AOAM530aCb8SrF7rIJTHYX5qToocmFe0J/Dn7c0/j7XP7w2Xr9fn8ezb mY2/k0ZrKNdQJS2Ti5iF4GZKXFf9rshxNAVZytebfIm0FFG9QtdiUjyvgKVXDvQ4ohNCPcXbcSD 4pYcsvVO/5jNW2PqqAU40IA== X-Received: by 2002:a5d:5051:0:b0:20a:e005:cca3 with SMTP id h17-20020a5d5051000000b0020ae005cca3mr8676765wrt.560.1651488595402; Mon, 02 May 2022 03:49:55 -0700 (PDT) X-Google-Smtp-Source: ABdhPJznK7MTrWwTm+wtuLbm1AtWbuZrM/hnsYDFHNU2iLGSqHpPTjz/i73gvAXGyZ+XCXckc/fOrQ== X-Received: by 2002:a5d:5051:0:b0:20a:e005:cca3 with SMTP id h17-20020a5d5051000000b0020ae005cca3mr8676723wrt.560.1651488595161; Mon, 02 May 2022 03:49:55 -0700 (PDT) Received: from [192.168.1.129] ([92.176.231.205]) by smtp.gmail.com with ESMTPSA id w7-20020adf8bc7000000b0020c5253d8f9sm6707583wra.69.2022.05.02.03.49.51 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 02 May 2022 03:49:54 -0700 (PDT) Message-ID: <1d2a03f7-d4f6-66ac-6e2e-adbd2eaf7d90@redhat.com> Date: Mon, 2 May 2022 12:49:50 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.0 Subject: Re: [PATCH 0/3] drm: Allow simpledrm to setup its emulated FB as firmware provided Content-Language: en-US To: Thomas Zimmermann , linux-kernel@vger.kernel.org Cc: Alexandre Belloni , Emma Anholt , Chun-Kuang Hu , David Airlie , spice-devel@lists.freedesktop.org, Laurentiu Palcu , Liviu Dudau , Alexandre Torgue , Gurchetan Singh , Paul Cercueil , Nirmoy Das , Gerd Hoffmann , Shawn Guo , linux-stm32@st-md-mailman.stormreply.com, Jerome Brunet , Marek Vasut , Hao Fang , linux-aspeed@lists.ozlabs.org, Samuel Holland , Kevin Hilman , Neil Armstrong , Alexey Brodkin , Michal Simek , Jernej Skrabec , Xinliang Liu , Chen-Yu Tsai , =?UTF-8?Q?Noralf_Tr=c3=b8nnes?= , Joel Stanley , Dave Airlie , Xinwei Kong , virtualization@lists.linux-foundation.org, NXP Linux Team , Maxime Coquelin , Raphael Gallais-Pou , Martin Blumenstingl , Chen Feng , Sascha Hauer , Alison Wang , Lucas De Marchi , linux-mips@vger.kernel.org, Sam Ravnborg , Hans de Goede , linux-mediatek@lists.infradead.org, dri-devel@lists.freedesktop.org, Alain Volmat , Matthias Brugger , linux-amlogic@lists.infradead.org, Mario Limonciello , Evan Quan , Pengutronix Kernel Team , Yong Wu , linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, amd-gfx@lists.freedesktop.org, Tomi Valkeinen , Hyun Kwon , Boris Brezillon , Andrew Jeffery , Greg Kroah-Hartman , Yannick Fertre , "Pan, Xinhui" , Nicolas Ferre , Claudiu Beznea , linux-renesas-soc@vger.kernel.org, Solomon Chiu , Kieran Bingham , Philippe Cornu , Alex Deucher , Tian Tao , Jyri Sarha , =?UTF-8?Q?Christian_K=c3=b6nig?= , Laurent Pinchart References: <20220502084830.285639-1-javierm@redhat.com> From: Javier Martinez Canillas In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-mips@vger.kernel.org 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 > > Thanks a lot! > Thanks for the prompt review! -- Best regards, Javier Martinez Canillas Linux Engineering Red Hat