linux-kernel - Re: [RFC] Mediatek SCPSYS power domain support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <7hoao16gvm.fsf@deeprootsystems.com>
Date:	Mon, 09 Mar 2015 13:43:57 -0700
From:	Kevin Hilman <khilman@...nel.org>
To:	Sascha Hauer <s.hauer@...gutronix.de>
Cc:	linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org,
	linux-kernel@...r.kernel.org,
	Matthias Brugger <matthias.bgg@...il.com>,
	linux-mediatek@...ts.infradead.org, kernel@...gutronix.de
Subject: Re: [RFC] Mediatek SCPSYS power domain support

Sascha Hauer <s.hauer@...gutronix.de> writes:

> s series adds support for the MediaTek SCPSYS unit.
>
> The SCPSYS unit handles several power management related tasks such
> as thermal measurement, DVFS, interrupt filter and low level sleep
> control.
>
> The initial support only contains the generic power domain handling.
> This is needed to turn on power to the different power domains.
>
> The driver is quite straight forward now. Due to the lack of a better
> place I have put it to drivers/soc/mediatek. As the SCPSYS unit has
> several other tasks that also do not fit into some specific subsystem
> this probably is a good place for this driver.
>
> What is not clear to me is how the Kconfig dependencies should be
> handled. Some domains are disabled by bootloader default, so without
> this driver for example sound won't work. So should PM, PM_GENERIC_DOMAINS
> and this driver be made mandatory for MT8173? Or should this driver
> when PM is disabled be compiled as a stub which only enables all
> domains? I don't have a good idea how things should work out here.

IMO, a stub that enables all the domains is what you want.

Kevin
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/