[RFC PATCH 3/5] dt-bindings: thermal: Add optional properties of Mediatek thermal controller

Rob Herring
2016-01-22T22:31:36Z

On Fri, Jan 22, 2016 at 04:40:27PM +0800, Pi-Cheng Chen wrote:
> This adds optional properties of Mediatek thermal controller which are
> required by SVS engine integrated with Mediatek thermal controller.
>
> Signed-off-by: Pi-Cheng Chen <pi-cheng.chen@linaro.org>
> ---
> .../bindings/thermal/mediatek-thermal.txt | 23 ++++++++++++++++------
> 1 file changed, 17 insertions(+), 6 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt
> index 81f9a51..acaacaa 100644
> --- a/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt
> +++ b/Documentation/devicetree/bindings/thermal/mediatek-thermal.txt
> @@ -7,6 +7,11 @@ this device needs phandles to the AUXADC. Also it controls a mux in the
> apmixedsys register space via AHB bus accesses, so a phandle to the APMIXEDSYS
> is also needed.
>
> +There is another hardware engine, SVS (Smart Voltage Scaling) which shares the
> +same block of banked registers with Mediatek thermal controller. Hence the
> +driver of SVS is integrated with the driver of Mediatek thermal controller. The
> +properties required by SVS engine are optional for Mediatek thermal controller.
> +
> Required properties:
> - compatible: "mediatek,mt8173-thermal"
> - reg: Address range of the thermal controller
> @@ -21,9 +26,15 @@ Required properties:
> - #thermal-sensor-cells : Should be 0. See ./thermal.txt for a description.
>
> Optional properties:
> -- nvmem-cells: A phandle to the calibration data provided by a nvmem device. If
> - unspecified default values shall be used.
> -- nvmem-cell-names: Should be "calibration-data"
> +- clocks, clock-names: Clocks that are optional for the thermal controller.
> + Specify to enable SVS engine.
> + "svs_pll": The PLL clock should be switched to during
> + initialization stage of SVS engine.
> + "svs_mux": The MUX clock controls the clock input of SVS engine.
> +- nvmem-cells: A list of phandles to the calibration data provided by a nvmem
> + device. If unspecified default values shall be used. The SVS
> + engine will be disabled if no SVS calibration data is specified.
> +- nvmem-cell-names: Should be "calibration-data" and "svs-calibration-data"
>
> Example:
>
> @@ -33,11 +44,11 @@ Example:
> reg = <0 0x1100b000 0 0x1000>;
> interrupts = <0 70 IRQ_TYPE_LEVEL_LOW>;
> clocks = <&pericfg CLK_PERI_THERM>, <&pericfg CLK_PERI_AUXADC>;
> - clock-names = "therm", "auxadc";
> + clock-names = "therm", "auxadc", ;

Huh?

> resets = <&pericfg MT8173_PERI_THERM_SW_RST>;
> reset-names = "therm";
> mediatek,auxadc = <&auxadc>;
> mediatek,apmixedsys = <&apmixedsys>;
> - nvmem-cells = <&thermal_calibration_data>;
> - nvmem-cell-names = "calibration-data";
> + nvmem-cells = <&thermal_calibration_data>, <&svs-calibration>;
> + nvmem-cell-names = "calibration-data", "svs-calibration-data";
> };
> --
> 1.9.1
>