All of lore.kernel.org
 help / color / mirror / Atom feed
From: nicolas.ferre@atmel.com (Nicolas Ferre)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 01/18] power: reset: Add if statement isntead of multiple depends on
Date: Tue, 8 Jul 2014 19:33:24 +0200	[thread overview]
Message-ID: <53BC2B64.8070502@atmel.com> (raw)
In-Reply-To: <1404743131-24569-2-git-send-email-maxime.ripard@free-electrons.com>

Nit-pik: In subject: s/isntead/instead/

On 07/07/2014 16:25, Maxime Ripard :
> All the config option so far are depending on the POWER_RESET symbol
> 
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> ---
>  drivers/power/reset/Kconfig | 17 +++++++++--------
>  1 file changed, 9 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/power/reset/Kconfig b/drivers/power/reset/Kconfig
> index bdcf5173e377..19d546c5edfa 100644
> --- a/drivers/power/reset/Kconfig
> +++ b/drivers/power/reset/Kconfig
> @@ -6,15 +6,17 @@ menuconfig POWER_RESET
>  
>  	  Say Y here to enable board reset and power off
>  
> +if POWER_RESET
> +
>  config POWER_RESET_AS3722
>  	bool "ams AS3722 power-off driver"
> -	depends on MFD_AS3722 && POWER_RESET
> +	depends on MFD_AS3722
>  	help
>  	  This driver supports turning off board via a ams AS3722 power-off.
>  
>  config POWER_RESET_AXXIA
>  	bool "LSI Axxia reset driver"
> -	depends on POWER_RESET && ARCH_AXXIA
> +	depends on ARCH_AXXIA
>  	help
>  	  This driver supports restart for Axxia SoC.
>  
> @@ -22,7 +24,7 @@ config POWER_RESET_AXXIA
>  
>  config POWER_RESET_GPIO
>  	bool "GPIO power-off driver"
> -	depends on OF_GPIO && POWER_RESET
> +	depends on OF_GPIO
>  	help
>  	  This driver supports turning off your board via a GPIO line.
>  	  If your board needs a GPIO high/low to power down, say Y and
> @@ -30,13 +32,13 @@ config POWER_RESET_GPIO
>  
>  config POWER_RESET_MSM
>  	bool "Qualcomm MSM power-off driver"
> -	depends on POWER_RESET && ARCH_QCOM
> +	depends on ARCH_QCOM
>  	help
>  	  Power off and restart support for Qualcomm boards.
>  
>  config POWER_RESET_QNAP
>  	bool "QNAP power-off driver"
> -	depends on OF_GPIO && POWER_RESET && PLAT_ORION
> +	depends on OF_GPIO && PLAT_ORION
>  	help
>  	  This driver supports turning off QNAP NAS devices by sending
>  	  commands to the microcontroller which controls the main power.
> @@ -54,14 +56,13 @@ config POWER_RESET_RESTART
>  config POWER_RESET_SUN6I
>  	bool "Allwinner A31 SoC reset driver"
>  	depends on ARCH_SUNXI
> -	depends on POWER_RESET
>  	help
>  	  Reboot support for the Allwinner A31 SoCs.
>  
>  config POWER_RESET_VEXPRESS
>  	bool "ARM Versatile Express power-off and reset driver"
>  	depends on ARM || ARM64
> -	depends on POWER_RESET && VEXPRESS_CONFIG
> +	depends on VEXPRESS_CONFIG
>  	help
>  	  Power off and reset support for the ARM Ltd. Versatile
>  	  Express boards.
> @@ -69,7 +70,6 @@ config POWER_RESET_VEXPRESS
>  config POWER_RESET_XGENE
>  	bool "APM SoC X-Gene reset driver"
>  	depends on ARM64
> -	depends on POWER_RESET
>  	help
>  	  Reboot support for the APM SoC X-Gene Eval boards.
>  
> @@ -80,3 +80,4 @@ config POWER_RESET_KEYSTONE
>  	help
>  	  Reboot support for the KEYSTONE SoCs.
>  
> +endif
> 


