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 93217C433EF for ; Fri, 18 Mar 2022 10:21:16 +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:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=8RqUiuXKPIT3dNtdMV7BWQxwOigh8EpTKpTL01ShYdE=; b=thcnaDgEz2unfL kiUHNA1scQPJ79Pq5M/wHHN2sFmhHbszSockQdV4f7NBe9GJy7ZEPuL/vV/3RbONAQMjWP1qwdz7M u/jJRtFCuVrqeAozyFWKhSD4rsICdqQ4fmRkY89/IQNYTifWCFz7Z/kymfLsy+M9Hjlx1Z1Z0ZbGD ic29STgjB8vrMTL0/k+X4rTaoS5rwstu47skkxTpP/ooMVjESTswuIMIuvgoy/kHMKRSWIyYhxScS RDrxjG5jMnVpFcjJCdfKgP8D4K+xVayIC2rS19+Te2a/gxDzIP+cWRdH19uMC6DiZd6oQwHus0W5A 5+835vWtZpQpfgdZsYKQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nV9iM-0017Xa-W9; Fri, 18 Mar 2022 10:20:11 +0000 Received: from mail-wm1-x329.google.com ([2a00:1450:4864:20::329]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nV9iH-0017V3-8o for linux-arm-kernel@lists.infradead.org; Fri, 18 Mar 2022 10:20:07 +0000 Received: by mail-wm1-x329.google.com with SMTP id bg31-20020a05600c3c9f00b00381590dbb33so4555815wmb.3 for ; Fri, 18 Mar 2022 03:20:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to; bh=YYBm1RX0czLMaWTTQInKeCgdVYhgkLcZ7jFkas/Rfhs=; b=AUl19gddWU+TGkn91rwswKtkRuunsoS1xg6IvUppECEb8oSzjSfXJv2d87cqXYs3Yy 95nkaF7kHo7GBGVTNaBNkuO6Rp5JM8YHyT8tVX8NcbKasWURoUqgjvOhVr+6gsjwxf67 6VqySP3QaYydQWitA3XNyI6fTecEOPfuXE2b/2PqMFm9CN17Tvi8nhCUC2lm71oY1H+o 2GWqd42sqZxKapISXzA8vBpYH53HlV8UMGJz1fKwf0WuqukOHYuMyfohAKbTkm1O00uV wU2oONr93WQy0UguF3b7ebHHu3pc+OUgVdXyQBp284tjYnM5VG8acVPte/GGuH+vp6P+ tK6A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to; bh=YYBm1RX0czLMaWTTQInKeCgdVYhgkLcZ7jFkas/Rfhs=; b=sgQT0zMqhn2ixnfzrvB4p4DkuxG7Iw+EPdMhyjgtrID+6/hu5fZjMpPIkCx9LX6VoO ycff/Y/tOM5UVsRCgAefYGMnmXXMMTxMRFMmEFufpkn0/sZVq6RvsQk/jOc7I0TZa4tD wU8BMWb5y1yed1+8PVg5IOH2yyLx90P4agACuD7VzsJaVueED3W4nkdHLPhRQPfDGA/c knUl19YGkWrc4dGcSIjwD84zpx6XLzPIQNM55B/zIdkRiezi0eHNfyrefrLyNrNFV6Yc k+49q9jpmjXDG7okdgaQEbuDk3rnJP2kqZKJrBRQhFhSfUBr5/OSHVxtrXasd7qZvgAe 9kOg== X-Gm-Message-State: AOAM530nhZfDiPstnvoj9HMGKSen1DxOIzWL5D7WT1XsuBLGbMjgIPkc GFJW4e4GbrUyNCyQ/Chi+xwprA== X-Google-Smtp-Source: ABdhPJykd7CP6qm5/RS39Zs+vu5rb4PuIDjEBZiEVPnAb1oCr3QXbjSKOIF+Kzx93wjR66zRLBNlTA== X-Received: by 2002:a7b:c7c3:0:b0:389:cbf1:fadf with SMTP id z3-20020a7bc7c3000000b00389cbf1fadfmr15021009wmk.147.1647598802494; Fri, 18 Mar 2022 03:20:02 -0700 (PDT) Received: from Red ([2a01:cb1d:3d5:a100:264b:feff:fe03:2806]) by smtp.googlemail.com with ESMTPSA id f10-20020adffcca000000b00203e6a34d3esm5889436wrs.99.2022.03.18.03.20.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 18 Mar 2022 03:20:01 -0700 (PDT) Date: Fri, 18 Mar 2022 11:20:00 +0100 From: LABBE Corentin To: Johan Jonker Cc: heiko@sntech.de, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, robh+dt@kernel.org, krzysztof.kozlowski@canonical.com, herbert@gondor.apana.org.au, davem@davemloft.net Subject: Re: [PATCH v3] dt-bindings: crypto: convert rockchip-crypto to yaml Message-ID: References: <20220211115925.3382735-1-clabbe@baylibre.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220318_032005_407033_3F061E4D X-CRM114-Status: GOOD ( 25.24 ) 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="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Le Tue, Feb 15, 2022 at 03:07:56PM +0100, Johan Jonker a =E9crit : > Hi Heiko, > = > On 2/11/22 12:59, Corentin Labbe wrote: > > Convert rockchip-crypto to yaml > > = > > Signed-off-by: Corentin Labbe > > --- > > Changes since v1: > > - fixed example > > - renamed to a new name > > - fixed some maxItems > > = > > Change since v2: > > - Fixed maintainers section > > = > > .../crypto/rockchip,rk3288-crypto.yaml | 66 +++++++++++++++++++ > > .../bindings/crypto/rockchip-crypto.txt | 28 -------- > > 2 files changed, 66 insertions(+), 28 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/crypto/rockchip,r= k3288-crypto.yaml > = > rockchip,crypto.yaml > = > > delete mode 100644 Documentation/devicetree/bindings/crypto/rockchip-c= rypto.txt > > = > > diff --git a/Documentation/devicetree/bindings/crypto/rockchip,rk3288-c= rypto.yaml b/Documentation/devicetree/bindings/crypto/rockchip,rk3288-crypt= o.yaml > > new file mode 100644 > > index 000000000000..2e1e9fa711c4 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/crypto/rockchip,rk3288-crypto.y= aml > > @@ -0,0 +1,66 @@ > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/crypto/rockchip,rk3288-crypto.yaml# > = > rockchip,crypto.yaml > = > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Rockchip Electronics And Security Accelerator > > + > > +maintainers: > > + - Heiko Stuebner > > + > > +properties: > > + compatible: > = > oneOf: > - const: rockchip,rk3288-crypto > - items: > - enum: > - rockchip,rk3228-crypto > - rockchip,rk3328-crypto > - rockchip,rk3368-crypto > - rockchip,rk3399-crypto > - const: rockchip,rk3288-crypto > = > rk3288 was the first in line that had support, so we use that as fall > back string. > = > > + const: rockchip,rk3288-crypto > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + clocks: > > + items: > > + - description: clock data > > + - description: clock data > > + - description: clock crypto accelerator > = > > + - description: clock dma > = > remove ??? > = > > + > > + clock-names: > > + items: > > + - const: aclk > > + - const: hclk > > + - const: sclk > = > > + - const: apb_pclk > = > remove ??? > = > Similar to the rk3568 pclk_xpcs discussion ACLK_DMAC1 belongs to the > dmac_bus_s node and should have been enabled by the DMA driver I think. > Could you advise if this is correct or should we remove parsing/enabling > ACLK_DMAC1 in rk3288_crypto.c in order to it easier > porting/adding/syncing nodes for other SoC types? > = > Johan > = Hello I came back on this as I got access to a rk3288-miqi, and crypto does not w= ork at all. This is due to ACLK_DMAC1 not being enabled. While not touching it work on rk3399 and rk3328, rk3288 seems to need it. Probably the DMA controller goes to sleep under PM. Any idea on how to create a dependency so dma controller does not sleep ? _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel