lkml.org 
[lkml]   [2017]   [May]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject答复: 答复: [PATCH 1/2] PCI: dwc: kirin: add PCIe Driver for HiSilicon Kirin SoC
Date


-----邮件原件-----
发件人: Niklas Cassel [mailto:niklas.cassel@axis.com]
发送时间: 2017年5月18日 16:57
收件人: Guodong Xu; songxiaowei
抄送: Arnd Bergmann; Bjorn Helgaas; Kishon; Jingoo Han; Tomasz Nowicki; Keith Busch; Duc Dang; liudongdong (C); Chenfeng (puck); Wangbinghui; Suzhuangluan; Linux Kernel Mailing List; linux-pci
主题: Re: 答复: [PATCH 1/2] PCI: dwc: kirin: add PCIe Driver for HiSilicon Kirin SoC



On 05/18/2017 08:23 AM, Guodong Xu wrote:
> On Thu, May 18, 2017 at 9:50 AM, songxiaowei <songxiaowei@hisilicon.com> wrote:
>> Hi Arnd,
>>
>>
>> On Fri, May 12, 2017 at 3:51 AM, Song Xiaowei <songxiaowei@hisilicon.com> wrote:
>>> From: songxiaowei <songxiaowei@hisilicon.com>
>>
>> Looks good overall, just a few details:
>>
>> Please fix your ~/.gitconfig to contain the same real name ("Song Xiaowei"
>> instead of "songxiaowei") that you use for sending the emails.
>>
>> [Xiaowei Song] I'll fix this issue and update the patch.
>>
>>> +
>>> +static int kirin_pcie_phy_init(struct kirin_pcie *kirin_pcie) {
>> ...
>>> + reg_val = kirin_apb_phy_readl(kirin_pcie, 0x400);
>>> + while (reg_val & pipe_clk_stable) {
>>> + udelay(100);
>>> + if (time == 0) {
>>> + dev_err(kirin_pcie->pci->dev, "PIPE clk is not stable\n");
>>> + return -EINVAL;
>>> + }
>>> + time--;
>>> + reg_val = kirin_apb_phy_readl(kirin_pcie, 0x400);
>>> + }
>>
>> If this is not called with interrupts disabled, please use a sleeping function (e.g. msleep(1)) as the delay and compare against ktime_before() to see how much total time has expired when waiting for a timeout, instead of using a counter.
>>
>> [Xiaowei Song] This issue was fixed in the patch sent at 2017-5-15, and I'll send it again.
>
> Xiaowei,
>
> Please make sure you add [PATCH v..] version information when you
> making future updates. Eg.
>
> $ git format-patch --subject-prefix="PATCH v3"

I actually prefer

git format-patch -v 3

that way you don't accidentally get something other than "PATCH".



Another very, very minor remark.
It's usually a good idea to have a comma after each initializer, that way you don't have to change multiple lines when adding/removing something.

[songxiaowei] I should fix this point immediately, Thank you a lot.

+struct platform_driver kirin_pcie_driver = {
+ .probe = kirin_pcie_probe,
+ .driver = {
+ .name = "Kirin-pcie",
+ .of_match_table = kirin_pcie_match,
+ .suppress_bind_attrs = true
+ },
+};


Best regards,
Xiaowei
\
 
 \ /
  Last update: 2017-05-18 11:14    [W:0.528 / U:1.208 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site