AW: [PATCH 2/2] arm64: dts: imx8mm: Add support for emtrion emCON-MX8M Mini

[Mueller, Reinhold]

Hi Matti,

thanks for the feedback.
Please look at the comment below.

Regards
Reinhold


Reinhold Mueller
Software engineer


emtrion GmbH
Am Hasenbiel 6 | 76297 Stutensee | Germany

Phone +49 7244 62694 20
Fax +49 7244 62694 19
Email Reinhold.Mueller@xxxxxxxxxx
Online www.emtrion.de




emtrion GmbH • Amtsgericht Mannheim • HRB 110 300 • Geschäftsführer: Ramona Maurer, Achmed Haddou • Umsatzsteueridentifikationsnummer:DE813694260 • Impressum: www.emtrion.de/de/impressum.html

HINWEIS: Personenbezogene Daten, die Sie per E-Mail an uns übermitteln, werden bei uns gespeichert und verarbeitet. Informationen zu unseren gesetzlichen Informationspflichten, zu uns und unseren Dienstleistungen finden Sie in unseren Datenschutzhinweisen.
Diese E-Mail kann vertrauliche und / oder rechtlich geschützte Informationen enthalten. Wenn Sie nicht der richtige Adressat sind, oder diese E-Mail irrtümlich erhalten haben, informieren Sie bitte den Absender und vernichten diese Mail. Das unerlaubte kopieren, sowie die unbefugte Weitergabe dieser Mail ist nicht gestattet.
> -----Ursprüngliche Nachricht-----
> Von: Matti Vaittinen <mazziesaccount@xxxxxxxxx>
> Gesendet: Freitag, 10. Dezember 2021 21:05
> An: Mueller, Reinhold <Reinhold.Mueller@xxxxxxxxxx>;
> robh+dt@xxxxxxxxxx
> Cc: shawnguo@xxxxxxxxxx; s.hauer@xxxxxxxxxxxxxx;
> kernel@xxxxxxxxxxxxxx; festevam@xxxxxxxxx; linux-imx@xxxxxxx;
> devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; inux-arm-
> kernel@xxxxxxxxxxxxxxxxxxx
> Betreff: Re: [PATCH 2/2] arm64: dts: imx8mm: Add support for emtrion
> emCON-MX8M Mini
>
> Hi dee Ho Reinhold,
>
>
> On 12/9/21 14:09, reinhold.mueller@xxxxxxxxxxx wrote:
> > From: Reinhold Mueller <reinhold.mueller@xxxxxxxxxxx>
> >
> > This patch adds support for the emtrion GmbH emCON-MX8M Mini
> modules.
> > They are available with NXP i.MX 8M Mini equipped with 2 or 4 GB
> Memory.
> >
> > The devicetree imx8mm-emcon.dtsi is the common part providing all
> > module components and the basic support for the SoC. The support for the
> > avari baseboard in the developer-kit configuration is provided by the
> > emcon-avari dts files.
> >
> > Signed-off-by: Reinhold Mueller <reinhold.mueller@xxxxxxxxxxx>
> > ---
> >   arch/arm64/boot/dts/freescale/Makefile        |   3 +-
> >   .../boot/dts/freescale/imx8mm-emcon-avari.dts |  23 +
> >   .../dts/freescale/imx8mm-emcon-avari.dtsi     | 141 ++++
> >   .../boot/dts/freescale/imx8mm-emcon.dtsi      | 645
> ++++++++++++++++++
> >   4 files changed, 811 insertions(+), 1 deletion(-)
> >   create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-emcon-
> avari.dts
> >   create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-emcon-
> avari.dtsi
> >   create mode 100644 arch/arm64/boot/dts/freescale/imx8mm-emcon.dtsi
> >
>
> > +&i2c3 {
> > +clock-frequency = <400000>;
> > +pinctrl-names = "default";
> > +pinctrl-0 = <&pinctrl_i2c3>;
> > +status = "okay";
> > +
> > +bd71847:pmic@4b {
> > +compatible = "rohm,bd71847";
> > +reg = <0x4b>;
> > +pinctrl-0 = <&pinctrl_pmic>;
> > +interrupt-parent = <&gpio3>;
> > +interrupts = <2 GPIO_ACTIVE_LOW>;
>
> I assume this should be IRQ_TYPE_LEVEL_LOW
>
You are right. I will update the patchset.

> Best Regards
> -- Matti Vaittinen