netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Andrew Lunn <andrew@lunn.ch>,
	"Madalin Bucur (OSS)" <madalin.bucur@oss.nxp.com>
Cc: "maxime.ripard@bootlin.com" <maxime.ripard@bootlin.com>,
	"mark.rutland@arm.com" <mark.rutland@arm.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"netdev@vger.kernel.org" <netdev@vger.kernel.org>
Subject: Re: [RFC] obsolete references to Documentation/devicetree/bindings/net/ethernet.txt
Date: Wed, 8 Jan 2020 10:24:43 -0600	[thread overview]
Message-ID: <20200108162443.GA16710@bogus> (raw)
In-Reply-To: <20191226110821.GE1480@lunn.ch>

On Thu, Dec 26, 2019 at 12:08:21PM +0100, Andrew Lunn wrote:
> On Mon, Dec 23, 2019 at 09:00:50AM +0000, Madalin Bucur (OSS) wrote:
> > Hi,
> > 
> > this patch removed the ethernet.txt content and added the information found in
> > it in the ethernet-controller.yaml (Documentation/devicetree/bindings/net):
> > 
> >   commit 9d3de3c58347623b5e71d554628a1571cd3142ad
> >   Author: Maxime Ripard <maxime.ripard@bootlin.com>
> >   Date:   Thu Jun 27 17:31:43 2019 +0200
> > 
> >       dt-bindings: net: Add YAML schemas for the generic Ethernet options
> > 
> >       The Ethernet controllers have a good number of generic options that can be
> >       needed in a device tree. Add a YAML schemas for those.
> > 
> >       Reviewed-by: Rob Herring <robh@kernel.org>
> >       Signed-off-by: Maxime Ripard <maxime.ripard@bootlin.com>
> >       Reviewed-by: Andrew Lunn <andrew@lunn.ch>
> >       Signed-off-by: Rob Herring <robh@kernel.org>
> > 
> > There are still many references to the previous ethernet.txt document:
> > 
> >   $ grep ethernet.txt Documentation/devicetree/bindings/net/ -r | wc -l
> >   96
> > 
> > Should those be updated too or it's enough to rely on the current content
> > of the previous ethernet.txt file:
> > 
> >   $ cat Documentation/devicetree/bindings/net/ethernet.txt
> >   This file has moved to ethernet-controller.yaml.

If this wasn't enough, we wouldn't have put that there.

> 
> Hi Madalin
> 
> Feel free to fix the references if you want. Probably a perl one liner
> should do most of the work.

The above was done to avoid some churn. The real fix is convert bindings 
to DT schema. The references should get dropped as part of that.

Rob

      reply	other threads:[~2020-01-08 16:24 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-12-23  9:00 [RFC] obsolete references to Documentation/devicetree/bindings/net/ethernet.txt Madalin Bucur (OSS)
2019-12-26 11:08 ` Andrew Lunn
2020-01-08 16:24   ` Rob Herring [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=20200108162443.GA16710@bogus \
    --to=robh@kernel.org \
    --cc=andrew@lunn.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=madalin.bucur@oss.nxp.com \
    --cc=mark.rutland@arm.com \
    --cc=maxime.ripard@bootlin.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).