From mboxrd@z Thu Jan  1 00:00:00 1970
From: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
Subject: Re: [PATCH 2/4] arm64: add basic DTS for i.MX8MQ
Date: Mon, 22 Jan 2018 16:11:42 +0100
Message-ID: <CAK8P3a0Yq1WuUYih6WKbbjoGRwq-rCQ8OEpuyu0=-SOVV2MNLg@mail.gmail.com>
References: <20180117183244.28303-1-l.stach@pengutronix.de>
 <20180117183244.28303-2-l.stach@pengutronix.de> <20180122144729.GA27764@dragon>
Mime-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
Return-path: <devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
In-Reply-To: <20180122144729.GA27764@dragon>
Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
To: Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Lucas Stach <l.stach-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, Catalin Marinas <catalin.marinas-5wv7dgnIgG8@public.gmane.org>, Will Deacon <will.deacon-5wv7dgnIgG8@public.gmane.org>, Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>, Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>, Fabio Estevam <fabio.estevam-3arQi8VN3Tc@public.gmane.org>, Linux ARM <linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>, DTML <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, Sascha Hauer <kernel-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>, patchwork-lst-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org
List-Id: devicetree@vger.kernel.org

On Mon, Jan 22, 2018 at 3:47 PM, Shawn Guo <shawnguo-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> wrote:
> On Wed, Jan 17, 2018 at 07:32:42PM +0100, Lucas Stach wrote:
>> This adds the basic DTS for the i.MX8MQ.
>> For now only the following peripherals are supported:
>> - IOMUXC (pin controller)
>> - CCM (clock controller)
>> - GPIO
>> - UART
>> - uSDHC (SD/eMMC controller)
>> - FEC (ethernet controller)
>> - i2c
>>
>> This is enough to get a very basic board support up and running.
>>
>> One known limitation is that the driver for the GPC interrupt
>> controller is still missing, rendering the CPU sleep states unusable
>> as there is nothing waking them up anymore. This will be fixed in
>> due course.
>>
>> Signed-off-by: Lucas Stach <l.stach-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
>> ---
>>  arch/arm64/boot/dts/Makefile             |   1 +
>>  arch/arm64/boot/dts/nxp/imx8mq-pinfunc.h | 632 +++++++++++++++++++++++++++++++
>>  arch/arm64/boot/dts/nxp/imx8mq.dtsi      | 418 ++++++++++++++++++++
>
> So we will have the following two folders for SoCs from the same company.
>
>  arch/arm64/boot/dts/nxp/
>  arch/arm64/boot/dts/freescale/

It might get worse once the sale to Qualcomm goes through and maybe if
Broadcom makes another bid for Qualcomm in a few years ;-)

> I'm a bit uncomfortable with that.  @Arnd, do you have an opinion how we
> should handle them?

It's a recurring problem of course. I have a small preference for just adding
the i.mx files to arch/arm64/boot/dts/freescale/, but I'd leave it up to you.

        Arnd
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html