Re: [PATCH V2 2/2] arm64: dts: ti: Add support for J721E EAIK

[Nishanth Menon <nm@ti.com>]

On 22:41-20210607, Sinthu Raja M wrote:
> On Mon, Jun 7, 2021 at 7:05 PM Nishanth Menon <nm@ti.com> wrote:
> >
> > On 15:03-20210607, Sinthu Raja wrote:
> > > From: Sinthu Raja <sinthu.raja@ti.com>
> > >
> > > J721E EdgeAI Kit (EAIK) is a low cost, small form factor board designed
> > > for TI’s J721E SoC. TI’s J721E SoC comprises of dual core A72, high
> > > performance vision accelerators, video codec accelerators, latest C71x
> > > and C66x DSP, high bandwidth real-time IPs for capture and display, GPU,
> > > dedicated safety island and security accelerators. The SoC is power
> > > optimized to provide best in class performance for perception, sensor
> > > fusion, localization, path planning tasks in robotics, industrial and
> > > automotive applications.
> > >
> > > J721E EAIK supports the following interfaces:
> > > * 4 GB LPDDR4 RAM
> > > * x1 Gigabit Ethernet interface
> > > * x1 USB 3.0 Type-C port
> > > * x3 USB 3.0 Type-A ports
> > > * x1 UHS-1 capable µSD card slot
> > > * x1 PCIe M.2 E Key with x1 USB2.0, x1 MCASP, x1 MMC, x1 UART
> > > * x1 PCIe M.2 M Key
> > > * 512 Mbit OSPI flash
> > > * x4 UART through UART-USB bridge
> > > * x4 CAN-FD interface
> > > * x1 DP interface
> > > * x1 HDMI interface
> > > * x2 CSI2 Camera interface (RPi and TI Camera connector)
> > > * 40-pin Raspberry Pi compatible GPIO header
> > > * Compact TI 20-Pin connector for JTAG debug
> > > * Interface for remote automation. Includes:
> > >    * power measurement and reset control
> > >    * boot mode change
> >
> > please reduce the description above, also add a url for the board.
> 
> Sure will reduce the description. The URL is not available until the
> board launch.
> 

[...]

> >
> > please also check other nodes that needs to be disabled or configured.
> 
> The intention of the patch is to keep only the initial support for the
> EAIK board. Subsequent patches will be posted as the testing
> completes.

For the nodes that are present in k.org, we should probably make up our
minds as to what needs to be the configuration. While I understand
incremental patches, please look through the list to make sure low
hanging items are addressed at the earliest. Commit message makes one
feel that there is tons of nodes that are probably functional, but your
response makes me feel, that is probably not the case - it is hard to
distinguish between the two.


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D