From: "Frank Wunderlich" <frank-w@public-files.de>
To: "Ran Bi" <ran.bi@mediatek.com>
Cc: "Lee Jones" <lee.jones@linaro.org>,
"Rob Herring" <robh+dt@kernel.org>,
"Mark Rutland" <mark.rutland@arm.com>,
"Matthias Brugger" <matthias.bgg@gmail.com>,
"Sean Wang" <sean.wang@mediatek.com>,
"Sebastian Reichel" <sre@kernel.org>,
"Alessandro Zummo" <a.zummo@towertech.it>,
"Alexandre Belloni" <alexandre.belloni@bootlin.com>,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
linux-mediatek@lists.infradead.org, linux-kernel@vger.kernel.org,
linux-pm@vger.kernel.org, linux-rtc@vger.kernel.org,
"Eddie Huang" <eddie.huang@mediatek.com>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Richard Fontana" <rfontana@redhat.com>,
"Allison Randal" <allison@lohutok.net>,
"David S . Miller" <davem@davemloft.net>,
"Mauro Carvalho Chehab" <mchehab+samsung@kernel.org>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Rob Herring" <robh@kernel.org>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Nicolas Ferre" <nicolas.ferre@microchip.com>,
"Paul E . McKenney" <paulmck@linux.ibm.com>,
"Josef Friedl" <josef.friedl@speed.at>,
"Yingjoe Chen" <yingjoe.chen@mediatek.com>,
"Ran Bi" <ran.bi@mediatek.com>
Subject: Aw: Re: [PATCH v2 5/7] power: reset: add driver for mt6323 poweroff
Date: Thu, 4 Jul 2019 13:06:31 +0200 [thread overview]
Message-ID: <trinity-03acdbbe-a94a-439c-a65c-88c9e23a0ae7-1562238391586@3c-app-gmx-bs27> (raw)
In-Reply-To: <1562234589.19751.16.camel@mhfsdcap03>
> Gesendet: Donnerstag, 04. Juli 2019 um 12:03 Uhr
> Von: "Ran Bi" <ran.bi@mediatek.com>
> We had implement MT8173 poweroff function in arm-trusted-firmware's PSCI
> plat_system_off() function. MT8173 SoC is using PMIC MT6397. (Ref:
> https://github.com/ARM-software/arm-trusted-firmware/blob/master/plat/mediatek/mt8173/plat_pm.c and https://github.com/ARM-software/arm-trusted-firmware/blob/master/plat/mediatek/mt8173/drivers/rtc) Do you think it's better to implement poweroff function into arm-trusted-firmware compared to hijack pm_poweroff() function in Kernel? Right now, we are doing the upstream of other PMIC chip like MT6358's poweroff function in arm-trusted-firmware too.
ATF imho only used for arm64, my board is 32bit armv7 and i (currently) do not boot up with ATF
regards Frank
WARNING: multiple messages have this Message-ID (diff)
From: "Frank Wunderlich" <frank-w-SipyoOjXbOMoAHOVJHB0wA@public.gmane.org>
Cc: Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Alexandre Belloni
<alexandre.belloni-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>,
Linus Walleij
<linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Richard Fontana
<rfontana-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>,
Mauro Carvalho Chehab
<mchehab+samsung-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
"Paul E . McKenney"
<paulmck-tEXmvtCZX7AybS5Ee8rs3A@public.gmane.org>,
Lee Jones <lee.jones-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
linux-rtc-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
Yingjoe Chen
<yingjoe.chen-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Nicolas Ferre
<nicolas.ferre-UWL1GkI3JZL3oGB3hsPCZA@public.gmane.org>,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-pm-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Ran Bi <ran.bi-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Sean Wang <sean.wang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
linux-mediatek-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
Matthias Brugger
<matthias.bgg-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Thomas Gleixner <tglx-hfZtesqFncYOwBW4kG4KsQ@public.gmane.org>,
Eddie Huang <eddie.huang-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>,
Allison Randal <allison-wcfNWazeolPR7s880joybQ@public.gmane.org>,
Alessandro Zummo <a.z>
Subject: Aw: Re: [PATCH v2 5/7] power: reset: add driver for mt6323 poweroff
Date: Thu, 4 Jul 2019 13:06:31 +0200 [thread overview]
Message-ID: <trinity-03acdbbe-a94a-439c-a65c-88c9e23a0ae7-1562238391586@3c-app-gmx-bs27> (raw)
In-Reply-To: <1562234589.19751.16.camel@mhfsdcap03>
> Gesendet: Donnerstag, 04. Juli 2019 um 12:03 Uhr
> Von: "Ran Bi" <ran.bi-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
> We had implement MT8173 poweroff function in arm-trusted-firmware's PSCI
> plat_system_off() function. MT8173 SoC is using PMIC MT6397. (Ref:
> https://github.com/ARM-software/arm-trusted-firmware/blob/master/plat/mediatek/mt8173/plat_pm.c and https://github.com/ARM-software/arm-trusted-firmware/blob/master/plat/mediatek/mt8173/drivers/rtc) Do you think it's better to implement poweroff function into arm-trusted-firmware compared to hijack pm_poweroff() function in Kernel? Right now, we are doing the upstream of other PMIC chip like MT6358's poweroff function in arm-trusted-firmware too.
ATF imho only used for arm64, my board is 32bit armv7 and i (currently) do not boot up with ATF
regards Frank
WARNING: multiple messages have this Message-ID (diff)
From: "Frank Wunderlich" <frank-w@public-files.de>
To: "Ran Bi" <ran.bi@mediatek.com>
Cc: Mark Rutland <mark.rutland@arm.com>,
Alexandre Belloni <alexandre.belloni@bootlin.com>,
Linus Walleij <linus.walleij@linaro.org>,
linux-kernel@vger.kernel.org,
Richard Fontana <rfontana@redhat.com>,
Mauro Carvalho Chehab <mchehab+samsung@kernel.org>,
"Paul E . McKenney" <paulmck@linux.ibm.com>,
Lee Jones <lee.jones@linaro.org>,
linux-rtc@vger.kernel.org, Rob Herring <robh@kernel.org>,
linux-arm-kernel@lists.infradead.org,
Yingjoe Chen <yingjoe.chen@mediatek.com>,
devicetree@vger.kernel.org, linux-pm@vger.kernel.org,
Ran Bi <ran.bi@mediatek.com>, Sean Wang <sean.wang@mediatek.com>,
Rob Herring <robh+dt@kernel.org>,
linux-mediatek@lists.infradead.org,
Matthias Brugger <matthias.bgg@gmail.com>,
Thomas Gleixner <tglx@linutronix.de>,
Eddie Huang <eddie.huang@mediatek.com>,
Allison Randal <allison@lohutok.net>,
Alessandro Zummo <a.zummo@towertech.it>,
Josef Friedl <josef.friedl@speed.at>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Sebastian Reichel <sre@kernel.org>,
"David S . Miller" <davem@davemloft.net>
Subject: Aw: Re: [PATCH v2 5/7] power: reset: add driver for mt6323 poweroff
Date: Thu, 4 Jul 2019 13:06:31 +0200 [thread overview]
Message-ID: <trinity-03acdbbe-a94a-439c-a65c-88c9e23a0ae7-1562238391586@3c-app-gmx-bs27> (raw)
In-Reply-To: <1562234589.19751.16.camel@mhfsdcap03>
> Gesendet: Donnerstag, 04. Juli 2019 um 12:03 Uhr
> Von: "Ran Bi" <ran.bi@mediatek.com>
> We had implement MT8173 poweroff function in arm-trusted-firmware's PSCI
> plat_system_off() function. MT8173 SoC is using PMIC MT6397. (Ref:
> https://github.com/ARM-software/arm-trusted-firmware/blob/master/plat/mediatek/mt8173/plat_pm.c and https://github.com/ARM-software/arm-trusted-firmware/blob/master/plat/mediatek/mt8173/drivers/rtc) Do you think it's better to implement poweroff function into arm-trusted-firmware compared to hijack pm_poweroff() function in Kernel? Right now, we are doing the upstream of other PMIC chip like MT6358's poweroff function in arm-trusted-firmware too.
ATF imho only used for arm64, my board is 32bit armv7 and i (currently) do not boot up with ATF
regards Frank
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-07-04 11:07 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-03 16:48 [PATCH v2 0/7] implement poweroff for mt6323/6397 Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` [PATCH v2 1/7] docs: dt-bindings: add poweroff Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-04 18:54 ` Alexandre Belloni
2019-07-04 18:54 ` Alexandre Belloni
2019-07-04 18:54 ` Alexandre Belloni
2019-07-03 16:48 ` [PATCH v2 2/7] rtc: mt6397: move some common definitions into rtc.h Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-04 9:11 ` Matthias Brugger
2019-07-04 9:11 ` Matthias Brugger
2019-07-04 9:11 ` Matthias Brugger
2019-07-04 11:49 ` Aw: " Frank Wunderlich
2019-07-04 11:49 ` Frank Wunderlich
2019-07-04 11:49 ` Frank Wunderlich
2019-07-03 16:48 ` [PATCH v2 3/7] rtc: mt6397: improvements of rtc driver Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-04 9:13 ` Matthias Brugger
2019-07-04 9:13 ` Matthias Brugger
2019-07-04 9:13 ` Matthias Brugger
2019-07-04 11:08 ` Aw: " Frank Wunderlich
2019-07-04 11:08 ` Frank Wunderlich
2019-07-04 11:08 ` Frank Wunderlich
2019-07-04 20:43 ` Alexandre Belloni
2019-07-04 20:43 ` Alexandre Belloni
2019-07-04 20:43 ` Alexandre Belloni
2019-07-05 15:35 ` Aw: " Frank Wunderlich
2019-07-05 15:35 ` Frank Wunderlich
2019-07-05 15:35 ` Frank Wunderlich
2019-07-05 21:24 ` Alexandre Belloni
2019-07-05 21:24 ` Alexandre Belloni
2019-07-05 21:24 ` Alexandre Belloni
2019-07-06 16:15 ` Aw: " Frank Wunderlich
2019-07-06 16:15 ` Frank Wunderlich
2019-07-06 16:15 ` Frank Wunderlich
2019-07-06 20:04 ` Alexandre Belloni
2019-07-06 20:04 ` Alexandre Belloni
2019-07-06 20:04 ` Alexandre Belloni
2019-07-03 16:48 ` [PATCH v2 4/7] mfd: mt6323: some improvements of mt6397-core Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` [PATCH v2 5/7] power: reset: add driver for mt6323 poweroff Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-04 9:15 ` Matthias Brugger
2019-07-04 9:15 ` Matthias Brugger
2019-07-04 9:15 ` Matthias Brugger
[not found] ` <d162c95d-773e-f364-0e06-07f67c5b0cbc-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2019-07-04 10:55 ` Frank Wunderlich
2019-07-04 10:03 ` Ran Bi
2019-07-04 10:03 ` Ran Bi
2019-07-04 10:03 ` Ran Bi
2019-07-04 11:06 ` Frank Wunderlich [this message]
2019-07-04 11:06 ` Aw: " Frank Wunderlich
2019-07-04 11:06 ` Frank Wunderlich
2019-07-03 16:48 ` [PATCH v2 6/7] MAINTAINERS: add Mediatek shutdown drivers Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` [PATCH v2 7/7] arm: dts: mt6323: add keys, power-controller, rtc and codec Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
2019-07-03 16:48 ` Frank Wunderlich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=trinity-03acdbbe-a94a-439c-a65c-88c9e23a0ae7-1562238391586@3c-app-gmx-bs27 \
--to=frank-w@public-files.de \
--cc=a.zummo@towertech.it \
--cc=alexandre.belloni@bootlin.com \
--cc=allison@lohutok.net \
--cc=davem@davemloft.net \
--cc=devicetree@vger.kernel.org \
--cc=eddie.huang@mediatek.com \
--cc=gregkh@linuxfoundation.org \
--cc=josef.friedl@speed.at \
--cc=lee.jones@linaro.org \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-rtc@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=mchehab+samsung@kernel.org \
--cc=nicolas.ferre@microchip.com \
--cc=paulmck@linux.ibm.com \
--cc=ran.bi@mediatek.com \
--cc=rfontana@redhat.com \
--cc=robh+dt@kernel.org \
--cc=robh@kernel.org \
--cc=sean.wang@mediatek.com \
--cc=sre@kernel.org \
--cc=tglx@linutronix.de \
--cc=yingjoe.chen@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.