Date: Thu, 10 Aug 2017 10:03:41 -0700
From: Tony Lindgren <tony@atomide.com>
To: Rob Herring <robh@kernel.org>
Cc: SZ Lin <sz.lin@moxa.com>, mark.rutland@arm.com, linux@armlinux.org.uk,
        bcousson@baylibre.com, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-omap@vger.kernel.org
Subject: Re: [PATCH v2] arm, am335x: add support for Moxa UC-8100-ME-T open
 platform
Message-ID: <20170810170341.GM8569@atomide.com>
References: <20170802065757.1074-1-sz.lin@moxa.com>
 <20170810155246.pfaii3zb5jblluwy@rob-hp-laptop>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20170810155246.pfaii3zb5jblluwy@rob-hp-laptop>
User-Agent: Mutt/1.8.3 (2017-05-23)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 548
Lines: 24

* Rob Herring <robh@kernel.org> [170810 08:53]:
> On Wed, Aug 02, 2017 at 02:57:57PM +0800, SZ Lin wrote:
> > Add support for Moxa UC-8100-ME-T open platform
...

> > +	vbat: fixedregulator@0 {
> 
> Unit address without reg property is not valid (build with W=2). Perhaps 
> vbat-regulator instead.
...

> > +	/* Power supply provides a fixed 3.3V @3A */
> > +	vmmcsd_fixed: fixedregulator@1 {
> 
> Ditto
> 
> With those fixed,
> 
> Acked-by: Rob Herring <robh@kernel.org>

I've fixed it up here and will apply into omap-for-v4.14/dt
thanks.

Tony