All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lukasz Majewski <l.majewski@samsung.com>
To: Abhilash Kesavan <a.kesavan@samsung.com>
Cc: rui.zhang@intel.com, edubezval@gmail.com, kgene.kim@samsung.com,
	linux-pm@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	b.zolnierkie@samsung.com, amit.daniel@samsung.com,
	kesavan.abhilash@gmail.com
Subject: Re: [PATCH 1/2] drivers: thermal: Remove ARCH_HAS_BANDGAP dependency for samsung
Date: Mon, 08 Dec 2014 10:43:33 +0100	[thread overview]
Message-ID: <20141208104333.0e044c58@amdc2363> (raw)
In-Reply-To: <1417502073-6322-1-git-send-email-a.kesavan@samsung.com>

Hi Abhilash,

> As samsung thermal support is enabled only for ARCH_EXYNOS, there is
> no need to select ARCH_HAS_BANDGAP from the arch-specific code.
> Removing this dependency will also allow the driver to be enabled on
> 64-bit SoCs.
> 
> Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
> ---
> This was based on the discussion regarding adding ARCH_HAS_BANDGAP
> symbol for arm64 here:
> http://www.spinics.net/lists/arm-kernel/msg377560.html
> 
>  drivers/thermal/samsung/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/thermal/samsung/Kconfig
> b/drivers/thermal/samsung/Kconfig index f760389..c43306e 100644
> --- a/drivers/thermal/samsung/Kconfig
> +++ b/drivers/thermal/samsung/Kconfig
> @@ -1,6 +1,6 @@
>  config EXYNOS_THERMAL
>  	tristate "Exynos thermal management unit driver"
> -	depends on ARCH_HAS_BANDGAP && OF
> +	depends on OF
>  	help
>  	  If you say yes here you get support for the TMU (Thermal
> Management Unit) driver for SAMSUNG EXYNOS series of SoCs. This
> driver initialises

Reviewed-by: Lukasz Majewski <l.majewski@samsung.com>
Acked-by: Lukasz Majewski <l.majewski@samsung.com>

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

WARNING: multiple messages have this Message-ID (diff)
From: l.majewski@samsung.com (Lukasz Majewski)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/2] drivers: thermal: Remove ARCH_HAS_BANDGAP dependency for samsung
Date: Mon, 08 Dec 2014 10:43:33 +0100	[thread overview]
Message-ID: <20141208104333.0e044c58@amdc2363> (raw)
In-Reply-To: <1417502073-6322-1-git-send-email-a.kesavan@samsung.com>

Hi Abhilash,

> As samsung thermal support is enabled only for ARCH_EXYNOS, there is
> no need to select ARCH_HAS_BANDGAP from the arch-specific code.
> Removing this dependency will also allow the driver to be enabled on
> 64-bit SoCs.
> 
> Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
> ---
> This was based on the discussion regarding adding ARCH_HAS_BANDGAP
> symbol for arm64 here:
> http://www.spinics.net/lists/arm-kernel/msg377560.html
> 
>  drivers/thermal/samsung/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/thermal/samsung/Kconfig
> b/drivers/thermal/samsung/Kconfig index f760389..c43306e 100644
> --- a/drivers/thermal/samsung/Kconfig
> +++ b/drivers/thermal/samsung/Kconfig
> @@ -1,6 +1,6 @@
>  config EXYNOS_THERMAL
>  	tristate "Exynos thermal management unit driver"
> -	depends on ARCH_HAS_BANDGAP && OF
> +	depends on OF
>  	help
>  	  If you say yes here you get support for the TMU (Thermal
> Management Unit) driver for SAMSUNG EXYNOS series of SoCs. This
> driver initialises

Reviewed-by: Lukasz Majewski <l.majewski@samsung.com>
Acked-by: Lukasz Majewski <l.majewski@samsung.com>

-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

  parent reply	other threads:[~2014-12-08  9:44 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-12-02  6:34 [PATCH 1/2] drivers: thermal: Remove ARCH_HAS_BANDGAP dependency for samsung Abhilash Kesavan
2014-12-02  6:34 ` Abhilash Kesavan
2014-12-02  6:34 ` [PATCH 2/2] ARM: EXYNOS: Remove ARCH_HAS_BANDGAP selection Abhilash Kesavan
2014-12-02  6:34   ` Abhilash Kesavan
2014-12-02 14:23   ` Bartlomiej Zolnierkiewicz
2014-12-02 14:23     ` Bartlomiej Zolnierkiewicz
2014-12-08  9:44   ` Lukasz Majewski
2014-12-08  9:44     ` Lukasz Majewski
2014-12-02 14:17 ` [PATCH 1/2] drivers: thermal: Remove ARCH_HAS_BANDGAP dependency for samsung Bartlomiej Zolnierkiewicz
2014-12-02 14:17   ` Bartlomiej Zolnierkiewicz
2014-12-08  9:43 ` Lukasz Majewski [this message]
2014-12-08  9:43   ` Lukasz Majewski
2014-12-10  3:56   ` Abhilash Kesavan
2014-12-10  3:56     ` Abhilash Kesavan
2014-12-10 14:41     ` Eduardo Valentin
2014-12-10 14:41       ` Eduardo Valentin

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=20141208104333.0e044c58@amdc2363 \
    --to=l.majewski@samsung.com \
    --cc=a.kesavan@samsung.com \
    --cc=amit.daniel@samsung.com \
    --cc=b.zolnierkie@samsung.com \
    --cc=edubezval@gmail.com \
    --cc=kesavan.abhilash@gmail.com \
    --cc=kgene.kim@samsung.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-pm@vger.kernel.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.