From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=194.117.254.33 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1794339720; b=FjM8X8PwOJJeg0OUC//QsL4d7PTV0hl51u4G2amyMEdnJEVoHakZndIEGVr5U9Glq+9g ugHNHh5pFB+3MMTCA/DHWv6HBhYzoopACb2Vqq/vPMxqb0NYstvqKLdV82CncO1cfML95 BH/7DmphtYGY0UbZwCNquVAoDQtZypPiJR5BxDbB5isbckYppoW9JnkuQziotuqZUJKKX txXgF8C95sVhztDoRugz+HuNF6qdhkM3k3qm9x3GRjFqA+2XpJ3sykVhP/krEAg6pUOSE NGCSHyw5Zzi+nOrLoCVp/LBoaUtrunTclBu/RRt8GeKJtzVfBzL4S9D8HpsEqb9vTtQ== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1794339720; h=DMARC-Filter:DKIM-Signature:Received:Received:X-UD-Smtp-Session:Date: From:To:Cc:Subject:Message-ID:References:MIME-Version:Content-Type: Content-Disposition:In-Reply-To; bh=Pqtl4npWkphouNTE/kHIzbcAETdRulwAMb+Ntlny9Ss=; b=aHfEcKJBIQeBkNQWoXAanflinm13UiJ0yfjJ1er5I67iOPOuk1jJilJRDnFC4sNMXeEI /gCDKteqBD3LML7D57M6FWdvLgcd3+skf7Gaq1Wc5KxwDOgtl02rdc68DWY/F0RRv2h+9 NMgpTFdRA3791sGh502IrkGOvmsnRaHLCR45jHkPcWHb4QvPiPBX9WAkxBGdDnPOVNYQz ECnEAds++xr+y9PUMOAfe5Ta+BUD7SNcbbKTjmp8EZWbTP7vM8Gishy/8RNwb2O1nb0ws wUFOfkzbzxIZ1upL85Mj1dbmxd9frpBfE+oTs5piZ/FwDCG8miqzpQeZCdRsNOaSuvQ== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none header.from=sang-engineering.com; spf=pass smtp.mailfrom=sang-engineering.com; dkim=pass header.d=sang-engineering.com header.i=@sang-engineering.com header.a=rsa-sha256 header.s=k1 header.b=QWWCNMYD; arc=none smtp.remote-ip=194.117.254.33 Received: from mail.zeus03.de (zeus03.de [194.117.254.33]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D51B3606A31C for ; Tue, 10 Nov 2026 19:41:55 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org D51B3606A31C Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=sang-engineering.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=sang-engineering.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=sang-engineering.com header.i=@sang-engineering.com header.a=rsa-sha256 header.s=k1 header.b=QWWCNMYD DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= sang-engineering.com; h=date:from:to:cc:subject:message-id :references:mime-version:content-type:in-reply-to; s=k1; bh=Pqtl 4npWkphouNTE/kHIzbcAETdRulwAMb+Ntlny9Ss=; b=QWWCNMYDGBox/JT2AfaP Gw5F1mJYcpz3xFj7cax/hxHCr0IHvNVEJ+00CPZmkOGGXzEY6i5p2WwbnFjRpRGZ nMx2MGFuvkSzhldqAFgylB8vrzUQQrtodFGLAh17OOhd6BOcTYBT7gpq/bzfPJJn SFLj5xdrPFE4kdTpw5ueDc0ZtTXhKnV7jBPj9J2/+HyONAWFBbdlgchZ7ldmQjpS bcLVBbnN/kSRBtDmcLhbb/MHRXe7OIlIBb27APdtFsDQotP+g3qUHjQd5vlUhoNw Kgb31cLkJemTwzxKL/+p2y7RipS0f5USd/K9mIc5ghb25zNiZZMsAiS6xBMxgYcb YQ== Received: (qmail 346477 invoked from network); 10 Nov 2026 20:41:46 +0100 Received: by mail.zeus03.de with ESMTPSA (TLS_AES_256_GCM_SHA384 encrypted, authenticated); 10 Nov 2026 20:41:46 +0100 X-UD-Smtp-Session: l3s3148p1@h2bMRPFfjtwujnsO Date: Tue, 10 Nov 2026 20:41:43 +0100 From: Wolfram Sang To: Krzysztof Kozlowski Cc: Jonathan Corbet , Shuah Khan , Randy Dunlap , workflows@vger.kernel.org, linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, Andy Shevchenko Subject: Re: [PATCH v3] docs: submitting-patches: Clarify partial review tags Message-ID: References: <20261110141429.149451-2-krzysztof.kozlowski@oss.qualcomm.com> X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20261110141429.149451-2-krzysztof.kozlowski@oss.qualcomm.com> On Tue, Nov 10, 2026 at 03:14:30PM +0100, Krzysztof Kozlowski wrote: > Refactoring of driver subsystems or some core driver parts often come > with change of the actual drivers. If that happens within one patch, > people giving "Reviewed-by" tag only to piece of the patch, like one > driver, and ignoring the rest including the main concept of the patch, > are not really reviewing "the patch". > > As explained in the Submitting Patches document, the "Reviewer's > statement of oversight" is for patch as in "entire patch" meaning: > > (a) I have carried out a technical review of this patch to > evaluate its appropriateness and readiness for inclusion into > the mainline kernel. > > The sentence is explicit, exhaustive and complete, meaning it does not > allow implicit variations like: "this patch or part of this patch". > > Reviewing pieces of the patch while ignoring the main part of it, is > much closer to Acknowledgment, thus Ack or at least "Reviewed-by" should > be followed by '# subpart' suffix. > > Reviewed-by: Andy Shevchenko > Signed-off-by: Krzysztof Kozlowski Reviewed-by: Wolfram Sang