Hi Tony,
> Am 26.08.2019 um 16:55 schrieb Tony Lindgren <tony@xxxxxxxxxxx>:
>
> * Tony Lindgren <tony@xxxxxxxxxxx> [190817 06:56]:
>> * Stephen Boyd <sboyd@xxxxxxxxxx> [190815 18:24]:
>>> Quoting Tony Lindgren (2019-08-14 06:14:06)
>>>> diff --git a/arch/arm/boot/dts/omap54xx-clocks.dtsi b/arch/arm/boot/dts/omap54xx-clocks.dtsi
>>>> --- a/arch/arm/boot/dts/omap54xx-clocks.dtsi
>>>> +++ b/arch/arm/boot/dts/omap54xx-clocks.dtsi
>>>> @@ -1146,6 +1146,20 @@
>>>> };
>>>> };
>>>>
>>>> + gpu_cm: gpu_cm@1500 {
>>>
>>> Node names shouldn't have underscores. Maybe clock-controller?
>>
>> OK yeah clock-controller sounds good to me.
>
> Below is this one updated to use clock-controller naming.
>
> Regards,
>
> Tony
>
> 8< ---------------------------
> From tony Mon Sep 17 00:00:00 2001
> From: Tony Lindgren <tony@xxxxxxxxxxx>
> Date: Wed, 14 Aug 2019 05:18:16 -0700
> Subject: [PATCH] ARM: dts: Configure sgx for omap5
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
>
> I've tested that the interconnect target module enables and idles
> just fine when probed with ti-sysc with PM runtime control via sys:
>
> # echo on > $(find /sys -name control | grep \/5600)
Patch has arrived in v5.4-rc1 but does not work in that environment for me:
root@letux:~# echo on > $(find /sys -name control | grep \/5600)
[ 213.490926] clock-controller:clk:0000:0: failed to enable
root@letux:~# uname -a
Linux letux 5.4.0-rc1+ #1083 SMP Sat Oct 5 17:47:13 CEST 2019 armv7l GNU/Linux
root@letux:~# find /sys -name control | grep \/5600
/sys/devices/platform/44000000.ocp/5600fe00.target-module/power/control
root@letux:~#
I have tested our own tree but also mainline (plus some minimal
patch set to run the Pyra) using omap2plus_defconfig.
Does it work on omap5uevm?
Is there some related patch needed that did not yet arrive in v5.4-rc1?
BR and thanks,
Nikolaus
