From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Calvin Johnson <calvin.johnson@oss.nxp.com>
Cc: Florian Fainelli <f.fainelli@gmail.com>,
linux.cj@gmail.com, Jeremy Linton <jeremy.linton@arm.com>,
Andrew Lunn <andrew@lunn.ch>,
Andy Shevchenko <andy.shevchenko@gmail.com>,
Russell King - ARM Linux admin <linux@armlinux.org.uk>,
Cristi Sovaiala <cristian.sovaiala@nxp.com>,
Florin Laurentiu Chiculita <florinlaurentiu.chiculita@nxp.com>,
Ioana Ciornei <ioana.ciornei@nxp.com>,
Madalin Bucur <madalin.bucur@oss.nxp.com>,
Laurentiu Tudor <laurentiu.tudor@nxp.com>,
linux-acpi@vger.kernel.org,
Diana Madalina Craciun <diana.craciun@nxp.com>,
linux-arm-kernel@lists.infradead.org,
Pankaj Bansal <pankaj.bansal@nxp.com>,
linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
Varun Sethi <V.Sethi@nxp.com>, Marcin Wojtas <mw@semihalf.com>,
Makarand Pawagi <makarand.pawagi@nxp.com>,
"Rajesh V . Bikkina" <rajesh.bikkina@nxp.com>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
Heikki Krogerus <heikki.krogerus@linux.intel.com>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [net-next PATCH v1 1/2] device property: Introduce fwnode_phy_find_device()
Date: Fri, 24 Apr 2020 11:34:30 +0200 [thread overview]
Message-ID: <20200424093430.GA376180@kroah.com> (raw)
In-Reply-To: <20200424092651.GA4501@lsv03152.swis.in-blr01.nxp.com>
On Fri, Apr 24, 2020 at 02:56:51PM +0530, Calvin Johnson wrote:
> On Thu, Apr 23, 2020 at 08:45:03PM -0700, Florian Fainelli wrote:
> >
> >
> > On 4/23/2020 8:16 PM, Calvin Johnson wrote:
> > > Define fwnode_phy_find_device() to iterate an mdiobus and find the
> > > phy device of the provided phy fwnode.
> > >
> > > Signed-off-by: Calvin Johnson <calvin.johnson@oss.nxp.com>
> >
> > If you forget to update the MAINTAINERS file, or do not place this code
> > under drivers/net/phy/* or drivers/of/of_mdio.c then this is going to
> > completely escape the sight of the PHYLIB/PHYLINK maintainers...
>
> Did you mean the following change?
>
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -6354,6 +6354,7 @@ F:
> Documentation/devicetree/bindings/net/ethernet-phy.yaml
> F: Documentation/devicetree/bindings/net/mdio*
> F: Documentation/devicetree/bindings/net/qca,ar803x.yaml
> F: Documentation/networking/phy.rst
> +F: drivers/base/property.c
> F: drivers/net/phy/
> F: drivers/of/of_mdio.c
I really doubt the phy maintainers want to maintain all of property.c,
right? Please be kinder...
greg k-h
WARNING: multiple messages have this Message-ID (diff)
From: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
To: Calvin Johnson <calvin.johnson@oss.nxp.com>
Cc: Andrew Lunn <andrew@lunn.ch>,
Heikki Krogerus <heikki.krogerus@linux.intel.com>,
"Rafael J. Wysocki" <rafael@kernel.org>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Cristi Sovaiala <cristian.sovaiala@nxp.com>,
Ioana Ciornei <ioana.ciornei@nxp.com>,
Florian Fainelli <f.fainelli@gmail.com>,
"Rajesh V . Bikkina" <rajesh.bikkina@nxp.com>,
Pankaj Bansal <pankaj.bansal@nxp.com>,
Russell King - ARM Linux admin <linux@armlinux.org.uk>,
Diana Madalina Craciun <diana.craciun@nxp.com>,
linux-acpi@vger.kernel.org,
Andy Shevchenko <andy.shevchenko@gmail.com>,
Florin Laurentiu Chiculita <florinlaurentiu.chiculita@nxp.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Madalin Bucur <madalin.bucur@oss.nxp.com>,
Makarand Pawagi <makarand.pawagi@nxp.com>,
Varun Sethi <V.Sethi@nxp.com>,
Thomas Gleixner <tglx@linutronix.de>,
Marcin Wojtas <mw@semihalf.com>,
linux-arm-kernel@lists.infradead.org,
Laurentiu Tudor <laurentiu.tudor@nxp.com>,
netdev@vger.kernel.org,
Dmitry Torokhov <dmitry.torokhov@gmail.com>,
linux-kernel@vger.kernel.org,
Jeremy Linton <jeremy.linton@arm.com>,
linux.cj@gmail.com
Subject: Re: [net-next PATCH v1 1/2] device property: Introduce fwnode_phy_find_device()
Date: Fri, 24 Apr 2020 11:34:30 +0200 [thread overview]
Message-ID: <20200424093430.GA376180@kroah.com> (raw)
In-Reply-To: <20200424092651.GA4501@lsv03152.swis.in-blr01.nxp.com>
On Fri, Apr 24, 2020 at 02:56:51PM +0530, Calvin Johnson wrote:
> On Thu, Apr 23, 2020 at 08:45:03PM -0700, Florian Fainelli wrote:
> >
> >
> > On 4/23/2020 8:16 PM, Calvin Johnson wrote:
> > > Define fwnode_phy_find_device() to iterate an mdiobus and find the
> > > phy device of the provided phy fwnode.
> > >
> > > Signed-off-by: Calvin Johnson <calvin.johnson@oss.nxp.com>
> >
> > If you forget to update the MAINTAINERS file, or do not place this code
> > under drivers/net/phy/* or drivers/of/of_mdio.c then this is going to
> > completely escape the sight of the PHYLIB/PHYLINK maintainers...
>
> Did you mean the following change?
>
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -6354,6 +6354,7 @@ F:
> Documentation/devicetree/bindings/net/ethernet-phy.yaml
> F: Documentation/devicetree/bindings/net/mdio*
> F: Documentation/devicetree/bindings/net/qca,ar803x.yaml
> F: Documentation/networking/phy.rst
> +F: drivers/base/property.c
> F: drivers/net/phy/
> F: drivers/of/of_mdio.c
I really doubt the phy maintainers want to maintain all of property.c,
right? Please be kinder...
greg k-h
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-04-24 9:34 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-24 3:16 [net-next PATCH v1 0/2] Introduce new APIs to support phylink and phy layers Calvin Johnson
2020-04-24 3:16 ` Calvin Johnson
2020-04-24 3:16 ` [net-next PATCH v1 1/2] device property: Introduce fwnode_phy_find_device() Calvin Johnson
2020-04-24 3:16 ` Calvin Johnson
2020-04-24 3:45 ` Florian Fainelli
2020-04-24 3:45 ` Florian Fainelli
2020-04-24 9:26 ` Calvin Johnson
2020-04-24 9:26 ` Calvin Johnson
2020-04-24 9:34 ` Greg Kroah-Hartman [this message]
2020-04-24 9:34 ` Greg Kroah-Hartman
2020-04-24 9:54 ` Andy Shevchenko
2020-04-24 9:54 ` Andy Shevchenko
2020-04-24 13:41 ` Calvin Johnson
2020-04-24 13:41 ` Calvin Johnson
2020-04-24 10:04 ` Heikki Krogerus
2020-04-24 10:04 ` Heikki Krogerus
2020-04-24 3:16 ` [net-next PATCH v1 2/2] phylink: introduce phylink_fwnode_phy_connect() Calvin Johnson
2020-04-24 3:16 ` Calvin Johnson
2020-04-24 9:56 ` Andy Shevchenko
2020-04-24 9:56 ` Andy Shevchenko
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=20200424093430.GA376180@kroah.com \
--to=gregkh@linuxfoundation.org \
--cc=V.Sethi@nxp.com \
--cc=andrew@lunn.ch \
--cc=andriy.shevchenko@linux.intel.com \
--cc=andy.shevchenko@gmail.com \
--cc=calvin.johnson@oss.nxp.com \
--cc=cristian.sovaiala@nxp.com \
--cc=diana.craciun@nxp.com \
--cc=dmitry.torokhov@gmail.com \
--cc=f.fainelli@gmail.com \
--cc=florinlaurentiu.chiculita@nxp.com \
--cc=heikki.krogerus@linux.intel.com \
--cc=ioana.ciornei@nxp.com \
--cc=jeremy.linton@arm.com \
--cc=laurentiu.tudor@nxp.com \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux.cj@gmail.com \
--cc=linux@armlinux.org.uk \
--cc=madalin.bucur@oss.nxp.com \
--cc=makarand.pawagi@nxp.com \
--cc=mw@semihalf.com \
--cc=netdev@vger.kernel.org \
--cc=pankaj.bansal@nxp.com \
--cc=rafael@kernel.org \
--cc=rajesh.bikkina@nxp.com \
--cc=sakari.ailus@linux.intel.com \
--cc=tglx@linutronix.de \
/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.