diff mbox

ARM: dts: mxs: Fix the RTC compatible prop on M28EVK

Message ID 1390519416-6760-1-git-send-email-marex@denx.de (mailing list archive)
State New, archived
Headers show

Commit Message

Marek Vasut Jan. 23, 2014, 11:23 p.m. UTC
The compatible property should be m41t62, not mt41t62, so fix this.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Shawn Guo <shawn.guo@linaro.org>
---
 arch/arm/boot/dts/imx28-m28evk.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marek Vasut Feb. 20, 2014, 12:31 p.m. UTC | #1
On Friday, January 24, 2014 at 12:23:36 AM, Marek Vasut wrote:
> The compatible property should be m41t62, not mt41t62, so fix this.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Shawn Guo <shawn.guo@linaro.org>

Bump ?

Best regards,
Marek Vasut
Shawn Guo Feb. 21, 2014, 2:09 a.m. UTC | #2
On Thu, Feb 20, 2014 at 01:31:37PM +0100, Marek Vasut wrote:
> On Friday, January 24, 2014 at 12:23:36 AM, Marek Vasut wrote:
> > The compatible property should be m41t62, not mt41t62, so fix this.
> > 
> > Signed-off-by: Marek Vasut <marex@denx.de>
> > Cc: Shawn Guo <shawn.guo@linaro.org>
> 
> Bump ?

Applied.  Sorry for the delay.

Shawn
Marek Vasut Feb. 21, 2014, 3:14 a.m. UTC | #3
On Friday, February 21, 2014 at 03:09:16 AM, Shawn Guo wrote:
> On Thu, Feb 20, 2014 at 01:31:37PM +0100, Marek Vasut wrote:
> > On Friday, January 24, 2014 at 12:23:36 AM, Marek Vasut wrote:
> > > The compatible property should be m41t62, not mt41t62, so fix this.
> > > 
> > > Signed-off-by: Marek Vasut <marex@denx.de>
> > > Cc: Shawn Guo <shawn.guo@linaro.org>
> > 
> > Bump ?
> 
> Applied.  Sorry for the delay.

Sure, no problem. Thanks. Shall I submit this for -stable too ?

Best regards,
Marek Vasut
Shawn Guo Feb. 21, 2014, 3:31 a.m. UTC | #4
On Fri, Feb 21, 2014 at 04:14:00AM +0100, Marek Vasut wrote:
> On Friday, February 21, 2014 at 03:09:16 AM, Shawn Guo wrote:
> > On Thu, Feb 20, 2014 at 01:31:37PM +0100, Marek Vasut wrote:
> > > On Friday, January 24, 2014 at 12:23:36 AM, Marek Vasut wrote:
> > > > The compatible property should be m41t62, not mt41t62, so fix this.
> > > > 
> > > > Signed-off-by: Marek Vasut <marex@denx.de>
> > > > Cc: Shawn Guo <shawn.guo@linaro.org>
> > > 
> > > Bump ?
> > 
> > Applied.  Sorry for the delay.
> 
> Sure, no problem. Thanks. Shall I submit this for -stable too ?

Oh, at this time which already passes -rc3, I didn't categorize it as a
critical fix (build error, kernel dumps, system hang) that we should
send right away for 3.14 and -stable.  If you do, you can try to resend
it to arm-soc folks with my ACK.

Shawn
Marek Vasut Feb. 24, 2014, 6:02 p.m. UTC | #5
On Friday, February 21, 2014 at 04:31:06 AM, Shawn Guo wrote:
> On Fri, Feb 21, 2014 at 04:14:00AM +0100, Marek Vasut wrote:
> > On Friday, February 21, 2014 at 03:09:16 AM, Shawn Guo wrote:
> > > On Thu, Feb 20, 2014 at 01:31:37PM +0100, Marek Vasut wrote:
> > > > On Friday, January 24, 2014 at 12:23:36 AM, Marek Vasut wrote:
> > > > > The compatible property should be m41t62, not mt41t62, so fix this.
> > > > > 
> > > > > Signed-off-by: Marek Vasut <marex@denx.de>
> > > > > Cc: Shawn Guo <shawn.guo@linaro.org>
> > > > 
> > > > Bump ?
> > > 
> > > Applied.  Sorry for the delay.
> > 
> > Sure, no problem. Thanks. Shall I submit this for -stable too ?
> 
> Oh, at this time which already passes -rc3, I didn't categorize it as a
> critical fix (build error, kernel dumps, system hang) that we should
> send right away for 3.14 and -stable.  If you do, you can try to resend
> it to arm-soc folks with my ACK.

So this will go for 3.15 ? I will submit it for stable once it hits mainline, 
it's not critical.

Thanks!

Best regards,
Marek Vasut
Marek Vasut July 22, 2014, 2:11 a.m. UTC | #6
On Friday, February 21, 2014 at 04:31:06 AM, Shawn Guo wrote:
> On Fri, Feb 21, 2014 at 04:14:00AM +0100, Marek Vasut wrote:
> > On Friday, February 21, 2014 at 03:09:16 AM, Shawn Guo wrote:
> > > On Thu, Feb 20, 2014 at 01:31:37PM +0100, Marek Vasut wrote:
> > > > On Friday, January 24, 2014 at 12:23:36 AM, Marek Vasut wrote:
> > > > > The compatible property should be m41t62, not mt41t62, so fix this.
> > > > > 
> > > > > Signed-off-by: Marek Vasut <marex@denx.de>
> > > > > Cc: Shawn Guo <shawn.guo@linaro.org>
> > > > 
> > > > Bump ?
> > > 
> > > Applied.  Sorry for the delay.
> > 
> > Sure, no problem. Thanks. Shall I submit this for -stable too ?
> 
> Oh, at this time which already passes -rc3, I didn't categorize it as a
> critical fix (build error, kernel dumps, system hang) that we should
> send right away for 3.14 and -stable.  If you do, you can try to resend
> it to arm-soc folks with my ACK.

Is there anything special to get this into 3.14-stable now or shall I just 
follow the general stable submission guidelines ?

Best regards,
Marek Vasut
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx28-m28evk.dts b/arch/arm/boot/dts/imx28-m28evk.dts
index 8536b0d..f0ad7b9 100644
--- a/arch/arm/boot/dts/imx28-m28evk.dts
+++ b/arch/arm/boot/dts/imx28-m28evk.dts
@@ -194,7 +194,7 @@ 
 				};
 
 				rtc: rtc@68 {
-					compatible = "stm,mt41t62";
+					compatible = "stm,m41t62";
 					reg = <0x68>;
 				};
 			};