Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1446335 > unrolled thread

[PATCH] ARM: exynos_defconfig: Enable PM_DEBUG

Started byKrzysztof Kozlowski <k.kozlowski@samsung.com>
First post2016-07-19 13:10 +0200
Last post2016-07-20 16:00 +0200
Articles 3 — 3 participants

Back to article view | Back to linux.kernel


Contents

  [PATCH] ARM: exynos_defconfig: Enable PM_DEBUG Krzysztof Kozlowski <k.kozlowski@samsung.com> - 2016-07-19 13:10 +0200
    Re: [PATCH] ARM: exynos_defconfig: Enable PM_DEBUG Javier Martinez Canillas <javier@osg.samsung.com> - 2016-07-19 17:00 +0200
      Re: [PATCH] ARM: exynos_defconfig: Enable PM_DEBUG Kukjin Kim <kgene@kernel.org> - 2016-07-20 16:00 +0200

#1446335 — [PATCH] ARM: exynos_defconfig: Enable PM_DEBUG

FromKrzysztof Kozlowski <k.kozlowski@samsung.com>
Date2016-07-19 13:10 +0200
Subject[PATCH] ARM: exynos_defconfig: Enable PM_DEBUG
Message-ID<rWB4t-3hE-9@gated-at.bofh.it>
PM_DEBUG and PM_ADVANCED_DEBUG bring some information useful for
development and they should not cause any significant overhead.  Enable
them in default configuration so debugging would be slightly easier.

Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
---
 arch/arm/configs/exynos_defconfig | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig
index a5aa44c56bfe..d7366891e9cf 100644
--- a/arch/arm/configs/exynos_defconfig
+++ b/arch/arm/configs/exynos_defconfig
@@ -35,6 +35,8 @@ CONFIG_ARM_EXYNOS_CPUIDLE=y
 CONFIG_VFP=y
 CONFIG_NEON=y
 CONFIG_KERNEL_MODE_NEON=y
+CONFIG_PM_DEBUG=y
+CONFIG_PM_ADVANCED_DEBUG=y
 CONFIG_NET=y
 CONFIG_PACKET=y
 CONFIG_UNIX=y
-- 
1.9.1

[toc] | [next] | [standalone]


#1446537

FromJavier Martinez Canillas <javier@osg.samsung.com>
Date2016-07-19 17:00 +0200
Message-ID<rWEF4-5jj-17@gated-at.bofh.it>
In reply to#1446335
Hello Krzysztof,

On 07/19/2016 07:09 AM, Krzysztof Kozlowski wrote:
> PM_DEBUG and PM_ADVANCED_DEBUG bring some information useful for
> development and they should not cause any significant overhead.  Enable
> them in default configuration so debugging would be slightly easier.
> 
> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
> ---

I agree that it's good to have these enabled to make debugging easier.

Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>

Best regards,
-- 
Javier Martinez Canillas
Open Source Group
Samsung Research America

[toc] | [prev] | [next] | [standalone]


#1447242

FromKukjin Kim <kgene@kernel.org>
Date2016-07-20 16:00 +0200
Message-ID<rX0cy-28F-23@gated-at.bofh.it>
In reply to#1446537
On 07/19/16 23:49, Javier Martinez Canillas wrote:
> Hello Krzysztof,
> 
> On 07/19/2016 07:09 AM, Krzysztof Kozlowski wrote:
>> PM_DEBUG and PM_ADVANCED_DEBUG bring some information useful for
>> development and they should not cause any significant overhead.  Enable
>> them in default configuration so debugging would be slightly easier.
>>
>> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>> ---
> 
> I agree that it's good to have these enabled to make debugging easier.
> 
I'm back ;-)

I agreed...so,
Acked-by: Kukjin Kim <kgene@kernel.org>

BTW, I have not seen the kernel log with PM_ADVANCED_DEBUG. Can anyone
show me the log?...to be honest I need to setup the dev. environment on
my desk at this moment...

Thanks,
Kukjin

> Reviewed-by: Javier Martinez Canillas <javier@osg.samsung.com>
> 
> Best regards,

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web