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 52BF0C4332F for ; Wed, 23 Nov 2022 13:07:13 +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=4wxesCukdjGPvbP1OxwU6nsciaO54CcQg17vArmB7Js=; b=k5ZQ9JiwRfFFJm eL9TPOglUyoxouqbSAG2h2kgHRcbrhkXLO2lKO/QqNYO+yqY2cP0Mp+4qnmcS13zztRr18eUraAcw K/SaCFAEFQjf8esg0GxDTrPFbUgaj4g9vyqesUSzQK6a5YpV/5wuJfXlUbAIAPOfTzsGlXdcHPXE+ /Ar7FsjMM8o+Ha/Zkhlkp3PVT1CikDW/wVZFChztm2oUhr94Za9y09bTtgajZRJli/Mojhz7heGOI 0RTO0sFyJ1/+znMuLkDTzYPJ6nmaOPes2efGPVMzjCriJeKKbWAvNzWvySHPMGoDR+Omhj4ROdumb ZCBheTJrPcR8/G+jJvzQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oxpRW-00H7Za-5R; Wed, 23 Nov 2022 13:05:34 +0000 Received: from mail-lj1-x22c.google.com ([2a00:1450:4864:20::22c]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oxpRS-00H7X5-7v for linux-arm-kernel@lists.infradead.org; Wed, 23 Nov 2022 13:05:32 +0000 Received: by mail-lj1-x22c.google.com with SMTP id z4so4400366ljq.6 for ; Wed, 23 Nov 2022 05:05:30 -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=M6G1EumI7FZLI8e5i3/vnjgK7flMtfRUkVTQawLF8ZY=; b=vnjhnZcYZOSnL800CVahuhYmBNc7/C4VM+zLXePAG9oZ557PbOQaCNT/qd9DM5rd0X 7jQFCm/KOFIiQSdRAOA8Hs2Sq5AEmg7uK4Xf0KfNwxTAAMkZ12EGDNfo/MOrNSVV7ZgA Pv7aNS7LXiNVZ/+EwFVmMWxHICksLY9CFyrlC1vTxs1AetYHpjQtWvTTnVXAfeKnRXfW kWH7Ns2l/A6+bQcb3k9ZVUvrupxwLJMxwN16xa6LR4aDMsizEcT0Mk8FvXPu8wd5SVKD wvzJUmmqFGQHA3L8Oo0QFzuHel/LnawDOozqRGTBQLtPV5XuzzXcR8DsGGj3gE7xvWIv T0GA== 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=M6G1EumI7FZLI8e5i3/vnjgK7flMtfRUkVTQawLF8ZY=; b=a/hEYixrx+7hVcQx54vE5OGOBtuNdWSpPEkKa3w2KJMCxFD2GU0rNXx5DE4+nZtIuo zNv1CNzwKJrnyW2qYwA+f3jxy/yz7MzvOXfFceZAIlKePrNA8wTyQcoFEWRx4n7Mg8bL Ti3/EC0IsdT7Wpo6PH6KS2WKKQguJnG1j0lj4//hyCnkrUaDgUu/AcgZZyBDOcRu1TgO Ax/qly9yxxaMKfDAJfHciQ/tzXSZsZQzvNdnF7681ruvStmvaRJODf9sUcjme2c0T1Xr /D0IK3App98hFZKCJqhi2Z3ulrP08fId72PM6c8ciC9x7lsibNa76yfbkfCeSIzo/+4a llJA== X-Gm-Message-State: ANoB5plnh2w44lbGW7aNtBDqjGeDQsRZkslEKEr4fsiAVha9X82O5B7K FSR2A5lhHA3iWiFSiREFM6z3fg== X-Google-Smtp-Source: AA0mqf6n/fSR8MwVcDjqufCotAGGQFmimX9GhxJ8X6nI8RJNkuZVynASdsA/m+MunzeilINIsiuhJw== X-Received: by 2002:a05:651c:2123:b0:277:328:affb with SMTP id a35-20020a05651c212300b002770328affbmr4326766ljq.376.1669208728764; Wed, 23 Nov 2022 05:05:28 -0800 (PST) Received: from [192.168.0.20] (088156142067.dynamic-2-waw-k-3-2-0.vectranet.pl. [88.156.142.67]) by smtp.gmail.com with ESMTPSA id y25-20020a2e7d19000000b0026dd4be2290sm2181049ljc.90.2022.11.23.05.05.27 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 23 Nov 2022 05:05:28 -0800 (PST) Message-ID: <06fa6b50-bd69-c118-4c82-89d153fe049e@linaro.org> Date: Wed, 23 Nov 2022 14:05:27 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.5.0 Subject: Re: [PATCH V5 1/4] clk: meson: S4: add support for Amlogic S4 SoC PLL clock driver and bindings Content-Language: en-US To: Yu Tu , 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 , Jerome Brunet , Kevin Hilman , Michael Turquette , Stephen Boyd , Krzysztof Kozlowski , Martin Blumenstingl Cc: kelvin.zhang@amlogic.com References: <20221123021346.18136-1-yu.tu@amlogic.com> <20221123021346.18136-2-yu.tu@amlogic.com> <92b570ea-3ddc-8e91-5a7a-ed601bb7c02c@amlogic.com> From: Krzysztof Kozlowski In-Reply-To: <92b570ea-3ddc-8e91-5a7a-ed601bb7c02c@amlogic.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221123_050530_320268_DFAC7137 X-CRM114-Status: GOOD ( 18.72 ) 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 23/11/2022 12:16, Yu Tu wrote: > Hi Krzysztof, > Thank you for your reply. > > On 2022/11/23 18:08, Krzysztof Kozlowski wrote: >> [ EXTERNAL EMAIL ] >> >> On 23/11/2022 03:13, Yu Tu wrote: >>> Add the S4 PLL clock controller found and bindings in the s4 SoC family. >>> >>> Signed-off-by: Yu Tu >>> --- >>> .../bindings/clock/amlogic,s4-pll-clkc.yaml | 51 + >> >> This is v5 and still bindings are here? Bindings are always separate >> patches. Use subject prefixes matching the subsystem (git log --oneline >> -- ...). >> >> And this was split, wasn't it? What happened here?!? > > Put bindings and clock driver patch together from Jerome. Maybe you can > read this chat history. > https://lore.kernel.or/all/1jy1v6z14n.fsf@starbuckisacylon.baylibre.com/ Link does not explain me anything. It mentions series, which is totally different than mixing it one patch! Anyway you should have warnings from checkpatch. Bindings are always separate patches. (...) >> >>> + >>> +properties: >>> + compatible: >>> + const: amlogic,s4-pll-clkc >>> + >>> + reg: >>> + maxItems: 1 >>> + >>> + clocks: >>> + maxItems: 1 >>> + >>> + clock-names: >>> + items: >>> + - const: xtal >>> + >>> + "#clock-cells": >>> + const: 1 >>> + >>> +required: >>> + - compatible >>> + - reg >>> + - clocks >>> + - clock-names >>> + - "#clock-cells" >>> + >>> +additionalProperties: false >>> + >>> +examples: >>> + - | >>> + clkc_pll: clock-controller@fe008000 { >>> + compatible = "amlogic,s4-pll-clkc"; >>> + reg = <0xfe008000 0x1e8>; >>> + clocks = <&xtal>; >>> + clock-names = "xtal"; >>> + #clock-cells = <1>; >>> + }; >> >> >>> +#endif /* __MESON_S4_PLL_H__ */ >>> diff --git a/include/dt-bindings/clock/amlogic,s4-pll-clkc.h b/include/dt-bindings/clock/amlogic,s4-pll-clkc.h >>> new file mode 100644 >>> index 000000000000..345f87023886 >>> --- /dev/null >>> +++ b/include/dt-bindings/clock/amlogic,s4-pll-clkc.h >> >> This belongs to bindings patch, not driver. >> >>> @@ -0,0 +1,30 @@ >>> +/* SPDX-License-Identifier: (GPL-2.0+ OR MIT) */ >>> +/* >>> + * Copyright (c) 2021 Amlogic, Inc. All rights reserved. >>> + * Author: Yu Tu >>> + */ >>> + >>> +#ifndef _DT_BINDINGS_CLOCK_AMLOGIC_S4_PLL_CLKC_H >>> +#define _DT_BINDINGS_CLOCK_AMLOGIC_S4_PLL_CLKC_H >>> + >>> +/* >>> + * CLKID index values >>> + */ >>> + >>> +#define CLKID_FIXED_PLL 1 >>> +#define CLKID_FCLK_DIV2 3 >> >> Indexes start from 0 and are incremented by 1. Not by 2. >> >> NAK. > > I remember Jerome discussing this with you.You can look at this > submission history. > https://lore.kernel.org/all/c088e01c-0714-82be-8347-6140daf56640@linaro.org/ You pointed to my arguments, so what is this proving? That you ignored feedback? Or was there some other mail? Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel