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 A1CFFC04A68 for ; Thu, 28 Jul 2022 11:51:46 +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:MIME-Version:Message-ID:In-reply-to: Date:Subject:To:From:References:Reply-To:Cc:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=77v3gJ0fkuqpGt4n/5b8gDsO/uKWn26Rq7HUBv5qIWQ=; b=J4IiD8etq300yK qYhcbEPVVsZS/65+2p8pqNABD3+LXUOD7sVXJ+/xJG9qGq5wxnos7dDy0fpApy/dDsEnChQnoBfQS HgBwiBueBJFV1w7fpR6EM8OmPxTIHdP7MDkbsQlIenIobPzT4Km2kg/A4DQGGX6Gr/KciHJPW01vE dwG2AE06rTwBpC1QqvLZUBrnnCN4l335iF/e6bufFzKjCKzljybzyf9q7H7FDsJ+SAWg88Ub4v6I1 vXH1Mgo/4BfPzbu/naFbcmUkLUUdKimO5oZTFvFOf1YMFOE1yJBfryJHHxQIxWZrFn9A8NrIsIq2j gsTPsaZqgfpFSk+Og2Ww==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oH221-008uHr-RZ; Thu, 28 Jul 2022 11:50:21 +0000 Received: from mail-wm1-x330.google.com ([2a00:1450:4864:20::330]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oH21x-008uFB-5M for linux-arm-kernel@lists.infradead.org; Thu, 28 Jul 2022 11:50:19 +0000 Received: by mail-wm1-x330.google.com with SMTP id x23-20020a05600c179700b003a30e3e7989so934047wmo.0 for ; Thu, 28 Jul 2022 04:50:16 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20210112.gappssmtp.com; s=20210112; h=references:user-agent:from:to:subject:date:in-reply-to:message-id :mime-version; bh=FxwqpjhD9/gLaQrnTT9TOXORIlpv5bjNu3/8g5til8A=; b=nTEylWFWV5aRYXQnjm2kiRjVJlk7CEQjzrUBHIDJnxn+cY0Vj8Q7ezPN7FgIiJK/+v ine2IGE+ebrioKCcdP9pPI+ICu9WtJvFA+PUXlNmr8SoGEJwVEgcyeHUlpHnXNjI/HmY pLvlZNwoi1m0h5/WHucjBDABR1SPLS8arubysiloaSV92rt9yiaiMG3l9yCH5l/aQ3/9 s+IwJzB984E9vKSpdLwuxLICOc1sgE4UtWWOy3UrfagJPHpMHyDvMoK4x3Cyq1tORPeE gbi0d45zdtr7iycIpE5AhCbM8wEKpE9OrycAhoQLkNBUjTUoAZMsEcibmKdyhO0fjDjX 21yw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:references:user-agent:from:to:subject:date :in-reply-to:message-id:mime-version; bh=FxwqpjhD9/gLaQrnTT9TOXORIlpv5bjNu3/8g5til8A=; b=T1fWSaoKWw0+gr35+zZDgYeuNsL8XlwiRJ+qrSrq5A8Es3BUWMzN+2oGGk4M0qGUep C0R9XWIvkAJ4JoPwVR/ETpTON+kPVP3hwPbZVRMgJFxpT+nRTKirVB4gWXQqeBYg7rMG Wy1qGRGmSvk/T63QSeeDuFKGBZkC6FGeg2p9hNZIIiTqBrAVtr1Isn8ViwZmZeiGXBZe bjFVDkDa/2f/QZI9E+yRJJYp31IF9XUfmx4IFtbOvyTjXn8P/HuxJpRnPibMHSTT5Ndg zag3h/OmI+GLNbFSZ3XwQbEoI0YtO27zpA0gXqVxEc8I88X9gmd2r4yT80Cgy/VV47Hq iDZA== X-Gm-Message-State: AJIora8BajAQrm5bCF7sltIlUhSj1ZDlc8PEsL5ffshRJOfGgYVbiwIo j+Fm8/z5swnWmGV2ytPii/Il1Q== X-Google-Smtp-Source: AGRyM1v0j0TKqWnZsAUWLQ3B4oY6N7U+8YetS6WSCx/Q+1LpG7g7jTIsnd/U+8JWEBXprZeoB78iww== X-Received: by 2002:a7b:c453:0:b0:3a3:1c65:ff97 with SMTP id l19-20020a7bc453000000b003a31c65ff97mr6046013wmi.180.1659009015199; Thu, 28 Jul 2022 04:50:15 -0700 (PDT) Received: from localhost (laubervilliers-658-1-213-31.w90-63.abo.wanadoo.fr. [90.63.244.31]) by smtp.gmail.com with ESMTPSA id o5-20020a05600c510500b003a2d6c623f3sm5639135wms.19.2022.07.28.04.50.13 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Jul 2022 04:50:14 -0700 (PDT) References: <20220728054202.6981-1-yu.tu@amlogic.com> <20220728054202.6981-2-yu.tu@amlogic.com> <82e3fd36-df96-a555-4cea-47fabd26502b@linaro.org> <74cd833a-4773-eeb0-80aa-75ea1cdc093e@amlogic.com> <39395257-703b-a5e9-17c3-80f79f67fdc7@linaro.org> User-agent: mu4e 1.8.6; emacs 27.1 From: Jerome Brunet To: Yu Tu , Krzysztof Kozlowski , linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-amlogic@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Rob Herring , Neil Armstrong , Kevin Hilman , Michael Turquette , Stephen Boyd , Krzysztof Kozlowski , Martin Blumenstingl Subject: Re: [PATCH V2 1/3] dt-bindings: clk: meson: add S4 SoC clock controller bindings Date: Thu, 28 Jul 2022 13:48:14 +0200 In-reply-to: Message-ID: <1jbkt9focq.fsf@starbuckisacylon.baylibre.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220728_045017_428472_E6FA8F62 X-CRM114-Status: GOOD ( 11.34 ) 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 Thu 28 Jul 2022 at 18:19, Yu Tu wrote: > On 2022/7/28 18:09, Krzysztof Kozlowski wrote: >> [ EXTERNAL EMAIL ] >> On 28/07/2022 12:05, Yu Tu wrote: >>> Hi Krzysztof, >>> Thanks for your reply. >>> >>> On 2022/7/28 16:41, Krzysztof Kozlowski wrote: >>>> [ EXTERNAL EMAIL ] >>>> >>>> On 28/07/2022 07:42, Yu Tu wrote: >>>>> Add new clock controller compatible and dt-bindings header for the >>>>> Everything-Else domain of the S4 SoC. >>>>> >>>>> Signed-off-by: Yu Tu >>>> >>>> >>>> >>>>> diff --git a/MAINTAINERS b/MAINTAINERS >>>>> index c1abc53f9e91..f872d0c0c253 100644 >>>>> --- a/MAINTAINERS >>>>> +++ b/MAINTAINERS >>>>> @@ -1775,6 +1775,7 @@ F: Documentation/devicetree/bindings/clock/amlogic* >>>>> F: drivers/clk/meson/ >>>>> F: include/dt-bindings/clock/gxbb* >>>>> F: include/dt-bindings/clock/meson* >>>>> +F: include/dt-bindings/clock/s4-clkc.h >>>>> ARM/Amlogic Meson SoC Crypto Drivers >>>>> M: Corentin Labbe >>>>> diff --git a/include/dt-bindings/clock/s4-clkc.h b/include/dt-bindings/clock/s4-clkc.h >>>>> new file mode 100644 >>>>> index 000000000000..b686c8877419 >>>>> --- /dev/null >>>>> +++ b/include/dt-bindings/clock/s4-clkc.h >>>> >>>> Filename with vendor prefix, so: >>>> amlogic,s4-clkc.h >>> It's fine with me. It's mainly Jerome's opinion. >> To clarify: I understand such naming might bring inconsistency, but we >> want to bring some order in the bindings directories. They keep growing >> and at some point the model names might start conflicting. > If Jerome agrees, I will change it according to your opinion and make > another edition. I'm aligned with Krzysztof on this. Please add the vendor prefix. It was mistake to omit the vendor prefix. Unfortunately, I don't think we can fix the old bindings now. > >> >> Best regards, >> Krzysztof >> . _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel