[PATCH v2 1/2] dt-bindings: arm: qcom: Update Devicetree identifiers
Amrit Anand
quic_amrianan at quicinc.com
Mon May 6 22:07:06 PDT 2024
On 3/14/2024 7:59 PM, Caleb Connolly wrote:
>
> On 14/03/2024 12:11, Amrit Anand wrote:
>> Update existing documentation for qcom,msm-id (interchangeably:
>> qcom,soc-id) and qcom,board-id. Add support for qcom,pmic-id, qcom,oem-id
>> to support multi-DTB selection on Qualcomm's boards.
>>
>> "qcom,soc-id", "qcom,board-id" and "qcom,pmic-id" are tuples of two 32-bit
>> values. The "qcom,oem-id" is a tuple of one 32-bit value.
>> Introduce macros to help generate SOC, board, PMIC and OEM identifiers.
>> QCOM_SOC_ID and QCOM_SOC_REVISION can be used to generate qcom,msm-id.
>> QCOM_BOARD_ID and QCOM_BOARD_SUBTYPE can be used to generate qcom,board-id.
>> QCOM_PMIC_SID and QCOM_PMIC_MODEL can be used to generate qcom,pmic-id.
>> QCOM_OEM_ID can be used to generate qcom,oem-id.
>>
>> Add entries for different types of SoC, boards, DDR type, Boot device
>> type which are currently used by Qualcomm based bootloader.
>>
>> Signed-off-by: Amrit Anand <quic_amrianan at quicinc.com>
>> ---
>> include/dt-bindings/arm/qcom,ids.h | 86 ++++++++++++++++++++++++++++++++++----
>> 1 file changed, 77 insertions(+), 9 deletions(-)
>>
>> diff --git a/include/dt-bindings/arm/qcom,ids.h b/include/dt-bindings/arm/qcom,ids.h
>> index f724834..dc24151 100644
>> --- a/include/dt-bindings/arm/qcom,ids.h
>> +++ b/include/dt-bindings/arm/qcom,ids.h
>> @@ -8,9 +8,14 @@
>> #define _DT_BINDINGS_ARM_QCOM_IDS_H
>>
>> /*
>> - * The MSM chipset and hardware revision used by Qualcomm bootloaders, DTS for
>> - * older chipsets (qcom,msm-id) and in socinfo driver:
>> + * The MSM chipset ID (soc-id) used by Qualcomm bootloaders,
>> + * and in socinfo driver:
>> + * where, "a" indicates Qualcomm supported chipsets, example MSM8260, MSM8660 etc
>> */
>> +
>> +#define QCOM_SOC_ID(a) ((QCOM_ID_##a) && 0xffff)
> How is QCOM_SOC_ID(SC7280) ever better than QCOM_ID_SC7280? What does
> this mask give you? It's going into a 32-bit field and not being OR'd
> with anything else...
Ok, understood. Seems better.
>> +
>> +
>> #define QCOM_ID_MSM8260 70
>> #define QCOM_ID_MSM8660 71
>> #define QCOM_ID_APQ8060 86
>> @@ -266,16 +271,79 @@
>> #define QCOM_ID_IPQ5302 595
>> #define QCOM_ID_IPQ5300 624
>>
>> + /* The SOC revision used by Qualcomm bootloaders (soc-revision) */
>> +
>> +#define QCOM_SOC_REVISION(a) (a & 0xff)
>> +
>> /*
>> - * The board type and revision information, used by Qualcomm bootloaders and
>> - * DTS for older chipsets (qcom,board-id):
>> + * The board type and revision information (board-id), used by Qualcomm bootloaders
>> + * where, "a" indicates board type which can be either MTP, QRD etc
>> */
>> +
>> #define QCOM_BOARD_ID(a, major, minor) \
>> - (((major & 0xff) << 16) | ((minor & 0xff) << 8) | QCOM_BOARD_ID_##a)
>> + (((major & 0xff) << 16) | ((minor & 0xff) << 8) | ((QCOM_BOARD_ID_##a) & 0xff))
>> +
>> +#define QCOM_BOARD_ID_MTP 0x8
>> +#define QCOM_BOARD_ID_LIQUID 0x9
>> +#define QCOM_BOARD_ID_DRAGONBOARD 0xA
>> +#define QCOM_BOARD_ID_QRD 0x11
> This replaces the previous definition of QCOM_BOARD_ID_QRD which was
> decimal 11, now you're defining it as hex 0x11? I'm guessing this was a
> typo.
Will correct it.
Thanks,
Amrit
>> +#define QCOM_BOARD_ID_ADP 0x19
>> +#define QCOM_BOARD_ID_HDK 0x1F
>> +#define QCOM_BOARD_ID_ATP 0x21
>> +#define QCOM_BOARD_ID_IDP 0x22
>> +#define QCOM_BOARD_ID_SBC 0x24
>> +#define QCOM_BOARD_ID_QXR 0x26
>> +#define QCOM_BOARD_ID_X100 0x26
>> +#define QCOM_BOARD_ID_CRD 0x28
>> +
>> +/*
>> + * The platform subtype is used by Qualcomm bootloaders and
>> + * DTS (board-subtype)
>> + * where, "a" indicates boot device type, it can be EMMC,
>> + * UFS, NAND or OTHER (which can be used for default).
>> + * "b" indicates DDR type which can be 128MB, 256MB,
>> + * 512MB, 1024MB, 2048MB, 3072MB, 4096MB or ANY
>> + * (which can be used for default).
>> + */
>> +#define QCOM_BOARD_SUBTYPE(a, b, SUBTYPE) \
>> + (((QCOM_BOARD_BOOT_##a & 0xf) << 16) | ((QCOM_BOARD_DDRTYPE_##b & 0x7) << 8) | \
>> + (SUBTYPE & 0xff))
>> +
>> +/* Board DDR Type where each value indicates higher limit */
>> +#define QCOM_BOARD_DDRTYPE_ANY 0x0
>> +#define QCOM_BOARD_DDRTYPE_128M 0x1
>> +#define QCOM_BOARD_DDRTYPE_256M 0x2
>> +#define QCOM_BOARD_DDRTYPE_512M 0x3
>> +#define QCOM_BOARD_DDRTYPE_1024M 0x4
>> +#define QCOM_BOARD_DDRTYPE_2048M 0x5
>> +#define QCOM_BOARD_DDRTYPE_3072M 0x6
>> +#define QCOM_BOARD_DDRTYPE_4096M 0x7
>>
>> -#define QCOM_BOARD_ID_MTP 8
>> -#define QCOM_BOARD_ID_DRAGONBOARD 10
>> -#define QCOM_BOARD_ID_QRD 11
>> -#define QCOM_BOARD_ID_SBC 24
>> +/* Board Boot Device Type */
>> +#define QCOM_BOARD_BOOT_EMMC 0x0
>> +#define QCOM_BOARD_BOOT_UFS 0x1
>> +#define QCOM_BOARD_BOOT_NAND 0x2
>> +#define QCOM_BOARD_BOOT_OTHER 0x3
>> +
>> +/*
>> + * The PMIC slave id is used by Qualcomm bootloaders to
>> + * indicates which PMIC is attached (pmic-sid)
>> + */
>> +
>> +#define QCOM_PMIC_SID(a) (a & 0xff)
>> +
>> +/*
>> + * The PMIC ID is used by Qualcomm bootloaders to describe the ID
>> + * of PMIC attached to bus described by SID (pmic-model)
>> + */
>> +
>> +#define QCOM_PMIC_MODEL(ID, major, minor) \
>> + (((major & 0xff) << 16) | ((minor & 0xff) << 8) | (ID & 0xff))
>> +
>> +/*
>> + * The OEM ID consists of 32 bit value to support OEM boards where they
>> + * have slight differences on top of Qualcomm's standard boards
>> + */
>> +#define QCOM_OEM_ID(a) (a & 0xffffffff)
>>
>> #endif /* _DT_BINDINGS_ARM_QCOM_IDS_H */
More information about the linux-riscv
mailing list