From: Lokesh Vutla <email@example.com> To: Nishanth Menon <firstname.lastname@example.org> Cc: Device Tree Mailing List <email@example.com>, Grygorii Strashko <firstname.lastname@example.org>, Sekhar Nori <email@example.com>, Tero Kristo <firstname.lastname@example.org>, Rob Herring <email@example.com>, Linux ARM Mailing List <firstname.lastname@example.org> Subject: Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Date: Mon, 7 Sep 2020 19:53:19 +0530 [thread overview] Message-ID: <email@example.com> (raw) In-Reply-To: <20200907141427.ti6r3h6namv2hezw@akan> On 07/09/20 7:44 pm, Nishanth Menon wrote: > On 17:32-20200907, Lokesh Vutla wrote: >> Hi, >> >> On 27/08/20 12:21 pm, Lokesh Vutla wrote: >>> This series adds initial support for latest new SoC, J7200, from Texas Instruments. >>> >>> The J7200 SoC is a part of the K3 Multicore SoC architecture platform. >>> It is targeted for for automotive gateway, vehicle compute systems, >>> Vehicle-to-Vehicle (V2V) and Vehicle-to-Everything (V2X) applications. >>> The SoC aims to meet the complex processing needs of modern embedded products. >>> >>> See J7200 Technical Reference Manual (SPRUIU1, June 2020) >>> for further details: https://www.ti.com/lit/pdf/spruiu1 >>> >>> Changes since v1: >>> - Swapped Patch 1 and 2 as suggested by Nishanth. >>> - Added description for each SoC in yaml bindings. >>> >>> Testing: >>> - ./scripts/checkpatch --strict >>> - Few warningns about Line length exceeding 100 columns. >>> But these are corresponding to comments >>> - v8make dtbs_check >>> - DT_SCHEMA_FLAGS="-u" >>> DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" >>> v8make dtbs_check >>> - DT_SCHEMA_FLAGS="-u" >>> DT_SCHEMA_FILES="Documentation/devicetree/bindings/arm/ti/k3.yaml" >>> v8make dt_binding_check >> >> This series has been lying around for soo long with no major comments. It will >> be nice to get this merged. I understand we are waiting for Acks on yaml >> documentation but it would be bad to miss a merge window for a new platform >> because we are waiting for Acks on yaml conversion. > > I do require Rob / DT maintainer to ack the DT yaml conversion and the > j7200 binding addition. > > Besides yaml and compatibility acks, there are a few ancillary > comments to fix up.. Kconfig -> I think we should either stay with > status quo and create a new config option per SoC OR rename the > config to be generic (using j7200 with j721e SoC config is not very Please suggest your preference here. I guess separate defconfig for J7200? > consistent). In addition, around the stuff that is going to next in > parallel, the dts is generating additional warnings as well (DSS hmm..there is no DSS being added in this series. There is one checkpatch warning for PATCH 1: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst This can be ignored. > etc). I think it might be easier if we wait for DT maintainer ack on > bindings prior to giving further cosmetic comments (To allow for any > additional changes to come in to -next). > okay. Thanks and regards, Lokesh _______________________________________________ linux-arm-kernel mailing list firstname.lastname@example.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-09-07 14:24 UTC|newest] Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-08-27 6:51 Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 1/4] dt-bindings: arm: ti: Convert K3 board/soc bindings to DT schema Lokesh Vutla 2020-08-28 0:41 ` Nishanth Menon 2020-08-28 3:14 ` Lokesh Vutla 2020-08-28 3:47 ` Suman Anna 2020-08-28 13:07 ` Nishanth Menon 2020-09-04 11:55 ` Nishanth Menon 2020-09-04 7:15 ` Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 2/4] dt-bindings: arm: ti: Add bindings for J7200 SoC Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 3/4] arm64: dts: ti: Add support " Lokesh Vutla 2020-08-27 17:04 ` Suman Anna 2020-09-08 11:47 ` Nishanth Menon 2020-08-31 9:13 ` Peter Ujfalusi 2020-09-02 3:51 ` Lokesh Vutla 2020-08-27 6:51 ` [PATCH v2 4/4] arm64: dts: ti: Add support for J7200 Common Processor Board Lokesh Vutla 2020-09-08 11:57 ` Nishanth Menon 2020-08-27 9:09 ` [PATCH v2 0/4] arm64: Initial support for Texas Instrument's J7200 Platform Grygorii Strashko 2020-09-07 12:02 ` Lokesh Vutla 2020-09-07 14:14 ` Nishanth Menon 2020-09-07 14:23 ` Lokesh Vutla [this message] 2020-09-07 23:48 ` Nishanth Menon 2020-09-08 9:55 ` Tero Kristo 2020-09-08 11:25 ` Nishanth Menon
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCH v2 0/4] arm64: Initial support for Texas Instrument'\''s J7200 Platform' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).