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@xxxxxxxxxxxxxx>
> ---
> 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@xxxxxxxxxxxxxx>
> + */
> +
> + /*
> + * 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
>
