RE: [PATCH v1 0/5] Add Keystone PCIe controller driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



>-----Original Message-----
>From: Jingoo Han [mailto:jg1.han@xxxxxxxxxxx]
>Sent: Thursday, May 15, 2014 8:49 PM
>To: Karicheri, Muralidharan
>Cc: linux-kernel@xxxxxxxxxxxxxxx; linux-pci@xxxxxxxxxxxxxxx; linux-arm-
>kernel@xxxxxxxxxxxxxxxxxxx; Shilimkar, Santosh; 'Russell King'; 'Grant Likely'; 'Rob Herring';
>'Mohit Kumar'; 'Bjorn Helgaas'; 'Jingoo Han'; 'Pratyush Anand'; 'Richard Zhu'; ABRAHAM,
>KISHON VIJAY; 'Marek Vasut'
>Subject: Re: [PATCH v1 0/5] Add Keystone PCIe controller driver
>
>On Friday, May 16, 2014 1:01 AM, Murali Karicheri wrote:
>>
>> This patch adds a PCIe controller driver for Keystone SoCs. This is
>> based on the origin RFC patch that I had sent earlier. I have
>> incorporated following comments:-
>>
>>  - Add a interrupt controller node for Legacy irq chip and use
>>    interrupt map/map-mask property to map legacy IRQs A/B/C/D
>>  - Add a Phy driver to replace the original serdes driver
>>  - Move common applicaiton register handling code to a separate
>>    file to allow re-use across other platforms that use older
>>    DW PCIe h/w
>>  - PCI quirk for maximum read request size. Check and override only
>>    if the maximum is higher than what controller can handle.
>>  - Converted to a module platform driver.
>>
>> CC: Santosh Shilimkar <santosh.shilimkar@xxxxxx>
>> CC: Russell King <linux@xxxxxxxxxxxxxxxx>
>> CC: Grant Likely <grant.likely@xxxxxxxxxx>
>> CC: Rob Herring <robh+dt@xxxxxxxxxx>
>> CC: Mohit Kumar <mohit.kumar@xxxxxx>
>> CC: Jingoo Han <jg1.han@xxxxxxxxxxx>
>> CC: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
>>
>
>Your patches modify 'pcie-designware.c', and affects other PCIe drivers using designware
>PCIe Core IP. Please add the following people to CC list. They are also related to the
>designware PCIe.
>
>  Pratyush Anand <pratyush.anand@xxxxxx>
>  Richard Zhu <r65037@xxxxxxxxxxxxx>
>  Kishon Vijay Abraham I <kishon@xxxxxx>
>  Marek Vasut <marex@xxxxxxx>
>
Will forward the patches to the above list as well.

Thanks

Murali
>Best regards,
>Jingoo Han
>
>>
>> Murali Karicheri (5):
>>   ARM: keystone: add pcie related options
>>   pci: designware: enhancements to support keystone pcie
>>   phy: pci serdes phy driver for keystone
>>   pci: dw: add common functions to support old hw based pci driver
>>   pci: keystone: add pcie driver based on designware core driver
>>
>>  .../devicetree/bindings/pci/pcie-keystone.txt      |   68 ++++
>>  arch/arm/mach-keystone/Kconfig                     |    2 +
>>  drivers/pci/host/Kconfig                           |   12 +
>>  drivers/pci/host/Makefile                          |    2 +
>>  drivers/pci/host/pci-dw-old-msi.c                  |  150 ++++++++
>>  drivers/pci/host/pci-dw-old.c                      |  371 ++++++++++++++++++
>>  drivers/pci/host/pci-dw-old.h                      |   30 ++
>>  drivers/pci/host/pci-keystone.c                    |  400 ++++++++++++++++++++
>>  drivers/pci/host/pcie-designware.c                 |  101 +++--
>>  drivers/pci/host/pcie-designware.h                 |   42 +-
>>  drivers/pci/quirks.c                               |   13 +
>>  drivers/phy/Kconfig                                |    6 +
>>  drivers/phy/Makefile                               |    1 +
>>  drivers/phy/phy-keystone.c                         |  230 +++++++++++
>>  14 files changed, 1388 insertions(+), 40 deletions(-)  create mode
>> 100644 Documentation/devicetree/bindings/pci/pcie-keystone.txt
>>  create mode 100644 drivers/pci/host/pci-dw-old-msi.c  create mode
>> 100644 drivers/pci/host/pci-dw-old.c  create mode 100644
>> drivers/pci/host/pci-dw-old.h  create mode 100644
>> drivers/pci/host/pci-keystone.c  create mode 100644
>> drivers/phy/phy-keystone.c
>>
>> --
>> 1.7.9.5

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




[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux