* [PATCH] PM / devfreq: fix Kconfig indent style
@ 2016-08-25 12:06 ` Jisheng Zhang
2016-08-26 2:22 ` Chanwoo Choi
2016-08-26 2:42 ` MyungJoo Ham
0 siblings, 2 replies; 5+ messages in thread
From: Jisheng Zhang @ 2016-08-25 12:06 UTC (permalink / raw)
To: myungjoo.ham, kyungmin.park; +Cc: linux-pm, linux-kernel, Jisheng Zhang
Use tab rather than space to indent, and tab + two spaces to indent
help message.
Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
---
drivers/devfreq/Kconfig | 16 ++++++++--------
1 file changed, 8 insertions(+), 8 deletions(-)
diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
index a5be56e..86d06ab 100644
--- a/drivers/devfreq/Kconfig
+++ b/drivers/devfreq/Kconfig
@@ -91,14 +91,14 @@ config ARM_EXYNOS_BUS_DEVFREQ
This does not yet operate with optimal voltages.
config ARM_TEGRA_DEVFREQ
- tristate "Tegra DEVFREQ Driver"
- depends on ARCH_TEGRA_124_SOC
- select DEVFREQ_GOV_SIMPLE_ONDEMAND
- select PM_OPP
- help
- This adds the DEVFREQ driver for the Tegra family of SoCs.
- It reads ACTMON counters of memory controllers and adjusts the
- operating frequencies and voltages with OPP support.
+ tristate "Tegra DEVFREQ Driver"
+ depends on ARCH_TEGRA_124_SOC
+ select DEVFREQ_GOV_SIMPLE_ONDEMAND
+ select PM_OPP
+ help
+ This adds the DEVFREQ driver for the Tegra family of SoCs.
+ It reads ACTMON counters of memory controllers and adjusts the
+ operating frequencies and voltages with OPP support.
source "drivers/devfreq/event/Kconfig"
--
2.9.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] PM / devfreq: fix Kconfig indent style
2016-08-25 12:06 ` [PATCH] PM / devfreq: fix Kconfig indent style Jisheng Zhang
@ 2016-08-26 2:22 ` Chanwoo Choi
2016-08-26 2:42 ` MyungJoo Ham
1 sibling, 0 replies; 5+ messages in thread
From: Chanwoo Choi @ 2016-08-26 2:22 UTC (permalink / raw)
To: Jisheng Zhang, myungjoo.ham, kyungmin.park; +Cc: linux-pm, linux-kernel
Hi Jisheng,
Looks good to me. But, you need to rebase it
on latest devfreq.git because the related patch
is already merged[1] for COMPILE_TEST.
So, the merge conflict may happen.
[1] https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/commit/?h=for-rc&id=c2f9bf4e49c12a3d9fcbee2cd48c9f5d769d8cc8
If you resend it with rebase on devfreq.git,
Feel free to add my reviewed-by tag:
Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
On 2016년 08월 25일 21:06, Jisheng Zhang wrote:
> Use tab rather than space to indent, and tab + two spaces to indent
> help message.
>
> Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
> ---
> drivers/devfreq/Kconfig | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
> index a5be56e..86d06ab 100644
> --- a/drivers/devfreq/Kconfig
> +++ b/drivers/devfreq/Kconfig
> @@ -91,14 +91,14 @@ config ARM_EXYNOS_BUS_DEVFREQ
> This does not yet operate with optimal voltages.
>
> config ARM_TEGRA_DEVFREQ
> - tristate "Tegra DEVFREQ Driver"
> - depends on ARCH_TEGRA_124_SOC
> - select DEVFREQ_GOV_SIMPLE_ONDEMAND
> - select PM_OPP
> - help
> - This adds the DEVFREQ driver for the Tegra family of SoCs.
> - It reads ACTMON counters of memory controllers and adjusts the
> - operating frequencies and voltages with OPP support.
> + tristate "Tegra DEVFREQ Driver"
> + depends on ARCH_TEGRA_124_SOC
> + select DEVFREQ_GOV_SIMPLE_ONDEMAND
> + select PM_OPP
> + help
> + This adds the DEVFREQ driver for the Tegra family of SoCs.
> + It reads ACTMON counters of memory controllers and adjusts the
> + operating frequencies and voltages with OPP support.
>
> source "drivers/devfreq/event/Kconfig"
>
>
--
Best Regards,
Chanwoo Choi
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: Re: [PATCH] PM / devfreq: fix Kconfig indent style
2016-08-25 12:06 ` [PATCH] PM / devfreq: fix Kconfig indent style Jisheng Zhang
2016-08-26 2:22 ` Chanwoo Choi
@ 2016-08-26 2:42 ` MyungJoo Ham
2016-08-26 2:45 ` Jisheng Zhang
1 sibling, 1 reply; 5+ messages in thread
From: MyungJoo Ham @ 2016-08-26 2:42 UTC (permalink / raw)
To: Chanwoo Choi, Jisheng Zhang, Kyungmin Park; +Cc: linux-pm, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 635 bytes --]
> Hi Jisheng,
>
> Looks good to me. But, you need to rebase it
> on latest devfreq.git because the related patch
> is already merged[1] for COMPILE_TEST.
> So, the merge conflict may happen.
>
> [1] https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/commit/?h=for-rc&id=c2f9bf4e49c12a3d9fcbee2cd48c9f5d769d8cc8
>
> If you resend it with rebase on devfreq.git,
> Feel free to add my reviewed-by tag:
> Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
Don't worry.
It is already rebased and queued up at
https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-rc
Cheers,
MyungJoo
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] PM / devfreq: fix Kconfig indent style
2016-08-26 2:42 ` MyungJoo Ham
@ 2016-08-26 2:45 ` Jisheng Zhang
0 siblings, 0 replies; 5+ messages in thread
From: Jisheng Zhang @ 2016-08-26 2:45 UTC (permalink / raw)
To: MyungJoo Ham, Chanwoo Choi; +Cc: Kyungmin Park, linux-pm, linux-kernel
Hi MyungJoo, Chanwoo,
On Fri, 26 Aug 2016 02:42:05 +0000 MyungJoo Ham wrote:
> > Hi Jisheng,
> >
> > Looks good to me. But, you need to rebase it
> > on latest devfreq.git because the related patch
> > is already merged[1] for COMPILE_TEST.
> > So, the merge conflict may happen.
> >
> > [1] https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/commit/?h=for-rc&id=c2f9bf4e49c12a3d9fcbee2cd48c9f5d769d8cc8
> >
> > If you resend it with rebase on devfreq.git,
> > Feel free to add my reviewed-by tag:
> > Reviewed-by: Chanwoo Choi <cw00.choi@samsung.com>
>
> Don't worry.
>
> It is already rebased and queued up at
> https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-rc
>
thanks so much!
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH] PM / devfreq: fix Kconfig indent style
[not found] <CGME20160825121032epcas1p43f3290ba0047692fde5256503badb5bd@epcas1p4.samsung.com>
@ 2016-08-26 1:30 ` MyungJoo Ham
0 siblings, 0 replies; 5+ messages in thread
From: MyungJoo Ham @ 2016-08-26 1:30 UTC (permalink / raw)
To: Jisheng Zhang, Kyungmin Park; +Cc: linux-pm, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 351 bytes --]
> Use tab rather than space to indent, and tab + two spaces to indent
> help message.
>
> Signed-off-by: Jisheng Zhang <jszhang@marvell.com>
> ---
> drivers/devfreq/Kconfig | 16 ++++++++--------
> 1 file changed, 8 insertions(+), 8 deletions(-)
>
Thanks!
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
Cheers,
MyungJoo
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-08-26 2:53 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CGME20160825121110epcas1p4e388645a033341ce2a85e39115e9bb57@epcas1p4.samsung.com>
2016-08-25 12:06 ` [PATCH] PM / devfreq: fix Kconfig indent style Jisheng Zhang
2016-08-26 2:22 ` Chanwoo Choi
2016-08-26 2:42 ` MyungJoo Ham
2016-08-26 2:45 ` Jisheng Zhang
[not found] <CGME20160825121032epcas1p43f3290ba0047692fde5256503badb5bd@epcas1p4.samsung.com>
2016-08-26 1:30 ` MyungJoo Ham
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).