Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Fri, 23 Jul 2021 04:00:13 +0200
From:   Krzysztof =?utf-8?Q?Wilczy=C5=84ski?= <kw@linux.com>
To:     "Cai,Huoqing" <caihuoqing@baidu.com>
Cc:     "Derrick, Jonathan" <jonathan.derrick@intel.com>,
        "linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] PCI: vmd: Make use of PCI_DEVICE_DATA() helper
 function
Message-ID: <20210723020013.GA2170028@rocinante>
References: <20210722112954.477-1-caihuoqing@baidu.com>
 <20210722112954.477-3-caihuoqing@baidu.com>
 <f2aeb584-6293-78ce-e5aa-4bde34045a86@intel.com>
 <e44664317c2949e794497dc5f903f2a8@baidu.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <e44664317c2949e794497dc5f903f2a8@baidu.com>
Precedence: bulk

Hi,

Thank you for sending the patch over!

> 	PATCH[2/2] has some extra indentation, please don't apply it , 

You might know this already, but just in case make sure to run the
"checkpatch.pl" script over the patch before submission.

> I'll send PATCH V2.

You mean v3?  Make sure to include a changelog, if possible.

A small nitpick: the PCI_DEVICE_DATA() is technically a macro rather
than a function, so you could update both the subject line and the
commit message accordingly, if you want.

Also, since you are about to send another version, add period at the end
of the sentence in the commit message.

Aside of the above, it's a nice refactor, thank you!

	Krzysztof