linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* Fwd: [PATCH v2 0/8] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3
@ 2016-05-02  5:48 MyungJoo Ham
  2016-05-02  5:55 ` Krzysztof Kozlowski
  2016-05-03  2:17 ` Chanwoo Choi
  0 siblings, 2 replies; 5+ messages in thread
From: MyungJoo Ham @ 2016-05-02  5:48 UTC (permalink / raw)
  To: 최찬우, 박경민,
	크쉬시토프,
	tomasz.figa, Sylwester Nawrocki, kgene
  Cc: robh, rjw, robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
	galak, linux, linux.amoon, m.reichl, tjakobi,
	대인기,
	linux-kernel, linux-pm, linux-samsung-soc, linux-arm-kernel,
	devicetree

Sender : 최찬우<cw00.choi@samsung.com> S5(책임)/Change Agent/Tizen Platform Lab(S/W센터)/삼성전자
Date : 2016-04-15 15:32 (GMT+09:00)
> 
> This patchset support the AMBA bus frequency scaling on Exynos5422-based
> Odroid-XU3 board. But, this series only support the bus frequency scaling
> for INT (Internal) block using VDD_INT power line.

Along with Devfreq/Passive patchset, the patchset is merged in
devfreq for-4.7 tree.

https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-4.7

@Chanwoo: If there are commits to be merged by a tree other than Linux/PM,
please let me know. I'll send out pull-request soon.


Cheers,
MyungJoo

> 
> Also, to support the bus frequency scaling for Exynos542x SoC,
> Exynos542x SoC has the specific 'NoC (Network on Chip) Probe' device
> to measure the transfered data traffic on NoC (Network on Chip)
> instead of PPMU (Platform Performance Monitoring Unit). NoC Probe device
> provide the utilization for INT block of Exynos542x SoC.
> 
> The generic exynos-bus frequency driver uses the 'NoC Probe' devfreq-event
> device (drivers/devfreq/event/exynos-nocp.c) without any modification.
> Just add the phandle of 'NoC Probe' dt node to bus dt node.
> 
> Depend on:
> This patchset depends on patch[1] which support the generic exynos-bus
> frequency driver.
> [1] http://www.spinics.net/lists/arm-kernel/msg495976.html
> - [PATCH v9 00/20] PM / devferq: Add generic exynos bus frequency driver and new passive governor
> 
> 

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Fwd: [PATCH v2 0/8] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3
  2016-05-02  5:48 Fwd: [PATCH v2 0/8] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3 MyungJoo Ham
@ 2016-05-02  5:55 ` Krzysztof Kozlowski
  2016-05-02  6:06   ` Krzysztof Kozlowski
  2016-05-03  2:17 ` Chanwoo Choi
  1 sibling, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2016-05-02  5:55 UTC (permalink / raw)
  To: myungjoo.ham, 최찬우, 박경민,
	tomasz.figa, Sylwester Nawrocki, kgene
  Cc: robh, rjw, robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
	galak, linux, linux.amoon, m.reichl, tjakobi,
	대인기,
	linux-kernel, linux-pm, linux-samsung-soc, linux-arm-kernel,
	devicetree

On 05/02/2016 07:48 AM, MyungJoo Ham wrote:
> Sender : 최찬우<cw00.choi@samsung.com> S5(책임)/Change Agent/Tizen Platform Lab(S/W센터)/삼성전자
> Date : 2016-04-15 15:32 (GMT+09:00)
>>
>> This patchset support the AMBA bus frequency scaling on Exynos5422-based
>> Odroid-XU3 board. But, this series only support the bus frequency scaling
>> for INT (Internal) block using VDD_INT power line.
> 
> Along with Devfreq/Passive patchset, the patchset is merged in
> devfreq for-4.7 tree.
> 
> https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-4.7
> 
> @Chanwoo: If there are commits to be merged by a tree other than Linux/PM,
> please let me know. I'll send out pull-request soon.

As we discussed before, don't apply the DTS changes. The DTS are
independent so they should go through samsung/arm-soc.

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Fwd: [PATCH v2 0/8] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3
  2016-05-02  5:55 ` Krzysztof Kozlowski
@ 2016-05-02  6:06   ` Krzysztof Kozlowski
  2016-05-02  6:24     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2016-05-02  6:06 UTC (permalink / raw)
  To: myungjoo.ham, 최찬우, 박경민,
	tomasz.figa, Sylwester Nawrocki, kgene
  Cc: robh, rjw, robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
	galak, linux, linux.amoon, m.reichl, tjakobi,
	대인기,
	linux-kernel, linux-pm, linux-samsung-soc, linux-arm-kernel,
	devicetree

On 05/02/2016 07:55 AM, Krzysztof Kozlowski wrote:
> On 05/02/2016 07:48 AM, MyungJoo Ham wrote:
>> Sender : 최찬우<cw00.choi@samsung.com> S5(책임)/Change Agent/Tizen Platform Lab(S/W센터)/삼성전자
>> Date : 2016-04-15 15:32 (GMT+09:00)
>>>
>>> This patchset support the AMBA bus frequency scaling on Exynos5422-based
>>> Odroid-XU3 board. But, this series only support the bus frequency scaling
>>> for INT (Internal) block using VDD_INT power line.
>>
>> Along with Devfreq/Passive patchset, the patchset is merged in
>> devfreq for-4.7 tree.
>>
>> https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-4.7
>>
>> @Chanwoo: If there are commits to be merged by a tree other than Linux/PM,
>> please let me know. I'll send out pull-request soon.
> 
> As we discussed before, don't apply the DTS changes. The DTS are
> independent so they should go through samsung/arm-soc.

Another point is that you applied patches without your signed-off-by...
that is not the proper way. If you touch a patch or apply it, you must
signed it off.

BR,
Krzysztof

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Fwd: [PATCH v2 0/8] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3
  2016-05-02  6:06   ` Krzysztof Kozlowski
@ 2016-05-02  6:24     ` Krzysztof Kozlowski
  0 siblings, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2016-05-02  6:24 UTC (permalink / raw)
  To: myungjoo.ham, 최찬우, 박경민,
	tomasz.figa, Sylwester Nawrocki, kgene
  Cc: robh, rjw, robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
	galak, linux, linux.amoon, m.reichl, tjakobi,
	대인기,
	linux-kernel, linux-pm, linux-samsung-soc, linux-arm-kernel,
	devicetree

On 05/02/2016 08:06 AM, Krzysztof Kozlowski wrote:
> On 05/02/2016 07:55 AM, Krzysztof Kozlowski wrote:
>> On 05/02/2016 07:48 AM, MyungJoo Ham wrote:
>>> Sender : 최찬우<cw00.choi@samsung.com> S5(책임)/Change Agent/Tizen Platform Lab(S/W센터)/삼성전자
>>> Date : 2016-04-15 15:32 (GMT+09:00)
>>>>
>>>> This patchset support the AMBA bus frequency scaling on Exynos5422-based
>>>> Odroid-XU3 board. But, this series only support the bus frequency scaling
>>>> for INT (Internal) block using VDD_INT power line.
>>>
>>> Along with Devfreq/Passive patchset, the patchset is merged in
>>> devfreq for-4.7 tree.
>>>
>>> https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-4.7
>>>
>>> @Chanwoo: If there are commits to be merged by a tree other than Linux/PM,
>>> please let me know. I'll send out pull-request soon.

... and now I noticed that you also applied clock patches. Please don't.
They are for some time in clk tree. Sylwester prepared a tag for that,
if you need one.

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: Fwd: [PATCH v2 0/8] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3
  2016-05-02  5:48 Fwd: [PATCH v2 0/8] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3 MyungJoo Ham
  2016-05-02  5:55 ` Krzysztof Kozlowski
@ 2016-05-03  2:17 ` Chanwoo Choi
  1 sibling, 0 replies; 5+ messages in thread
From: Chanwoo Choi @ 2016-05-03  2:17 UTC (permalink / raw)
  To: myungjoo.ham, 박경민,
	크쉬시토프,
	tomasz.figa, Sylwester Nawrocki, kgene
  Cc: robh, rjw, robh+dt, pawel.moll, mark.rutland, ijc+devicetree,
	galak, linux, linux.amoon, m.reichl, tjakobi,
	대인기,
	linux-kernel, linux-pm, linux-samsung-soc, linux-arm-kernel,
	devicetree

Dear Myungjoo,

On 2016년 05월 02일 14:48, MyungJoo Ham wrote:
> Sender : 최찬우<cw00.choi@samsung.com> S5(책임)/Change Agent/Tizen Platform Lab(S/W센터)/삼성전자
> Date : 2016-04-15 15:32 (GMT+09:00)
>>
>> This patchset support the AMBA bus frequency scaling on Exynos5422-based
>> Odroid-XU3 board. But, this series only support the bus frequency scaling
>> for INT (Internal) block using VDD_INT power line.
> 
> Along with Devfreq/Passive patchset, the patchset is merged in
> devfreq for-4.7 tree.
> 
> https://git.kernel.org/cgit/linux/kernel/git/mzx/devfreq.git/log/?h=for-4.7
> 
> @Chanwoo: If there are commits to be merged by a tree other than Linux/PM,
> please let me know. I'll send out pull-request soon.

On Krzysztof Kozlowski's reply, dts patches will be merged on samsung-soc tree.
and then the clock patches about exynos5420 was already merged to samsung-clk tree
by Sylwester.

Only devfreq patches should be merged on devfreq tree.

We already discussed this topic on offline. I expect that you will again handle
this patchset according to reply of both Krzysztof and me.

Best Regards,
Chanwoo Choi

> 
> 
> Cheers,
> MyungJoo
> 
>>
>> Also, to support the bus frequency scaling for Exynos542x SoC,
>> Exynos542x SoC has the specific 'NoC (Network on Chip) Probe' device
>> to measure the transfered data traffic on NoC (Network on Chip)
>> instead of PPMU (Platform Performance Monitoring Unit). NoC Probe device
>> provide the utilization for INT block of Exynos542x SoC.
>>
>> The generic exynos-bus frequency driver uses the 'NoC Probe' devfreq-event
>> device (drivers/devfreq/event/exynos-nocp.c) without any modification.
>> Just add the phandle of 'NoC Probe' dt node to bus dt node.
>>
>> Depend on:
>> This patchset depends on patch[1] which support the generic exynos-bus
>> frequency driver.
>> [1] http://www.spinics.net/lists/arm-kernel/msg495976.html
>> - [PATCH v9 00/20] PM / devferq: Add generic exynos bus frequency driver and new passive governor
>>
>>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2016-05-03  2:17 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-05-02  5:48 Fwd: [PATCH v2 0/8] PM / devfreq: Add NoCP devfreq-event and support busfreq on exyno5422-odroidxu3 MyungJoo Ham
2016-05-02  5:55 ` Krzysztof Kozlowski
2016-05-02  6:06   ` Krzysztof Kozlowski
2016-05-02  6:24     ` Krzysztof Kozlowski
2016-05-03  2:17 ` Chanwoo Choi

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).