Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <20191120220559.18914-1-c-robey@ti.com> <41c73bc1-99ae-6797-5bb7-7acc0f6518c0@ti.com>
 <20191212170452.GV35479@atomide.com>
In-Reply-To: <20191212170452.GV35479@atomide.com>
From:   Drew Fustini <pdp7pdp7@gmail.com>
Date:   Tue, 31 Mar 2020 20:38:38 +0200
Message-ID: <CAEf4M_C05Hwc_BEL6MaFNNEW0Cf2kc-LvMi9qdKxL7hVAFFDGQ@mail.gmail.com>
Subject: Re: [PATCH v2] ARM: dts: am5729: beaglebone-ai: adding device tree
To:     Tony Lindgren <tony@atomide.com>
Cc:     Grygorii Strashko <grygorii.strashko@ti.com>,
        Caleb Robey <c-robey@ti.com>, linux-omap@vger.kernel.org,
        Jason Kridner <jkridner@gmail.com>,
        Lokesh Vutla <lokeshvutla@ti.com>, Jason Kridner <jdk@ti.com>,
        Faiz Abbas <faiz_abbas@ti.com>,
        Andreas Dannenberg <dannenberg@ti.com>,
        Jean-Jacques Hiblot <jjhiblot@ti.com>,
        Praneeth Bajjuri <praneeth@ti.com>,
        "Andrew F . Davis" <afd@ti.com>, Tom Rini <trini@konsulko.com>,
        Robert Nelson <robertcnelson@gmail.com>,
        Rob Herring <robh+dt@kernel.org>,
        Mark Rutland <mark.rutland@arm.com>,
        =?UTF-8?Q?Beno=C3=AEt_Cousson?= <bcousson@baylibre.com>,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        Drew Fustini <drew@beagleboard.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk

On Thu, Dec 12, 2019 at 6:06 PM Tony Lindgren <tony@atomide.com> wrote:
> > On 21/11/2019 00:05, Caleb Robey wrote:
> > > From: Jason Kridner <jdk@ti.com>
> > >
> > > BeagleBoard.org BeagleBone AI is an open source hardware single
> > > board computer based on the Texas Instruments AM5729 SoC featuring
> > > dual-core 1.5GHz Arm Cortex-A15 processor, dual-core C66 digital
> > > signal processor (DSP), quad-core embedded vision engine (EVE),
> > > Arm Cortex-M4 processors, dual programmable realtime unit
> > > industrial control subsystems and more. The board features 1GB
> > > DDR3L, USB3.0 Type-C, USB HS Type-A, microHDMI, 16GB eMMC flash,
> > > 1G Ethernet, 802.11ac 2/5GHz, Bluetooth, and BeagleBone expansion
> > > headers.
> > >
> > > For more information, refer to:
> > > https://beaglebone.ai
> > >
> > > This patch introduces the BeagleBone AI device tree.
> > >
> > > Note that the device use the "ti,tpd12s016" component which is
> > > software compatible with "ti,tpd12s015". Thus we only use the
> > > latter driver.
> >
> > Ah. thanks. I see my comments resolved here.
> > no more comments to net part from my side.
>
> Just FYI, Jason had one pending comment on the earlier version
> about the compatible property to use.
>
> So I'm assuming there will be a new version posted, tagging
> this one as read.

It came to my attention today when talking with Jason Kridner and
Robert Nelson that we did not get the BeagleBone AI device tree
upstream yet.

I am having trouble identifying what the pending comment was from the
original patch series.

Was it related to this compatible string?

> +       compatible = "beagleboard.org,am5729-beagleboneai", "ti,am5728",

thanks,
drew

--
Drew Fustini
BeagleBoard.org Foundation
https://beagleboard.org/about