On Tue, Mar 16, 2021 at 2:56 PM Ulf Hansson <ulf.hansson@xxxxxxxxxx> wrote: > It looks like we have a couple of options to support this. I suggest > we consider the two below, but perhaps others (Arnd/Linus?) have > better ideas? Admittedly it's a bit hard to shoehorn this in as it is not a standard resource (clk, regulator, genpd, reset, gpio...) There is drivers/soc and then you end up with the same custom abstraction that qcom is using. The upside to using that is that we can #ifdef it to static stubs in the .h file if this SoC is not used, so I would go for that. See for example qcom_scm_ice_invalidate_key() used from drivers/firmware/qcom_scm.c, header is at include/linux/qcom_scm.h and here you find: #if IS_ENABLED(CONFIG_QCOM_SCM) and if not, there are some stubs. Yours, Linus Walleij
