Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1556945 > unrolled thread

Re: [PATCH v2 0/3] PCI: designware/dra7xx: misc fixes and cleanups

Started byBjorn Helgaas <helgaas@kernel.org>
First post2017-01-11 22:30 +0100
Last post2017-01-12 18:10 +0100
Articles 5 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v2 0/3] PCI: designware/dra7xx: misc fixes and cleanups Bjorn Helgaas <helgaas@kernel.org> - 2017-01-11 22:30 +0100
    Re: [PATCH v2 0/3] PCI: designware/dra7xx: misc fixes and cleanups Kishon Vijay Abraham I <kishon@ti.com> - 2017-01-12 06:30 +0100
      Re: [PATCH v2 0/3] PCI: designware/dra7xx: misc fixes and cleanups Bjorn Helgaas <helgaas@kernel.org> - 2017-01-12 14:20 +0100
        Re: [PATCH v2 0/3] PCI: designware/dra7xx: misc fixes and cleanups Kishon Vijay Abraham I <kishon@ti.com> - 2017-01-12 17:40 +0100
          Re: [PATCH v2 0/3] PCI: designware/dra7xx: misc fixes and cleanups Kishon Vijay Abraham I <kishon@TI.COM> - 2017-01-12 18:10 +0100

#1556945 — Re: [PATCH v2 0/3] PCI: designware/dra7xx: misc fixes and cleanups

FromBjorn Helgaas <helgaas@kernel.org>
Date2017-01-11 22:30 +0100
SubjectRe: [PATCH v2 0/3] PCI: designware/dra7xx: misc fixes and cleanups
Message-ID<sYyCZ-3As-33@gated-at.bofh.it>
On Fri, Dec 30, 2016 at 03:26:11PM +0530, Kishon Vijay Abraham I wrote:
> This series contains
>         *) a patch to cleanup dra7xx probe
>         *) a patch to force DRA7xx controller to work in GEN1 mode
>         *) a patch to fix dra7xx driver so that even if CONFIG_PCI_MSI
>            is enabled, dra7xx driver can service legacy interrupts
>            (provided the device support only legacy interrupts).
> 
> Changes from v1:
> *) rebased to 4.10-rc1 (and [1])
> *) use existing dt property "max-link-speed" instead of creating a new
>    TI specific dt property.
> 
> [1] -> https://lkml.org/lkml/2016/12/28/34
> 
> Kishon Vijay Abraham I (3):
>   PCI: dra7xx: simplify the probe code
>   PCI: dra7xx: Add support to force RC to work in GEN1 mode
>   PCI: dra7xx: Enable MSI and legacy interrupts simultaneously
> 
>  drivers/pci/controller/pci-dra7xx.c |   81 ++++++++++++++++++++---------------
>  1 file changed, 47 insertions(+), 34 deletions(-)

Applied to pci/host-dra7xx for v4.11.  I haven't applied the directory
rename yet, so I'll fix that up as necessary.

Bjorn

[toc] | [next] | [standalone]


#1557123

FromKishon Vijay Abraham I <kishon@ti.com>
Date2017-01-12 06:30 +0100
Message-ID<sYG7v-8s2-3@gated-at.bofh.it>
In reply to#1556945
Hi Bjorn,

On Thursday 12 January 2017 02:51 AM, Bjorn Helgaas wrote:
> On Fri, Dec 30, 2016 at 03:26:11PM +0530, Kishon Vijay Abraham I wrote:
>> This series contains
>>         *) a patch to cleanup dra7xx probe
>>         *) a patch to force DRA7xx controller to work in GEN1 mode
>>         *) a patch to fix dra7xx driver so that even if CONFIG_PCI_MSI
>>            is enabled, dra7xx driver can service legacy interrupts
>>            (provided the device support only legacy interrupts).
>>
>> Changes from v1:
>> *) rebased to 4.10-rc1 (and [1])
>> *) use existing dt property "max-link-speed" instead of creating a new
>>    TI specific dt property.
>>
>> [1] -> https://lkml.org/lkml/2016/12/28/34
>>
>> Kishon Vijay Abraham I (3):
>>   PCI: dra7xx: simplify the probe code
>>   PCI: dra7xx: Add support to force RC to work in GEN1 mode
>>   PCI: dra7xx: Enable MSI and legacy interrupts simultaneously
>>
>>  drivers/pci/controller/pci-dra7xx.c |   81 ++++++++++++++++++++---------------
>>  1 file changed, 47 insertions(+), 34 deletions(-)
> 
> Applied to pci/host-dra7xx for v4.11.  I haven't applied the directory
> rename yet, so I'll fix that up as necessary.

