Re: [PATCH 5/9] arm64: dts: rockchip: add rk3568 tsadc nodes

From: Piotr Oniszczuk
Date: Mon Jan 17 2022 - 09:13:26 EST




> Wiadomość napisana przez Peter Geis <pgwipeout@xxxxxxxxx> w dniu 17.01.2022, o godz. 14:49:
>
>>
>> Peter,
>>
>> I'm trying to use this series to get thermal on rk3566 based tvbox (x96-x6).
>> Driver loads but i'm getting following error:
>>
>> [ 132.873979] rockchip-thermal fe710000.tsadc: failed to register sensor 0: -517
>> [ 132.874650] rockchip-thermal fe710000.tsadc: failed to register sensor[0] : error = -517
>>
>> Maybe you have some hints here?
>
> Error -517 means -EPROBE_DEFER, a device it relies on hasn't probed.
> TSADC doesn't directly rely on anything that shouldn't already be
> enabled. (Pinctrl, clocks, and grf all break a lot more than tsadc if
> disabled)
> Does your kernel config have all of the rockchip elements enabled?
>

Peter,
For sure I can't say 'yes' with 100% confidence.
I'm trying do my best for rk3566 (currently have working: SD/Eth/HDMI/Sound/USB2port0/BT).

Maybe you may point me pls for good reference of rk356x defconfig?

btw: a bit of context:
In my application (https://github.com/warpme/minimyth2) i have target to have single OS binary for all supported Allwinner/Amlogic/Rockchip/Broadcom devices.
By this kernel config i'm using is painfully assembled to have minimal working kernel for all targets.
It's a painful road....but still want to go this route for: minimal image size; shortest build time & one-for-all binary.
my current config: https://github.com/warpme/minimyth2/blob/master/script/kernel/linux-5.16/files/linux-5.16-arm64-armv8.config

btw2: for rk3566 i'm using majority of your patches!
Great work of you!