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 DEAFCC3DA7F for ; Wed, 7 Aug 2024 18:33:57 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:Message-ID:References:In-Reply-To:Subject:Cc:To:From:Date: MIME-Version:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=uPhLM9T/fLlMrEKInffkblL2I9r8PE3dC5RrRLFYkOs=; b=z/Nmq8LIbB0P5A22DFSGcb4epF 2tNyXAzzmWr7JZPgyd2lav4ZHTI+ai1CRn6SE/SzzivOQx4EkVOvWTCTElxTO5yFKavNn1LJNjyjp xtgF9ujMZsQismJgRld3unIaYCfogZ8QgD9FLuMvH4nIWUNysK9MzJeihgikAX7/tvpS+OrfX5uD5 lfqQXSGYdEacp5kqj0TBelNi2tmqZFZb+dVTib5t+xEJ8XQTfoQDys6zJ86tC/1WknKecBAxdWOx+ I3UzovHU9ugWn8QxG5MooWveoFs3vl/KqXTpjiAjxMpbJyL28dpQcSFIQrwHZny4oilkijtpkJqgn 1Yy+sjDA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sblTm-00000005yFY-2LDh; Wed, 07 Aug 2024 18:33:46 +0000 Received: from mail.manjaro.org ([116.203.91.91]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1sblSv-00000005y5x-3GIc; Wed, 07 Aug 2024 18:33:13 +0000 MIME-Version: 1.0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=manjaro.org; s=2021; t=1723055571; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=uPhLM9T/fLlMrEKInffkblL2I9r8PE3dC5RrRLFYkOs=; b=d5BHXrjsF/rxyvxtJfaz4qEvZkd3JMM+ozTmqbuuIdwjZMdba3mn7mk4lJoWgUmk10AWzd NmxxpoVYDrBoVE6QNfr8Qf2N7jUzF9xX4YbSWd8iEfHmUglvQxNnEPbxTbJdzoHdEVnF35 FUK88vDOzbPfw0MMgDgvd34/4WLxq/blweX53gPIzZqEs51hYPD67tFMtDrzFS0UdtdmrC 5HvLm+Ygswi3cYCjzKAN32JB0RiQmFhE5mOric0kzwWUhA399cN+SdDzgIKOtrIPMfChGU +3hssPajcaN3xqvldJGqLaN9Df6I2NrsVhEr14iC3dt0lU9wa3YfrAur3QkvXg== Date: Wed, 07 Aug 2024 20:32:51 +0200 From: Dragan Simic To: Florian Klink Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , Sebastian Reichel , Kever Yang , Muhammed Efe Cetin , FUKAUMI Naoki , =?UTF-8?Q?Tam=C3=A1s_Sz=C5=B1cs?= , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] arm64: dts: rockchip: add rfkill node for M.2 E wifi on orangepi-5-plus In-Reply-To: References: <20240807162001.1737829-1-flokli@flokli.de> <20240807170030.1747381-1-flokli@flokli.de> Message-ID: <1b2e1b1b321f84f5cbff2ae18c3eba2e@manjaro.org> X-Sender: dsimic@manjaro.org Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Authentication-Results: ORIGINATING; auth=pass smtp.auth=dsimic@manjaro.org smtp.mailfrom=dsimic@manjaro.org X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240807_113254_135927_EAB5B6BC X-CRM114-Status: GOOD ( 18.58 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 2024-08-07 20:14, Florian Klink wrote: > On Wed, Aug 07, 2024 at 07:24:27PM GMT, Dragan Simic wrote: >> On 2024-08-07 19:00, Florian Klink wrote: >>> This follows the same logic as 82d40b141a4c ("arm64: dts: rockchip: >>> add >>> rfkill node for M.2 Key E WiFi on rock-5b"). >>> >>> On the orangepi-5-plus, there's also a GPIO pin connecting the WiFi >>> enable signal inside the M.2 Key E slot. >>> >>> The exact GPIO PIN can be validated in the Armbian rk-5.10-rkr4 >>> kernel >>> rk3588-orangepi-5-plus.dtsi file [1], which contains a `wifi_disable` >>> node referencing RK_PC4 on &gpio0. >>> >>> Signed-off-by: Florian Klink >>> Tested-by: Florian Klink >> >> I forgot to mention that providing a Tested-by tag is redundant when >> there's already a Signed-off-by tag, because the latter already >> implies >> the former. > > This came after I sent the v3. Generally I wish people would test > things > - though too often it's not. I explicitly tested this to work (with a > wifi module added to that slot being unblock-able afterwards), and > wanted to point that out, thus adding the Tested-by. In general, some time should be allowed between sending consecutive versions of the same patch, so people can provide their feedback. When it comes to testing the submitted patches, please note that signing off a patch implies that the signer has already, to the best of their abilities, made sure that the patch works as described and expected. With all that in mind, please allow me to repeat that a Tested-by tag should not be provided from the same person that the Signed-off-by tag is already coming from. It's simply redundant. > DCO 1.1 doesn't say anything about Tested-by, it's mostly legalese > about > being allowed to send out the patch, and understanding the consequences > regarding licensing. It doesn't require the person adding their > Signed-Off-By to have tested it. Well, not all rules are to be followed blindly, and some documentation perhaps needs updating or expanding to be more precise. On top of that, having something absent from the documentation doesn't necessarily mean that some additional rules don't apply. It's many times simply about applying common sense.