[PATCH 4/5] ARM: dts: rockchip: enable saradc for rk3288-evb

[547884940@qq.com (chen)]

Hi heiko,


Heiko Stuebner wrote on 2017?07?01? 19:06:
> Hi Jacob,
>
> Am Samstag, 1. Juli 2017, 10:02:46 CEST schrieb Jacob Chen:
>> saradc in rk3288-evb use 1.8v ref.
>>
>> Signed-off-by: Jacob Chen <jacob-chen@iotwrt.com>
>> ---
>>   arch/arm/boot/dts/rk3288-evb.dtsi | 14 ++++++++++++++
>>   1 file changed, 14 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/rk3288-evb.dtsi b/arch/arm/boot/dts/rk3288-evb.dtsi
>> index f226ca7..9cee856 100644
>> --- a/arch/arm/boot/dts/rk3288-evb.dtsi
>> +++ b/arch/arm/boot/dts/rk3288-evb.dtsi
>> @@ -129,6 +129,15 @@
>>   		};
>>   	};
>>   
>> +	vccadc_ref: vccadc-ref {
>> +		compatible = "regulator-fixed";
>> +		regulator-name = "vcc1v8_sys";
>> +		regulator-always-on;
>> +		regulator-boot-on;
>> +		regulator-min-microvolt = <1800000>;
>> +		regulator-max-microvolt = <1800000>;
>> +	};
> this looks wrong. Regulators should always be named with the name
> used in schematics and also, according to the rk3288-evb schematics
> I have here, the saradc uses vcc_18 directly which comes from either
> the rk808 or act8846 pmics (depending on the board version).
>

OK,  I don't have software to check that schematics...
Agree and i will send a new patch.

>> +
>>   	/* This turns on USB vbus for both host0 (ehci) and host1 (dwc2) */
>>   	vcc_host: vcc-host-regulator {
>>   		compatible = "regulator-fixed";
>> @@ -218,6 +227,11 @@
>>   	status = "okay";
>>   };
>>   
>> +&saradc {
>> +	vref-supply = <&vccadc_ref>;
> So taking the above into account
> 	vref-supply = <&vcc_18>; ?
>
>
> Heiko
>
>> +	status = "okay";
>> +};
>> +
>>   &sdmmc {
>>   	bus-width = <4>;
>>   	cap-mmc-highspeed;
>>
>
>