Re: [PATCH v5 2/2] phy: cygnus: pcie: Add Cygnus PCIe PHY support
From: Kishon Vijay Abraham I
Date: Tue Oct 06 2015 - 15:26:30 EST
Hi.
On Wednesday 07 October 2015 12:53 AM, Florian Fainelli wrote:
> On 06/10/15 12:20, Kishon Vijay Abraham I wrote:
>> Hi,
>>
>> On Tuesday 06 October 2015 10:43 PM, Ray Jui wrote:
>>> Hi Kishon,
>>>
>>> On 10/6/2015 7:25 AM, Kishon Vijay Abraham I wrote:
>>>> Hi,
>>>>
>>>> On Monday 21 September 2015 10:47 PM, Ray Jui wrote:
>>>>> This patch adds the PCIe PHY support for the Broadcom PCIe RC interface
>>>>> on Cygnus
>>>>>
>>>>> Signed-off-by: Ray Jui <rjui@xxxxxxxxxxxx>
>>>>> Reviewed-by: Arun Parameswaran <aparames@xxxxxxxxxxxx>
>>>>> Reviewed-by: JD (Jiandong) Zheng <jdzheng@xxxxxxxxxxxx>
>>>>> Reviewed-by: Scott Branden <sbranden@xxxxxxxxxxxx>
>>>>
>>>>
>>>> merged this driver to linux-phy tree. Can you send a separate patch
>>>> adding yourself as the maintainer of this driver?
>>>>
>>>> Thanks
>>>> Kishon
>>>
>>> We have a pool of maintainers to maintain all iProc based device drivers
>>> (including this PCIe PHY driver for Cygnus):
>>>
>>> BROADCOM IPROC ARM ARCHITECTURE
>>> M: Ray Jui <rjui@xxxxxxxxxxxx>
>>> M: Scott Branden <sbranden@xxxxxxxxxxxx>
>>> M: Jon Mason <jonmason@xxxxxxxxxxxx>
>>> L: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx (moderated for non-subscribers)
>>> L: bcm-kernel-feedback-list@xxxxxxxxxxxx
>>> T: git git://github.com/broadcom/cygnus-linux.git
>>> S: Maintained
>>> N: iproc
>>> N: cygnus
>>> N: nsp
>>> ...
>>> ...
>>>
>>> That should be sufficient, correct?
>>
>> yes. I think you might still have to add this?
>> F: drivers/phy/phy-bcm-cygnus-pcie.c
>
> The regular expression should already cover that file, what would be the
> benefit of explicitly listing this additional file?
>
Indeed. Sorry for the noise.
Thanks
Kishon
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/