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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8BDB3C54EBE for ; Fri, 13 Jan 2023 08:34:17 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S240194AbjAMIeP (ORCPT ); Fri, 13 Jan 2023 03:34:15 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:42404 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S241173AbjAMIdy (ORCPT ); Fri, 13 Jan 2023 03:33:54 -0500 Received: from mail-ej1-x62b.google.com (mail-ej1-x62b.google.com [IPv6:2a00:1450:4864:20::62b]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 10F9B44C78 for ; Fri, 13 Jan 2023 00:33:36 -0800 (PST) Received: by mail-ej1-x62b.google.com with SMTP id cf18so44238268ejb.5 for ; Fri, 13 Jan 2023 00:33:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=69vPfjdndDvs8QEdWRfbzSSvuuOt62gz3qB9+kdUnVA=; b=G0yy8cPfoiavoMyPQc3/K7FUfmxncwLLWeYLWTYO7Eh5k23/uG7jZmmlXAU3i+2m5u T+4X9VzZxFiLZoEf0+m1uc3vT7FIYF3QS+8pUFkgqjMDrHwpuir72UVHlU7pmG+wzh3d qTTgJ2qHORnGEzxoCQtk/C/gVDmiDZF8yfSukEJPxDQs1+JJuKVHhrzuPsvYWvnOdLaa Tw84jFKE34wPD5VnjHCMf/2swGQlwXzn/k1FP6xV2KWTBcsxzHeV9O9lZV5dd+lOMXzH BEBSkqPa8beFeQwqw/NarHzFP/8Mq19qyWTFQZ2roV9WU5I7QDf82+VcOg+qvEg/jm3Z YhPA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=69vPfjdndDvs8QEdWRfbzSSvuuOt62gz3qB9+kdUnVA=; b=Swg6FPPuc7mVJ0+DlmPwu32uGjxGrWTIjaG+Fo4pIr40AMNfuvldYhCaUnZHwxvezM B8ZT+j48z6Muu9CElIImubxgxzsyyCi+pQT/uLnGCeq0Vbsq5T0SvebjEdNGdlLAPOyu KNdSyCJjo5gXOftTEPjcXtD+HyppivQSCoPcGZ3e2wyKog7k/0+5XQLACnOfBXomtcX/ xIOsgbeI7Yu1WmajubdpbPmuxBQI1HT1UYK5wNkdD88cB6vd+Jx53TStxzeYDnLBK55A dJZW/S45m3JkahXg84n+tQuU6dqrZse9wB4uel4BlKMPFPdXW1HmS2iUpu4OumGNnFkd cN9g== X-Gm-Message-State: AFqh2krpHePT1yu2VbBzIl0REVvSrWHDnmKRC6LjTb1m7eaZ/2DpBuDe Zy6A4PqYTOlIAuVT1Z4uhNAUww== X-Google-Smtp-Source: AMrXdXs2ZJAqttVWYRXGxoX3QUG9Uarxkn9ckYoeRGcdElRtpW1qBalWUSAxDqAu65QxD49NUMt6mQ== X-Received: by 2002:a17:907:cbc6:b0:7c0:8371:97aa with SMTP id vk6-20020a170907cbc600b007c0837197aamr71708773ejc.28.1673598814566; Fri, 13 Jan 2023 00:33:34 -0800 (PST) Received: from [192.168.1.109] ([178.197.216.144]) by smtp.gmail.com with ESMTPSA id sb25-20020a1709076d9900b007b2a58e31dasm8336587ejc.145.2023.01.13.00.33.33 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 13 Jan 2023 00:33:34 -0800 (PST) Message-ID: <20651db4-b0e8-ee3e-a752-c1d4ff14b53f@linaro.org> Date: Fri, 13 Jan 2023 09:33:32 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 Subject: Re: [PATCH v2 1/3] dt-bindings: crypto: sun8i-ce: Add compatible for D1 Content-Language: en-US To: Herbert Xu , Samuel Holland Cc: Corentin Labbe , "David S . Miller" , Chen-Yu Tsai , Jernej Skrabec , Krzysztof Kozlowski , Rob Herring , Albert Ou , Conor Dooley , Palmer Dabbelt , Paul Walmsley , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org, linux-sunxi@lists.linux.dev References: <20221231220146.646-1-samuel@sholland.org> <20221231220146.646-2-samuel@sholland.org> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org On 13/01/2023 04:51, Herbert Xu wrote: > On Sat, Dec 31, 2022 at 04:01:43PM -0600, Samuel Holland wrote: >> D1 has a crypto engine similar to the one in other Allwinner SoCs. >> Like H6, it has a separate MBUS clock gate. >> >> It also requires the internal RC oscillator to be enabled for the TRNG >> to return data, presumably because noise from the oscillator is used as >> an entropy source. This is likely the case for earlier variants as well, >> but it really only matters for H616 and newer SoCs, as H6 provides no >> way to disable the internal oscillator. >> >> Signed-off-by: Samuel Holland >> --- >> I noticed that the vendor driver has code to explicitly enable IOSC when >> using the TRNG on A83T (search SS_TRNG_OSC_ADDR), but that is covered by >> a different binding/driver in mainline. >> >> Changes in v2: >> - Add TRNG clock >> >> .../bindings/crypto/allwinner,sun8i-ce.yaml | 33 ++++++++++++++----- >> 1 file changed, 25 insertions(+), 8 deletions(-) > > This doesn't have an ack from Rob Herring. Would you like me > to apply just the crypto patch by itself? But it has my Reviewed-by, which is equivalent. Please take it via crypto with the driver change. Best regards, Krzysztof