From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f174.google.com (mail-qk1-f174.google.com [209.85.222.174]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 813562D3740 for ; Wed, 10 Sep 2025 19:17:21 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757531843; cv=none; b=PfZnONiqj2c6G7Uu71gVA17R+19yXu6KLGHmYZhRBYYcz0c5d/QotZdQ32fzWfR+oJZPKCgeFUt2OxWbRqsB2p4PlJkmg2dMBzEI9SyrDuHbjfyPMzIXC7zLTODG5CDwmyrMQ57xoJJIk8LMKWcUcLfuYB/xNurHV9aTZMkd3lc= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1757531843; c=relaxed/simple; bh=SCwHNEdMA8MEn00BiFAEH+58S7M3dIzW3LxE3UwmJtE=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=udyQ8gXkjWEWvjvMupeUKVtX2erqLpXQdvmK9MlETRHqH1W0Ro0mpXIUk8VQBZUGwFmayy2iWRcnm6p0ioM+S/Nj4WCLU2HO+dxny0aY1q2ba2yNHSFXFOtaFFAJSKoXEPSeHsvwKX7jFZK+fotKIwvnPhuESF4jYNgTgOr49uw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=CH06cekx; arc=none smtp.client-ip=209.85.222.174 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="CH06cekx" Received: by mail-qk1-f174.google.com with SMTP id af79cd13be357-81076e81a23so837945685a.3 for ; Wed, 10 Sep 2025 12:17:21 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1757531840; x=1758136640; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:feedback-id:from:to:cc:subject:date :message-id:reply-to; bh=7snMmseLVBcfubyQhVWOsGdoU6rRK3K5ZVTv8Xsb4Kw=; b=CH06cekxPOyCvZx6yIU5SYzBawHUX7+Ou9ol6PsbLwgQMoxVU/XSh5CCgN4hFYWiOf anUv3FzI2ew7vmF2m6/M5LEWmQuVo8ri6AhRqBL7L7o6yr4NxAFfBg6lvAOWynrhsCXB Kiq0OfhRAa/gLlZLDrWduev0vokcxAu5cifZX8A4B76uQqZs3wpuv9xuqLyL/6XSGo84 7bS8rPRw5Xm//UADI0zPEusNm3gj7DJv1tWZsCRkqHM7RLXClDmNIPewLje2KvcG3A/g vbwrREuN1aDQwzBnXaa741+z1/D57fOVQBdMLAzSXtPwx94a7yk4Nqou5OOpx207vnHS EcEw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1757531840; x=1758136640; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:feedback-id:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=7snMmseLVBcfubyQhVWOsGdoU6rRK3K5ZVTv8Xsb4Kw=; b=gIsqeQGshCR9Pdn65CLmadgp3KBk0PaGWKnvHhoy+FJHuhv3RcMKOxFUlksOata+h9 holf8mPh484QuVyT35rtOGp9S5X+xor2TIA31JhRknbECUJh9FJoNtw/PhFKvK1mb009 kzLU5vAlM9gkC8hDGOvBygPO9d3sh0h/SoWjCnMiLPWkddQwktbqBQkAYsCLE/YyeK6c 4b/1LohEkBv7+sioySPgeX+QFPq8uGVKBHk+twb1dJsYBXYsFY9X3dmPX79q0OVx0zER 2vq/OLbCuOr2gbFXaoSAgNR2Vj1GLxDK8DR+onAaHa3YzZR/+u05cmlW4+D9JJi0J6yM b/RQ== X-Forwarded-Encrypted: i=1; AJvYcCWMzOgoKsvITfl+peX3+FksMHKDMnTwPHRFPiGkpbVnHuqLwcsIVgX3XdQRQg2926vDeeu+Z0aj81Mt2/Q=@vger.kernel.org X-Gm-Message-State: AOJu0Yx5bo7bXtIHTAWJZsZhvaibSgLkt5MkT4uRiCnUh8DEMHPqjaU1 p2FTaUbHLhcaddDDxyszUHI3A8j7AdidaXdYWS/DaAJNMMkQqxCKQF2z X-Gm-Gg: ASbGncvw6ybwIAGFeM/jvBcd6bDSf2TmKOY2AXHaLQHBN05dGpajJwY9vsm0ipTa8hM 1STNrOvhcT7edcoczVw3nOsYRzyjs+K0wAvcW/tnhq3GvB9jGHMibJQFwArCvrw47O3vMVKnERB sJMeTPH6MkVgz0IEwJjWPWJMOB/L4AjlIZ9WmWCzyRWrfc0d4GpESi+H/gLgLBwnw9j3+7O6B7H SMLaAv887SmF65jmBp17KCHR6VPbKepP3fkgcV6ukZtxRLmFWKk8OSEEe96eqDLpOrY8gEEKx26 0ivlPSHL0D8ZEYVp5WbDVN+ZLn+mEB3hdEXuozfKdPOknYGzQDi5fWN2i4ljV7/JQKiz6hrAvMw pQc1/mDXj/XE/45y4vCCxd0syjblnESO28RQnHOWzOoyF9LF1/eb8wHBx8kpdvRvEIB2ufcZVsE fAifi54njbHoJKuYD3O5kMSAYjUC+EdNtJXA== X-Google-Smtp-Source: AGHT+IFLKs6Z9Get2FJtBN4ng6d/qG2NCMDW+y5NcESyPYI9AIJUrHThVuklIbRDXM58AvJuPVzh/A== X-Received: by 2002:a05:620a:1aa0:b0:81f:65ec:81c4 with SMTP id af79cd13be357-81f65ec834cmr243699785a.30.1757531840153; Wed, 10 Sep 2025 12:17:20 -0700 (PDT) Received: from fauth-a2-smtp.messagingengine.com (fauth-a2-smtp.messagingengine.com. [103.168.172.201]) by smtp.gmail.com with ESMTPSA id af79cd13be357-81b5f715c0bsm339500585a.61.2025.09.10.12.17.19 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 10 Sep 2025 12:17:19 -0700 (PDT) Received: from phl-compute-03.internal (phl-compute-03.internal [10.202.2.43]) by mailfauth.phl.internal (Postfix) with ESMTP id 0ABE3F40068; Wed, 10 Sep 2025 15:17:19 -0400 (EDT) Received: from phl-mailfrontend-01 ([10.202.2.162]) by phl-compute-03.internal (MEProxy); Wed, 10 Sep 2025 15:17:19 -0400 X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgeeffedrtdeggddvgedutdcutefuodetggdotefrod ftvfcurfhrohhfihhlvgemucfhrghsthforghilhdpuffrtefokffrpgfnqfghnecuuegr ihhlohhuthemuceftddtnecusecvtfgvtghiphhivghnthhsucdlqddutddtmdenucfjug hrpeffhffvvefukfhfgggtuggjsehttdertddttddunecuhfhrohhmpeeuohhquhhnucfh vghnghcuoegsohhquhhnrdhfvghnghesghhmrghilhdrtghomheqnecuggftrfgrthhtvg hrnhepieffjeehjeeludekfeeikefgteevjeeitdevkefghfetkedvieejfeegtdffvefg necuffhomhgrihhnpehkvghrnhgvlhdrohhrghenucevlhhushhtvghrufhiiigvpedtne curfgrrhgrmhepmhgrihhlfhhrohhmpegsohhquhhnodhmvghsmhhtphgruhhthhhpvghr shhonhgrlhhithihqdeiledvgeehtdeigedqudejjeekheehhedvqdgsohhquhhnrdhfvg hngheppehgmhgrihhlrdgtohhmsehfihigmhgvrdhnrghmvgdpnhgspghrtghpthhtohep udelpdhmohguvgepshhmthhpohhuthdprhgtphhtthhopegurghnihgvlhdrrghlmhgvih gurgestgholhhlrggsohhrrgdrtghomhdprhgtphhtthhopehmthhurhhquhgvthhtvges sggrhihlihgsrhgvrdgtohhmpdhrtghpthhtohepshgsohihugeskhgvrhhnvghlrdhorh hgpdhrtghpthhtohepohhjvggurgeskhgvrhhnvghlrdhorhhgpdhrtghpthhtoheprghl vgigrdhgrgihnhhorhesghhmrghilhdrtghomhdprhgtphhtthhopehgrghrhiesghgrrh ihghhuohdrnhgvthdprhgtphhtthhopegsjhhorhhnfegpghhhsehprhhothhonhhmrghi lhdrtghomhdprhgtphhtthhopehlohhsshhinheskhgvrhhnvghlrdhorhhgpdhrtghpth htoheprgdrhhhinhgusghorhhgsehkvghrnhgvlhdrohhrgh X-ME-Proxy: Feedback-ID: iad51458e:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Wed, 10 Sep 2025 15:17:18 -0400 (EDT) Date: Wed, 10 Sep 2025 12:17:17 -0700 From: Boqun Feng To: Daniel Almeida Cc: Michael Turquette , Stephen Boyd , Miguel Ojeda , Alex Gaynor , Gary Guo , =?iso-8859-1?Q?Bj=F6rn?= Roy Baron , Benno Lossin , Andreas Hindborg , Alice Ryhl , Trevor Gross , Danilo Krummrich , "Rafael J. Wysocki" , Viresh Kumar , linux-clk@vger.kernel.org, rust-for-linux@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org Subject: Re: [PATCH v2 1/2] rust: clk: implement Send and Sync Message-ID: References: <20250910-clk-type-state-v2-0-1b97c11bb631@collabora.com> <20250910-clk-type-state-v2-1-1b97c11bb631@collabora.com> <3D936C1B-FBA9-4964-859C-84BB665BBE3B@collabora.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline In-Reply-To: <3D936C1B-FBA9-4964-859C-84BB665BBE3B@collabora.com> On Wed, Sep 10, 2025 at 03:47:30PM -0300, Daniel Almeida wrote: > Hi Boqun, > > > On 10 Sep 2025, at 14:49, Boqun Feng wrote: > > > > On Wed, Sep 10, 2025 at 02:28:27PM -0300, Daniel Almeida wrote: > >> From: Alice Ryhl > >> > >> These traits are required for drivers to embed the Clk type in their own > >> data structures because driver data structures are usually required to > >> be Send. See e.g. [1] for the kind of workaround that drivers currently > >> need due to lacking this annotation. > >> > >> Link: https://lore.kernel.org/rust-for-linux/20250812-tyr-v2-1-9e0f3dc9da95@collabora.com/ [1] > >> Signed-off-by: Daniel Almeida > >> Signed-off-by: Alice Ryhl > >> Reviewed-by: Danilo Krummrich > >> Acked-by: Viresh Kumar > >> Reviewed-by: Daniel Almeida > > > > This tag list looks a bit weird to me. Why is there a SoB from you > > before Alice's SoB? At least for the usage I'm familiar with, outside > > the case of Co-developed-bys, multiple SoBs is used for recording how > > the patches are routed. For example, if I have a patch that has my SoB > > and I send it to you, you queue in your tree and then send out to other > > maintainers for merging, in general you would put your SoB after mine in > > that case. But I don't think that's case here? Alice's patch has only > > her SoB: > > > > https://lore.kernel.org/rust-for-linux/20250904-clk-send-sync-v1-1-48d023320eb8@google.com/ > > > > What's the intention of the SoB tag here? > > > > Otherwise the patch looks good to me. If we get the tag list resolved, > > feel free to add: > > > > Reviewed-by: Boqun Feng > > > > Regards, > > Boqun > > > > You have to include your SOB when submitting patches from others. > > This is something I tend to forget often, so I made sure it was there. The > order may be indeed off though. > I mean you can just use `b4 shazam -s` to apply the patch and add your SoB at the end ;-) Regards, Boqun > - Daniel >