From mboxrd@z Thu Jan 1 00:00:00 1970 From: Shawn Guo Subject: Re: [PATCH v2 0/7] Change snvs rtc and poweroff to use syscon add pwrkey driver Date: Wed, 20 May 2015 22:22:07 +0800 Message-ID: <20150520142205.GA24844@dragon> References: <1432055082-28983-1-git-send-email-Frank.Li@freescale.com> <20150520015851.GB1071@dragon> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pd0-f169.google.com ([209.85.192.169]:33534 "EHLO mail-pd0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752736AbbETOWt (ORCPT ); Wed, 20 May 2015 10:22:49 -0400 Received: by pdbqa5 with SMTP id qa5so70634502pdb.0 for ; Wed, 20 May 2015 07:22:49 -0700 (PDT) Content-Disposition: inline In-Reply-To: Sender: linux-input-owner@vger.kernel.org List-Id: linux-input@vger.kernel.org To: Zhi Li Cc: Shawn Guo , "Frank.Li@freescale.com" , Dmitry Torokhov , Rob Herring , a.zummo@towertech.it, p.zabel@pengutronix.de, alexandre.belloni@free-electrons.com, "devicetree@vger.kernel.org" , rtc-linux@googlegroups.com, linux-pm@vger.kernel.org, linux-input@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" On Tue, May 19, 2015 at 09:44:46PM -0500, Zhi Li wrote: > On Tue, May 19, 2015 at 8:58 PM, Shawn Guo wrote: > > On Wed, May 20, 2015 at 01:04:35AM +0800, Frank.Li@freescale.com wrote: > >> From: Frank Li > >> > >> Change from v1 to v2 > >> - Change simple-bus to simple-mfd > >> - RTC remove lock. > >> - RTC remove comments "These register offsets are relative to LP (Low Power) range" > >> - power off, remove extra space > >> > >> SNVS is MPF module. > >> include rtc, power off, ON/OFF power key > >> > >> arm: imx: power-off: change to syscon to access register > >> rtc: arm: imx: snvs: change use syscon to access register > >> Above two patch change existed power-off and rtc use syscon > >> > >> Document: dt: fsl: snvs: change support syscon > >> arm: dts: imx: update snvs to use syscon access register > >> Update dt file and document > >> > >> input: keyboard: imx: add snvs power key driver > >> document: devicetree: input: imx: i.mx snvs power device tree bindings > >> arm: dts: imx6sx: enable snvs power key > >> Added on/off key driver > >> > >> Frank Li (6): > >> arm: imx: power-off: change to syscon to access register > >> rtc: arm: imx: snvs: change use syscon to access register > >> Document: dt: fsl: snvs: change support syscon > >> arm: dts: imx: update snvs to use syscon access register > >> document: devicetree: input: imx: i.mx snvs power device tree bindings > >> arm: dts: imx6sx: enable snvs power key > > > > So, the series will break existing DTBs for RTC and poweroff, right? > > If this is unavoidable, should we at least have some warning messages > > telling users about his breakage? > > How to do that? > Just added it commit message or need runtime warning? runtime, so that users can know why their installed DTBs break with the new kernel. Shawn From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pa0-f53.google.com (mail-pa0-f53.google.com. [209.85.220.53]) by gmr-mx.google.com with ESMTPS id x3si1925738pdh.2.2015.05.20.07.22.49 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 20 May 2015 07:22:49 -0700 (PDT) Received: by pacwv17 with SMTP id wv17so68864007pac.2 for ; Wed, 20 May 2015 07:22:49 -0700 (PDT) Date: Wed, 20 May 2015 22:22:07 +0800 From: Shawn Guo To: Zhi Li Cc: Shawn Guo , "Frank.Li@freescale.com" , Dmitry Torokhov , Rob Herring , a.zummo@towertech.it, p.zabel@pengutronix.de, alexandre.belloni@free-electrons.com, "devicetree@vger.kernel.org" , rtc-linux@googlegroups.com, linux-pm@vger.kernel.org, linux-input@vger.kernel.org, "linux-arm-kernel@lists.infradead.org" Subject: [rtc-linux] Re: [PATCH v2 0/7] Change snvs rtc and poweroff to use syscon add pwrkey driver Message-ID: <20150520142205.GA24844@dragon> References: <1432055082-28983-1-git-send-email-Frank.Li@freescale.com> <20150520015851.GB1071@dragon> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 In-Reply-To: Reply-To: rtc-linux@googlegroups.com List-ID: List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , On Tue, May 19, 2015 at 09:44:46PM -0500, Zhi Li wrote: > On Tue, May 19, 2015 at 8:58 PM, Shawn Guo wrote: > > On Wed, May 20, 2015 at 01:04:35AM +0800, Frank.Li@freescale.com wrote: > >> From: Frank Li > >> > >> Change from v1 to v2 > >> - Change simple-bus to simple-mfd > >> - RTC remove lock. > >> - RTC remove comments "These register offsets are relative to LP (Low Power) range" > >> - power off, remove extra space > >> > >> SNVS is MPF module. > >> include rtc, power off, ON/OFF power key > >> > >> arm: imx: power-off: change to syscon to access register > >> rtc: arm: imx: snvs: change use syscon to access register > >> Above two patch change existed power-off and rtc use syscon > >> > >> Document: dt: fsl: snvs: change support syscon > >> arm: dts: imx: update snvs to use syscon access register > >> Update dt file and document > >> > >> input: keyboard: imx: add snvs power key driver > >> document: devicetree: input: imx: i.mx snvs power device tree bindings > >> arm: dts: imx6sx: enable snvs power key > >> Added on/off key driver > >> > >> Frank Li (6): > >> arm: imx: power-off: change to syscon to access register > >> rtc: arm: imx: snvs: change use syscon to access register > >> Document: dt: fsl: snvs: change support syscon > >> arm: dts: imx: update snvs to use syscon access register > >> document: devicetree: input: imx: i.mx snvs power device tree bindings > >> arm: dts: imx6sx: enable snvs power key > > > > So, the series will break existing DTBs for RTC and poweroff, right? > > If this is unavoidable, should we at least have some warning messages > > telling users about his breakage? > > How to do that? > Just added it commit message or need runtime warning? runtime, so that users can know why their installed DTBs break with the new kernel. Shawn -- -- You received this message because you are subscribed to "rtc-linux". Membership options at http://groups.google.com/group/rtc-linux . Please read http://groups.google.com/group/rtc-linux/web/checklist before submitting a driver. --- You received this message because you are subscribed to the Google Groups "rtc-linux" group. To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com. For more options, visit https://groups.google.com/d/optout. From mboxrd@z Thu Jan 1 00:00:00 1970 From: shawn.guo@linaro.org (Shawn Guo) Date: Wed, 20 May 2015 22:22:07 +0800 Subject: [PATCH v2 0/7] Change snvs rtc and poweroff to use syscon add pwrkey driver In-Reply-To: References: <1432055082-28983-1-git-send-email-Frank.Li@freescale.com> <20150520015851.GB1071@dragon> Message-ID: <20150520142205.GA24844@dragon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, May 19, 2015 at 09:44:46PM -0500, Zhi Li wrote: > On Tue, May 19, 2015 at 8:58 PM, Shawn Guo wrote: > > On Wed, May 20, 2015 at 01:04:35AM +0800, Frank.Li at freescale.com wrote: > >> From: Frank Li > >> > >> Change from v1 to v2 > >> - Change simple-bus to simple-mfd > >> - RTC remove lock. > >> - RTC remove comments "These register offsets are relative to LP (Low Power) range" > >> - power off, remove extra space > >> > >> SNVS is MPF module. > >> include rtc, power off, ON/OFF power key > >> > >> arm: imx: power-off: change to syscon to access register > >> rtc: arm: imx: snvs: change use syscon to access register > >> Above two patch change existed power-off and rtc use syscon > >> > >> Document: dt: fsl: snvs: change support syscon > >> arm: dts: imx: update snvs to use syscon access register > >> Update dt file and document > >> > >> input: keyboard: imx: add snvs power key driver > >> document: devicetree: input: imx: i.mx snvs power device tree bindings > >> arm: dts: imx6sx: enable snvs power key > >> Added on/off key driver > >> > >> Frank Li (6): > >> arm: imx: power-off: change to syscon to access register > >> rtc: arm: imx: snvs: change use syscon to access register > >> Document: dt: fsl: snvs: change support syscon > >> arm: dts: imx: update snvs to use syscon access register > >> document: devicetree: input: imx: i.mx snvs power device tree bindings > >> arm: dts: imx6sx: enable snvs power key > > > > So, the series will break existing DTBs for RTC and poweroff, right? > > If this is unavoidable, should we at least have some warning messages > > telling users about his breakage? > > How to do that? > Just added it commit message or need runtime warning? runtime, so that users can know why their installed DTBs break with the new kernel. Shawn