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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id ECE2CEB64DC for ; Thu, 29 Jun 2023 17:00:23 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 1F37710E19E; Thu, 29 Jun 2023 17:00:23 +0000 (UTC) Received: from 189.cn (ptr.189.cn [183.61.185.101]) by gabe.freedesktop.org (Postfix) with ESMTP id 7A4B610E19E; Thu, 29 Jun 2023 17:00:21 +0000 (UTC) HMM_SOURCE_IP: 10.64.8.31:48394.1297546617 HMM_ATTACHE_NUM: 0000 HMM_SOURCE_TYPE: SMTP Received: from clientip-114.242.206.180 (unknown [10.64.8.31]) by 189.cn (HERMES) with SMTP id 05A4C10020D; Fri, 30 Jun 2023 01:00:13 +0800 (CST) Received: from ([114.242.206.180]) by gateway-151646-dep-75648544bd-xp9j7 with ESMTP id 2713aa1564624f348fd6c097b85f39d9 for helgaas@kernel.org; Fri, 30 Jun 2023 01:00:17 CST X-Transaction-ID: 2713aa1564624f348fd6c097b85f39d9 X-Real-From: 15330273260@189.cn X-Receive-IP: 114.242.206.180 X-MEDUSA-Status: 0 Message-ID: Date: Fri, 30 Jun 2023 01:00:10 +0800 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 Subject: Re: [PATCH v7 6/8] PCI/VGA: Introduce is_boot_device function callback to vga_client_register To: Bjorn Helgaas , Sui Jingfeng References: <20230629155436.GA397963@bhelgaas> Content-Language: en-US From: Sui Jingfeng <15330273260@189.cn> In-Reply-To: <20230629155436.GA397963@bhelgaas> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: linux-fbdev@vger.kernel.org, Pan Xinhui , kvm@vger.kernel.org, nouveau@lists.freedesktop.org, Lijo Lazar , dri-devel@lists.freedesktop.org, YiPeng Chai , Mario Limonciello , Likun Gao , Yi Liu , Karol Herbst , amd-gfx@lists.freedesktop.org, Jason Gunthorpe , Ben Skeggs , linux-pci@vger.kernel.org, Kevin Tian , Bokun Zhang , intel-gfx@lists.freedesktop.org, Alex Williamson , Abhishek Sahu , Maxime Ripard , Rodrigo Vivi , Bjorn Helgaas , Tvrtko Ursulin , Yishai Hadas , Cornelia Huck , linux-kernel@vger.kernel.org, Thomas Zimmermann , Alex Deucher , Christian Konig , Hawking Zhang Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Hi, On 2023/6/29 23:54, Bjorn Helgaas wrote: > On Thu, Jun 22, 2023 at 01:08:15PM +0800, Sui Jingfeng wrote: >> Hi, >> >> >> A nouveau developer(Lyude) from redhat send me a R-B, >> >> Thanks for the developers of nouveau project. >> >> >> Please allow me add a link[1] here. >> >> >> [1] https://lore.kernel.org/all/0afadc69f99a36bc9d03ecf54ff25859dbc10e28.camel@redhat.com/ > 1) Thanks for this. If you post another version of this series, > please pick up Lyude's Reviewed-by and include it in the relevant > patches (as long as you haven't made significant changes to the > code Lyude reviewed). Yes, no significant changes. Just fix typo. I also would like to add support for other DRM drivers. But I think this deserve another patch. > Whoever applies this should automatically > pick up Reviewed-by/Ack/etc that are replies to the version being > applied, but they won't go through previous revisions to find them. > > 2) Please mention the commit to which the series applies. I tried to > apply this on v6.4-rc1, but it doesn't apply cleanly. Since I'm a graphic driver developer, I'm using drm-tip. I just have already pulled, it still apply cleanly on drm-tip. > 3) Thanks for including cover letters in your postings. Please > include a little changelog in the cover letter so we know what > changed between v6 and v7, etc. No change between v6 and v7, it seems that it is because the mailbox don't allow me to sending too many mails a day. so some of the patch is failed to delivery because out of quota. > 4) Right now we're in the middle of the v6.5 merge window, so new > content, e.g., this series, is too late for v6.5. Most > maintainers, including me, wait to merge new content until the > merge window closes and a new -rc1 is tagged. This merge window > should close on July 9, and people will start merging content for > v6.6, typically based on v6.5-rc1. I'm wondering Would you will merge all of the patches in this series (e.g. including the patch for drm/amdgpu(7/8) and drm/radeon(8/8)) ? Or just part of them? Emm, I don't know because my patch seems across different subsystem of Linux kernel. There is also a developer for AMDGPU (Mario) give me a R-B for the patch-0002 of this series. So, at least, PATCH-0001, PATCH-0002, PATCH-0003, PATCH-0004, PATCH-0006 are already OK(got reviewed by). Those 5 patch are already qualified to be merged, I think. I means that if you could merge those 5 patch first, then there no need to send another version again. I will refine the rest patch with more details and description. I'm fear of making too much noise. > Bjorn