On 11.01.2023 03:18, Dmitry Baryshkov wrote:
On 10/01/2023 20:54, Konrad Dybcio wrote:If we both mean the "speed bin"-dependent values, the driver
On 10.01.2023 18:56, Konrad Dybcio wrote:
From: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxxx>Need to add
Add the bindings for the CPR3 driver to the documentation.
Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxxx>
[Konrad: Add type reference to acc-syscon; update AGdR's email]
Signed-off-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>
---
qcom,opp-oloop-vadj
qcom,opp-cloop-vadj
And note that at least for CPR3 they are different between fusing revisions. I see that for CPRh (esp. for 8998v2) they are the same, but this is not the case for 8996 (CPR3).
reads the fuse value but currently does nothing. My guess would
be that Angelo omitted it, as - just like you pointed out - MSM8998
(and SDM660 for that matter) don't really use it. I suppose I could
take care of that in bindings by making this an array and handle it
separately in a different patchset, as the per-revision values
aren't *that much* different, and again aren't really of concern for
the first round of supported SoCs.