On 07/02/2023 14:09, bchihi@xxxxxxxxxxxx wrote: > From: Balsam CHIHI <bchihi@xxxxxxxxxxxx> > > Add LVTS thermal controllers dt-binding definition for mt8192 and mt8195. > > +allOf: > + - $ref: thermal-sensor.yaml# > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - mediatek,mt8192-lvts-ap > + - mediatek,mt8192-lvts-mcu > + then: > + properties: > + nvmem-cells: > + maxItems: 1 > + > + nvmem-cell-names: > + maxItems: 1 > + > + - if: > + properties: > + compatible: > + contains: > + enum: > + - mediatek,mt8195-lvts-ap > + - mediatek,mt8195-lvts-mcu > + then: > + properties: > + nvmem-cells: minItems: 2 > + maxItems: 2 > + > + nvmem-cell-names: minItems: 2 > + maxItems: 2 > + > +required: > + - compatible > + - reg > + - interrupts (...) > diff --git a/include/dt-bindings/thermal/mediatek,lvts-thermal.h b/include/dt-bindings/thermal/mediatek,lvts-thermal.h > new file mode 100644 > index 000000000000..4f2082065a31 > --- /dev/null > +++ b/include/dt-bindings/thermal/mediatek,lvts-thermal.h > @@ -0,0 +1,19 @@ > +/* SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) */ > +/* > + * Copyright (c) 2023 MediaTek Inc. > + * Author: Balsam CHIHI <bchihi@xxxxxxxxxxxx> > + */ > + > +#ifndef __MEDIATEK_LVTS_DT_H > +#define __MEDIATEK_LVTS_DT_H > + > +#define MT8195_MCU_BIG_CPU0 0 > +#define MT8195_MCU_BIG_CPU1 1 > +#define MT8195_MCU_BIG_CPU2 2 > +#define MT8195_MCU_BIG_CPU3 3 > +#define MT8195_MCU_LITTLE_CPU0 4 These changed for some reason. Why? The indentation is now broken (although not visible above quote, but visible in diff and code). Best regards, Krzysztof