All of lore.kernel.org
 help / color / mirror / Atom feed
From: Peter Korsgaard <jacmet@sunsite.dk>
To: "AnilKumar, Chimata" <anilkumar@ti.com>
Cc: "a.zummo@towertech.it" <a.zummo@towertech.it>,
	"sameo@linux.intel.com" <sameo@linux.intel.com>,
	"tony@atomide.com" <tony@atomide.com>,
	Colin Foe-Parker <colin.foeparker@logicpd.com>,
	"rtc-linux@googlegroups.com" <rtc-linux@googlegroups.com>,
	"devicetree-discuss@lists.ozlabs.org"
	<devicetree-discuss@lists.ozlabs.org>,
	"broonie@opensource.wolfsonmicro.com"
	<broonie@opensource.wolfsonmicro.com>,
	"rob.herring@calxeda.com" <rob.herring@calxeda.com>,
	"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2 1/4] mfd: tps65217: Set PMIC to shutdowm on PWR_EN toggle
Date: Mon, 19 Nov 2012 13:17:59 +0100	[thread overview]
Message-ID: <87r4nplrbc.fsf@dell.be.48ers.dk> (raw)
In-Reply-To: <331ABD5ECB02734CA317220B2BBEABC13EA760FD@DBDE01.ent.ti.com> (Chimata AnilKumar's message of "Mon, 19 Nov 2012 11:36:57 +0000")

>>>>> "A" == AnilKumar, Chimata <anilkumar@ti.com> writes:

Hi,

 >> I don't feel this property name is very clear. Perhaps something about
 >> power button or push button monitor as it is called in the datasheet?

 A> Here I am writing "TPS65217_STATUS_OFF" to status register, BIT(7).
 A> Description of this bit field: "OFF bit. Set this bit to 1 to enter
 A> OFF state when PWR_EN pin is pulled low. Bit is automatically reset
 A> to 0."

 A> PWR_EN pin pull-down is doing by RTC module.

Ahh yes, sorry -ENOCOFFEE. I somehow misread the datasheet and looked at
bit 0 instead.

-- 
Bye, Peter Korsgaard

WARNING: multiple messages have this Message-ID (diff)
From: jacmet@sunsite.dk (Peter Korsgaard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 1/4] mfd: tps65217: Set PMIC to shutdowm on PWR_EN toggle
Date: Mon, 19 Nov 2012 13:17:59 +0100	[thread overview]
Message-ID: <87r4nplrbc.fsf@dell.be.48ers.dk> (raw)
In-Reply-To: <331ABD5ECB02734CA317220B2BBEABC13EA760FD@DBDE01.ent.ti.com> (Chimata AnilKumar's message of "Mon, 19 Nov 2012 11:36:57 +0000")

>>>>> "A" == AnilKumar, Chimata <anilkumar@ti.com> writes:

Hi,

 >> I don't feel this property name is very clear. Perhaps something about
 >> power button or push button monitor as it is called in the datasheet?

 A> Here I am writing "TPS65217_STATUS_OFF" to status register, BIT(7).
 A> Description of this bit field: "OFF bit. Set this bit to 1 to enter
 A> OFF state when PWR_EN pin is pulled low. Bit is automatically reset
 A> to 0."

 A> PWR_EN pin pull-down is doing by RTC module.

Ahh yes, sorry -ENOCOFFEE. I somehow misread the datasheet and looked at
bit 0 instead.

-- 
Bye, Peter Korsgaard

  reply	other threads:[~2012-11-19 12:17 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-16 12:08 [PATCH v2 0/4] pm: Add power off control AnilKumar Ch
2012-11-16 12:08 ` AnilKumar Ch
     [not found] ` <1353067706-29491-1-git-send-email-anilkumar-l0cyMroinI0@public.gmane.org>
2012-11-16 12:08   ` [PATCH v2 1/4] mfd: tps65217: Set PMIC to shutdowm on PWR_EN toggle AnilKumar Ch
2012-11-16 12:08     ` AnilKumar Ch
2012-11-16 18:53     ` Kevin Hilman
2012-11-16 18:53       ` Kevin Hilman
2012-11-19 10:50     ` Peter Korsgaard
2012-11-19 10:50       ` Peter Korsgaard
     [not found]       ` <87a9udn9xo.fsf-D6SC8u56vOOJDPpyT6T3/w@public.gmane.org>
2012-11-19 11:36         ` AnilKumar, Chimata
2012-11-19 11:36           ` AnilKumar, Chimata
2012-11-19 12:17           ` Peter Korsgaard [this message]
2012-11-19 12:17             ` Peter Korsgaard
2012-11-16 12:08   ` [PATCH v2 2/4] rtc: OMAP: Add system pm_power_off to rtc driver AnilKumar Ch
2012-11-16 12:08     ` AnilKumar Ch
2012-11-16 12:08   ` [PATCH v2 3/4] ARM: dts: AM33XX: Set pmic-shutdown-controller for BeagleBone AnilKumar Ch
2012-11-16 12:08     ` AnilKumar Ch
2012-11-16 12:08   ` [PATCH v2 4/4] ARM: dts: AM33XX: Enable system power off control in am335x-bone AnilKumar Ch
2012-11-16 12:08     ` AnilKumar Ch
2012-11-16 19:10     ` Kevin Hilman
2012-11-16 19:10       ` Kevin Hilman
     [not found]       ` <87r4ntcqk3.fsf-1D3HCaltpLuhEniVeURVKkEOCMrvLtNR@public.gmane.org>
2012-11-19  4:14         ` AnilKumar, Chimata
2012-11-19  4:14           ` AnilKumar, Chimata

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=87r4nplrbc.fsf@dell.be.48ers.dk \
    --to=jacmet@sunsite.dk \
    --cc=a.zummo@towertech.it \
    --cc=anilkumar@ti.com \
    --cc=broonie@opensource.wolfsonmicro.com \
    --cc=colin.foeparker@logicpd.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=rob.herring@calxeda.com \
    --cc=rtc-linux@googlegroups.com \
    --cc=sameo@linux.intel.com \
    --cc=tony@atomide.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.