-- 
Nicolas Ferre

WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Ferre <nicolas.ferre-AIFe0yeh4nAAvxtiuMwx3w@public.gmane.org>
To: Maxime Ripard
	<maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	linux-PelNFVqkFnVyf+4FbqDuWQ@public.gmane.org,
	Jean-Christophe Plagniol-Villard
	<plagnioj-sclMFOaUSTBWk0Htik3J/w@public.gmane.org>,
	dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org,
	dbaryshkov-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
Cc: Boris Brezillon
	<boris-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	Alexandre Belloni
	<alexandre.belloni-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	Thomas Petazzoni
	<thomas-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH v2 01/18] power: reset: Add if statement isntead of multiple depends on
Date: Tue, 8 Jul 2014 19:33:24 +0200	[thread overview]
Message-ID: <53BC2B64.8070502@atmel.com> (raw)
In-Reply-To: <1404743131-24569-2-git-send-email-maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>

Nit-pik: In subject: s/isntead/instead/

On 07/07/2014 16:25, Maxime Ripard :
> All the config option so far are depending on the POWER_RESET symbol
> 
> Signed-off-by: Maxime Ripard <maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
> ---
>  drivers/power/reset/Kconfig | 17 +++++++++--------
>  1 file changed, 9 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/power/reset/Kconfig b/drivers/power/reset/Kconfig
> index bdcf5173e377..19d546c5edfa 100644
> --- a/drivers/power/reset/Kconfig
> +++ b/drivers/power/reset/Kconfig
> @@ -6,15 +6,17 @@ menuconfig POWER_RESET
>  
>  	  Say Y here to enable board reset and power off
>  
> +if POWER_RESET
> +
>  config POWER_RESET_AS3722
>  	bool "ams AS3722 power-off driver"
> -	depends on MFD_AS3722 && POWER_RESET
> +	depends on MFD_AS3722
>  	help
>  	  This driver supports turning off board via a ams AS3722 power-off.
>  
>  config POWER_RESET_AXXIA
>  	bool "LSI Axxia reset driver"
> -	depends on POWER_RESET && ARCH_AXXIA
> +	depends on ARCH_AXXIA
>  	help
>  	  This driver supports restart for Axxia SoC.
>  
> @@ -22,7 +24,7 @@ config POWER_RESET_AXXIA
>  
>  config POWER_RESET_GPIO
>  	bool "GPIO power-off driver"
> -	depends on OF_GPIO && POWER_RESET
> +	depends on OF_GPIO
>  	help
>  	  This driver supports turning off your board via a GPIO line.
>  	  If your board needs a GPIO high/low to power down, say Y and
> @@ -30,13 +32,13 @@ config POWER_RESET_GPIO
>  
>  config POWER_RESET_MSM
>  	bool "Qualcomm MSM power-off driver"
> -	depends on POWER_RESET && ARCH_QCOM
> +	depends on ARCH_QCOM
>  	help
>  	  Power off and restart support for Qualcomm boards.
>  
>  config POWER_RESET_QNAP
>  	bool "QNAP power-off driver"
> -	depends on OF_GPIO && POWER_RESET && PLAT_ORION
> +	depends on OF_GPIO && PLAT_ORION
>  	help
>  	  This driver supports turning off QNAP NAS devices by sending
>  	  commands to the microcontroller which controls the main power.
> @@ -54,14 +56,13 @@ config POWER_RESET_RESTART
>  config POWER_RESET_SUN6I
>  	bool "Allwinner A31 SoC reset driver"
>  	depends on ARCH_SUNXI
> -	depends on POWER_RESET
>  	help
>  	  Reboot support for the Allwinner A31 SoCs.
>  
>  config POWER_RESET_VEXPRESS
>  	bool "ARM Versatile Express power-off and reset driver"
>  	depends on ARM || ARM64
> -	depends on POWER_RESET && VEXPRESS_CONFIG
> +	depends on VEXPRESS_CONFIG
>  	help
>  	  Power off and reset support for the ARM Ltd. Versatile
>  	  Express boards.
> @@ -69,7 +70,6 @@ config POWER_RESET_VEXPRESS
>  config POWER_RESET_XGENE
>  	bool "APM SoC X-Gene reset driver"
>  	depends on ARM64
> -	depends on POWER_RESET
>  	help
>  	  Reboot support for the APM SoC X-Gene Eval boards.
>  
> @@ -80,3 +80,4 @@ config POWER_RESET_KEYSTONE
>  	help
>  	  Reboot support for the KEYSTONE SoCs.
>  
> +endif
> 


