Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> 於 2023年9月13日 週三 下午6:04寫道：
>
> On 12/09/2023 16:05, yangchen.openbmc@xxxxxxxxx wrote:
> > From: Yang Chen <yangchen.openbmc@xxxxxxxxx>
> >
> > Add linux device tree entry related to the Minerva Chassis Management
> > Controller (CMC) specific devices connected to the Aspeed SoC (AST2600).
> >
> > Signed-off-by: Yang Chen <yangchen.openbmc@xxxxxxxxx>
> > ---
> > arch/arm/boot/dts/aspeed/Makefile | 1 +
> > .../aspeed-bmc-facebook-minerva-cmc.dts | 265 ++++++++++++++++++
> > 2 files changed, 266 insertions(+)
> > create mode 100644 arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-minerva-cmc.dts
> >
> > diff --git a/arch/arm/boot/dts/aspeed/Makefile b/arch/arm/
>
>
> > +
> > +&i2c15 {
> > + status = "okay";
> > +
> > + eeprom@50 {
> > + compatible = "atmel,24c128";
> > + reg = <0x50>;
> > + };
> > +};
> > +
> > +&adc0 {
> > + ref_voltage = <2500>;
>
> There is no such property in the bindings.
Thanks for your advice, I will change this property to
"aspeed,int-vref-microvolt" which is defined in the binding in the v2
patch.
>
> > + status = "okay";
> > + pinctrl-0 = <&pinctrl_adc0_default &pinctrl_adc1_default
> > + &pinctrl_adc2_default &pinctrl_adc3_default
> > + &pinctrl_adc4_default &pinctrl_adc5_default
> > + &pinctrl_adc6_default &pinctrl_adc7_default>;
> > +};
> > +
> > +&adc1 {
> > + ref_voltage = <2500>;
>
> NAK, there is no such property.
Thanks for your advice, I will change this property to
"aspeed,int-vref-microvolt" which is defined in the binding in the v2
patch.
>
>
>
> Best regards,
> Krzysztof
>
