Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1751863AbbFXH5O (ORCPT <rfc822;w@1wt.eu>);
	Wed, 24 Jun 2015 03:57:14 -0400
Received: from mail-vn0-f49.google.com ([209.85.216.49]:46673 "EHLO
	mail-vn0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1750906AbbFXH5F (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 24 Jun 2015 03:57:05 -0400
MIME-Version: 1.0
In-Reply-To: <1435130188-4087-1-git-send-email-linux.amoon@gmail.com>
References: <1435130188-4087-1-git-send-email-linux.amoon@gmail.com>
Date: Wed, 24 Jun 2015 16:57:04 +0900
X-Google-Sender-Auth: fyaLnaKvMA2D1q6KOVCnZDt_BgA
Message-ID: <CAJKOXPewv=_aOjf=ot-n8hsE=FEmLzHGsafSQiRngLhMU2BjEg@mail.gmail.com>
Subject: Re: [PATCH] regulator: s2mps11: Added shutdown function to poweroff Odroid-XU3
From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
To: Anand Moon <linux.amoon@gmail.com>
Cc: Sangbeom Kim <sbkim73@samsung.com>, Liam Girdwood <lgirdwood@gmail.com>,
        Mark Brown <broonie@kernel.org>, linux-kernel@vger.kernel.org,
        linux-samsung-soc@vger.kernel.org
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 2721
Lines: 68

2015-06-24 16:16 GMT+09:00 Anand Moon <linux.amoon@gmail.com>:
> Added .shutdown function to s2mps11 to help poweroff the board succefully.

s/succefully/successfully/

> The device drivers set the register to turn off the PMIC.

Driver or drivers?

>
> Signed-off-by: Anand Moon <linux.amoon@gmail.com>
> ---
> Changes fixes the poweroff
> root@odroidxu3:~# poweroff
>
> Broadcast message from root@odroidxu3
>         (/dev/ttySAC2) at 13:08 ...
>
> The system is going down for power off NOW!
> root@odroidxu3:~# wait-for-state stop/waiting
>  * Stopping rsync daemon rsync                                           [ OK ]
>  * Stopping RDP Session manager                                          [ OK ]
>  * Stopping NTP server ntpd                                              [ OK ]
>  * Asking all remaining processes to terminate...                        [ OK ]
>  * All processes ended within 1 seconds...                               [ OK ]
> nm-dispatcher.action: Caught signal 15, shutting down...
> ModemManager[2134]: <warn>  Could not acquire the 'org.freedesktop.ModemManager1' service name
>
> ModemManager[2134]: <info>  ModemManager is shut down
>
>  * Unmounting temporary filesystems...                                   [ OK ]
>  * Deactivating swap...                                                  [ OK ]
>  * Unmounting local filesystems...                                       [ OK ]
>  * Will now halt
> [  209.020280] reboot: Power down
> [  209.122039] Power down failed, please power off system manually.
> ---
>  drivers/regulator/s2mps11.c | 8 ++++++++
>  1 file changed, 8 insertions(+)
>
> diff --git a/drivers/regulator/s2mps11.c b/drivers/regulator/s2mps11.c
> index ff82811..871f7b8 100644
> --- a/drivers/regulator/s2mps11.c
> +++ b/drivers/regulator/s2mps11.c
> @@ -1060,6 +1060,13 @@ out:
>         return ret;
>  }
>
> +static void s2mps11_pmic_shutdown(struct platform_device *pdev)
> +{
> +       struct sec_pmic_dev *iodev = dev_get_drvdata(pdev->dev.parent);
> +
> +       regmap_update_bits(iodev->regmap_pmic, S2MPS11_REG_CTRL1, 0xff, 0x00);

This looks odd to me and interesting in the same time...
1. Why clearing all of the fields from the register? Don't you want to
clear only one of it?
2. What exactly you want to do here? What is expected behaviour?
3. How this relates to PWRHOLD coming from AP to the PMIC?
4. Why this is needed only for this driver and only for this board (Odroid XU3)?

Best regards,
Krzysztof
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/