[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <5c91b8dd-5342-0eb2-aef4-dbb865d6dfda@linux.dev>
Date: Thu, 20 Jul 2023 17:17:31 +0800
From: Sui Jingfeng <sui.jingfeng@...ux.dev>
To: Bjorn Helgaas <helgaas@...nel.org>
Cc: David Airlie <airlied@...il.com>, amd-gfx@...ts.freedesktop.org,
dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
intel-gfx@...ts.freedesktop.org, kvm@...r.kernel.org,
linux-fbdev@...r.kernel.org,
Sui Jingfeng <suijingfeng@...ngson.cn>,
linux-pci@...r.kernel.org
Subject: Re: [PATCH v3 0/9] PCI/VGA: Improve the default VGA device selection
Hi,
On 2023/7/20 03:32, Bjorn Helgaas wrote:
>> drm/amdgpu: Implement the is_primary_gpu callback of
>> vga_client_register()
>> drm/radeon: Add an implement for the is_primary_gpu function callback
>> drm/i915: Add an implement for the is_primary_gpu hook
>> drm/ast: Register as a vga client to vgaarb by calling
>> vga_client_register()
>> drm/loongson: Add an implement for the is_primary_gpu function
>> callback
> There's unnecessary variation in the subject lines (and the commit
> logs) of these patches. If they all do the same thing but in
> different drivers, it's useful if the patches all*look* the same.
>
> You might be able to write these subjects as
> "Implement .is_primary_gpu() callback" for brevity.
This is a very good suggestion, I will adopt this.
Thanks, really.
Powered by blists - more mailing lists