I'd have liked that to be merged as well. Is that because Arnd hasn't Acked
that one yet?

Thanks
Kishon

[toc] | [prev] | [next] | [standalone]


#1557433

FromBjorn Helgaas <helgaas@kernel.org>
Date2017-01-12 14:20 +0100
Message-ID<sYNsm-4qh-21@gated-at.bofh.it>
In reply to#1557123
On Thu, Jan 12, 2017 at 10:52:56AM +0530, Kishon Vijay Abraham I wrote:
> Hi Bjorn,
> 
> On Thursday 12 January 2017 02:51 AM, Bjorn Helgaas wrote:
> > On Fri, Dec 30, 2016 at 03:26:11PM +0530, Kishon Vijay Abraham I wrote:
> >> This series contains
> >>         *) a patch to cleanup dra7xx probe
> >>         *) a patch to force DRA7xx controller to work in GEN1 mode
> >>         *) a patch to fix dra7xx driver so that even if CONFIG_PCI_MSI
> >>            is enabled, dra7xx driver can service legacy interrupts
> >>            (provided the device support only legacy interrupts).
> >>
> >> Changes from v1:
> >> *) rebased to 4.10-rc1 (and [1])
> >> *) use existing dt property "max-link-speed" instead of creating a new
> >>    TI specific dt property.
> >>
> >> [1] -> https://lkml.org/lkml/2016/12/28/34
> >>
> >> Kishon Vijay Abraham I (3):
> >>   PCI: dra7xx: simplify the probe code
> >>   PCI: dra7xx: Add support to force RC to work in GEN1 mode
> >>   PCI: dra7xx: Enable MSI and legacy interrupts simultaneously
> >>
> >>  drivers/pci/controller/pci-dra7xx.c |   81 ++++++++++++++++++++---------------
> >>  1 file changed, 47 insertions(+), 34 deletions(-)
> > 
> > Applied to pci/host-dra7xx for v4.11.  I haven't applied the directory
> > rename yet, so I'll fix that up as necessary.
> 
> I'd have liked that to be merged as well. Is that because Arnd hasn't Acked
> that one yet?

Nope, I expect to do the rename in some form.  I'm just trying to
clear out the easy stuff first before moving on to the harder patches
in the queue.

Bjorn

[toc] | [prev] | [next] | [standalone]


#1557596

FromKishon Vijay Abraham I <kishon@ti.com>
Date2017-01-12 17:40 +0100
Message-ID<sYQzV-6d7-47@gated-at.bofh.it>
In reply to#1557433
Hi,

On Thursday 12 January 2017 06:45 PM, Bjorn Helgaas wrote:
> On Thu, Jan 12, 2017 at 10:52:56AM +0530, Kishon Vijay Abraham I wrote:
>> Hi Bjorn,
>>
>> On Thursday 12 January 2017 02:51 AM, Bjorn Helgaas wrote:
>>> On Fri, Dec 30, 2016 at 03:26:11PM +0530, Kishon Vijay Abraham I wrote:
>>>> This series contains
>>>>         *) a patch to cleanup dra7xx probe
>>>>         *) a patch to force DRA7xx controller to work in GEN1 mode
>>>>         *) a patch to fix dra7xx driver so that even if CONFIG_PCI_MSI
>>>>            is enabled, dra7xx driver can service legacy interrupts
>>>>            (provided the device support only legacy interrupts).
>>>>
>>>> Changes from v1:
>>>> *) rebased to 4.10-rc1 (and [1])
>>>> *) use existing dt property "max-link-speed" instead of creating a new
>>>>    TI specific dt property.
>>>>
>>>> [1] -> https://lkml.org/lkml/2016/12/28/34
>>>>
>>>> Kishon Vijay Abraham I (3):
>>>>   PCI: dra7xx: simplify the probe code
>>>>   PCI: dra7xx: Add support to force RC to work in GEN1 mode
>>>>   PCI: dra7xx: Enable MSI and legacy interrupts simultaneously
>>>>
>>>>  drivers/pci/controller/pci-dra7xx.c |   81 ++++++++++++++++++++---------------
>>>>  1 file changed, 47 insertions(+), 34 deletions(-)
>>>
>>> Applied to pci/host-dra7xx for v4.11.  I haven't applied the directory
>>> rename yet, so I'll fix that up as necessary.
>>
>> I'd have liked that to be merged as well. Is that because Arnd hasn't Acked
>> that one yet?
> 
> Nope, I expect to do the rename in some form.  I'm just trying to
> clear out the easy stuff first before moving on to the harder patches
> in the queue.

