Re: [PATCH v2] ARM: dts: am5729: beaglebone-ai: adding device tree

[Tony Lindgren <tony@atomide.com>]

* Drew Fustini <pdp7pdp7@gmail.com> [200331 18:39]:
> 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",

Yes I think Jason had some comment on the compatible string
to use.

Regards,

Tony