From: Kishon Vijay Abraham I <kishon@ti.com>
To: Florian Fainelli <f.fainelli@gmail.com>, Ray Jui <rjui@broadcom.com>
Cc: Arnd Bergmann <arnd@arndb.de>, <linux-kernel@vger.kernel.org>,
"JD (Jiandong) Zheng" <jdzheng@broadcom.com>,
Arun Parameswaran <arunp@broadcom.com>,
<bcm-kernel-feedback-list@broadcom.com>
Subject: Re: [PATCH v5 2/2] phy: cygnus: pcie: Add Cygnus PCIe PHY support
Date: Wed, 7 Oct 2015 00:55:07 +0530 [thread overview]
Message-ID: <56142013.2010006@ti.com> (raw)
In-Reply-To: <56141FA0.2080901@gmail.com>
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@broadcom.com>
>>>>> Reviewed-by: Arun Parameswaran <aparames@broadcom.com>
>>>>> Reviewed-by: JD (Jiandong) Zheng <jdzheng@broadcom.com>
>>>>> Reviewed-by: Scott Branden <sbranden@broadcom.com>
>>>>
>>>>
>>>> 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@broadcom.com>
>>> M: Scott Branden <sbranden@broadcom.com>
>>> M: Jon Mason <jonmason@broadcom.com>
>>> L: linux-arm-kernel@lists.infradead.org (moderated for non-subscribers)
>>> L: bcm-kernel-feedback-list@broadcom.com
>>> 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
next prev parent reply other threads:[~2015-10-06 19:25 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-09-21 17:17 [PATCH v5 0/2] Add Cygnus PCIe Serdes/PHY support Ray Jui
2015-09-21 17:17 ` [PATCH v5 1/2] dt-bindings: Add Cygnus PCIe PHY binding doc Ray Jui
2015-09-21 17:17 ` [PATCH v5 2/2] phy: cygnus: pcie: Add Cygnus PCIe PHY support Ray Jui
2015-10-06 14:25 ` Kishon Vijay Abraham I
2015-10-06 17:13 ` Ray Jui
2015-10-06 19:20 ` Kishon Vijay Abraham I
2015-10-06 19:23 ` Florian Fainelli
2015-10-06 19:25 ` Kishon Vijay Abraham I [this message]
2015-10-06 19:23 ` Ray Jui
2015-10-06 19:25 ` Kishon Vijay Abraham I
2015-09-28 16:03 ` [PATCH v5 0/2] Add Cygnus PCIe Serdes/PHY support Ray Jui
2015-09-29 13:24 ` Kishon Vijay Abraham I
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56142013.2010006@ti.com \
--to=kishon@ti.com \
--cc=arnd@arndb.de \
--cc=arunp@broadcom.com \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=f.fainelli@gmail.com \
--cc=jdzheng@broadcom.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rjui@broadcom.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.