-- 
Nicolas Ferre
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Ferre <nicolas.ferre@atmel.com>
To: Maxime Ripard <maxime.ripard@free-electrons.com>,
	<linux@maxim.org.za>,
	Jean-Christophe Plagniol-Villard <plagnioj@jcrosoft.com>,
	<dwmw2@infradead.org>, <dbaryshkov@gmail.com>
Cc: Boris Brezillon <boris@free-electrons.com>,
	Alexandre Belloni <alexandre.belloni@free-electrons.com>,
	Thomas Petazzoni <thomas@free-electrons.com>,
	<linux-arm-kernel@lists.infradead.org>,
	<linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>
Subject: Re: [PATCH v2 01/18] power: reset: Add if statement isntead of multiple depends on
Date: Tue, 8 Jul 2014 19:33:24 +0200	[thread overview]
Message-ID: <53BC2B64.8070502@atmel.com> (raw)
In-Reply-To: <1404743131-24569-2-git-send-email-maxime.ripard@free-electrons.com>

Nit-pik: In subject: s/isntead/instead/

On 07/07/2014 16:25, Maxime Ripard :
> All the config option so far are depending on the POWER_RESET symbol
> 
> Signed-off-by: Maxime Ripard <maxime.ripard@free-electrons.com>
> ---
>  drivers/power/reset/Kconfig | 17 +++++++++--------
>  1 file changed, 9 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/power/reset/Kconfig b/drivers/power/reset/Kconfig
> index bdcf5173e377..19d546c5edfa 100644
> --- a/drivers/power/reset/Kconfig
> +++ b/drivers/power/reset/Kconfig
> @@ -6,15 +6,17 @@ menuconfig POWER_RESET
>  
>  	  Say Y here to enable board reset and power off
>  
> +if POWER_RESET
> +
>  config POWER_RESET_AS3722
>  	bool "ams AS3722 power-off driver"
> -	depends on MFD_AS3722 && POWER_RESET
> +	depends on MFD_AS3722
>  	help
>  	  This driver supports turning off board via a ams AS3722 power-off.
>  
>  config POWER_RESET_AXXIA
>  	bool "LSI Axxia reset driver"
> -	depends on POWER_RESET && ARCH_AXXIA
> +	depends on ARCH_AXXIA
>  	help
>  	  This driver supports restart for Axxia SoC.
>  
> @@ -22,7 +24,7 @@ config POWER_RESET_AXXIA
>  
>  config POWER_RESET_GPIO
>  	bool "GPIO power-off driver"
> -	depends on OF_GPIO && POWER_RESET
> +	depends on OF_GPIO
>  	help
>  	  This driver supports turning off your board via a GPIO line.
>  	  If your board needs a GPIO high/low to power down, say Y and
> @@ -30,13 +32,13 @@ config POWER_RESET_GPIO
>  
>  config POWER_RESET_MSM
>  	bool "Qualcomm MSM power-off driver"
> -	depends on POWER_RESET && ARCH_QCOM
> +	depends on ARCH_QCOM
>  	help
>  	  Power off and restart support for Qualcomm boards.
>  
>  config POWER_RESET_QNAP
>  	bool "QNAP power-off driver"
> -	depends on OF_GPIO && POWER_RESET && PLAT_ORION
> +	depends on OF_GPIO && PLAT_ORION
>  	help
>  	  This driver supports turning off QNAP NAS devices by sending
>  	  commands to the microcontroller which controls the main power.
> @@ -54,14 +56,13 @@ config POWER_RESET_RESTART
>  config POWER_RESET_SUN6I
>  	bool "Allwinner A31 SoC reset driver"
>  	depends on ARCH_SUNXI
> -	depends on POWER_RESET
>  	help
>  	  Reboot support for the Allwinner A31 SoCs.
>  
>  config POWER_RESET_VEXPRESS
>  	bool "ARM Versatile Express power-off and reset driver"
>  	depends on ARM || ARM64
> -	depends on POWER_RESET && VEXPRESS_CONFIG
> +	depends on VEXPRESS_CONFIG
>  	help
>  	  Power off and reset support for the ARM Ltd. Versatile
>  	  Express boards.
> @@ -69,7 +70,6 @@ config POWER_RESET_VEXPRESS
>  config POWER_RESET_XGENE
>  	bool "APM SoC X-Gene reset driver"
>  	depends on ARM64
> -	depends on POWER_RESET
>  	help
>  	  Reboot support for the APM SoC X-Gene Eval boards.
>  
> @@ -80,3 +80,4 @@ config POWER_RESET_KEYSTONE
>  	help
>  	  Reboot support for the KEYSTONE SoCs.
>  
> +endif
> 


