From: Andrew Lunn <andrew@lunn.ch>
To: "Ardelean, Alexandru" <alexandru.Ardelean@analog.com>
Cc: "hkallweit1@gmail.com" <hkallweit1@gmail.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"netdev@vger.kernel.org" <netdev@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"f.fainelli@gmail.com" <f.fainelli@gmail.com>,
"davem@davemloft.net" <davem@davemloft.net>
Subject: Re: [PATCH 3/4] net: phy: adin: implement support for 1588 start-of-packet indication
Date: Thu, 16 Jan 2020 15:02:36 +0100 [thread overview]
Message-ID: <20200116140236.GH19046@lunn.ch> (raw)
In-Reply-To: <efab72f360a2043bc8cf545dcc7f24d00f3269c6.camel@analog.com>
On Thu, Jan 16, 2020 at 01:58:55PM +0000, Ardelean, Alexandru wrote:
> On Thu, 2020-01-16 at 14:55 +0100, Andrew Lunn wrote:
> > [External]
> >
> > On Thu, Jan 16, 2020 at 11:14:53AM +0200, Alexandru Ardelean wrote:
> > > The ADIN1300 & ADIN1200 PHYs support detection of IEEE 1588 time stamp
> > > packets. This mechanism can be used to signal the MAC via a pulse-
> > > signal
> > > when the PHY detects such a packet.
> >
> > Do you have patches for a MAC driver? I want to see how this connects
> > together.
>
> Nope.
>
> I admit that on the MAC side, I'm not yet familiar how this is integrated.
> I'd need to study this more in-depth.
O.K.
Then i suggest you post patch #1 as a single patch. And then work on
the MAC side, and post both MAC and PHY as a complete and tested
patchset.
Thanks
Andrew
next prev parent reply other threads:[~2020-01-16 14:02 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-16 9:14 [PATCH 0/4] net: phy: adin: implement support for 1588 start-of-packet indication Alexandru Ardelean
2020-01-16 9:14 ` [PATCH 1/4] net: phy: adin: const-ify static data Alexandru Ardelean
2020-01-16 13:30 ` Andrew Lunn
2020-01-16 9:14 ` [PATCH 2/4] net: phy: adin: rename struct adin_hw_stat -> adin_map Alexandru Ardelean
2020-01-16 13:38 ` Andrew Lunn
2020-01-16 13:57 ` Ardelean, Alexandru
2020-01-16 9:14 ` [PATCH 3/4] net: phy: adin: implement support for 1588 start-of-packet indication Alexandru Ardelean
2020-01-16 13:55 ` Andrew Lunn
2020-01-16 13:58 ` Ardelean, Alexandru
2020-01-16 14:02 ` Andrew Lunn [this message]
2020-01-16 14:03 ` Ardelean, Alexandru
2020-01-16 9:14 ` [PATCH 4/4] dt-bindings: net: adin: document 1588 TX/RX SOP bindings Alexandru Ardelean
2020-01-16 13:43 ` Andrew Lunn
2020-01-16 14:00 ` Ardelean, Alexandru
2020-01-22 2:05 ` Rob Herring
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=20200116140236.GH19046@lunn.ch \
--to=andrew@lunn.ch \
--cc=alexandru.Ardelean@analog.com \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=f.fainelli@gmail.com \
--cc=hkallweit1@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=netdev@vger.kernel.org \
/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.