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 6F4FDC4332F for ; Wed, 21 Dec 2022 11:44:03 +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=DKpZ1cFdmNfK8CHE46Um5hOhb08QHrXAbtLAT2kV4mQ=; b=IvZb8ovWuQ751M pvrajdzOhQs/OFgBwwcHamecIb2c2NFOLEUNRaKzoNZyI4e7sI5h4Wq8DkF9IeDTDP1RYrGIP6M3v o7HfBlIvU2ETOEoVE9x96m2ikj2kq9kwmzfJ9lZ/E96Q5e5jELdSSBcsf73COKoo8vPOWm2AoZ+Sf zAZnSWzpbYcN1Uyg4WJZisoxH9waXz48HSKJT0D+kLVPrJo8IqChzO2aQtaInJ6wQ0o9vhqtdnmnw DOogPH9ozewaLY8Y4OZWKSFgCgU0+OjVEns58Y25cMJQ6yRt6DkxnB89zweAm6S+9xGHlp7qscs0P szvATuHrHpSLSqPd40RA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1p7xUs-00EK4n-EO; Wed, 21 Dec 2022 11:42:54 +0000 Received: from mail-lf1-x130.google.com ([2a00:1450:4864:20::130]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1p7xTG-00EIlC-Nk for linux-arm-kernel@lists.infradead.org; Wed, 21 Dec 2022 11:41:16 +0000 Received: by mail-lf1-x130.google.com with SMTP id o6so18207236lfi.5 for ; Wed, 21 Dec 2022 03:41:13 -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=TFr+KkgF0f2zsju3on50AY6Jqv8wMu+vCgWh7SKVHvU=; b=a2EUh3XRGsS46OpUUPOejw8y2YIi+wRZ3E9NbbtKa16cGTd+sMjTKjBtYMbjgClYen +fLzFb+nskcXKLmkZaNaO1ieTjC+oJ3wy7O4NDF+1zmHsj0UFJBGRaxsxxjHm2Yrr7J/ HD/Y25cJ58bdyREnEe1nqQJQsBPe03KSi5r2ovgkObqPEBGGvQKY5Oj5nUOHlPp9e6Ly reFgdBt6JNfcY4LfLAoObxVIG0a8qYY4XQkCM0vIKeD+uAsmm47CgeFDTUI/Drrp4+Jw cww7q/I/Ds/Rw7sa+vjwQS2GQzZEbwHWGG22HCGeJknxoJYOfvuhzNza/DEpSqvag4BG xkLw== 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=TFr+KkgF0f2zsju3on50AY6Jqv8wMu+vCgWh7SKVHvU=; b=pCEVCNiSlEnDpJSawYlk4EXbByOCyz9yGwtSO/uCEM2zDbbRYoCLkUs+azL3LhILkB plf3d3yI+ZDQw2493zk6cBfcdtnRHjELOAJNlEPJWxqfNF6fQd6pf4o4bRyXYt5YIV8v vJ3ABMeBLtQiZ2TyuDMFeIxHJ989oJyfw9NIOlNEZPjbwG21fgaNKYrmj3FhEvjiOJoj vC5jUewBDB5NrdOfvRLfWl/AdEXm1bSiIEUKH5wXA4abVL/gfmF1C3ittK3wjuNCmODb mU8xFY9fH1rTpoGs0ThDDSoqrkPNv/bEE2anreubOI5JFn7vFU4ApA74ZUYJlvxm1UOL jDhA== X-Gm-Message-State: AFqh2krHuOKuOtb1tR+yov97oQEU6aAFpQa6cLa8T3ZWbiLI0ssw0Y0d Skj1xxWz4XdjcmkgpRcPOGkA5ebL8f6TCIup X-Google-Smtp-Source: AMrXdXsyPsMudczLvkigBGwO5mqibV7RZCAK5DqU/SA/VLHDDLkZTKPd3hjMSTk2tDvZ0oatgeXkNw== X-Received: by 2002:a05:6512:3e0c:b0:4b5:a5c7:3286 with SMTP id i12-20020a0565123e0c00b004b5a5c73286mr681061lfv.9.1671622872355; Wed, 21 Dec 2022 03:41:12 -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 f14-20020a056512360e00b004b551cdfe13sm1800382lfs.279.2022.12.21.03.41.11 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Wed, 21 Dec 2022 03:41:11 -0800 (PST) Message-ID: <3f7dc970-4cc0-7d8e-b33f-83c9dd368b11@linaro.org> Date: Wed, 21 Dec 2022 12:41:10 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.0 Subject: Re: [PATCH v4 2/2] dt-bindings: arm: rockchip: Add EmbedFire LubanCat 1 Content-Language: en-US To: ty Cc: Heiko Stuebner , Rob Herring , Peter Geis , Krzysztof Kozlowski , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, DHDAXCW References: <5a7907e3-f79e-5de9-ed1b-ec24ea52c437@linaro.org> From: Krzysztof Kozlowski In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221221_034115_038963_14C4F84A X-CRM114-Status: GOOD ( 25.52 ) 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 21/12/2022 11:46, ty wrote: > Dear Kozlowski, > > Sorry that we didn't fully get the idea in your previous comments. > >> >> On 21/12/2022 06:52, Yuteng Zhong wrote: >>> from: DHDAXCW >>> >>> LubanCat 1 is a Rockchip RK3566 SBC based >>> is developed by EmbedFire Electronics Co., Ltd. >>> Mini Linux Card Type Cheap Computer Development Board >> >> This is a friendly reminder during the review process. >> >> It seems my previous comments were not fully addressed. Maybe my >> feedback got lost between the quotes, maybe you just forgot to apply it. >> Please go back to the previous discussion and either implement all >> requested changes or keep discussing them. >> >> Thank you. > > Your pervious comment were: > >> Please wrap commit message according to Linux coding style / submission >> process: >> https://elixir.bootlin.com/linux/v5.18-rc4/source/Documentation/process/submitting-patches.rst#L586 > > So I think there must be something wrong in the message but I didn't > notice. This message has a `from` line, and the body was wrapped at 75 > columns. Please help me make it better. You do not wrap at 75, but at some other place. Just put it to editor and check where is wrapping (count characters). > > > >>> >>> Signed-off-by: Yuteng Zhong >>> Signed-off-by: DHDAXCW >>> --- >>> Documentation/devicetree/bindings/arm/rockchip.yaml | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml >>> index 88ff4422a8c1..84d39a3a8843 100644 >>> --- a/Documentation/devicetree/bindings/arm/rockchip.yaml >>> +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml >>> @@ -781,6 +781,11 @@ properties: >>> - const: rockchip,rk3568-bpi-r2pro >>> - const: rockchip,rk3568 >>> >>> + - description: EmbedFire LubanCat 1 >>> + items: >>> + - const: embedfire,lubancat-1 >>> + - const: rockchip,rk3566 >> >> Here as well. > > This change was addressing the previous comment in V2: > >> This is separate patch. Does not look like properly ordered. Don't add >> stuff at the end of files/lists. > > The previous PATCH v2 merged changes in 3 diffent files: > > 1. arch/arm64/boot/dts/rockchip/Makefile > 2. arch/arm64/boot/dts/rockchip/rk3566-lubancat-1.dts > 3. Documentation/devicetree/bindings/arm/rockchip.yaml > > What we do in this PATCH v4 was separated the 3 into a separated patch. So how did you solve the comment "Does not look like properly ordered. Don't add stuff at the end of files/lists."? What did you change to solve it? Best regards, Krzysztof _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel