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

Groups > linux.kernel > #1691147 > unrolled thread

Re: [PATCH v2 0/5] arm64: dts: rockchip: support mail and IPA thermal for rk3399

Started byCaesar Wang <wxt@rock-chips.com>
First post2017-07-19 11:10 +0200
Last post2017-07-19 23:10 +0200
Articles 2 — 2 participants

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

  Re: [PATCH v2 0/5] arm64: dts: rockchip: support mail and IPA thermal  for rk3399 Caesar Wang <wxt@rock-chips.com> - 2017-07-19 11:10 +0200
    Re: [PATCH v2 0/5] arm64: dts: rockchip: support mail and IPA thermal for rk3399 Heiko Stuebner <heiko@sntech.de> - 2017-07-19 23:10 +0200

#1691147 — Re: [PATCH v2 0/5] arm64: dts: rockchip: support mail and IPA thermal for rk3399

FromCaesar Wang <wxt@rock-chips.com>
Date2017-07-19 11:10 +0200
SubjectRe: [PATCH v2 0/5] arm64: dts: rockchip: support mail and IPA thermal for rk3399
Message-ID<u4T9y-6rf-49@gated-at.bofh.it>
Hi Rob & Heiko,

Do we have the chance to merge these patches?

I'm try to bring up the display and run webgl for testing with my github on
https://github.com/Caesar-github/rockchip/commits/gru/next-stable-chromeos

I believe the Rocky@RK will post patches to support the other SoCs after.

-Caesar

在 2017年07月17日 16:14, Caesar Wang 写道:
> This series patches supported the mail in devicetree and used the
> thermal IPA by default.
>
> Verified with kernel is based on Linus's master branch and Heiko's
> v4.14-armsoc-tmp/dts64 branch. ( The Linux version 4.12.0 for now).
>
> The most rockchip SoCs will be supported with IPA mode for thermal
> in later.
>
> ---
>
> History version:
> 1. The first version found on
> https://www.spinics.net/lists/arm-kernel/msg593118.html
>
> Tested on Kevin board with bringing up ChromeOS.
> OS VERSION:
>      CHROMEOS_RELEASE_DESCRIPTION=9693.1.0 (Official Build) dev-channel kevin test
>
> BIOS VERSION:
>      Google_Kevin.8785.211.2017_06_20_1043
>
> EC VERSION:
>      Build info:    kevin_v1.10.217-24514961d 2017-07-03 07:46:36 wxt@nb
>
> With the ARM's lastest mali driver TX011-SW-99002-r18p0-01rel0 on
> https://developer.arm.com/products/software/mali-drivers/midgard-kernel
>
>  From the bootup log:
>      localhost devfreq0 # dmesg |grep mali
>      [    0.933334] mali ff9a0000.gpu: GPU identified as 0x0860 r2p0 status 0
>      [    0.940830] mali ff9a0000.gpu: Protected mode not available
>      [    0.947334] mali ff9a0000.gpu: Using configured power model mali-simple-power-model, and fallback mali-simple-power-model
>      [    0.960083] mali ff9a0000.gpu: Probed as mali0
>
>      localhost devfreq0 # pwd
>      /sys/devices/platform/ff9a0000.gpu/devfreq/devfreq0
>      localhost devfreq0 # ls
>      available_frequencies  device    min_freq          subsystem    uevent
>      available_governors    governor  polling_interval  target_freq  userspace
>      cur_freq               max_freq  power             trans_stat
>
>      localhost ff9a0000.gpu # ls
>      core_availability_policy  gpuinfo               modalias       soft_job_timeout
>      core_mask                 js_scheduling_period  of_node        subsystem
>      devfreq                   js_timeouts           pm_poweroff    uevent
>      driver                    mem_pool_max_size     power
>      driver_override           mem_pool_size         power_policy
>      dvfs_period               misc                  reset_timeout
> ---
>
> And for thermal with IPA.
> Try to run 'md5sum /dev/zero &' and octane/benchmark scripts to go up the temperature.
>  From the scripts to have a look at the actual control.
> "
> while true; do grep "" /sys/class/thermal/thermal_zone[0-1]/temp /sys/devices/system/cpu/cpu[0-5]/cpufreq/scaling_cur_freq /sys/devices/platform/ff9a0000.gpu/devfreq/devfreq0/cur_freq;date;sleep .5; done &
> "
>
> -Caesar
>
>
> Changes in v2:
> As Heiko comments on https://patchwork.kernel.org/patch/9835939/
> - interrupt-name use the lower case.
> - use the correct compatible "arm,mali-t860"
> - remove the clock name, since the mali only have one input clock.
> - foo@ will produce warnings when used without reg property.
> - update the commit to explain the two passive trip points changed.
>
> Caesar Wang (5):
>    dt-bindings: gpu: add the RK3399 mali for rockchip specifics
>    dt-bindings: gpu: add a power_model optional properties for MALI
>    arm64: dts: rockchip: add ARM Mali GPU node for RK3399 SoCs
>    arm64: dts: rockchip: enable the GPU for RK3399-GRU
>    arm64: dts: rockchip: update the thermal zones for RK3399 SoCs
>
>   .../devicetree/bindings/gpu/arm,mali-midgard.txt   | 13 ++++
>   arch/arm64/boot/dts/rockchip/rk3399-gru.dtsi       |  5 ++
>   arch/arm64/boot/dts/rockchip/rk3399-op1-opp.dtsi   | 33 ++++++++++
>   arch/arm64/boot/dts/rockchip/rk3399-opp.dtsi       | 33 ++++++++++
>   arch/arm64/boot/dts/rockchip/rk3399.dtsi           | 74 ++++++++++++----------
>   5 files changed, 125 insertions(+), 33 deletions(-)
>

