Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp3671262imm; Mon, 30 Jul 2018 01:04:48 -0700 (PDT) X-Google-Smtp-Source: AAOMgpfKdbmeFHseGvFl9eCK4aS+WuRjz5rVGHHH258rLc8dbow8fd1VvCmbLKwOJTAXToWcN5ot X-Received: by 2002:a63:d5b:: with SMTP id 27-v6mr15212110pgn.107.1532937888657; Mon, 30 Jul 2018 01:04:48 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1532937888; cv=none; d=google.com; s=arc-20160816; b=RKc+k72ZeF5pNnn1ecR91bIJFnFeAE9v15BcRyYYar+2Xku8OFjPNrZAbbgHwbAtdp ari11WI3wl5Cu7RTmuKlFjvK0WugHl3iJo+pC9Cs7jK4Xwf75GAu9L9BrG7u1gXnJ+GE H1dhynuN1yb3JoKpo9TWHc6FcOkpPLEAk4jsowpW94gypuy58xB0T0U4eAsjXTkSO5sI LSlU05xIwgYMcUc0gt3i/gMjkvjFRu9+ta7HnO/fGW07Z7oY44AhvtloEeNoBB1wKIKt OhiYRzZG1lV3VLsaNl5uTqPFGf6VTVAyql93qx2LAtdglmnFWmrxjLnCnlvKmcbTvKTV 3Z/g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:in-reply-to:mime-version:user-agent:date :message-id:from:references:cc:to:subject:arc-authentication-results; bh=o5As/twZbXKgACkaBRUxrT4J5SFG9Nar3oF/9vAIpPg=; b=ONu/WzFVB4CW3H0TKS/VkAiZXuexDxHsUBu4yLRzlczZqOEIfMNQTFklCUXgPrVldf VpF5/WzXQuI+Yu7++TqU2zhf4yWf5REqUDi3uBfC9QKfELl/AP7y/TKGZfXHCziybP0m vWjdhdv5WRtSLbH3soy1qYRDzOnJFykr1Wf1hxQBb9VvKSJjiRgb0B6rVUKWr+dW09su kYP0znDIwbxa6a0GLlXTjP/NAQuZGKFerScSu/1wFkz7ITSQj1IMMC34z90DH/9nNdUo b1+e57e6cvcefElXycolNLl8vFDPq9lhr+B9bS/Ds4tr3EagIsDkIx2k+v4zl7PHOqx2 +rdQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id k125-v6si10041159pgk.315.2018.07.30.01.04.34; Mon, 30 Jul 2018 01:04:48 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726667AbeG3Jhc (ORCPT + 99 others); Mon, 30 Jul 2018 05:37:32 -0400 Received: from metis.ext.pengutronix.de ([85.220.165.71]:42163 "EHLO metis.ext.pengutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726490AbeG3Jhb (ORCPT ); Mon, 30 Jul 2018 05:37:31 -0400 Received: from soja.hi.pengutronix.de ([2001:67c:670:100:3ad5:47ff:feaf:13da]) by metis.ext.pengutronix.de with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.89) (envelope-from ) id 1fk39P-0000SY-5q; Mon, 30 Jul 2018 10:03:31 +0200 Subject: Re: [PATCH v8 1/6] ARM: imx6q: provide documentation for new fsl,pmic-stby-poweroff property To: Robin Gong , Lucas Stach , Shawn Guo , Mark Brown , "Rafael J. Wysocki" Cc: Mark Rutland , "devicetree@vger.kernel.org" , Michael Turquette , Stephen Boyd , "linux-kernel@vger.kernel.org" , Liam Girdwood , Rob Herring , dl-linux-imx , "kernel@pengutronix.de" , "A.s. Dong" , Fabio Estevam , Russell King , Andrew Morton , Leonard Crestez , "linux-clk@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" References: <20180726092220.17250-1-o.rempel@pengutronix.de> <20180726092220.17250-2-o.rempel@pengutronix.de> <84c1921c-922e-4b5d-b688-db89047fa1db@pengutronix.de> <1532680211.32306.36.camel@pengutronix.de> From: Oleksij Rempel Message-ID: <02f4b642-14a8-f2db-ea23-c42a4f6f1652@pengutronix.de> Date: Mon, 30 Jul 2018 10:03:29 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="0ZXQ3BVq0ww7d6z67tFll8hPf7AVFeQ0a" X-SA-Exim-Connect-IP: 2001:67c:670:100:3ad5:47ff:feaf:13da X-SA-Exim-Mail-From: o.rempel@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: linux-kernel@vger.kernel.org Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --0ZXQ3BVq0ww7d6z67tFll8hPf7AVFeQ0a Content-Type: multipart/mixed; boundary="4qY4n4uWgIxPC5PJDnntgkyLxmMu2YwQx"; protected-headers="v1" From: Oleksij Rempel To: Robin Gong , Lucas Stach , Shawn Guo , Mark Brown , "Rafael J. Wysocki" Cc: Mark Rutland , "devicetree@vger.kernel.org" , Michael Turquette , Stephen Boyd , "linux-kernel@vger.kernel.org" , Liam Girdwood , Rob Herring , dl-linux-imx , "kernel@pengutronix.de" , "A.s. Dong" , Fabio Estevam , Russell King , Andrew Morton , Leonard Crestez , "linux-clk@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" Message-ID: <02f4b642-14a8-f2db-ea23-c42a4f6f1652@pengutronix.de> Subject: Re: [PATCH v8 1/6] ARM: imx6q: provide documentation for new fsl,pmic-stby-poweroff property References: <20180726092220.17250-1-o.rempel@pengutronix.de> <20180726092220.17250-2-o.rempel@pengutronix.de> <84c1921c-922e-4b5d-b688-db89047fa1db@pengutronix.de> <1532680211.32306.36.camel@pengutronix.de> In-Reply-To: --4qY4n4uWgIxPC5PJDnntgkyLxmMu2YwQx Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: quoted-printable On 27.07.2018 10:58, Robin Gong wrote: >=20 >=20 >> -----Original Message----- >> From: Lucas Stach [mailto:l.stach@pengutronix.de] >> Sent: 2018=E5=B9=B47=E6=9C=8827=E6=97=A5 16:30 >> To: Robin Gong ; Oleksij Rempel >> ; Shawn Guo ; Mark >> Brown ; Rafael J. Wysocki >> Cc: Mark Rutland ; devicetree@vger.kernel.org; >> Michael Turquette ; Stephen Boyd >> ; linux-kernel@vger.kernel.org; Liam Girdwood >> ; Rob Herring ; dl-linux-imx >> ; kernel@pengutronix.de; A.s. Dong >> ; Fabio Estevam ; Russell= >> King ; Andrew Morton ; >> Leonard Crestez ; linux-clk@vger.kernel.org; >> linux-arm-kernel@lists.infradead.org >> Subject: Re: [PATCH v8 1/6] ARM: imx6q: provide documentation for new >> fsl,pmic-stby-poweroff property >> >> Hi Robin, >> >> Am Freitag, den 27.07.2018, 01:51 +0000 schrieb Robin Gong: >> [...] >>>>>> --- >>>>>> =C2=A0Documentation/devicetree/bindings/clock/imx6q-clock.txt | 8 >>>>>> ++++++++ >>>>>> =C2=A01 file changed, 8 insertions(+) >>>>>> >>>>>> diff --git >>>>>> a/Documentation/devicetree/bindings/clock/imx6q-clock.txt >>>>>> b/Documentation/devicetree/bindings/clock/imx6q-clock.txt >>>>>> index a45ca67a9d5f..e1308346e00d 100644 >>>>>> --- a/Documentation/devicetree/bindings/clock/imx6q-clock.txt >>>>>> +++ b/Documentation/devicetree/bindings/clock/imx6q-clock.txt >>>>>> @@ -6,6 +6,14 @@ Required properties: >>>>>> =C2=A0- interrupts: Should contain CCM interrupt >>>>>> =C2=A0- #clock-cells: Should be <1> >>>>>> >>>>>> +Optional properties: >>>>>> +- fsl,pmic-stby-poweroff: Configure CCM to assert PMIC_STBY_REQ >>>>>> +signal >>>>>> +=C2=A0=C2=A0on power off. >>>>>> +=C2=A0=C2=A0Use this property if the SoC should be powered off by= >>>>>> +external power >>>>>> +=C2=A0=C2=A0management IC (PMIC) triggered via PMIC_STBY_REQ sign= al. >>>>> >>>>> PMIC_ON_REQ didn't connect to any pin of PMIC in your case? >>>> >>>> No. First, it was only one customer specific issue. After some >>>> research I found even publicly available boards (for example >>>> RioTboard) which has same/similar design. After seeing this in imx6 >>>> documentation as valid power off way, I have no doubts - there shoul= d be >> even more devices doin this in the wild. >>> >>> Not sure why the customer didn't follow reference design, since >>> PMIC_ON_REQ can provide power off/on feature by pressing ONOFF key >>> which connected ONOFF pin of i.mx6(ONOFF will toggle PMIC_ON_REQ to >>> power off/on PMIC). The official power off/on way (PMIC_ON_REQ) can >>> also power off all power rails of PFUZE except snvs as your patch did= >>> on PMIC_STBY_REQ.=C2=A0=C2=A0PMIC_STBY_REQ is used to notify pmic swi= tch power >>> mode (PFM/APS) or decrease voltage to save power in kernel suspend, n= ot >> power off....I am not sure if we need this patchset to 'workaround' th= e board >> issue. >>> >> Not all boards follow the reference design, that's a fact of life. >> >> Please look at the i.MX6Q reference manual. The sequence implemented i= n this >> patchset can be found as a valid way to power off the system in >> "60.4.3 Power mode transitions" "Normal ON to OFF with external PMIC",= so >> there is hardly any way to argue that this is a board specific quirk. = This is one of >> the Freescale/NXP recommended sequences to turn off the system. > Okay, but could you add one more comment for this solution? RTC alarm a= nd ONOFF > Button wakeup feature can't be support in this case. Enough to add it in to changelog? or should it go to the binding documentation? --4qY4n4uWgIxPC5PJDnntgkyLxmMu2YwQx-- --0ZXQ3BVq0ww7d6z67tFll8hPf7AVFeQ0a Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAEBCAAdFiEEpENFL0P3hvQ7p0DDdQOiSHVI77QFAltexlEACgkQdQOiSHVI 77TWfAgAjRB1PciUy1OHomvrxni8EkaYEB80z17EC958utpp0mPJwNuFY56/8HFJ oCrfPZ7bn+/N0K5iCATabDyC4ylmjEGboht6je0eOXtoOqlWU/e+wMRp/yNSlRsw 1/mQAHKWuVIlBopDYuIrNHxR9fYcmJCcOzGEb3u9vFeEJL9G8jda7IgG9ToeKTlm YW9F5TmlabqPZ57WLDK9AXzpW+3W1jm57VCUZesIkIyoNRYboKhPbvpS1UOr2NHi rZcwhrIG5CIMWnbxgWAlnNhhG2lF+w1D3wQdqURZns6RmNeDQUQ6OL0bIkLZYy+m U7/apbmR1x8qalt+rjgpuFFw9CcxoQ== =LKCD -----END PGP SIGNATURE----- --0ZXQ3BVq0ww7d6z67tFll8hPf7AVFeQ0a--