[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <a540daba-d61f-1de0-a3dd-eab733a25706@189.cn>
Date: Wed, 7 Jun 2023 14:16:31 +0800
From: Sui Jingfeng <15330273260@....cn>
To: Bjorn Helgaas <helgaas@...nel.org>
Cc: Alex Deucher <alexander.deucher@....com>,
Christian Konig <christian.koenig@....com>,
Pan Xinhui <Xinhui.Pan@....com>,
David Airlie <airlied@...il.com>,
Daniel Vetter <daniel@...ll.ch>,
Jani Nikula <jani.nikula@...ux.intel.com>,
Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>,
Rodrigo Vivi <rodrigo.vivi@...el.com>,
Tvrtko Ursulin <tvrtko.ursulin@...ux.intel.com>,
Ben Skeggs <bskeggs@...hat.com>,
Karol Herbst <kherbst@...hat.com>,
Lyude Paul <lyude@...hat.com>,
Bjorn Helgaas <bhelgaas@...gle.com>,
Alex Williamson <alex.williamson@...hat.com>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>,
Thomas Zimmermann <tzimmermann@...e.de>,
Hawking Zhang <Hawking.Zhang@....com>,
Mario Limonciello <mario.limonciello@....com>,
Lijo Lazar <lijo.lazar@....com>,
YiPeng Chai <YiPeng.Chai@....com>,
Andrey Grodzovsky <andrey.grodzovsky@....com>,
Somalapuram Amaranath <Amaranath.Somalapuram@....com>,
Bokun Zhang <Bokun.Zhang@....com>,
Ville Syrjala <ville.syrjala@...ux.intel.com>,
Li Yi <liyi@...ngson.cn>,
Sui Jingfeng <suijingfeng@...ngson.cn>,
Jason Gunthorpe <jgg@...pe.ca>,
Kevin Tian <kevin.tian@...el.com>,
Cornelia Huck <cohuck@...hat.com>,
Yishai Hadas <yishaih@...dia.com>,
Abhishek Sahu <abhsahu@...dia.com>,
Yi Liu <yi.l.liu@...el.com>, kvm@...r.kernel.org,
nouveau@...ts.freedesktop.org, intel-gfx@...ts.freedesktop.org,
linux-kernel@...r.kernel.org, dri-devel@...ts.freedesktop.org,
loongson-kernel@...ts.loongnix.cn, amd-gfx@...ts.freedesktop.org,
linux-pci@...r.kernel.org
Subject: Re: [Intel-gfx] [PATCH v2 1/2] vgaarb: various coding style and
comments fix
Hi,
On 2023/6/7 03:49, Bjorn Helgaas wrote:
> Match the subject line style:
>
> $ git log --oneline drivers/pci/vgaarb.c
> f321c35feaee PCI/VGA: Replace full MIT license text with SPDX identifier
> d5109fe4d1ec PCI/VGA: Use unsigned format string to print lock counts
> 4e6c91847a7f PCI/VGA: Log bridge control messages when adding devices
> dc593fd48abb PCI/VGA: Remove empty vga_arb_device_card_gone()
> ...
>
> Subject line should be a summary of the commit log, not just "various
> style fixes". This one needs to say something about
> vga_str_to_iostate().
Ok, thanks for the educating .
> On Mon, Jun 05, 2023 at 04:58:30AM +0800, Sui Jingfeng wrote:
>> From: Sui Jingfeng <suijingfeng@...ngson.cn>
>>
>> To keep consistent with vga_iostate_to_str() function, the third argument
>> of vga_str_to_iostate() function should be 'unsigned int *'.
>>
>> Signed-off-by: Sui Jingfeng <suijingfeng@...ngson.cn>
>> ---
>> drivers/pci/vgaarb.c | 29 +++++++++++++++--------------
>> include/linux/vgaarb.h | 8 +++-----
>> 2 files changed, 18 insertions(+), 19 deletions(-)
>>
>> diff --git a/drivers/pci/vgaarb.c b/drivers/pci/vgaarb.c
>> index 5a696078b382..e40e6e5e5f03 100644
>> --- a/drivers/pci/vgaarb.c
>> +++ b/drivers/pci/vgaarb.c
>> @@ -61,7 +61,6 @@ static bool vga_arbiter_used;
>> static DEFINE_SPINLOCK(vga_lock);
>> static DECLARE_WAIT_QUEUE_HEAD(vga_wait_queue);
>>
>> -
>> static const char *vga_iostate_to_str(unsigned int iostate)
>> {
>> /* Ignore VGA_RSRC_IO and VGA_RSRC_MEM */
>> @@ -77,10 +76,12 @@ static const char *vga_iostate_to_str(unsigned int iostate)
>> return "none";
>> }
>>
>> -static int vga_str_to_iostate(char *buf, int str_size, int *io_state)
>> +static int vga_str_to_iostate(char *buf, int str_size, unsigned int *io_state)
>> {
>> - /* we could in theory hand out locks on IO and mem
>> - * separately to userspace but it can cause deadlocks */
>> + /*
>> + * we could in theory hand out locks on IO and mem
>> + * separately to userspace but it can cause deadlocks
>> + */
> Omit all the comment formatting changes. They are distractions from the
> vga_str_to_iostate() parameter change.
>
> I think this patch should be the single line change to the
> vga_str_to_iostate() prototype so it matches the callers.
>
> If you want to do the other comment formatting changes, they're fine,
> but they should be all together in a separate patch that clearly
> doesn't change the generated code.
Ok, no problem.
Will be improved at next version.
> Bjorn
Powered by blists - more mailing lists