Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1689695 > unrolled thread

Re: [PATCH v6 0/6] provide power off support for iMX6 with external PMIC

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH v6 0/6] provide power off support for iMX6 with external  PMIC Oleksij Rempel <ore@pengutronix.de> - 2017-07-18 07:00 +0200
    Re: [PATCH v6 0/6] provide power off support for iMX6 with external  PMIC Mark Brown <broonie@kernel.org> - 2017-07-20 14:40 +0200
      Re: [PATCH v6 0/6] provide power off support for iMX6 with external  PMIC Russell King - ARM Linux <linux@armlinux.org.uk> - 2017-07-20 15:20 +0200

#1689695 — Re: [PATCH v6 0/6] provide power off support for iMX6 with external PMIC

Hallo all,

Here Gentle Ping to the Old King

DT is already ACKed.

On 20.06.2017 16:09, Oleksij Rempel wrote:
> This patch series is providing power off support for Freescale/NXP iMX6 based
> boards with external power management integrated circuit (PMIC).
>
> changes:
> v6:
>  - rename imx6_pm_poweroff to imx6_pm_stby_poweroff
>  - fix "MPIC_STBY_REQ" typo in the comment.
>
> v5:
>  - remove useless includes from pm-imx6.c patch
>  - add Acked-by to "regulator: pfuze100: add fsl,pmic-stby-poweroff property"
>    patch
>
> v4:
>  - update comment in "regulator: pfuze100: add fsl,pmic-stby-poweroff ..."
>    patch
>  - add Acked-by to "ARM: imx6q: provide documentation for new ..."
>    patch
>
> v3:
>  - set pm_power_off_prepare = NULL on .remove.
>  - documentation and spelling fixes.
>  - use %pf instead of lookup_symbol_name.
>
> Oleksij Rempel (6):
>   ARM: imx6q: provide documentation for new fsl,pmic-stby-poweroff
>     property
>   ARM: imx6: register pm_power_off handler if "fsl,pmic-stby-poweroff"
>     is set
>   kernel/reboot.c: export pm_power_off_prepare
>   regulator: pfuze100: add fsl,pmic-stby-poweroff property
>   regulator: pfuze100-regulator: provide pm_power_off_prepare handler
>   ARM: dts: imx6: RIoTboard provide standby on power off option
>
>  .../devicetree/bindings/clock/imx6q-clock.txt      |  8 ++
>  .../devicetree/bindings/regulator/pfuze100.txt     |  7 ++
>  arch/arm/boot/dts/imx6dl-riotboard.dts             |  5 ++
>  arch/arm/mach-imx/pm-imx6.c                        | 25 ++++++
>  drivers/regulator/pfuze100-regulator.c             | 92 ++++++++++++++++++++++
>  kernel/reboot.c                                    |  1 +
>  6 files changed, 138 insertions(+)
>

[toc] | [next] | [standalone]

#1692885

[Multipart message — attachments visible in raw view] — view raw

On Tue, Jul 18, 2017 at 06:53:54AM +0200, Oleksij Rempel wrote:
> Hallo all,
> 
> Here Gentle Ping to the Old King
> 
> DT is already ACKed.

Please don't send content free pings and please allow a reasonable time
for review.  People get busy, go on holiday, attend conferences and so 
on so unless there is some reason for urgency (like critical bug fixes)
please allow at least a couple of weeks for review.  If there have been
review comments then people may be waiting for those to be addressed.

Sending content free pings adds to the mail volume (if they are seen at
all) which is often the problem and since they can't be reviewed
directly if something has gone wrong you'll have to resend the patches
anyway, though there are some other maintainers who like them - if in
doubt look at how patches for the subsystem are normally handled.

[toc] | [prev] | [next] | [standalone]

#1692921

On Thu, Jul 20, 2017 at 01:37:01PM +0100, Mark Brown wrote:
> On Tue, Jul 18, 2017 at 06:53:54AM +0200, Oleksij Rempel wrote:
> > Hallo all,
> > 
> > Here Gentle Ping to the Old King
> > 
> > DT is already ACKed.
> 
> Please don't send content free pings and please allow a reasonable time
> for review.  People get busy, go on holiday, attend conferences and so 
> on so unless there is some reason for urgency (like critical bug fixes)
> please allow at least a couple of weeks for review.  If there have been
> review comments then people may be waiting for those to be addressed.
> 
> Sending content free pings adds to the mail volume (if they are seen at
> all) which is often the problem and since they can't be reviewed
> directly if something has gone wrong you'll have to resend the patches
> anyway, though there are some other maintainers who like them - if in
> doubt look at how patches for the subsystem are normally handled.

It also doesn't help that there's little context about who Oleksij is
expecting a reply from.  There's 11 individuals in the To: header.
Is the reference above refering to me?  Shrug, I don't know.  I doubt
it because I'm not maintaining the imx6 SoC support.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web