From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [PATCH v5 33/48] staging: nvec: Register with kernel power-off handler Date: Sun, 09 Nov 2014 15:54:53 -0800 Message-ID: <545FFECD.9050600@roeck-us.net> References: <1415292213-28652-1-git-send-email-linux@roeck-us.net> <1415292213-28652-34-git-send-email-linux@roeck-us.net> <5008744.rVN94atT9b@ax5200p> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii"; Format="flowed" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <5008744.rVN94atT9b@ax5200p> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: driverdev-devel-bounces@linuxdriverproject.org Sender: "devel" To: Marc Dietrich , devel@driverdev.osuosl.org Cc: linux-tegra@vger.kernel.org, Greg Kroah-Hartman , linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org List-Id: linux-pm@vger.kernel.org On 11/09/2014 01:41 PM, Marc Dietrich wrote: > Am Donnerstag 06 November 2014, 08:43:17 schrieb Guenter Roeck: >> Register with kernel power-off handler instead of setting pm_power_off >> directly. Register with default priority since we don't know any better. >> >> Cc: Julian Andres Klode >> Cc: Marc Dietrich >> Cc: Greg Kroah-Hartman >> Acked-by: Greg Kroah-Hartman >> Signed-off-by: Guenter Roeck > ok, this version (V5) actually switches the device off in contrast to the > first implementation. Somehow I have to find out why it works at all without > irqs. > > Acked-by: Marc Dietrich > Hi Marc, The most recent version does not turn off interrupts as long as the infrastructure (architecture code, ie machine_power_off) doesn't turn it off. Thanks a lot for (re-)testing. Guenter