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 F1827C4321E for ; Fri, 2 Dec 2022 13:36:37 +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-Type: Content-Transfer-Encoding:Reply-To:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:References:Cc:To:Subject: From:MIME-Version:Date:Message-ID:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=A1UmMyzBm8MGVwZK8YtfHZfGWx7tYX6BKOvqEb8ulwo=; b=rrHXsWVbhLSjTxX99wQWWDk9N/ uTA7NoT4kn3qqyyg378FUXD6yioOMlEPTVm+wwcnZqTzVg4NhR7k8zHMXoMIIIB9d1R0iukYIVSNx bxm2ZfIoEBETgsST7BdaXVAUg3s0BeyD26fEN3GELIOhvhwyrWzEgslZHb+RyHNveqFCLd8ht58e1 55QqMVo+WkgMlIQAnoFY0FTHt09Iv1XR6WcALmfN86FkSv27n5h9Y6rjMewo1T1tkGEFBNWzRKfwp m9B7+zvPM/lv9A/7bDkvFABk1DQy+l1Bc8yOuB1r/QmFlmInIbZ7X0t49yc5CEf60dt3IUDuz3jlM f4vk/62g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p16DJ-00GckL-Ru; Fri, 02 Dec 2022 13:36:25 +0000 Received: from mail-wr1-x42b.google.com ([2a00:1450:4864:20::42b]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p16DA-00GcZx-Gy for linux-amlogic@lists.infradead.org; Fri, 02 Dec 2022 13:36:18 +0000 Received: by mail-wr1-x42b.google.com with SMTP id h7so1799300wrs.6 for ; Fri, 02 Dec 2022 05:36:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:from:to:cc:subject:date:message-id:reply-to; bh=NSQmW5YxjbuneNhJ6KbeJhq3/ChGxKuyY2+shwyWAFY=; b=hThn92958p8GAYwnggbWKyJot52EESD2qp4nNgtQPuVsGuUD08W0X1M0L5K5QNm4W7 YFpdSZJ22DlMnB4thycsIXgPH2hUH6T0PjSJSaZzDr7uzzusq6TTquOy7cqqHnwbtPc0 OJcf/4rRR3XjDZSaa1oaDZYUaGPiykA2x2ZMhx1PkEYPwoEExQmu7aVl9R31GWM9/LGY T77SepAnUB3XPNUJU7eIUbxf5feKUEH7NTivKdqAxlUNir1AZxBvR0Lsa70nSTmLORaQ 1qPfB++LV82kCLHqJhpFG4K/Q0UEavY513XrewGEaBcrTJDdJuikWhFzrLuC7Ac0zjWq JRTQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:organization:references:cc:to :content-language:subject:reply-to:from:user-agent:mime-version:date :message-id:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=NSQmW5YxjbuneNhJ6KbeJhq3/ChGxKuyY2+shwyWAFY=; b=4qeeGPsKjN+Rg/n06XMNH18yZltXQ9F3pZ+AqqHgH9sWunQqm6SaGdtll9qx/i51yK btv6cfZn1cR77H3eARLIcJWN89p4BTpRYzo0XF4bixh8AUmxfIW0gIW+DEWXf7FObzqf qwmH/fIyzy1BfcINaufKNeTHFQx5t7z+ZMvUwc/S8egDZg2xKc128AdNfsl4r79MIJAf bIQE/ZM8jDxrJ56dWBtPbRLVsCaCC/QtXwZ8tPE7hiXqD8cVl1YH7vS8QM5eCM73gaV6 DvrpYPcXCjK5KxS7NyJO9iSSA+ojwSmt0wDbNQS9CjgGDrqKPeuSw7ayfTk1F37b3jp7 z56g== X-Gm-Message-State: ANoB5pkmjtUtiE/uyZ9bcm28XKq5++79aIK3g1l465RG+6lv9P+QsBEY hzDbLEr2MI3tqeiM1k2tf4IraQ== X-Google-Smtp-Source: AA0mqf4lMbaEjBgYIV2OLBHivBa/8hU4TeVP1kZQZvX6vp9GnI6UFiFG6l+7NheRXkwYaUBfmjai9Q== X-Received: by 2002:a05:6000:1c1e:b0:242:1aec:7c5b with SMTP id ba30-20020a0560001c1e00b002421aec7c5bmr14270946wrb.386.1669988172939; Fri, 02 Dec 2022 05:36:12 -0800 (PST) Received: from ?IPV6:2a01:e0a:982:cbb0:c6e:eb0:b551:55ee? ([2a01:e0a:982:cbb0:c6e:eb0:b551:55ee]) by smtp.gmail.com with ESMTPSA id i14-20020a5d55ce000000b002422816aa25sm8571784wrw.108.2022.12.02.05.36.11 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 02 Dec 2022 05:36:12 -0800 (PST) Message-ID: <3b02ab6f-56c7-a877-1b2f-01fc1fb8f552@linaro.org> Date: Fri, 2 Dec 2022 14:36:11 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2 From: neil.armstrong@linaro.org Subject: Re: [PATCH v8 01/11] dt-bindings: clock: meson: add A1 PLL clock controller bindings Content-Language: en-US To: Dmitry Rokosov , Krzysztof Kozlowski Cc: Jerome Brunet , Rob Herring , devicetree@vger.kernel.org, sboyd@kernel.org, khilman@baylibre.com, kernel@sberdevices.ru, robh+dt@kernel.org, martin.blumenstingl@googlemail.com, linux-arm-kernel@lists.infradead.org, jian.hu@amlogic.com, linux-kernel@vger.kernel.org, krzysztof.kozlowski+dt@linaro.org, linux-amlogic@lists.infradead.org, rockosov@gmail.com, mturquette@baylibre.com, linux-clk@vger.kernel.org References: <20221201225703.6507-1-ddrokosov@sberdevices.ru> <20221201225703.6507-2-ddrokosov@sberdevices.ru> <166995398251.2089685.16059995540663317860.robh@kernel.org> <20221202095124.4ecpfqhtzf34lwbf@CAB-WSD-L081021> <1jfsdy83h3.fsf@starbuckisacylon.baylibre.com> <20221202112826.zkpjyb3f4e2755sh@CAB-WSD-L081021> Organization: Linaro Developer Services In-Reply-To: <20221202112826.zkpjyb3f4e2755sh@CAB-WSD-L081021> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221202_053616_607145_D179D5BF X-CRM114-Status: GOOD ( 14.06 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: neil.armstrong@linaro.org Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org On 02/12/2022 12:28, Dmitry Rokosov wrote: >>>>>> On Fri, 02 Dec 2022 01:56:53 +0300, Dmitry Rokosov wrote: >>>>>>> From: Jian Hu >>>>>>> >>>>>>> Add the documentation to support Amlogic A1 PLL clock driver, >>>>>>> and add A1 PLL clock controller bindings. >>>>>>> >>>>>>> Signed-off-by: Jian Hu >>>>>>> Signed-off-by: Dmitry Rokosov >>>>>>> --- >>>>>>> .../bindings/clock/amlogic,a1-pll-clkc.yaml | 52 +++++++++++++++++++ >>>>>>> include/dt-bindings/clock/a1-pll-clkc.h | 16 ++++++ >>>>>>> 2 files changed, 68 insertions(+) >>>>>>> create mode 100644 Documentation/devicetree/bindings/clock/amlogic,a1-pll-clkc.yaml >>>>>>> create mode 100644 include/dt-bindings/clock/a1-pll-clkc.h >>>>>>> >>>>>> >>>>>> My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check' >>>>>> on your patch (DT_CHECKER_FLAGS is new in v5.13): >>>>>> >>>>>> yamllint warnings/errors: >>>>>> ./Documentation/devicetree/bindings/clock/amlogic,a1-pll-clkc.yaml:26:6: [warning] wrong indentation: expected 6 but found 5 (indentation) >>>>>> >>>>>> dtschema/dtc warnings/errors: >>>>>> ./Documentation/devicetree/bindings/clock/amlogic,a1-pll-clkc.yaml: $id: relative path/filename doesn't match actual path or filename > > ... > >>>>> >>>>> Please find all fixes of above warnings and errors in the my patch >>>>> located at the link: >>>>> >>>>> https://lore.kernel.org/linux-amlogic/20221201225703.6507-9-ddrokosov@sberdevices.ru/ >>>> >>>> Why? This patch here is broken and it should be fixed. Don't apply >>>> broken patches... >>> >>> Dmitry is ressurecting a series that is several years old and not his to >>> begin with. >>> >>> He was unsure about take the code of somebody else. >>> To be fair, he even asked for advice on IRC about to proceed. >>> >>> Dmitry, as you may have guessed, for next revision, please fix Jian Hu >>> original patches in place, not with fixup patches. >>> >>> If your fixes are minor (not complete rewrite), please keep the original >>> author and add your SoB >> >> We never take intentionally wrong patches, e.g. code which does not even >> compile, and immediately fix it in next patch. >> >> Can you imagine adding such drivers? Which are broken in the commit they >> are added? >> >> So the patchset is old or abandoned, take ownership, add co-developed >> etc. Just don't add known broken code. > > Okay, I've got your point. It's reasonable. > I will fix Jian Hu's patches (squash with mine) and mark all of them > with co-developed and SoB Jian Hu tags. Thank you for explanation. It was clearly explained in the cover-letter, nobody expected these patches to be applied as-is. Using RFC would have been the best solution, but this situation is rather specific and he made the right decisions to trigger this current discussion toward an acceptable solution for everybody. Neil > _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic