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


Groups > linux.kernel > #1581664 > unrolled thread

Re: [PATCH V3 0/4] PCI: exynos: use the PHY generic framework

Started byBjorn Helgaas <helgaas@kernel.org>
First post2017-02-15 22:20 +0100
Last post2017-03-01 06:00 +0100
Articles 4 — 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 V3 0/4] PCI: exynos: use the PHY generic framework Bjorn Helgaas <helgaas@kernel.org> - 2017-02-15 22:20 +0100
    Re: [PATCH V3 0/4] PCI: exynos: use the PHY generic framework Jaehoon Chung <jh80.chung@samsung.com> - 2017-02-15 23:40 +0100
      Re: [PATCH V3 0/4] PCI: exynos: use the PHY generic framework Bjorn Helgaas <helgaas@kernel.org> - 2017-02-17 21:00 +0100
        Re: [PATCH V3 0/4] PCI: exynos: use the PHY generic framework Kishon Vijay Abraham I <kishon@ti.com> - 2017-03-01 06:00 +0100

#1581664 — Re: [PATCH V3 0/4] PCI: exynos: use the PHY generic framework

FromBjorn Helgaas <helgaas@kernel.org>
Date2017-02-15 22:20 +0100
SubjectRe: [PATCH V3 0/4] PCI: exynos: use the PHY generic framework
Message-ID<tbf9w-59d-15@gated-at.bofh.it>
On Mon, Feb 13, 2017 at 05:26:09PM +0900, Jaehoon Chung wrote:
> This patcheset is for using PHY generic framework.
> Current pci-exyns doesn't use the phy framework since there haven't been on
> PHY subsystem when Exynos5440 had bean upstreamed.
> It's making a difficult to upstream the other Exynos variants because of different
> PHY registers.
> 
> This patcheset has the below modifications:
> 1) Introduces the phy-exynos-pcie
> 2) Handles PHY register from PHY framework for pci-exynos
> 3) Modifies the dt-binding of pci-exynos
> 4) Maintains the backward compatibility
> 
> Changelog on V3:
> - Drops "ARM: dts: exynos5440: support the phy-pcie node of pcie"
>   : Will send this patch after applying this patchset.
> - Fixes typo
> - Based on latest PCI git repository (host-exynos branch)
> - Changes commit message
> - Removes the dependency
> 
> Changelog on V2:
> - Keep current codes for backward compatibility
> - Fixes some typos
> - Split the patches for removing the dependency
> - Removes the unnecessary codes
> - Change the patch's sequence
> - Based on latest PCI git repository.(next branch)
> 
> Jaehoon Chung (4):
>   Documetation: samsung-phy: add the exynos-pcie-phy binding
>   phy: phy-exynos-pcie: Add support for Exynos PCIe phy
>   Documetation: binding: modify the exynos5440 pcie binding
>   PCI: exynos: support the using PHY generic framework
> 
>  .../bindings/pci/samsung,exynos5440-pcie.txt       |  29 +++
>  .../devicetree/bindings/phy/samsung-phy.txt        |  17 ++
>  drivers/pci/host/pci-exynos.c                      |  54 +++-
>  drivers/phy/Kconfig                                |   8 +
>  drivers/phy/Makefile                               |   1 +
>  drivers/phy/phy-exynos-pcie.c                      | 285 +++++++++++++++++++++
>  6 files changed, 390 insertions(+), 4 deletions(-)
>  create mode 100644 drivers/phy/phy-exynos-pcie.c

I made the updates Vivek suggested and provisionally added these to my
pci/host-exynos branch to get some build testing.

Kishon, I guess it probably makes more sense for me to apply this
because even though the drivers/pci changes are small, I think they
depend on other pci-exynos.c changes that are on my branch but not in
Linus' tree yet.

Bjorn

[toc] | [next] | [standalone]


#1581726

FromJaehoon Chung <jh80.chung@samsung.com>
Date2017-02-15 23:40 +0100
Message-ID<tbgoW-5QG-19@gated-at.bofh.it>
In reply to#1581664
Dear Bjorn,

On 02/16/2017 06:11 AM, Bjorn Helgaas wrote:
> On Mon, Feb 13, 2017 at 05:26:09PM +0900, Jaehoon Chung wrote:
>> This patcheset is for using PHY generic framework.
>> Current pci-exyns doesn't use the phy framework since there haven't been on
>> PHY subsystem when Exynos5440 had bean upstreamed.
>> It's making a difficult to upstream the other Exynos variants because of different
>> PHY registers.
>>
>> This patcheset has the below modifications:
>> 1) Introduces the phy-exynos-pcie
>> 2) Handles PHY register from PHY framework for pci-exynos
>> 3) Modifies the dt-binding of pci-exynos
>> 4) Maintains the backward compatibility
>>
>> Changelog on V3:
>> - Drops "ARM: dts: exynos5440: support the phy-pcie node of pcie"
>>   : Will send this patch after applying this patchset.
>> - Fixes typo
>> - Based on latest PCI git repository (host-exynos branch)
>> - Changes commit message
>> - Removes the dependency
>>
>> Changelog on V2:
>> - Keep current codes for backward compatibility
>> - Fixes some typos
>> - Split the patches for removing the dependency
>> - Removes the unnecessary codes
>> - Change the patch's sequence
>> - Based on latest PCI git repository.(next branch)
>>
>> Jaehoon Chung (4):
>>   Documetation: samsung-phy: add the exynos-pcie-phy binding
>>   phy: phy-exynos-pcie: Add support for Exynos PCIe phy
>>   Documetation: binding: modify the exynos5440 pcie binding
>>   PCI: exynos: support the using PHY generic framework
>>
>>  .../bindings/pci/samsung,exynos5440-pcie.txt       |  29 +++
>>  .../devicetree/bindings/phy/samsung-phy.txt        |  17 ++
>>  drivers/pci/host/pci-exynos.c                      |  54 +++-
>>  drivers/phy/Kconfig                                |   8 +
>>  drivers/phy/Makefile                               |   1 +
>>  drivers/phy/phy-exynos-pcie.c                      | 285 +++++++++++++++++++++
>>  6 files changed, 390 insertions(+), 4 deletions(-)
>>  create mode 100644 drivers/phy/phy-exynos-pcie.c
> 
> I made the updates Vivek suggested and provisionally added these to my
> pci/host-exynos branch to get some build testing.

Thanks!

> 
> Kishon, I guess it probably makes more sense for me to apply this
> because even though the drivers/pci changes are small, I think they
> depend on other pci-exynos.c changes that are on my branch but not in
> Linus' tree yet.

I will send the patches relevant to Exynos5433(TM2) soon..
Thanks a lot.

Best Regards,
Jaehoon Chung

> 
> Bjorn
> 
> 
> 

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


#1583694

FromBjorn Helgaas <helgaas@kernel.org>
Date2017-02-17 21:00 +0100
Message-ID<tbWRd-72-31@gated-at.bofh.it>
In reply to#1581726
On Thu, Feb 16, 2017 at 07:33:15AM +0900, Jaehoon Chung wrote:
> Dear Bjorn,
> 
> On 02/16/2017 06:11 AM, Bjorn Helgaas wrote:
> > On Mon, Feb 13, 2017 at 05:26:09PM +0900, Jaehoon Chung wrote:
> >> This patcheset is for using PHY generic framework.
> >> Current pci-exyns doesn't use the phy framework since there haven't been on
> >> PHY subsystem when Exynos5440 had bean upstreamed.
> >> It's making a difficult to upstream the other Exynos variants because of different
> >> PHY registers.
> >>
> >> This patcheset has the below modifications:
> >> 1) Introduces the phy-exynos-pcie
> >> 2) Handles PHY register from PHY framework for pci-exynos
> >> 3) Modifies the dt-binding of pci-exynos
> >> 4) Maintains the backward compatibility
> >>
> >> Changelog on V3:
> >> - Drops "ARM: dts: exynos5440: support the phy-pcie node of pcie"
> >>   : Will send this patch after applying this patchset.
> >> - Fixes typo
> >> - Based on latest PCI git repository (host-exynos branch)
> >> - Changes commit message
> >> - Removes the dependency
> >>
> >> Changelog on V2:
> >> - Keep current codes for backward compatibility
> >> - Fixes some typos
> >> - Split the patches for removing the dependency
> >> - Removes the unnecessary codes
> >> - Change the patch's sequence
> >> - Based on latest PCI git repository.(next branch)
> >>
> >> Jaehoon Chung (4):
> >>   Documetation: samsung-phy: add the exynos-pcie-phy binding
> >>   phy: phy-exynos-pcie: Add support for Exynos PCIe phy
> >>   Documetation: binding: modify the exynos5440 pcie binding
> >>   PCI: exynos: support the using PHY generic framework
> >>
> >>  .../bindings/pci/samsung,exynos5440-pcie.txt       |  29 +++
> >>  .../devicetree/bindings/phy/samsung-phy.txt        |  17 ++
> >>  drivers/pci/host/pci-exynos.c                      |  54 +++-
> >>  drivers/phy/Kconfig                                |   8 +
> >>  drivers/phy/Makefile                               |   1 +
> >>  drivers/phy/phy-exynos-pcie.c                      | 285 +++++++++++++++++++++
> >>  6 files changed, 390 insertions(+), 4 deletions(-)
> >>  create mode 100644 drivers/phy/phy-exynos-pcie.c
> > 
> > I made the updates Vivek suggested and provisionally added these to my
> > pci/host-exynos branch to get some build testing.
> 
> Thanks!
> 
> > 
> > Kishon, I guess it probably makes more sense for me to apply this
> > because even though the drivers/pci changes are small, I think they
> > depend on other pci-exynos.c changes that are on my branch but not in
> > Linus' tree yet.

I haven't heard from Kishon, but I merged these to my -next branch for
v4.11.

Bjorn

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


#1589979

FromKishon Vijay Abraham I <kishon@ti.com>
Date2017-03-01 06:00 +0100
Message-ID<tg4wO-6c9-9@gated-at.bofh.it>
In reply to#1583694
Hi Bjorn,

On Saturday 18 February 2017 01:21 AM, Bjorn Helgaas wrote:
> On Thu, Feb 16, 2017 at 07:33:15AM +0900, Jaehoon Chung wrote:
>> Dear Bjorn,
>>
>> On 02/16/2017 06:11 AM, Bjorn Helgaas wrote:
>>> On Mon, Feb 13, 2017 at 05:26:09PM +0900, Jaehoon Chung wrote:
>>>> This patcheset is for using PHY generic framework.
>>>> Current pci-exyns doesn't use the phy framework since there haven't been on
>>>> PHY subsystem when Exynos5440 had bean upstreamed.
>>>> It's making a difficult to upstream the other Exynos variants because of different
>>>> PHY registers.
>>>>
>>>> This patcheset has the below modifications:
>>>> 1) Introduces the phy-exynos-pcie
>>>> 2) Handles PHY register from PHY framework for pci-exynos
>>>> 3) Modifies the dt-binding of pci-exynos
>>>> 4) Maintains the backward compatibility
>>>>
>>>> Changelog on V3:
>>>> - Drops "ARM: dts: exynos5440: support the phy-pcie node of pcie"
>>>>   : Will send this patch after applying this patchset.
>>>> - Fixes typo
>>>> - Based on latest PCI git repository (host-exynos branch)
>>>> - Changes commit message
>>>> - Removes the dependency
>>>>
>>>> Changelog on V2:
>>>> - Keep current codes for backward compatibility
>>>> - Fixes some typos
>>>> - Split the patches for removing the dependency
>>>> - Removes the unnecessary codes
>>>> - Change the patch's sequence
>>>> - Based on latest PCI git repository.(next branch)
>>>>
>>>> Jaehoon Chung (4):
>>>>   Documetation: samsung-phy: add the exynos-pcie-phy binding
>>>>   phy: phy-exynos-pcie: Add support for Exynos PCIe phy
>>>>   Documetation: binding: modify the exynos5440 pcie binding
>>>>   PCI: exynos: support the using PHY generic framework
>>>>
>>>>  .../bindings/pci/samsung,exynos5440-pcie.txt       |  29 +++
>>>>  .../devicetree/bindings/phy/samsung-phy.txt        |  17 ++
>>>>  drivers/pci/host/pci-exynos.c                      |  54 +++-
>>>>  drivers/phy/Kconfig                                |   8 +
>>>>  drivers/phy/Makefile                               |   1 +
>>>>  drivers/phy/phy-exynos-pcie.c                      | 285 +++++++++++++++++++++
>>>>  6 files changed, 390 insertions(+), 4 deletions(-)
>>>>  create mode 100644 drivers/phy/phy-exynos-pcie.c
>>>
>>> I made the updates Vivek suggested and provisionally added these to my
>>> pci/host-exynos branch to get some build testing.
>>
>> Thanks!
>>
>>>
>>> Kishon, I guess it probably makes more sense for me to apply this
>>> because even though the drivers/pci changes are small, I think they
>>> depend on other pci-exynos.c changes that are on my branch but not in
>>> Linus' tree yet.
> 
> I haven't heard from Kishon, but I merged these to my -next branch for
> v4.11.

I might have missed this one. Thanks for taking the patches.

Thanks
Kishon

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web