All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
To: Riku Voipio <riku.voipio@linaro.org>, linux-pm@vger.kernel.org
Cc: Zhang Rui <rui.zhang@intel.com>,
	Eduardo Valentin <edubezval@gmail.com>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2] thermal: allow building dove_thermal with mvebu
Date: Sat, 31 May 2014 14:54:40 +0200	[thread overview]
Message-ID: <5389D110.8020606@gmail.com> (raw)
In-Reply-To: <1401537027-32291-1-git-send-email-riku.voipio@linaro.org>

On 05/31/2014 01:50 PM, Riku Voipio wrote:
> DT-enabled Dove has moved from ARCH_DOVE in mach-dove to MACH_DOVE
> in mach-mvebu. As non-DT ARCH_DOVE will stay to rot for a while, add a new
> DT-only MACH_DOVE to thermal Kconfig.
> 
> This was originally supposed to go in via "ARM: dove: prepare new Dove DT Kconfig"
> patch from Sebastian Hesselbarth for 3.15, but slipped through the cracks.
> 
> I've tested on CuBox that without this patch you can't compile
> dove_thermal into a mach-mvebu based kernel, and with this patch I can
> build the driver and it works as expected run-time.

Riku,

looking through my mails, the corresponding patch should have
already been applied [1].

If not, you get my

Acked-by: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>

[1] https://lkml.kernel.org/r/1395295065.3571.10.camel@rzhang1-mobl4

> v2: non-ascii char creeped in somehow
> 
> Signed-off-by: Riku Voipio <riku.voipio@linaro.org>
> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Eduardo Valentin <edubezval@gmail.com>
> Cc: linux-pm@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> ---
>  drivers/thermal/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 2d51912..5783c72 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -151,7 +151,7 @@ config KIRKWOOD_THERMAL
>  
>  config DOVE_THERMAL
>  	tristate "Temperature sensor on Marvell Dove SoCs"
> -	depends on ARCH_DOVE
> +	depends on ARCH_DOVE || MACH_DOVE
>  	depends on OF
>  	help
>  	  Support for the Dove thermal sensor driver in the Linux thermal
> 


WARNING: multiple messages have this Message-ID (diff)
From: sebastian.hesselbarth@gmail.com (Sebastian Hesselbarth)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] thermal: allow building dove_thermal with mvebu
Date: Sat, 31 May 2014 14:54:40 +0200	[thread overview]
Message-ID: <5389D110.8020606@gmail.com> (raw)
In-Reply-To: <1401537027-32291-1-git-send-email-riku.voipio@linaro.org>

On 05/31/2014 01:50 PM, Riku Voipio wrote:
> DT-enabled Dove has moved from ARCH_DOVE in mach-dove to MACH_DOVE
> in mach-mvebu. As non-DT ARCH_DOVE will stay to rot for a while, add a new
> DT-only MACH_DOVE to thermal Kconfig.
> 
> This was originally supposed to go in via "ARM: dove: prepare new Dove DT Kconfig"
> patch from Sebastian Hesselbarth for 3.15, but slipped through the cracks.
> 
> I've tested on CuBox that without this patch you can't compile
> dove_thermal into a mach-mvebu based kernel, and with this patch I can
> build the driver and it works as expected run-time.

Riku,

looking through my mails, the corresponding patch should have
already been applied [1].

If not, you get my

Acked-by: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>

[1] https://lkml.kernel.org/r/1395295065.3571.10.camel at rzhang1-mobl4

> v2: non-ascii char creeped in somehow
> 
> Signed-off-by: Riku Voipio <riku.voipio@linaro.org>
> Cc: Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>
> Cc: Zhang Rui <rui.zhang@intel.com>
> Cc: Eduardo Valentin <edubezval@gmail.com>
> Cc: linux-pm at vger.kernel.org
> Cc: linux-arm-kernel at lists.infradead.org
> ---
>  drivers/thermal/Kconfig | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
> index 2d51912..5783c72 100644
> --- a/drivers/thermal/Kconfig
> +++ b/drivers/thermal/Kconfig
> @@ -151,7 +151,7 @@ config KIRKWOOD_THERMAL
>  
>  config DOVE_THERMAL
>  	tristate "Temperature sensor on Marvell Dove SoCs"
> -	depends on ARCH_DOVE
> +	depends on ARCH_DOVE || MACH_DOVE
>  	depends on OF
>  	help
>  	  Support for the Dove thermal sensor driver in the Linux thermal
> 

  reply	other threads:[~2014-05-31 12:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-05-30 12:28 [PATCH] thermal: allow building dove_thermal with mvebu Riku Voipio
2014-05-30 12:28 ` Riku Voipio
2014-05-31 11:50 ` [PATCH v2] " Riku Voipio
2014-05-31 11:50   ` Riku Voipio
2014-05-31 12:54   ` Sebastian Hesselbarth [this message]
2014-05-31 12:54     ` Sebastian Hesselbarth
     [not found]     ` <CAAqcGHkcnm_JjyzWewEcdGuGyyBwtDT_6EWxFzKSqS7NEUaerA@mail.gmail.com>
2014-07-01  2:04       ` Zhang Rui
2014-07-01  8:50         ` Sebastian Hesselbarth
2014-07-15 14:00           ` Riku Voipio

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=5389D110.8020606@gmail.com \
    --to=sebastian.hesselbarth@gmail.com \
    --cc=edubezval@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=riku.voipio@linaro.org \
    --cc=rui.zhang@intel.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.