From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 4765FC433EF for ; Wed, 9 Feb 2022 21:30:40 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To: Subject:MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=HzWwRx3R5SgJ8srmpVxBKQtbE5QbdZ4XoQlkmSxPGfc=; b=t8vFPrsuidtUla N6vEzidxgBbLB4+WqEW93mkG5SKCTWnNPzEpoMwcTlydYVyVlJGbsVill0kt2bpJBGnqlvZTlq5cj QmNRKRgvS3dlvV0dWhEVID/mqfywZVPEXwpRlxG+IcTdYpegPnjIPwR3Arak+B21VT9i/mL61+boH kax39yhdYUVnO/LyGo8iQIr5He9N8u8RHsMb4H1fqH3hL4X0a9YAL2tXXxOwEOTo+B3wdwiUC4wl4 p3OywvW/DF7i+4m9MZH/cGnvtaPkQhKGb2ucJHwEZGY7TI+xyj4FyebxnryCqG1oCXLbZx47MpnUk 133GTRazHVyjtCd/1OHQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nHuWd-001gTM-UJ; Wed, 09 Feb 2022 21:29:20 +0000 Received: from smtp-relay-internal-1.canonical.com ([185.125.188.123]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nHuWa-001gSS-Nh for linux-arm-kernel@lists.infradead.org; Wed, 09 Feb 2022 21:29:18 +0000 Received: from mail-ej1-f69.google.com (mail-ej1-f69.google.com [209.85.218.69]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-internal-1.canonical.com (Postfix) with ESMTPS id 825C94004C for ; Wed, 9 Feb 2022 21:29:07 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1644442147; bh=9TShvtjM40hZA5vpKCtm1+G42669wQg8EHY6WDDuENg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=vUIyZwCkJDhG7fVXJZNYs0ErDukIkhwILoXZtwIpNCMuxlBiPHlY++J+9EfRjh+58 ouXj08Qn8Oodkz1OqeMMUxkNEzDV9jJI8az6mmWayLxMJZRadZ3EiEZ/LT718cByD8 ze5aOFfsEAz9Lf4eD43tCxMKUrWq4duuza2Hb/rc3KKny1EO4d4bfowd3iwqNxIaMn JiZNY7D4YQHIaHUagJ00Dv4UB771b9pYxigW8nC9Eyovy1sOEX4i3AkwnT93J3iZ03 SeAnj3dP8iDhHyeqE2ROiVrTTTFyVeSdOnHLalDwF5jCM0Ym5w38mc/R+NqlZ0b3c3 zqjDXi40D13rA== Received: by mail-ej1-f69.google.com with SMTP id hr36-20020a1709073fa400b006cd2c703959so1755121ejc.14 for ; Wed, 09 Feb 2022 13:29:07 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=9TShvtjM40hZA5vpKCtm1+G42669wQg8EHY6WDDuENg=; b=v6UKmd2IKULSGs57ZR/ybCyh/gRvSTTOrxBDgCV8MvEEIOZBHZahPhRrGVtCxoLS6v O+PQVLwcFDdktli0T9iue8S9vQsZgnmz7J0C4WZseOJKuuHE5kg6lENYFvZx6Eyg98xJ lTBon9SM4ZSJBTN5Zoyocaus2iLQBCWXWj+P7UIu4kXd+QDHbBb655bolE7E0NXOYSow qjlSnE58m6OegcsFxTDvsrP8dW89sTkNkCqCFqFe4D5NZ2ogPRpSO8BDWPS1jMK26ZE3 fcughc/C8NB0GVEVyTgWQPjGwTFVp4JfeQ4IxpZpvIoHlL444t6iVZ88BowXeLHeI+ca agEA== X-Gm-Message-State: AOAM532Y1MWUCF70KIGpcBBkvZOXdYpn2uOFq25S/8ZHIU1JpFEbuPjS FKpv4PaPyyR71NQGXk60F4faZPXNpYp2Yp1C+1VLgBJxBBjNIHR7pt7+IWhOpEsFiKA0uRU9Lxv jhsVCbqnP0w5hxFJOrEz+IJEo4Runf9ecwCIptHKuFEi49DS54EwI X-Received: by 2002:a17:906:7489:: with SMTP id e9mr3719921ejl.113.1644442147089; Wed, 09 Feb 2022 13:29:07 -0800 (PST) X-Google-Smtp-Source: ABdhPJxjCqY2kC52KdzLBk3YrmqJ5TDW/193D4g8ooKrOH9ULeyClcLHYjH3c0tFzHy/8RpMgdHYEA== X-Received: by 2002:a17:906:7489:: with SMTP id e9mr3719894ejl.113.1644442146756; Wed, 09 Feb 2022 13:29:06 -0800 (PST) Received: from [192.168.0.97] (xdsl-188-155-168-84.adslplus.ch. [188.155.168.84]) by smtp.gmail.com with ESMTPSA id n25sm5339414eds.89.2022.02.09.13.29.05 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 09 Feb 2022 13:29:06 -0800 (PST) Message-ID: <8e987e4a-3667-b541-4d5f-d174ed054ebf@canonical.com> Date: Wed, 9 Feb 2022 22:29:05 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 Subject: Re: [PATCH 2/5] dt-bindings: arm: ti: Add bindings for AM625 SoC Content-Language: en-US To: Vignesh Raghavendra , Nishanth Menon , Tero Kristo , Rob Herring , Santosh Shilimkar Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20220208131827.1430086-1-vigneshr@ti.com> <20220208131827.1430086-3-vigneshr@ti.com> <9303ca0e-2a83-3c3e-0892-c74a820562a9@canonical.com> From: Krzysztof Kozlowski In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220209_132916_937905_2D554FD8 X-CRM114-Status: GOOD ( 24.00 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 09/02/2022 20:04, Vignesh Raghavendra wrote: > > > On 08/02/22 10:31 pm, Krzysztof Kozlowski wrote: >> On 08/02/2022 14:18, Vignesh Raghavendra wrote: >>> From: Nishanth Menon >>> >>> The AM62 SoC family is the follow on AM335x built on K3 Multicore SoC >>> architecture platform, providing ultra-low-power modes, dual display, >>> multi-sensor edge compute, security and other BOM-saving integration. >>> The AM62 SoC targets broad market to enable applications such as >>> Industrial HMI, PLC/CNC/Robot control, Medical Equipment, Building >>> Automation, Appliances and more. >>> >>> Some highlights of this SoC are: >>> >>> * Quad-Cortex-A53s (running up to 1.4GHz) in a single cluster. >>> Pin-to-pin compatible options for single and quad core are available. >>> * Cortex-M4F for general-purpose or safety usage. >>> * Dual display support, providing 24-bit RBG parallel interface and >>> OLDI/LVDS-4 Lane x2, up to 200MHz pixel clock support for 2K display >>> resolution. >>> * Selectable GPUsupport, up to 8GFLOPS, providing better user experience >>> in 3D graphic display case and Android. >>> * PRU(Programmable Realtime Unit) support for customized programmable >>> interfaces/IOs. >>> * Integrated Giga-bit Ethernet switch supporting up to a total of two >>> external ports (TSN capable). >>> * 9xUARTs, 5xSPI, 6xI2C, 2xUSB2, 3xCAN-FD, 3x eMMC and SD, GPMC for >>> NAND/FPGA connection, OSPI memory controller, 3xMcASP for audio, >>> 1x CSI-RX-4L for Camera, eCAP/eQEP, ePWM, among other peripherals. >>> * Dedicated Centralized System Controller for Security, Power, and >>> Resource Management. >>> * Multiple low power modes support, ex: Deep sleep,Standby, MCU-only, >>> enabling battery powered system design. >>> >>> AM625 is the first device of the family. Add DT bindings for the same. >> >> Don't paste the same huge commit description in several commits. > > Sorry, I think this is the first commit with full description. I will > probably trim 4/5 at bit > >> >>> >>> More details can be found in the Technical Reference Manual: >>> https://www.ti.com/lit/pdf/spruiv7 >>> >>> Signed-off-by: Nishanth Menon >>> Signed-off-by: Vignesh Raghavendra >>> --- >>> Documentation/devicetree/bindings/arm/ti/k3.yaml | 6 ++++++ >>> 1 file changed, 6 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/arm/ti/k3.yaml b/Documentation/devicetree/bindings/arm/ti/k3.yaml >>> index b03c10fa2e7a..64f3db3ea9dd 100644 >>> --- a/Documentation/devicetree/bindings/arm/ti/k3.yaml >>> +++ b/Documentation/devicetree/bindings/arm/ti/k3.yaml >>> @@ -53,6 +53,12 @@ properties: >>> - ti,am642-sk >>> - const: ti,am642 >>> >>> + - description: K3 AM625 SoC >>> + items: >>> + - enum: >>> + - ti,am625-sk >>> + - const: ti,am625 >> >> Why keeping it not alphabetically sorted? What sorting did you choose? >> > > Above list is not sorted alphabetically, I tried to keep similar SoCs > bunched together. AM625 and AM642 are of same family, hence chose to add > the new entry here. Then maybe it should be before AM642? > One alternative is to add it to end of the list (chronologically)? > Or I can add a patch to sort the list alphabetically first and then > introduce new compatible. Please let me know your preference? It's not that important, just wondering. I propose to avoid putting at the end, because this causes conflicts in case of concurrent work. If I had to choose, I would propose to sort SoCs by name. Either way is fine - with trimmed commit msg in patch 4 or here: Acked-by: Krzysztof Kozlowski Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel