lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date:   Wed, 1 Mar 2017 10:08:20 +0530
From:   Kishon Vijay Abraham I <kishon@...com>
To:     Bjorn Helgaas <helgaas@...nel.org>,
        Jaehoon Chung <jh80.chung@...sung.com>
CC:     <linux-pci@...r.kernel.org>, <bhelgaas@...gle.com>,
        <krzk@...nel.org>, <linux-kernel@...r.kernel.org>,
        <jingoohan1@...il.com>, <javier@....samsung.com>,
        <kgene@...nel.org>, <linux-samsung-soc@...r.kernel.org>,
        <cpgs@...sung.com>, <niyas.ahmed@...sung.com>,
        <alim.akhtar@...sung.com>, <pankaj.dubey@...sung.com>,
        <devicetree@...r.kernel.org>, <mark.rutland@....com>,
        <vivek.gautam@...eaurora.org>, <robh+dt@...nel.org>
Subject: Re: [PATCH V3 0/4] PCI: exynos: use the PHY generic framework

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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