From: Adrian Bunk <bunk@kernel.org>
To: Dan Murphy <dmurphy@ti.com>
Cc: Andrew Lunn <andrew@lunn.ch>,
Florian Fainelli <f.fainelli@gmail.com>,
Heiner Kallweit <hkallweit1@gmail.com>,
netdev@vger.kernel.org
Subject: Re: dp83867: Why does ti,fifo-depth set only TX, and why is it mandatory?
Date: Sat, 23 Nov 2019 23:00:01 +0200 [thread overview]
Message-ID: <20191123210001.GA20458@localhost> (raw)
In-Reply-To: <08b61f8c-bd7b-7ea7-2e47-50ddb540d67f@ti.com>
On Thu, Nov 21, 2019 at 02:24:01PM -0600, Dan Murphy wrote:
> Adrian
Hi Dan,
>...
> OK the HW team said that FIFO depth is no longer a mandatory field to be
> written for either RGMII or SGMII.
>
> So my suggestion here is that we deprecate, but support in the driver, the
> ti-fifo-depth, and add the already documented
>
> rx-fifo-depth and tx-fifo-depth as optional DT entries.
>
> So I can change the driver and DT docs and test the RGMII device as above as
> long as we are in agreement
sounds good to me.
> Dan
Thanks
Adrian
prev parent reply other threads:[~2019-11-23 21:00 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-11-14 16:24 dp83867: Why does ti,fifo-depth set only TX, and why is it mandatory? Adrian Bunk
2019-11-14 17:53 ` Dan Murphy
2019-11-14 19:47 ` Adrian Bunk
2019-11-14 20:01 ` Dan Murphy
2019-11-19 17:35 ` Dan Murphy
2019-11-21 20:24 ` Dan Murphy
2019-11-23 21:00 ` Adrian Bunk [this message]
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=20191123210001.GA20458@localhost \
--to=bunk@kernel.org \
--cc=andrew@lunn.ch \
--cc=dmurphy@ti.com \
--cc=f.fainelli@gmail.com \
--cc=hkallweit1@gmail.com \
--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.