* [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@vger.kernel.org, linux-kernel@vger.kernel.org [-- 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@vger.kernel.org, linux-kernel@vger.kernel.org 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
[parent not found: <CGME20160825121032epcas1p43f3290ba0047692fde5256503badb5bd@epcas1p4.samsung.com>]
* 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@vger.kernel.org, linux-kernel@vger.kernel.org [-- 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).