-- 
Nicolas Ferre

  reply	other threads:[~2014-07-08 17:33 UTC|newest]

Thread overview: 66+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-07 14:25 [PATCH v2 00/18] AT91: cleanup of the reset and poweroff code Maxime Ripard
2014-07-07 14:25 ` Maxime Ripard
2014-07-07 14:25 ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 01/18] power: reset: Add if statement isntead of multiple depends on Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-08 17:33   ` Nicolas Ferre [this message]
2014-07-08 17:33     ` Nicolas Ferre
2014-07-08 17:33     ` Nicolas Ferre
2014-07-07 14:25 ` [PATCH v2 02/18] AT91: setup: Switch to pr_fmt Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 03/18] AT91: G45: DT: Declare a second ram controller Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:57   ` Alexandre Belloni
2014-07-07 14:57     ` Alexandre Belloni
2014-07-07 14:57     ` Alexandre Belloni
2014-07-07 14:25 ` [PATCH v2 04/18] AT91: Rework ramc mapping code Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:59   ` Alexandre Belloni
2014-07-07 14:59     ` Alexandre Belloni
2014-07-07 14:59     ` Alexandre Belloni
2014-07-07 14:25 ` [PATCH v2 05/18] AT91: SAMA5D3: DT: Add shutdown controller Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 06/18] power: reset: Add AT91 reset driver Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-08 17:31   ` Nicolas Ferre
2014-07-08 17:31     ` Nicolas Ferre
2014-07-08 17:31     ` Nicolas Ferre
2014-07-07 14:25 ` [PATCH v2 07/18] AT91: DT: Remove the old-style reset probing Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 08/18] AT91: soc: Introduce register_devices callback Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 09/18] AT91: Probe the reset driver Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 10/18] AT91: Call at91_register_devices in the board files Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 11/18] AT91: Remove reset code from the machine code Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 12/18] power: reset: Add AT91 poweroff driver Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 13/18] AT91: DT: Remove poweroff DT probing Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 14/18] AT91: Register the poweroff driver Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 15/18] AT91: Remove poweroff code Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 16/18] AT91: pm: Remove show_reset_status function Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 17/18] AT91: Remove rstc and shdwnc global base addresses Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25 ` [PATCH v2 18/18] AT91: Remove rstc and shdwc headers Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 14:25   ` Maxime Ripard
2014-07-07 15:01 ` [PATCH v2 00/18] AT91: cleanup of the reset and poweroff code Alexandre Belloni
2014-07-07 15:01   ` Alexandre Belloni
2014-07-07 15:01   ` Alexandre Belloni
2014-07-08 17:31 ` Nicolas Ferre
2014-07-08 17:31   ` Nicolas Ferre
2014-07-08 17:31   ` Nicolas Ferre

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=53BC2B64.8070502@atmel.com \
    --to=nicolas.ferre@atmel.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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.