Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1166299

Re: [PATCH 5/5] ARM64: MediaTek MT8173: Add SCPSYS device node

Show all headers | View raw

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

> This adds the SCPSYS device node to the MT8173 dtsi file.
>
> Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
> ---
>  arch/arm64/boot/dts/mediatek/mt8173.dtsi | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> index 924fdb6..12430f0 100644
> --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi
> @@ -125,6 +125,16 @@
>  						<GIC_SPI 147 IRQ_TYPE_LEVEL_HIGH>;
>  		};
>  
> +		scpsys: scpsys@10006000 {
> +			compatible = "mediatek,mt8173-scpsys";
> +			#power-domain-cells = <1>;
> +			reg = <0 0x10006000 0 0x1000>;
> +			clocks = <&clk26m>,
> +				 <&topckgen CLK_TOP_MM_SEL>;

Neither your binding doc (nor the generic one) mentions these clock
properties or what they are used for.  They appear to define a clock
that must be enabled in order to for the power domain to be on.

Also the order here seems rather important and probably needs
documenting in the binding (e.g. the clk_id order is hard-coded in the
driver.)

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

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH v4] Mediatek SCPSYS power domain support Sascha Hauer <s.hauer@pengutronix.de> - 2015-06-09 10:50 +0200
  [PATCH 5/5] ARM64: MediaTek MT8173: Add SCPSYS device node Sascha Hauer <s.hauer@pengutronix.de> - 2015-06-09 10:50 +0200
    Re: [PATCH 5/5] ARM64: MediaTek MT8173: Add SCPSYS device node Kevin Hilman <khilman@kernel.org> - 2015-06-16 20:20 +0200
  [PATCH 4/5] ARM64: MediaTek: Add generic pm domain support Sascha Hauer <s.hauer@pengutronix.de> - 2015-06-09 10:50 +0200
  [PATCH 2/5] dt-bindings: soc: Add documentation for the MediaTek SCPSYS unit Sascha Hauer <s.hauer@pengutronix.de> - 2015-06-09 10:50 +0200
  [PATCH 3/5] soc: Mediatek: Add SCPSYS power domain driver Sascha Hauer <s.hauer@pengutronix.de> - 2015-06-09 10:50 +0200
    Re: [PATCH 3/5] soc: Mediatek: Add SCPSYS power domain driver Sascha Hauer <s.hauer@pengutronix.de> - 2015-06-15 09:50 +0200
  [PATCH 1/5] soc: mediatek: Add infracfg misc driver support Sascha Hauer <s.hauer@pengutronix.de> - 2015-06-09 10:50 +0200

csiph-web