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

Groups > linux.kernel > #1564645 > unrolled thread

[PATCH v4 0/4] PM / devfreq: Update the devfreq and devfreq-event device

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH v4 0/4] PM / devfreq: Update the devfreq and devfreq-event  device Chanwoo Choi <cw00.choi@samsung.com> - 2017-01-23 04:40 +0100
    [PATCH v4 2/4] PM / devfreq: exynos-ppmu: Show the registred device  for ppmu device Chanwoo Choi <cw00.choi@samsung.com> - 2017-01-23 04:40 +0100
    RE: [PATCH v4 1/4] PM / devfreq: Fix the wrong description for  userspace governor MyungJoo Ham <myungjoo.ham@samsung.com> - 2017-01-24 04:50 +0100
      Re: [PATCH v4 1/4] PM / devfreq: Fix the wrong description for  userspace governor "Rafael J. Wysocki" <rafael@kernel.org> - 2017-01-30 11:50 +0100
      RE: Re: [PATCH v4 1/4] PM / devfreq: Fix the wrong description for  userspace governor MyungJoo Ham <myungjoo.ham@samsung.com> - 2017-01-31 07:30 +0100
    RE: [PATCH v4 2/4] PM / devfreq: exynos-ppmu: Show the registred  device for ppmu device MyungJoo Ham <myungjoo.ham@samsung.com> - 2017-01-24 04:50 +0100
      Re: [PATCH v4 2/4] PM / devfreq: exynos-ppmu: Show the registred  device for ppmu device Chanwoo Choi <cw00.choi@samsung.com> - 2017-01-24 11:30 +0100

#1564645 — [PATCH v4 0/4] PM / devfreq: Update the devfreq and devfreq-event device

This patches update the devfreq and devfreq-event device. I add the summary
of each patch as following.
- Patch1 fixes the wrong description of governor_userspace.c.
- Patch2 show the information of registered PPMU devices.
- Patch3/4 modify the name of sysfs entry for devfreq/devfreq-event device

Changes from v3:
(https://lkml.org/lkml/2017/1/16/254)
- Patch3 initializes the init value of 'event_no' in order to remove
  the unneeded operation (-1) when calling the atomic_inc_return(&event_no).
- Patch4 uses the integer type for unique number of devfreq device.

Changes from v2:
(https://lkml.org/lkml/2016/12/28/102)
- On v2 patchset, patch1/2/3/5 were merged on devfreq git repo.
- Remain the warning message for exynos-ppmu.c
  when failed to get the clock of ppmu.
- Rebase these patches on devfreq git repo[1].

Changes from v1:
- Rebase these patches on v4.10-rc1.
- Include the separate patch[2] in these patches.

[1] https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/ (branch:
for-4.10-rc)
[2] https://lkml.org/lkml/2016/12/16/7
- [PATCH] PM / devfreq: Fix the wrong description for userspace governor

Chanwoo Choi (4):
  PM / devfreq: Fix the wrong description for userspace governor
  PM / devfreq: exynos-ppmu: Show the registred device for ppmu device
  PM / devfreq: Simplify the sysfs name of devfreq-event device
  PM / devfreq: Modify the device name as devfreq[X] for sysfs

 drivers/devfreq/devfreq-event.c      | 4 ++--
 drivers/devfreq/devfreq.c            | 4 +++-
 drivers/devfreq/event/exynos-ppmu.c  | 3 +++
 drivers/devfreq/governor_userspace.c | 3 +--
 4 files changed, 9 insertions(+), 5 deletions(-)

-- 
1.9.1

[toc] | [next] | [standalone]

#1564647 — [PATCH v4 2/4] PM / devfreq: exynos-ppmu: Show the registred device for ppmu device

This patch just adds the simple log to show the PPMU device's registration
during the kernel booting.

Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
---
 drivers/devfreq/event/exynos-ppmu.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/devfreq/event/exynos-ppmu.c b/drivers/devfreq/event/exynos-ppmu.c
index fb3706faf5bd..9b7350935b73 100644
--- a/drivers/devfreq/event/exynos-ppmu.c
+++ b/drivers/devfreq/event/exynos-ppmu.c
@@ -643,6 +643,9 @@ static int exynos_ppmu_probe(struct platform_device *pdev)
 				"failed to add devfreq-event device\n");
 			return PTR_ERR(edev[i]);
 		}
+
+		pr_info("exynos-ppmu: new PPMU device registered %s (%s)\n",
+			dev_name(&pdev->dev), desc[i].name);
 	}
 
 	clk_prepare_enable(info->ppmu.clk);
-- 
1.9.1

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

#1565460 — RE: [PATCH v4 1/4] PM / devfreq: Fix the wrong description for userspace governor

[Multipart message — attachments visible in raw view] — view raw

> This patch fixes the wrong description of governor_userspace.c
> and removes the unneeded blank line.
> 
> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> ---

Applied in for-next

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

#1569576 — Re: [PATCH v4 1/4] PM / devfreq: Fix the wrong description for userspace governor

On Tue, Jan 24, 2017 at 4:42 AM, MyungJoo Ham <myungjoo.ham@samsung.com> wrote:
>> This patch fixes the wrong description of governor_userspace.c
>> and removes the unneeded blank line.
>>
>> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
>> ---
>
> Applied in for-next


Quite frankly I'm not entirely sure what's going on in the devfreq land.

Some patches are ACKed by you, some of them are applied and it is hard
to say what the rule is.

Besides, it is quite late in the cycle, so it would be nice to receive
a pull request for the devfreq material already collected for 4.11.

Thanks,
Rafael

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

#1570434 — RE: Re: [PATCH v4 1/4] PM / devfreq: Fix the wrong description for userspace governor

[Multipart message — attachments visible in raw view] — view raw

> On Tue, Jan 24, 2017 at 4:42 AM, MyungJoo Ham <myungjoo.ham@samsung.com> wrote:
> >> This patch fixes the wrong description of governor_userspace.c
> >> and removes the unneeded blank line.
> >>
> >> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> >> ---
> >
> > Applied in for-next
> 
> 
> Quite frankly I'm not entirely sure what's going on in the devfreq land.
> 
> Some patches are ACKed by you, some of them are applied and it is hard
> to say what the rule is.
> 
> Besides, it is quite late in the cycle, so it would be nice to receive
> a pull request for the devfreq material already collected for 4.11.
> 
> Thanks,
> Rafael

- The patches that change ABI is being delayed
  for further discussions with Chanwoo.

- The patches that change mechanisms of handling passive governor
differently in devfreq.c require further updates from Chanwoo;
so they are being not applied for now as well.

If it appears that those two patchset are not going to make in time
(mid this week), I'll send out pull-request for 4.11 without them.


Cheers,
MyungJoo 

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

#1565461 — RE: [PATCH v4 2/4] PM / devfreq: exynos-ppmu: Show the registred device for ppmu device

[Multipart message — attachments visible in raw view] — view raw

> This patch just adds the simple log to show the PPMU device's registration
> during the kernel booting.
> 
> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
> ---
>  drivers/devfreq/event/exynos-ppmu.c | 3 +++
>  1 file changed, 3 insertions(+)
> 

Applied in for-next

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

#1565764 — Re: [PATCH v4 2/4] PM / devfreq: exynos-ppmu: Show the registred device for ppmu device

On 2017년 01월 24일 12:42, MyungJoo Ham wrote:
>> This patch just adds the simple log to show the PPMU device's registration
>> during the kernel booting.
>>
>> Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>
>> ---
>>  drivers/devfreq/event/exynos-ppmu.c | 3 +++
>>  1 file changed, 3 insertions(+)
>>
> 
> Applied in for-next

I'd like you to review the patch3 and patch4.
On v4, The patch4 is modified according to your comment.

-- 
Best Regards,
Chanwoo Choi
Samsung Electronics

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web