From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f51.google.com (mail-wm1-f51.google.com [209.85.128.51]) (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 2D25F32BF3D for ; Thu, 20 Nov 2025 10:32:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763634748; cv=none; b=kjH2hd7WR0T12cyRmiMvhyQljbdJmALEJY0g4uj3lueMwaA1Qf9p+wyflV7Q4g11A4W1Gwf7B8rGfJR3oqhP8yyuSzM2z6BxtUmURtTNZvnHSf49bz2vzyJB82zILzMUtY42nwUZq5IPONZ6iTOfIqHmXepzA34NjYI9jptoFDY= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1763634748; c=relaxed/simple; bh=OSupr0kJjnxwNvOhXhv6PH9+Mq/EXttUjlC7N2YQVtc=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=NHeNkWtaKriVvaXct1nBEaohQDLrOPzIMysQ2uhusGp//7Dx6kS7wllOoyEMnObekwffasa6QlO/NFT66SJKMfkkU68HMdW7Pe3WHUgXUL/jTvt5BczBny5fJNdEaPiHEMYKHTR1Wj+q9wsrM6WhnHpAR/PeMOIn6sbwOeQO5AA= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=UEseo7bM; arc=none smtp.client-ip=209.85.128.51 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="UEseo7bM" Received: by mail-wm1-f51.google.com with SMTP id 5b1f17b1804b1-477b1cc8fb4so4695165e9.1 for ; Thu, 20 Nov 2025 02:32:25 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1763634744; x=1764239544; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=OSupr0kJjnxwNvOhXhv6PH9+Mq/EXttUjlC7N2YQVtc=; b=UEseo7bMUCkbROB9ao2jiRTLBLtvfd2OvdCYzl4Zy3QS5JXaMdi9KNMjtZQKxTCXNu moVHOKFDA+BKfJBjLDc3pwTOdC0Asl1RSRh+FTaK7N/9jlpbzl7Hqg8wrP0HF6xy8H3m FVIw2tIYL0Jo6olgQpCW3SjDy3px2bQAmmmQZMcl23PJhdjq8G8/gVPs9dzQFCQGPU/r ZrHfaaIdZ1LOOPYphRxDy5vVtHSmwR4U/NpfbG89ibaYqB8rvUYM82XdyNnHH+UAaKLt AZIGMgTLFbdHhB8Q1iYq4xd5TJ0uZg/wKk1BDVb8zuOwuIIDT5sHxmnZMxi73gosAXEg 6dzg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1763634744; x=1764239544; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=OSupr0kJjnxwNvOhXhv6PH9+Mq/EXttUjlC7N2YQVtc=; b=NnNpIEq8SX+icp7XV2fp0Nk0yEkvgQXztv8O7YrU8pkeYnQvI2K6Rph5ZQzMUdt8Jy Kn4LtaDqWPCdJr/gcC9UlOPjXcEADvcL2MOXlh+Y/KHm8Y5gIIuqnyHJwJKgil/I/WBz X1tgIOjniTLjaCsdF9XqCcVUGRz0CQRMiSrwx268LVJ8SV4bfQU/oRrQ8XkkpcHFMFMd EP95w5vBN3zDXu6wbcpnxyJKEyC3/GPlfFboVtb3oncL6djECmu73oXOWSrqOBP2hFeI 4MGSKxjkwLrSbgPkxUGw8CleZBdy+b2U6BRIUEp7H+lzU77BiS1tMsbZnIVTOawNWSbL oLnw== X-Forwarded-Encrypted: i=1; AJvYcCXPfUGR5MJOB67X6HAVLaVBv/V55vq9AhDa2VbRIQcniHk/RFXS5eXE8IJknMhg5Lo0RNJiGMp1pGO6ZU0=@vger.kernel.org X-Gm-Message-State: AOJu0YwHyxj2gEGoEeOTgMdg8RYyhW0r/a9/QN7wZ6sKpQ38ehScD5ui dEpLv4Z8UnoRgZ8fE/VlkVs9I4PjtLQ564eYTeTa/Efs3/8HBZ5UBVUotYoqtQsu3/E= X-Gm-Gg: ASbGnctsYBm5IC4X46KUaMsQHS9J9Yprg0bdQCScbfb0gr7lZFFQNtA8CqQ/aGVwyQH 7uAA4JRwNl4wwYt/aKEfgrfa3dRHr7Bl8KGHJfKJ9/T32Q5np8P1J6urt0QSf1TR9x0qFCmU3wS MhngFA90tRzXPePl+/GR5wAA1uEqvv3zMcuOBZ7y5EmFx3O2+CqFjhoz0bRQgySFrcuVZQ/Khm5 QLo/ZspE1BGYjYROuYqZCld9d3lPgieESqKSRXr6aknWgtxa9HdfHa5gT7nPP0fGz69cQ/TRHsT mMMUEiUgmuxaswA1HhZG5cVzyYqkWLfZCG8BrA2aks2jVQBIuQlB9K/EQ4KoaXPSxXjwE/g0JZ6 BicmLQ5KBHkLmXV9TwLZYpuu7M7t8geIiA1Ob1ZQ6b/Y9VK5LcvxyZLoAFZXbcauCZGPR7qRdxR SnIDL0ZNgYlyliOMY8aihLw51fyBc= X-Google-Smtp-Source: AGHT+IECH0Ar6MJ7BtthyBsytzJSAtWIhKLrFKyKhjk8O39/oM0DhnAqKZk88fXqWIq9/kPk2wPzpg== X-Received: by 2002:a05:600c:474a:b0:46e:5100:326e with SMTP id 5b1f17b1804b1-477b8a98a93mr27846505e9.23.1763634744455; Thu, 20 Nov 2025 02:32:24 -0800 (PST) Received: from salami.lan ([212.129.72.102]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-477b106b03bsm102270655e9.9.2025.11.20.02.32.23 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 20 Nov 2025 02:32:23 -0800 (PST) Message-ID: <8ebfbfa9d592b7a5d965dadf5f3615a77c89cb03.camel@linaro.org> Subject: Re: [PATCH 0/3] Samsung mfd/rtc driver alarm IRQ simplification From: =?ISO-8859-1?Q?Andr=E9?= Draszik To: Lee Jones Cc: Krzysztof Kozlowski , Alexandre Belloni , Peter Griffin , Tudor Ambarus , Will McVicker , Juan Yescas , Douglas Anderson , kernel-team@android.com, Kaustabh Chakraborty , linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-rtc@vger.kernel.org Date: Thu, 20 Nov 2025 10:32:22 +0000 In-Reply-To: <20251120103104.GY1949330@google.com> References: <20251114-s5m-alarm-v1-0-c9b3bebae65f@linaro.org> <20251120103104.GY1949330@google.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.56.2-7 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Hi Lee, On Thu, 2025-11-20 at 10:31 +0000, Lee Jones wrote: > On Fri, 14 Nov 2025, Andr=C3=A9 Draszik wrote: >=20 > > Hi, > >=20 > > With the attached patches the Samsung s5m RTC driver is simplified a > > little bit with regards to alarm IRQ acquisition. > >=20 > > The end result is that instead of having a list of IRQ numbers for each > > variant (and a BUILD_BUG_ON() to ensure consistency), the RTC driver > > queries the 'alarm' platform resource from the parent (mfd cell). > >=20 > > Additionally, we can drop a now-useless field from runtime data, > > reducing memory consumption slightly. > >=20 > > The attached patches must be applied in-order. I would expect them all > > to go via the MFD tree. Alternatively, they could be applied one after > > another during multiple kernel release cycles, but that seems a > > needless complication. > >=20 > > Signed-off-by: Andr=C3=A9 Draszik > > --- > > Andr=C3=A9 Draszik (3): > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 mfd: sec: add rtc alarm IRQ as platform = device resource > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 rtc: s5m: query platform device IRQ reso= urce for alarm IRQ > > =C2=A0=C2=A0=C2=A0=C2=A0=C2=A0 mfd: sec: drop now unused struct sec_pmi= c_dev::irq_data > >=20 > > =C2=A0drivers/mfd/sec-common.c=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2= =A0=C2=A0 | 41 ++++++++++++++++++++++++++++++---------- > > =C2=A0drivers/mfd/sec-core.h=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0 |=C2=A0 2 +- > > =C2=A0drivers/mfd/sec-irq.c=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 | 10 ++-------- > > =C2=A0drivers/rtc/rtc-s5m.c=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0=C2=A0= =C2=A0=C2=A0=C2=A0=C2=A0 | 21 ++++++++------------ > > =C2=A0include/linux/mfd/samsung/core.h |=C2=A0 1 - > > =C2=A05 files changed, 42 insertions(+), 33 deletions(-) >=20 > This needs to be rebased now, right? Yes, I'll do that now. Thanks for merging the other patch. Cheers, Andre'