From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [PATCH 08/44] kernel: Move pm_power_off to common code Date: Fri, 10 Oct 2014 09:53:52 -0700 Message-ID: <54380F20.3060800@roeck-us.net> References: <1412659726-29957-1-git-send-email-linux@roeck-us.net> <1412659726-29957-9-git-send-email-linux@roeck-us.net> <20141009103847.GC6787@amd> <54368A30.9070101@roeck-us.net> <20141009202419.GA16885@amd> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-path: DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=roeck-us.net; s=default; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:References:Subject:CC:To:MIME-Version:From:Date:Message-ID; bh=GeTamrlv1VO56GLSNtO+MA4zt0M3dOMz1NxH7IEcuus=; b=bVGHuA50L6w5y+yDVA97S9MUCHZ6uapWZzYaHDfBFJFRw9VI8lqMKrvbsYukM4ngacQ7IaXezTy+319dAn24fIobLWWKiLsGE9cAMShrj5GckTI4WPsu4jngS9wvHPHc7bpkLiQju2WkSxyz79rcb6sCzqrJOm+MN3bnGhaM4Hs=; In-Reply-To: <20141009202419.GA16885@amd> Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: List-subscribe: List-owner: List-post: List-archive: Content-Type: text/plain; charset="us-ascii"; format="flowed" To: Pavel Machek Cc: linux-kernel@vger.kernel.org, adi-buildroot-devel@lists.sourceforge.net, devel@driverdev.osuosl.org, devicetree@vger.kernel.org, lguest@lists.ozlabs.org, linux-acpi@vger.kernel.org, linux-alpha@vger.kernel.org, linux-am33-list@redhat.com, linux-cris-kernel@axis.com, linux-efi@vger.kernel.org, linux-hexagon@vger.kernel.org, linux-m32r-ja@ml.linux-m32r.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, linux-tegra@vger.kernel.org, linux-xtensa@linux-xtensa.org, openipmi-developer@lists.sourceforge.net, user-mode-linux-devel@lists.sourceforge.net, linux-arm-kernel@lists.infradead.org, linux-c6x-dev@linux-c6x.org, linux-ia64@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-metag@vger.kernel.org, linux-mips@linux-mips.org, linux-parisc@vger.kernel.org, linux- On 10/09/2014 01:24 PM, Pavel Machek wrote: > Hi! > >>>> @@ -184,6 +179,8 @@ machine_halt(void) >>>> void >>>> machine_power_off(void) >>>> { >>>> + do_kernel_poweroff(); >>>> + >>> >>> poweroff -> power_off for consistency. >>> >> Dunno; matter of personal preference. I started with that, but ultimately went >> with poweroff to distinguish poweroff handler functions from existing code, >> specifically kernel_power_off(). > > That works for you, but once it is merged, it is ugly/confusing typo. > Pavel > Ok, no problem, I'll change it. Guenter From mboxrd@z Thu Jan 1 00:00:00 1970 From: Guenter Roeck Subject: Re: [PATCH 08/44] kernel: Move pm_power_off to common code Date: Fri, 10 Oct 2014 09:53:52 -0700 Message-ID: <54380F20.3060800@roeck-us.net> References: <1412659726-29957-1-git-send-email-linux@roeck-us.net> <1412659726-29957-9-git-send-email-linux@roeck-us.net> <20141009103847.GC6787@amd> <54368A30.9070101@roeck-us.net> <20141009202419.GA16885@amd> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20141009202419.GA16885@amd> Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: Pavel Machek Cc: linux-kernel@vger.kernel.org, adi-buildroot-devel@lists.sourceforge.net, devel@driverdev.osuosl.org, devicetree@vger.kernel.org, lguest@lists.ozlabs.org, linux-acpi@vger.kernel.org, linux-alpha@vger.kernel.org, linux-am33-list@redhat.com, linux-cris-kernel@axis.com, linux-efi@vger.kernel.org, linux-hexagon@vger.kernel.org, linux-m32r-ja@ml.linux-m32r.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, linux-tegra@vger.kernel.org, linux-xtensa@linux-xtensa.org, openipmi-developer@lists.sourceforge.net, user-mode-linux-devel@lists.sourceforge.net, linux-arm-kernel@lists.infradead.org, linux-c6x-dev@linux-c6x.org, linux-ia64@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-metag@vger.kernel.org, linux-mips@linux-mips.org, linux-parisc@vger.kernel.orglinux- List-Id: linux-acpi@vger.kernel.org On 10/09/2014 01:24 PM, Pavel Machek wrote: > Hi! > >>>> @@ -184,6 +179,8 @@ machine_halt(void) >>>> void >>>> machine_power_off(void) >>>> { >>>> + do_kernel_poweroff(); >>>> + >>> >>> poweroff -> power_off for consistency. >>> >> Dunno; matter of personal preference. I started with that, but ultimately went >> with poweroff to distinguish poweroff handler functions from existing code, >> specifically kernel_power_off(). > > That works for you, but once it is merged, it is ugly/confusing typo. > Pavel > Ok, no problem, I'll change it. Guenter From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Fri, 10 Oct 2014 18:57:57 +0200 (CEST) Received: from bh-25.webhostbox.net ([208.91.199.152]:33489 "EHLO bh-25.webhostbox.net" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S27011102AbaJJQ5zzNfYN (ORCPT ); Fri, 10 Oct 2014 18:57:55 +0200 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=roeck-us.net; s=default; h=Content-Transfer-Encoding:Content-Type:In-Reply-To:References:Subject:CC:To:MIME-Version:From:Date:Message-ID; bh=GeTamrlv1VO56GLSNtO+MA4zt0M3dOMz1NxH7IEcuus=; b=bVGHuA50L6w5y+yDVA97S9MUCHZ6uapWZzYaHDfBFJFRw9VI8lqMKrvbsYukM4ngacQ7IaXezTy+319dAn24fIobLWWKiLsGE9cAMShrj5GckTI4WPsu4jngS9wvHPHc7bpkLiQju2WkSxyz79rcb6sCzqrJOm+MN3bnGhaM4Hs=; Received: from mailnull by bh-25.webhostbox.net with sa-checked (Exim 4.82) (envelope-from ) id 1XcdVl-002oTV-NZ for linux-mips@linux-mips.org; Fri, 10 Oct 2014 16:57:49 +0000 Received: from 108-223-40-66.lightspeed.sntcca.sbcglobal.net ([108.223.40.66]:47224 helo=server.roeck-us.net) by bh-25.webhostbox.net with esmtpsa (TLSv1:DHE-RSA-AES128-SHA:128) (Exim 4.82) (envelope-from ) id 1XcdS5-002ity-ST; Fri, 10 Oct 2014 16:54:02 +0000 Message-ID: <54380F20.3060800@roeck-us.net> Date: Fri, 10 Oct 2014 09:53:52 -0700 From: Guenter Roeck User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.0 MIME-Version: 1.0 To: Pavel Machek CC: linux-kernel@vger.kernel.org, adi-buildroot-devel@lists.sourceforge.net, devel@driverdev.osuosl.org, devicetree@vger.kernel.org, lguest@lists.ozlabs.org, linux-acpi@vger.kernel.org, linux-alpha@vger.kernel.org, linux-am33-list@redhat.com, linux-cris-kernel@axis.com, linux-efi@vger.kernel.org, linux-hexagon@vger.kernel.org, linux-m32r-ja@ml.linux-m32r.org, linuxppc-dev@lists.ozlabs.org, linux-s390@vger.kernel.org, linux-tegra@vger.kernel.org, linux-xtensa@linux-xtensa.org, openipmi-developer@lists.sourceforge.net, user-mode-linux-devel@lists.sourceforge.net, linux-arm-kernel@lists.infradead.org, linux-c6x-dev@linux-c6x.org, linux-ia64@vger.kernel.org, linux-m68k@lists.linux-m68k.org, linux-metag@vger.kernel.org, linux-mips@linux-mips.org, linux-parisc@vger.kernel.org, linux-pm@vger.kernel.org, linux-sh@vger.kernel.org, xen-devel@lists.xenproject.org, Richard Henderson , Ivan Kokshaysky , Matt Turner , Vineet Gupta , Russell King , Catalin Marinas , Will Deacon , Haavard Skinnemoen , Hans-Christian Egtvedt , Steven Miao , Mark Salter , Aurelien Jacquiot , Mikael Starvik , Jesper Nilsson , David Howells , Richard Kuo , Tony Luck , Fenghua Yu , Hirokazu Takata , Geert Uytterhoeven , James Hogan , Michal Simek , Ralf Baechle , Koichi Yasutake , Jonas Bonn , "James E.J. Bottomley" , Helge Deller , Benjamin Herrenschmidt , Paul Mackerras , Michael Ellerman , Martin Schwidefsky , Heiko Carstens , Chen Liqin , Lennox Wu , "David S. Miller" , Chris Metcalf , Jeff Dike , Richard Weinberger , Guan Xuetao , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Konrad Rzeszutek Wilk , Boris Ostrovsky , David Vrabel , Chris Zankel , Max Filippov , "Rafael J. Wysocki" , Len Brown Subject: Re: [PATCH 08/44] kernel: Move pm_power_off to common code References: <1412659726-29957-1-git-send-email-linux@roeck-us.net> <1412659726-29957-9-git-send-email-linux@roeck-us.net> <20141009103847.GC6787@amd> <54368A30.9070101@roeck-us.net> <20141009202419.GA16885@amd> In-Reply-To: <20141009202419.GA16885@amd> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit X-Authenticated_sender: linux@roeck-us.net X-OutGoing-Spam-Status: No, score=0.0 X-CTCH-PVer: 0000001 X-CTCH-Spam: Suspect X-CTCH-VOD: Unknown X-CTCH-Flags: 512 X-CTCH-RefID: str=0001.0A020208.5438100D.0310,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=512,sb=0 X-CTCH-Score: 0.000 X-CTCH-ScoreCust: 0.000 X-CTCH-Rules: X-CTCH-SenderID: linux@roeck-us.net X-CTCH-SenderID-Flags: 0 X-CTCH-SenderID-TotalMessages: 54 X-CTCH-SenderID-TotalSpam: 0 X-CTCH-SenderID-TotalSuspected: 54 X-CTCH-SenderID-TotalConfirmed: 0 X-CTCH-SenderID-TotalBulk: 0 X-CTCH-SenderID-TotalVirus: 0 X-CTCH-SenderID-TotalRecipients: 0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - bh-25.webhostbox.net X-AntiAbuse: Original Domain - linux-mips.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - roeck-us.net X-Get-Message-Sender-Via: bh-25.webhostbox.net: mailgid no entry from get_relayhosts_entry X-Source: X-Source-Args: X-Source-Dir: Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 43212 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: linux@roeck-us.net Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips On 10/09/2014 01:24 PM, Pavel Machek wrote: > Hi! > >>>> @@ -184,6 +179,8 @@ machine_halt(void) >>>> void >>>> machine_power_off(void) >>>> { >>>> + do_kernel_poweroff(); >>>> + >>> >>> poweroff -> power_off for consistency. >>> >> Dunno; matter of personal preference. I started with that, but ultimately went >> with poweroff to distinguish poweroff handler functions from existing code, >> specifically kernel_power_off(). > > That works for you, but once it is merged, it is ugly/confusing typo. > Pavel > Ok, no problem, I'll change it. Guenter From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from bh-25.webhostbox.net (bh-25.webhostbox.net [208.91.199.152]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 57DA51A097C for ; Sat, 11 Oct 2014 03:58:50 +1100 (EST) Received: from mailnull by bh-25.webhostbox.net with sa-checked (Exim 4.82) (envelope-from ) id 1XcdWh-002pJt-9W for linuxppc-dev@lists.ozlabs.org; Fri, 10 Oct 2014 16:58:47 +0000 Message-ID: <54380F20.3060800@roeck-us.net> Date: Fri, 10 Oct 2014 09:53:52 -0700 From: Guenter Roeck MIME-Version: 1.0 To: Pavel Machek Subject: Re: [PATCH 08/44] kernel: Move pm_power_off to common code References: <1412659726-29957-1-git-send-email-linux@roeck-us.net> <1412659726-29957-9-git-send-email-linux@roeck-us.net> <20141009103847.GC6787@amd> <54368A30.9070101@roeck-us.net> <20141009202419.GA16885@amd> In-Reply-To: <20141009202419.GA16885@amd> Content-Type: text/plain; charset=windows-1252; format=flowed Cc: linux-m32r-ja@ml.linux-m32r.org, linux-mips@linux-mips.org, linux-efi@vger.kernel.org, linux-ia64@vger.kernel.org, Steven Miao , linux-xtensa@linux-xtensa.org, Boris Ostrovsky , Catalin Marinas , Will Deacon , David Howells , Max Filippov , Paul Mackerras , Ralf Baechle , "H. Peter Anvin" , Guan Xuetao , Thomas Gleixner , Lennox Wu , Hans-Christian Egtvedt , devel@driverdev.osuosl.org, linux-s390@vger.kernel.org, Jesper Nilsson , lguest@lists.ozlabs.org, Russell King , linux-c6x-dev@linux-c6x.org, Len Brown , "David S. Miller" , linux-hexagon@vger.kernel.org, Hirokazu Takata , linux-sh@vger.kernel.org, "James E.J. Bottomley" , linux-acpi@vger.kernel.org, Ingo Molnar , Geert Uytterhoeven , Mark Salter , xen-devel@lists.xenproject.org, Matt Turner , Chen Liqin , Jonas Bonn , Haavard Skinnemoen , devicetree@vger.kernel.org, James Hogan , user-mode-linux-devel@lists.sourceforge.net, linux-pm@vger.kernel.org, Aurelien Jacquiot , Heiko Carstens , Jeff Dike , adi-buildroot-devel@lists.sourceforge.net, Chris Metcalf , Konrad Rzeszutek Wilk , Mikael Starvik , Richard Weinberger , linux-m68k@lists.linux-m68k.org, linux-am33-list@redhat.com, Ivan Kokshaysky , linux-tegra@vger.kernel.org, openipmi-developer@lists.sourceforge.net, linux-metag@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Richard Henderson , Chris Zankel , Michal Simek , Tony Luck , linux-parisc@vger.kernel.org, linux-cris-kernel@axis.com, Vineet Gupta , "Rafael J. Wysocki" , linux-kernel@vger.kernel.org, Fenghua Yu , Richard Kuo , David Vrabel , linux-alpha@vger.kernel.org, Martin Schwidefsky , Koichi Yasutake , linuxppc-dev@lists.ozlabs.org, Helge Deller List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 10/09/2014 01:24 PM, Pavel Machek wrote: > Hi! > >>>> @@ -184,6 +179,8 @@ machine_halt(void) >>>> void >>>> machine_power_off(void) >>>> { >>>> + do_kernel_poweroff(); >>>> + >>> >>> poweroff -> power_off for consistency. >>> >> Dunno; matter of personal preference. I started with that, but ultimately went >> with poweroff to distinguish poweroff handler functions from existing code, >> specifically kernel_power_off(). > > That works for you, but once it is merged, it is ugly/confusing typo. > Pavel > Ok, no problem, I'll change it. Guenter