| Message ID | 20210525200246.118323-1-konrad.dybcio@somainline.org |
|---|---|
| State | Accepted |
| Commit | db718417e87f1eb4562d8bc8b02fb108e80d63a1 |
| Headers | show |
| Series | [1/7] arm64: dts: qcom: Add PMI8996 DTSI file | expand |
On Tue 25 May 15:02 CDT 2021, Konrad Dybcio wrote: > PMI8996 is *almost* the same hardware as PMI8994, say for some annoyances: > > - Boards equipped with PMI8996 now have to include pmic-id (which wasn't the case before) > - Different qpnp-ibb-discharge-resistor value (will be addressed after LABIBB is introduced) > - Different inhibit-derating-ua value (will be addressed after BCL is introduced) > - Different ramp_up_step value (will be addressed after [if?] QPNP Flash LED is introduced) > > Signed-off-by: Konrad Dybcio <konrad.dybcio@somainline.org> > --- > arch/arm64/boot/dts/qcom/pmi8996.dtsi | 15 +++++++++++++++ > 1 file changed, 15 insertions(+) > create mode 100644 arch/arm64/boot/dts/qcom/pmi8996.dtsi > > diff --git a/arch/arm64/boot/dts/qcom/pmi8996.dtsi b/arch/arm64/boot/dts/qcom/pmi8996.dtsi > new file mode 100644 > index 000000000000..31b47209e261 > --- /dev/null > +++ b/arch/arm64/boot/dts/qcom/pmi8996.dtsi > @@ -0,0 +1,15 @@ > +// SPDX-License-Identifier: GPL-2.0 Can we have this with BSD license (as well)? > +/* > + * Copyright (c) 2021, Konrad Dybcio <konrad.dybcio@somainline.org> > + */ > + > + /* > + * PMI8996 is a slight modification of PMI8994 with > + * some notable changes, like being the first PMIC > + * whose the bootloader has to check to continue booting > + * and a change to a LABIBB parameter. > + */ > + > +/ { > + qcom,pmic-id = <0x20009 0x10013 0 0>; > +}; So I'm supposed to include this and the pmi8994? I was expecting an include of pmi8994.dtsi from here and then some override of the necessary properties. Regards, Bjorn > -- > 2.31.1 >
diff --git a/arch/arm64/boot/dts/qcom/pmi8996.dtsi b/arch/arm64/boot/dts/qcom/pmi8996.dtsi new file mode 100644 index 000000000000..31b47209e261 --- /dev/null +++ b/arch/arm64/boot/dts/qcom/pmi8996.dtsi @@ -0,0 +1,15 @@ +// SPDX-License-Identifier: GPL-2.0 +/* + * Copyright (c) 2021, Konrad Dybcio <konrad.dybcio@somainline.org> + */ + + /* + * PMI8996 is a slight modification of PMI8994 with + * some notable changes, like being the first PMIC + * whose the bootloader has to check to continue booting + * and a change to a LABIBB parameter. + */ + +/ { + qcom,pmic-id = <0x20009 0x10013 0 0>; +};
PMI8996 is *almost* the same hardware as PMI8994, say for some annoyances: - Boards equipped with PMI8996 now have to include pmic-id (which wasn't the case before) - Different qpnp-ibb-discharge-resistor value (will be addressed after LABIBB is introduced) - Different inhibit-derating-ua value (will be addressed after BCL is introduced) - Different ramp_up_step value (will be addressed after [if?] QPNP Flash LED is introduced) Signed-off-by: Konrad Dybcio <konrad.dybcio@somainline.org> --- arch/arm64/boot/dts/qcom/pmi8996.dtsi | 15 +++++++++++++++ 1 file changed, 15 insertions(+) create mode 100644 arch/arm64/boot/dts/qcom/pmi8996.dtsi