Re: [PATCH 0/8] Power domains support for Exynos5433 SoCs

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Krzysztof,

On 2017-01-27 08:52, Krzysztof Kozlowski wrote:
On Fri, Jan 27, 2017 at 08:47:44AM +0100, Marek Szyprowski wrote:
On 2017-01-27 08:43, Krzysztof Kozlowski wrote:
On Wed, Jan 25, 2017 at 12:55:34PM +0100, Marek Szyprowski wrote:
This patchset is a final step to add support for all power domains
on Exynos5433 SoCs. This patchset contains patches for adding Exynos5433
support to Exynos power domain driver and definitions of all power
domains found in Exynos5433 SoCs.

Patches have been generated on top of linux-next from 25th January 2017.

This is a part of a larger task, which goal is to add support for power
domains on Exynos5433 SoCs / TM2 boards. All patches needed to get it
working have been pushed to the following git repo:
https://git.linaro.org/people/marek.szyprowski/linux-srpol.git v4.10-next-tm2-pd

To get everything working following patches/patchsets are needed on top
of the linux-next tree:
By "get everything working" you also mean that there is a runtime
dependency? In other words, can I apply DTS patches here?
IMHO we should wait with applying pm domains DTS patches. Without ALL other
patches
kernel will sooner or later panic with "(a)synchronous external abort" or
deadlock
in a different way.
Okay, makes sense. A partial solution would be to apply them with
status=disabled. After all it is still a valid description of a
hardware. However this would not bring much benefits anyway...

First we would need to learn the Exynos power domain driver to honor
"status=disabled" property, because right now it instantiate all domains
regardless the provided status ;) Feel free to prepare a patch for that.

I'm also not sure if driver core will handle such case properly. I remember
that it deferred probing of all devices when no power domain driver was
available, but this might be a bit different case than skipping domain
registration.

Beside the name of node I mentioned, please put all the power domains
sorted to each other by address. I see the big picture here:
https://git.linaro.org/people/marek.szyprowski/linux-srpol.git/tree/arch/arm64/boot/dts/exynos/exynos5433.dtsi?h=v4.10-next-tm2-pd
and it does not looks ordered.

I think I've already tried to keep them sorted by address, but maybe while
rebasing/updating/fixing something went wrong.

Best regards
--
Marek Szyprowski, PhD
Samsung R&D Institute Poland

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux