From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) (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 D88EA1E5B8B for ; Tue, 1 Apr 2025 16:28:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.173 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743524886; cv=none; b=uecc5F/WYDM3fZNHI/Vc+Yiyr/W6EyrByQWDYZMyuU8js0/n+qLtECI79+q33viccVCcgHLeFqQw26I4xuMujGuARACPwm+4Sa7fXr1aP2rvzlRQvn1nhb+x4nqpcdwDwd0Q6v8t+bOupZMfpsOJSQivXWmP8zAxzAsEHEvP6ww= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1743524886; c=relaxed/simple; bh=QyvPwe2n7SMJQvY20lH5qrJm2IZrAH6H92ENM40f7x4=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=m3jUS0UmtxZfzfsZXKkAWTbjItuibAHZfK/Z0awyPgAhxK7h/CcrtsXkkHU0nJ7ZtYb+2i+MHVv7F6xAqTi8kN2xJDD/FaYBf0skMB77TYwR9uQl0RRtu3sVTjCAgF9+oteTOYY+oJBQ2gUvpOFPf/ru7ZwiwJIU7iRta3u+RUI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=G/Tl31CL; arc=none smtp.client-ip=209.85.214.173 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="G/Tl31CL" Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-224341bbc1dso98481435ad.3 for ; Tue, 01 Apr 2025 09:28:04 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1743524884; x=1744129684; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=sEr9LMvpLbHE893cIqhG/JU8h0Dt2xG4p+araqhsX+g=; b=G/Tl31CLaQf9aFz7+CTDTug+UVm1CNVsHhLnpWq57mdwnmnAOORtjDSe5bqeniH/Li 61CeBh6vIylSUqtFueMX0T1ZzVommA5WRDHoLCxy8wPi+T2+/zeKt1KWsYsWtewxEND5 56OdweU6OmmQv9R9YLsYJf4FLnq9r9n0dW2RKIU71IFJKiThjMYj48ZvR7mbvapRBK92 9xAzRBM0YT0dTAtHdOyhC0eGJOd1u/nFOHnSYTauAzEQ3ZFqsGV6pS+rAHLwI+R5Urq5 1XhMfrcA+nddSbsFs5fBXLKkHvuuLgdieZbPQmt43fMHKg1HaJuKNAwtBCyXHRgoY4Lk G9ig== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1743524884; x=1744129684; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=sEr9LMvpLbHE893cIqhG/JU8h0Dt2xG4p+araqhsX+g=; b=i7lLyMuPyd3j8yzO2H0H6nrmjr8gSNu1DsrY6SDmgtwgJ4CFD7d1zpJLkNnbmrhREJ kleIhbpSibjcwsMbAT8h5qQtvDEXyRuh5YmjN7rAdWBam9MPEuOsHGxbJq9sSW+pemhg 4qshvVDCcQ500XUBATJHlrcd/RO1Rz5LBOkeNn/WUeTwwqwONTGo+xUvyWIQXVohOJwO Ch8FFvKxahxB68pWaoCBvEXAylUzn9jm9BEzKTVIG+KC1AyQJscG5XXlygQ1Q5pdIIlq bEwMQ/O4pwC7r5TlsBlt1iNvxGep7B/KTc42Z+3MoeTYwkOiP6t6rfXvlNIleSjt+27f SLJg== X-Forwarded-Encrypted: i=1; AJvYcCXe09+jEZkzOL3bfPtM3HbQPz8YblUnsbj1X3kHX6lkNasFuEGn456q+wW5nUrB0UBMkDqFlx+FtcZEtcA=@vger.kernel.org X-Gm-Message-State: AOJu0YwSX2jnfS44sPdzoq0g9nrxDHwSJ1TOUwM711HC5CJuHQKogr3o fAwERvbbr4oO3Od4NEHbSBU2lGSD+ne1rbt5QP9JDGAyo3UKlitwL5qNMeWM+g== X-Gm-Gg: ASbGncui0BmfgfykEpfy1FLOnvmfXxaBt0P8W8oPukfoqAomlPUbC3S9MEFvtVmZKIN oH2XtqX5KCF9gDns8K9Zaq7p7z1XRCx3JO1vUMxkFJmn0b52e+8s/AQUBSE7d/FSa8apzzISg2M AlNRG+EK4Zu8PU26B4hECBclzwU6Q1iPrezOqhuONYFhCRnzCapCqxsRr2Dq3enkOhjmC0dpNFZ Sr1SnLnlJXKmIQOpYcltHTEZ4sFAaCxquKOliSPSWP3dGxHmzEynbwVWZELxbmZEDTu+PwWylX6 SPEdeTl4hxgKFkqXRigR3/TbHl55y4u9DqDb/p4RW4rmkypxpo/YF8zq4h6n9zwHv1jSxO5GWW7 kdzRKBfXwPqevirA= X-Google-Smtp-Source: AGHT+IEyaXPv9oHXzdX7kp5aOc8Pw45evlmKNFGJN69UA0/YokAg60Q/XphIOEUon7k6t8bnNWSYvA== X-Received: by 2002:a17:902:ea02:b0:224:216e:332f with SMTP id d9443c01a7336-2292f9fc054mr220540675ad.48.1743524883898; Tue, 01 Apr 2025 09:28:03 -0700 (PDT) Received: from google.com (198.103.247.35.bc.googleusercontent.com. [35.247.103.198]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-2291f1f7376sm89939395ad.245.2025.04.01.09.28.03 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 01 Apr 2025 09:28:03 -0700 (PDT) Date: Tue, 1 Apr 2025 09:27:59 -0700 From: William McVicker To: Rob Herring Cc: Catalin Marinas , Will Deacon , Peter Griffin , =?iso-8859-1?Q?Andr=E9?= Draszik , Tudor Ambarus , Krzysztof Kozlowski , Conor Dooley , Alim Akhtar , Daniel Lezcano , Thomas Gleixner , Saravana Kannan , Krzysztof Kozlowski , kernel-team@android.com, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, Donghoon Yu , Youngmin Nam Subject: Re: [PATCH v1 5/6] clocksource/drivers/exynos_mct: Add module support Message-ID: References: <20250331230034.806124-1-willmcvicker@google.com> <20250331230034.806124-6-willmcvicker@google.com> <20250401023001.GA3186122-robh@kernel.org> 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=us-ascii Content-Disposition: inline In-Reply-To: <20250401023001.GA3186122-robh@kernel.org> On 03/31/2025, Rob Herring wrote: > On Mon, Mar 31, 2025 at 04:00:27PM -0700, Will McVicker wrote: > > From: Donghoon Yu > > > > On Arm64 platforms the Exynos MCT driver can be built as a module. On > > boot (and even after boot) the arch_timer is used as the clocksource and > > tick timer. Once the MCT driver is loaded, it can be used as the wakeup > > source for the arch_timer. > > > > Signed-off-by: Donghoon Yu > > Signed-off-by: Youngmin Nam > > [Original commit from https://android.googlesource.com/kernel/gs/+/8a52a8288ec7d88ff78f0b37480dbb0e9c65bbfd] > > Signed-off-by: Will McVicker > > --- > > drivers/clocksource/Kconfig | 3 +- > > drivers/clocksource/exynos_mct.c | 47 +++++++++++++++++++++++++++----- > > 2 files changed, 42 insertions(+), 8 deletions(-) > > [...] > > > +#ifdef MODULE > > +static int exynos4_mct_probe(struct platform_device *pdev) > > +{ > > + struct device_node *np = pdev->dev.of_node; > > + > > + if (of_machine_is_compatible("samsung,exynos4412-mct")) > > Your root node compatible has "samsung,exynos4412-mct"!? > > In any case, add a data ptr to of_device_id table and then use the match > data rather than comparing compatible strings again. Ah yes, you're right. Thanks for the suggestion! I'll update on v2. Regards, Will [...]