From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f51.google.com (mail-wr1-f51.google.com [209.85.221.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 4AF6B1B423D for ; Mon, 20 Jan 2025 10:57:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.51 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737370633; cv=none; b=TUUsiirZ6rNKLUz1Kgy5jdgCRiJnDM70a5eulsfbOt0LyQhO6UgfHZtJU5hDzyTWlOhrBl/LCSO9xBtZMQy9ARJ6Y8a0nvklu4RIskvIo3HjJSKTUS5AOHLP3tMoPk43bhG4pCCiBimGTqSVShhyj4w05j0VBCpo4f7OdublPUg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737370633; c=relaxed/simple; bh=hlbVUse1kdyH7T+j1cXIhKPxXlagveeorWZyug3mE/M=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=X6fe8jV42090zN3P81sm9mdxwaFRvqDboAgPS5I2f7346P4XyBApCg1m4TNcV/WhZgBrRHj3IMylS7eGRpl7HefhK01mKKkyPFhq65RtQIU6Gg3C9ynIlO8VQOLIdb+ygLajNNGS0Lrk9LitFpf0HobTLkFVM3eKgvGz3AnJXDo= 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=epuezM5O; arc=none smtp.client-ip=209.85.221.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="epuezM5O" Received: by mail-wr1-f51.google.com with SMTP id ffacd0b85a97d-38789e5b6a7so2400713f8f.1 for ; Mon, 20 Jan 2025 02:57:10 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1737370629; x=1737975429; 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=ZnCt5DjDOmwR4I1vAJ/enzzSs6/BjfG1IINpOsDqGQc=; b=epuezM5O+vv7k5klavkWOWcY9eDWFTGY10QTZewjv4Q+LeW3J5pOQxeIWtAu38KGSC Ycfc4kt5h4KlU/5a0thQX94m6Uj+7QsNIcUdGq3j3CfOk8TalIdGwcD0xiBVjL/fsUO4 kjyrpRYuu7es+FN+0BVUA6bzG9Ut3g5zcXbQERFQ/GzLHeAUSTQ/FQwhtlTR7aJ/BqF9 rph/YWienhn2B5R89UquAhsA6WUdLUrVeDnjETB+tA1E/MMFF9qzHNqQtNBRn7PDdLth H/IwBNuGR6xax6g+4YMqgABdg3seyXQ7UjGOO8tkMesT8bJjM0egQiK9tDwdk7xlRsPh 6cYw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1737370629; x=1737975429; 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=ZnCt5DjDOmwR4I1vAJ/enzzSs6/BjfG1IINpOsDqGQc=; b=fAVEclR/6YM4t0O/k52buM6tOqLegCF2yyQynSfM3a4o1sEb0VNva+6o8I/F2BNxU0 7WCVH+eiKxNl3SRW+Ba7wsQA7Xc6YBQHTnG7NObG7pljRCs8fH23XJPR8MFmOmCFd3+s p+pv//A7TlYljw2BA54dr/kX11ROFckVYQKBw59cf+AVkUZECr8vUM6PZFMNwEaLAM0G l0doZc1mHtRRcEvTUPohWXtfnB3AS2+GGkLbDoIbgnJ/TbBJtrI279/OCh/xNQiMdQtW aXoUNNguTxN9AXVX6Z5esNPlr3vCk0yJ84gB8LtC4Bh6CbmmK18fikILj+0KXd3y85jL ZXzA== X-Forwarded-Encrypted: i=1; AJvYcCUP1JpZiR4p8kwcg+8qpePDHGaPCzJMkuW+e9UMayF7D8UwhdHHmFmGjLocfZf/4ipHJ6pqMdIwQCmFHKc=@vger.kernel.org X-Gm-Message-State: AOJu0YxHFedrGvI2EUrVQIkV5kVfxStYTS6h80EBxHtaVy5KZCZNjhXp L0YLX/IHIjGyGGJhgD6wajU+RWumqqngATCa4KJOeW3jrz9S3KSHUm8F/dQBGrI= X-Gm-Gg: ASbGnctRHo5jgrMXcDuRVJwlMa5nvDanylS/080pN3Somu9OLTsONlXBqvEwp2yR+X+ Ut1XWXszuhZoP4v9hpOUx8I/oiDhT2oTP30ItlKxvF5dE6hXUIFG/0OeB2Sp5On3DdcNkXcA/OO Mp+7NJVW6JXQ2xHmd8IOS9dFmNMECto3728L0RE8Q0uwrZ779divVGb4YkX3J1Qmno1Gvdf1YL+ 9kvCUzeGic87PF9efZz0nxIJo5gVQbolw9BZqKzNShBVjCF//csLJaWqfvHYg9vemXae+acsh0= X-Google-Smtp-Source: AGHT+IFcWdDkIWIV3+3k8GBszP8VeZU7+D+s8s4XaEHc4R+KN6tCXBGU2C0qdKTOyDm73nBXW992qw== X-Received: by 2002:a5d:4b48:0:b0:38a:41a3:218 with SMTP id ffacd0b85a97d-38bf57a2612mr7545183f8f.36.1737370629606; Mon, 20 Jan 2025 02:57:09 -0800 (PST) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-38bf3221b7fsm9969683f8f.27.2025.01.20.02.57.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Jan 2025 02:57:09 -0800 (PST) Date: Mon, 20 Jan 2025 13:57:05 +0300 From: Dan Carpenter To: "Peng Fan (OSS)" Cc: Sudeep Holla , Cristian Marussi , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Alexandre Belloni , arm-scmi@vger.kernel.org, linux-arm-kernel@lists.infradead.org, imx@lists.linux.dev, linux-kernel@vger.kernel.org, linux-rtc@vger.kernel.org, Peng Fan Subject: Re: [PATCH 3/4] rtc: Introduce devm_rtc_allocate_device_priv Message-ID: References: <20250120-rtc-v1-0-08c50830bac9@nxp.com> <20250120-rtc-v1-3-08c50830bac9@nxp.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=us-ascii Content-Disposition: inline In-Reply-To: <20250120-rtc-v1-3-08c50830bac9@nxp.com> On Mon, Jan 20, 2025 at 10:25:35AM +0800, Peng Fan (OSS) wrote: > int __devm_rtc_register_device(struct module *owner, struct rtc_device *rtc) > diff --git a/drivers/rtc/dev.c b/drivers/rtc/dev.c > index c4a3ab53dcd4b7280a3a2981fe842729603a1feb..e0e1a488b795645d7c9453490d6cdba510cc5db5 100644 > --- a/drivers/rtc/dev.c > +++ b/drivers/rtc/dev.c > @@ -410,7 +410,8 @@ static long rtc_dev_ioctl(struct file *file, > } > default: > if (rtc->ops->param_get) > - err = rtc->ops->param_get(rtc->dev.parent, ¶m); > + err = rtc->ops->param_get(rtc->priv ? > + &rtc->dev : rtc->dev.parent, ¶m); This seems kind of horrible... I can't think of anywhere else which does something like this. It would almost be better to do something like: err = rtc->ops->param_get(rtc->priv ? (void *)rtc : rtc->dev.parent, ¶m); The advatange of this is that it looks totally horrible from the get go instead of only subtly wrong. And it would immediately crash if you got it wrong implementing the ->param_get() function pointer. regards, dan carpenter