devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Simon Horman <horms@verge.net.au>
To: Geert Uytterhoeven <geert+renesas@glider.be>
Cc: Mark Rutland <mark.rutland@arm.com>,
	devicetree@vger.kernel.org, Magnus Damm <magnus.damm@gmail.com>,
	linux-renesas-soc@vger.kernel.org,
	Rob Herring <robh+dt@kernel.org>,
	Sudeep Holla <sudeep.holla@arm.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 0/8] ARM: dts: renesas: Remove unit-addresses and regs from integrated caches
Date: Tue, 7 Mar 2017 07:46:04 +0100	[thread overview]
Message-ID: <20170307064603.GD14208@verge.net.au> (raw)
In-Reply-To: <1488818443-25196-1-git-send-email-geert+renesas@glider.be>

On Mon, Mar 06, 2017 at 05:40:35PM +0100, Geert Uytterhoeven wrote:
> 	Hi Simon, Magnus,
> 
> This patch series removes the bogus unit-addresses and reg properties
> from the device nodes representing Cortex-A15/A7 cache controllers.
> 
> Note that the latter were added to remove warnings from dtc when using
> W=1:
> 
>     Warning (unit_address_vs_reg): Node /cache-controller@0 has a unit name, but no reg property
> 
> In hindsight, adding the reg properties turned out to be the wrong fix.
> Indeed, the Cortex-A15/A7 cache controllers are integrated controllers,
> and thus the device nodes representing them should not have
> unit-addresses or reg properties.
> 
> This series does not have a runtime effect, as Linux doesn't rely much
> on the properties of the cache-controller nodes.
> 
> This series is the arm32 counterpart of "[PATCH 0/2] arm64: dts: renesas:
> Remove unit-addresses and regs from integrated caches", which you've just
> applied.
> 
> Thanks for applying!

Thanks, done.

      parent reply	other threads:[~2017-03-07  6:46 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-03-06 16:40 [PATCH 0/8] ARM: dts: renesas: Remove unit-addresses and regs from integrated caches Geert Uytterhoeven
2017-03-06 16:40 ` [PATCH 1/8] ARM: dts: r8a73a4: " Geert Uytterhoeven
2017-03-06 16:40 ` [PATCH 2/8] ARM: dts: r8a7743: Remove unit-address and reg from integrated cache Geert Uytterhoeven
2017-03-06 16:40 ` [PATCH 3/8] ARM: dts: r8a7745: " Geert Uytterhoeven
2017-03-06 16:40 ` [PATCH 4/8] ARM: dts: r8a7790: Remove unit-addresses and regs from integrated caches Geert Uytterhoeven
2017-03-06 16:40 ` [PATCH 6/8] ARM: dts: r8a7792: Remove unit-address and reg from integrated cache Geert Uytterhoeven
     [not found] ` <1488818443-25196-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2017-03-06 16:40   ` [PATCH 5/8] ARM: dts: r8a7791: " Geert Uytterhoeven
2017-03-06 16:40   ` [PATCH 7/8] ARM: dts: r8a7793: " Geert Uytterhoeven
2017-03-06 16:40   ` [PATCH 8/8] ARM: dts: r8a7794: " Geert Uytterhoeven
2017-03-07  6:46 ` Simon Horman [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=20170307064603.GD14208@verge.net.au \
    --to=horms@verge.net.au \
    --cc=devicetree@vger.kernel.org \
    --cc=geert+renesas@glider.be \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=sudeep.holla@arm.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 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).