cool, thanks. Let me know if I have to resend the patches based on one of your
branches (i.e after you apply the easy stuff).

Thanks
Kishon

[toc] | [prev] | [next] | [standalone]


#1557621

FromKishon Vijay Abraham I <kishon@TI.COM>
Date2017-01-12 18:10 +0100
Message-ID<sYR2V-6Ch-7@gated-at.bofh.it>
In reply to#1557596
Hi,

On Thursday 12 January 2017 06:53 PM, Kishon Vijay Abraham I wrote:
> Hi,
> 
> On Thursday 12 January 2017 06:45 PM, Bjorn Helgaas wrote:
>> On Thu, Jan 12, 2017 at 10:52:56AM +0530, Kishon Vijay Abraham I wrote:
>>> Hi Bjorn,
>>>
>>> On Thursday 12 January 2017 02:51 AM, Bjorn Helgaas wrote:
>>>> On Fri, Dec 30, 2016 at 03:26:11PM +0530, Kishon Vijay Abraham I wrote:
>>>>> This series contains
>>>>>         *) a patch to cleanup dra7xx probe
>>>>>         *) a patch to force DRA7xx controller to work in GEN1 mode
>>>>>         *) a patch to fix dra7xx driver so that even if CONFIG_PCI_MSI
>>>>>            is enabled, dra7xx driver can service legacy interrupts
>>>>>            (provided the device support only legacy interrupts).
>>>>>
>>>>> Changes from v1:
>>>>> *) rebased to 4.10-rc1 (and [1])
>>>>> *) use existing dt property "max-link-speed" instead of creating a new
>>>>>    TI specific dt property.
>>>>>
>>>>> [1] -> https://lkml.org/lkml/2016/12/28/34
>>>>>
>>>>> Kishon Vijay Abraham I (3):
>>>>>   PCI: dra7xx: simplify the probe code
>>>>>   PCI: dra7xx: Add support to force RC to work in GEN1 mode
>>>>>   PCI: dra7xx: Enable MSI and legacy interrupts simultaneously
>>>>>
>>>>>  drivers/pci/controller/pci-dra7xx.c |   81 ++++++++++++++++++++---------------
>>>>>  1 file changed, 47 insertions(+), 34 deletions(-)
>>>>
>>>> Applied to pci/host-dra7xx for v4.11.  I haven't applied the directory
>>>> rename yet, so I'll fix that up as necessary.
>>>
>>> I'd have liked that to be merged as well. Is that because Arnd hasn't Acked
>>> that one yet?
>>
>> Nope, I expect to do the rename in some form.  I'm just trying to
>> clear out the easy stuff first before moving on to the harder patches
>> in the queue.
> 
> cool, thanks. Let me know if I have to resend the patches based on one of your
> branches (i.e after you apply the easy stuff).

Btw, I had sent a new patch series [1] which had a couple of additional patches
(that was after creating drivers/pci/dwc/). Consider merging that too or I can
resend the missing patches.

Thanks
Kishon

[1] -> https://lkml.org/lkml/2017/1/11/238
> 
> Thanks
> Kishon
> 

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web