On Thu, 25 Jul 2019 at 10:34, Guillaume <guillaume.gardet@xxxxxxx> wrote: > > From: Guillaume GARDET <guillaume.gardet@xxxxxxx> > > Add nodes for GPU (Mali T604) to Exynos5250. > Tested with kmscube and glmark2-es2-drm on Chromebook snow. > Load tested on arndale board. > > Signed-off-by: Guillaume GARDET <guillaume.gardet@xxxxxxx> > > Cc: Kukjin Kim <kgene@xxxxxxxxxx> > Cc: Krzysztof Kozlowski <krzk@xxxxxxxxxx> > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > --- > V2 changes: > * move mali node as /soc sub-node > * move gpu_opp_table as mali sub-node > * minor style updates > * test on arndale board > * enable by default I was not specific enough. It can stay disabled in the DTSI because not all resources are provided. The board DTSes (all or ones which are tested) will enable it and provide missing resources - mali-supply. Just like: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=f0a6208b90bdd44f48f5718c8bb0eb1e763d14c0 > * add dt bindings, in a separate patch Put the bindings before DTS change. > > arch/arm/boot/dts/exynos5250.dtsi | 47 +++++++++++++++++++++++++++++++ > 1 file changed, 47 insertions(+) > > diff --git a/arch/arm/boot/dts/exynos5250.dtsi b/arch/arm/boot/dts/exynos5250.dtsi > index d5e0392b409e..f7f34d469bde 100644 > --- a/arch/arm/boot/dts/exynos5250.dtsi > +++ b/arch/arm/boot/dts/exynos5250.dtsi > @@ -328,6 +328,52 @@ > iommus = <&sysmmu_rotator>; > }; > > + mali: gpu@11800000 { > + compatible = "samsung,exynos5250-mali", "arm,mali-t604"; > + reg = <0x11800000 0x5000>; > + interrupts = <GIC_SPI 118 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 119 IRQ_TYPE_LEVEL_HIGH>, > + <GIC_SPI 117 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-names = "job", "mmu", "gpu"; > + clocks = <&clock CLK_G3D>; > + clock-names = "g3d"; According to bindings this is not proper clock name. I wonder why it was working... it seems the driver expects first clock to be always the core clock. Best regards, Krzysztof
