From mboxrd@z Thu Jan 1 00:00:00 1970 From: Felipe Balbi Subject: Re: [PATCH v2 1/2] arm: dts: am4372: let boards enable RTC and Watchdog Date: Wed, 18 Jun 2014 16:51:43 -0500 Message-ID: <20140618215143.GA8181@saruman.home> References: <1403106200-777-1-git-send-email-balbi@ti.com> <1403106200-777-2-git-send-email-balbi@ti.com> <53A1B677.9010909@ti.com> <20140618192553.GB4570@saruman.home> <53A20800.3070301@ti.com> Reply-To: Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="SUOF0GtieIMvvwua" Return-path: Content-Disposition: inline In-Reply-To: <53A20800.3070301@ti.com> Sender: linux-kernel-owner@vger.kernel.org To: Nishanth Menon Cc: balbi@ti.com, devicetree@vger.kernel.org, linux@arm.linux.org.uk, Tony Lindgren , Rajendra Nayak , Linux Kernel Mailing List , r.sricharan@ti.com, robh+dt@kernel.org, Benoit Cousson , galak@codeaurora.org, Linux OMAP Mailing List , Linux ARM Kernel Mailing List List-Id: devicetree@vger.kernel.org --SUOF0GtieIMvvwua Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Wed, Jun 18, 2014 at 04:43:28PM -0500, Nishanth Menon wrote: > On 06/18/2014 02:25 PM, Felipe Balbi wrote: > >Hi, > > > >On Wed, Jun 18, 2014 at 10:55:35AM -0500, Nishanth Menon wrote: > >>$subject - ARM: dts: > >> > >>On 06/18/2014 10:43 AM, Felipe Balbi wrote: > >>>by providing phandles to rtc and wdt nodes, we can > >>>let boards enable rtc and wdt as necessary. > >>> > >>>Signed-off-by: Felipe Balbi > >>>--- > >>> arch/arm/boot/dts/am4372.dtsi | 4 ++-- > >>> 1 file changed, 2 insertions(+), 2 deletions(-) > >>> > >>>diff --git a/arch/arm/boot/dts/am4372.dtsi b/arch/arm/boot/dts/am4372.= dtsi > >>>index 49fa596..e3830d4 100644 > >>>--- a/arch/arm/boot/dts/am4372.dtsi > >>>+++ b/arch/arm/boot/dts/am4372.dtsi > >>>@@ -270,7 +270,7 @@ > >>> ti,hwmods =3D "counter_32k"; > >>> }; > >>> > >>>- rtc@44e3e000 { > >>>+ rtc: rtc@44e3e000 { > >>> compatible =3D "ti,am4372-rtc","ti,da830-rtc"; > >>> reg =3D <0x44e3e000 0x1000>; > >>> interrupts =3D >>>@@ -279,7 +279,7 @@ > >>> status =3D "disabled"; > >>> }; > >>> > >>>- wdt@44e35000 { > >>>+ wdt: wdt@44e35000 { > >>> compatible =3D "ti,am4372-wdt","ti,omap3-wdt"; > >>> reg =3D <0x44e35000 0x1000>; > >>> interrupts =3D ; > >>> > >>we might want to fix up any other similar nodes in dtsi as well, no? > >> > >>examples: > >>cpu@0 > > > >and why would you want boards to mess with the cpu node ? > > > cpu-supply =3D <®ulator>; >=20 > for cpufreq. good point, but does thqt even work with AM437x devices in mainline? --=20 balbi --SUOF0GtieIMvvwua Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJTognvAAoJEIaOsuA1yqREBskQAKREx6HXEYwke4wo4t3xX8FF zFDeDNE+B/UCpydqykKMgzPToNkImg0y2DCIpwvej3Q8yUBZiu2OKA2RSMh6n+F0 zZViCUhGtZC0YN5pNLmPZtBwg4U9ALX9VgfJml2ex58z6fwyEsrt9v4TOUX8x9Xl ZZvN/xBMtMBaFE+GvVlNHHDd1NHLlnoMOJF6FUFuY8cI5G9kfmnQ19w99JmIXz4+ UrAe0KHe4xezJj/HYlAzbvZESVWs7kv4oQXn4xXK81XYClQ5P7w7ZoAeQ2hJMaCn iPlxMTa0XincnP9vRKn1FjnYTdlgdvo+FLY3YiNTVOy3W5Gv0JqTk2Aj9UPc0Zb7 nWTclbuYk7O5vWUsDUvPr2BF6+GJbn+7EuBXl/i4CZEwH6U0gEonoPqMKef7/dPs 1tTKYf9vxd4lrCRdQtNiAgY3ilUwNgtpawZqcQG1ro9K9/N7XxNt5yvy5LQGRwKL 2ZKdO9u7ve+p8qed/ZxUkBJUaQ1ToF5Rrc/qUTHz7qIvLoFiS3ImXFIEEBKfoIu+ u6imOilXZLcBKsZE9jUE1kZ43C715jqXHu5PmSIzMUuY0J2hMUO2lfhQZr+NV6U/ mfaFCJb5jhLEePIj38YJc7OIXP9LCrSXooIRKpqumn1cE1eXjyhZPvEwLugBvKJJ s6cK6OlZnMEG7QMIgU5n =g5Qg -----END PGP SIGNATURE----- --SUOF0GtieIMvvwua--