Re: [PATCH v4 0/4] PCI: st: provide support for dw pcie

2015-09-17 Thread Gabriel Fernandez
No problem

Thanks

Gabriel

On 17 September 2015 at 16:59, Bjorn Helgaas  wrote:
> Hi Gabriel,
>
> On Thu, Aug 27, 2015 at 02:34:13PM +0200, Gabriel Fernandez wrote:
>>
>> This patchset is based on v4.2-rc1 and is based on
>> [PATCH v8 0/6] PCI: hisi: Add PCIe host support for HiSilicon SoC Hip05
>> patchset from Zhou Wang.
>>
>> Changes in v4:
>>  - Remove pci: designware: remove my pci_common_init_dev() patch and
>>   use [PATCH v8 3/6] PCI: designware: Add ARM64 support instead.
>>   This patch is a good solution for me to disable IO support.
>>  - add __init to st_pcie_probe() and use module_init() instead
>>   device_initcall() to prevent the probe function from being
>>   deferred and to prevent module unloading.
>>
>> Changes in v3:
>>  - Remove power management functions (was not fully tested)
>>  - Remove configuration space range from dt binding
>>  - Remove pci_common_init_dev() call in pcie-designware.c to avoid
>>default IO space declaration.
>>
>> Changes in v2:
>>  - comestic corrections in device tree binding
>>  - add pci-st.c into MAINTAINERS
>>  - remove st_pcie_ops structure to avoid another level of indirection
>>  - remove nasty busy-loop
>>  - remove useless test using virt_to_phys()
>>  - move disable io support into dw-pcie driver
>>
>> I don't change the st_pcie_abort_handler() function because abort handling
>> is masked during boot.
>>
>>
>> This patch-set introduces a STMicroelectronics PCIe controller.
>> It's based on designware PCIe driver.
>>
>> Gabriel Fernandez (4):
>>   ARM: STi: Kconfig update for PCIe support
>>   PCI: st: Add Device Tree bindings for sti pcie
>>   PCI: st: Provide support for the sti PCIe controller
>>   MAINTAINERS: Add pci-st.c to ARCH/STI architecture
>
> Rob had some questions, so I'm waiting for an update that responds to
> those.  When you post that, can you also squash the last two patches
> together?
>
> Thanks,
>   Bjorn
>
>>  Documentation/devicetree/bindings/pci/st-pcie.txt |  53 ++
>>  MAINTAINERS   |   1 +
>>  arch/arm/mach-sti/Kconfig |   2 +
>>  drivers/pci/host/Kconfig  |   9 +
>>  drivers/pci/host/Makefile |   1 +
>>  drivers/pci/host/pci-st.c | 583 
>> ++
>>  6 files changed, 649 insertions(+)
>>  create mode 100644 Documentation/devicetree/bindings/pci/st-pcie.txt
>>  create mode 100644 drivers/pci/host/pci-st.c
>>
>> --
>> 1.9.1
>>
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v4 0/4] PCI: st: provide support for dw pcie

2015-09-17 Thread Bjorn Helgaas
Hi Gabriel,

On Thu, Aug 27, 2015 at 02:34:13PM +0200, Gabriel Fernandez wrote:
> 
> This patchset is based on v4.2-rc1 and is based on 
> [PATCH v8 0/6] PCI: hisi: Add PCIe host support for HiSilicon SoC Hip05
> patchset from Zhou Wang.
> 
> Changes in v4:
>  - Remove pci: designware: remove my pci_common_init_dev() patch and
>   use [PATCH v8 3/6] PCI: designware: Add ARM64 support instead.
>   This patch is a good solution for me to disable IO support.
>  - add __init to st_pcie_probe() and use module_init() instead
>   device_initcall() to prevent the probe function from being
>   deferred and to prevent module unloading.
> 
> Changes in v3:
>  - Remove power management functions (was not fully tested)
>  - Remove configuration space range from dt binding
>  - Remove pci_common_init_dev() call in pcie-designware.c to avoid 
>default IO space declaration. 
> 
> Changes in v2:
>  - comestic corrections in device tree binding
>  - add pci-st.c into MAINTAINERS
>  - remove st_pcie_ops structure to avoid another level of indirection
>  - remove nasty busy-loop
>  - remove useless test using virt_to_phys()
>  - move disable io support into dw-pcie driver
> 
> I don't change the st_pcie_abort_handler() function because abort handling
> is masked during boot.
> 
> 
> This patch-set introduces a STMicroelectronics PCIe controller.
> It's based on designware PCIe driver.
> 
> Gabriel Fernandez (4):
>   ARM: STi: Kconfig update for PCIe support
>   PCI: st: Add Device Tree bindings for sti pcie
>   PCI: st: Provide support for the sti PCIe controller
>   MAINTAINERS: Add pci-st.c to ARCH/STI architecture

Rob had some questions, so I'm waiting for an update that responds to
those.  When you post that, can you also squash the last two patches
together?

Thanks,
  Bjorn

>  Documentation/devicetree/bindings/pci/st-pcie.txt |  53 ++
>  MAINTAINERS   |   1 +
>  arch/arm/mach-sti/Kconfig |   2 +
>  drivers/pci/host/Kconfig  |   9 +
>  drivers/pci/host/Makefile |   1 +
>  drivers/pci/host/pci-st.c | 583 
> ++
>  6 files changed, 649 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/pci/st-pcie.txt
>  create mode 100644 drivers/pci/host/pci-st.c
> 
> -- 
> 1.9.1
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v4 0/4] PCI: st: provide support for dw pcie

2015-08-27 Thread Gabriel Fernandez

This patchset is based on v4.2-rc1 and is based on 
[PATCH v8 0/6] PCI: hisi: Add PCIe host support for HiSilicon SoC Hip05
patchset from Zhou Wang.

Changes in v4:
 - Remove pci: designware: remove my pci_common_init_dev() patch and
use [PATCH v8 3/6] PCI: designware: Add ARM64 support instead.
This patch is a good solution for me to disable IO support.
 - add __init to st_pcie_probe() and use module_init() instead
device_initcall() to prevent the probe function from being
deferred and to prevent module unloading.

Changes in v3:
 - Remove power management functions (was not fully tested)
 - Remove configuration space range from dt binding
 - Remove pci_common_init_dev() call in pcie-designware.c to avoid 
   default IO space declaration. 

Changes in v2:
 - comestic corrections in device tree binding
 - add pci-st.c into MAINTAINERS
 - remove st_pcie_ops structure to avoid another level of indirection
 - remove nasty busy-loop
 - remove useless test using virt_to_phys()
 - move disable io support into dw-pcie driver

I don't change the st_pcie_abort_handler() function because abort handling
is masked during boot.


This patch-set introduces a STMicroelectronics PCIe controller.
It's based on designware PCIe driver.

Gabriel Fernandez (4):
  ARM: STi: Kconfig update for PCIe support
  PCI: st: Add Device Tree bindings for sti pcie
  PCI: st: Provide support for the sti PCIe controller
  MAINTAINERS: Add pci-st.c to ARCH/STI architecture

 Documentation/devicetree/bindings/pci/st-pcie.txt |  53 ++
 MAINTAINERS   |   1 +
 arch/arm/mach-sti/Kconfig |   2 +
 drivers/pci/host/Kconfig  |   9 +
 drivers/pci/host/Makefile |   1 +
 drivers/pci/host/pci-st.c | 583 ++
 6 files changed, 649 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/pci/st-pcie.txt
 create mode 100644 drivers/pci/host/pci-st.c

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html