[toc] | [next] | [standalone]

#1692201 — Re: [PATCH v2 0/5] arm64: dts: rockchip: support mail and IPA thermal for rk3399

FromHeiko Stuebner <heiko@sntech.de>
Date2017-07-19 23:10 +0200
SubjectRe: [PATCH v2 0/5] arm64: dts: rockchip: support mail and IPA thermal for rk3399
Message-ID<u54oi-5KN-35@gated-at.bofh.it>
In reply to#1691147
Am Mittwoch, 19. Juli 2017, 17:06:59 CEST schrieb Caesar Wang:
> Hi Rob & Heiko,
> 
> Do we have the chance to merge these patches?

From the mali-thermal thingy, it doesn't look like Rob likes that IPA stuff.

So I'll most likely pick the other patches that rely on approved bindings
after I return from my vacation and leave the thermal issue until it can
be resolved.


Heiko

> 
> I'm try to bring up the display and run webgl for testing with my github on
> https://github.com/Caesar-github/rockchip/commits/gru/next-stable-chromeos
> 
> I believe the Rocky@RK will post patches to support the other SoCs after.
> 
> -Caesar
> 
> 在 2017年07月17日 16:14, Caesar Wang 写道:
> > This series patches supported the mail in devicetree and used the
> > thermal IPA by default.
> >
> > Verified with kernel is based on Linus's master branch and Heiko's
> > v4.14-armsoc-tmp/dts64 branch. ( The Linux version 4.12.0 for now).
> >
> > The most rockchip SoCs will be supported with IPA mode for thermal
> > in later.
> >
> > ---
> >
> > History version:
> > 1. The first version found on
> > https://www.spinics.net/lists/arm-kernel/msg593118.html
> >
> > Tested on Kevin board with bringing up ChromeOS.
> > OS VERSION:
> >      CHROMEOS_RELEASE_DESCRIPTION=9693.1.0 (Official Build) dev-channel kevin test
> >
> > BIOS VERSION:
> >      Google_Kevin.8785.211.2017_06_20_1043
> >
> > EC VERSION:
> >      Build info:    kevin_v1.10.217-24514961d 2017-07-03 07:46:36 wxt@nb
> >
> > With the ARM's lastest mali driver TX011-SW-99002-r18p0-01rel0 on
> > https://developer.arm.com/products/software/mali-drivers/midgard-kernel
> >
> >  From the bootup log:
> >      localhost devfreq0 # dmesg |grep mali
> >      [    0.933334] mali ff9a0000.gpu: GPU identified as 0x0860 r2p0 status 0
> >      [    0.940830] mali ff9a0000.gpu: Protected mode not available
> >      [    0.947334] mali ff9a0000.gpu: Using configured power model mali-simple-power-model, and fallback mali-simple-power-model
> >      [    0.960083] mali ff9a0000.gpu: Probed as mali0
> >
> >      localhost devfreq0 # pwd
> >      /sys/devices/platform/ff9a0000.gpu/devfreq/devfreq0
> >      localhost devfreq0 # ls
> >      available_frequencies  device    min_freq          subsystem    uevent
> >      available_governors    governor  polling_interval  target_freq  userspace
> >      cur_freq               max_freq  power             trans_stat
> >
> >      localhost ff9a0000.gpu # ls
> >      core_availability_policy  gpuinfo               modalias       soft_job_timeout
> >      core_mask                 js_scheduling_period  of_node        subsystem
> >      devfreq                   js_timeouts           pm_poweroff    uevent
> >      driver                    mem_pool_max_size     power
> >      driver_override           mem_pool_size         power_policy
> >      dvfs_period               misc                  reset_timeout
> > ---
> >
> > And for thermal with IPA.
> > Try to run 'md5sum /dev/zero &' and octane/benchmark scripts to go up the temperature.
> >  From the scripts to have a look at the actual control.
> > "
> > while true; do grep "" /sys/class/thermal/thermal_zone[0-1]/temp /sys/devices/system/cpu/cpu[0-5]/cpufreq/scaling_cur_freq /sys/devices/platform/ff9a0000.gpu/devfreq/devfreq0/cur_freq;date;sleep .5; done &
> > "
> >
> > -Caesar
> >
> >
> > Changes in v2:
> > As Heiko comments on https://patchwork.kernel.org/patch/9835939/
> > - interrupt-name use the lower case.
> > - use the correct compatible "arm,mali-t860"
> > - remove the clock name, since the mali only have one input clock.
> > - foo@ will produce warnings when used without reg property.
> > - update the commit to explain the two passive trip points changed.
> >
> > Caesar Wang (5):
> >    dt-bindings: gpu: add the RK3399 mali for rockchip specifics
> >    dt-bindings: gpu: add a power_model optional properties for MALI
> >    arm64: dts: rockchip: add ARM Mali GPU node for RK3399 SoCs
> >    arm64: dts: rockchip: enable the GPU for RK3399-GRU
> >    arm64: dts: rockchip: update the thermal zones for RK3399 SoCs
> >
> >   .../devicetree/bindings/gpu/arm,mali-midgard.txt   | 13 ++++
> >   arch/arm64/boot/dts/rockchip/rk3399-gru.dtsi       |  5 ++
> >   arch/arm64/boot/dts/rockchip/rk3399-op1-opp.dtsi   | 33 ++++++++++
> >   arch/arm64/boot/dts/rockchip/rk3399-opp.dtsi       | 33 ++++++++++
> >   arch/arm64/boot/dts/rockchip/rk3399.dtsi           | 74 ++++++++++++----------
> >   5 files changed, 125 insertions(+), 33 deletions(-)
> >
> 
> 
> 
>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web