From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (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 29A531F4272 for ; Tue, 21 Jan 2025 15:15:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.52 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737472550; cv=none; b=P5J0D4QldbY6tP0m0HmxBgI+FnRGa3UuYjsUS0VVaQXZGDtMTX2ntEz67rkwQkyUrEgJy3+pUzZwNH3dzpTsr2YgofkHlO5gSMXLo1GxmG5yuxHESq8D1jTvHgR7mv1A1yAWwDX0TuzG9ZduOYccADm+AjssGsVQJud4KKHZGDI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737472550; c=relaxed/simple; bh=VAOxaKNqT9HIaNEdqkFqK0wD2sSNhVsUuyIHNh+V45k=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=vBRGEsNISYylaRVmIT8LkpikX+lxOolV5TmLzRGgqNg5UUyyAsVGcAF+VbP5OT+bkh933ui1z+wMK/T4Km0sSLIueF2k7YFK4VvENx7ayn2wrcd4QAZ1W7gdRtFcGTFhuYeJMHVJNLNtUh8T+F//nuK86KPiKWD4KCsm+i26IxE= 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=wAhW53O7; arc=none smtp.client-ip=209.85.128.52 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="wAhW53O7" Received: by mail-wm1-f52.google.com with SMTP id 5b1f17b1804b1-436281c8a38so42052355e9.3 for ; Tue, 21 Jan 2025 07:15:47 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1737472546; x=1738077346; 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=SvugShSkE19vxPC4Zwoh2Wo9A7je9UiUf85WI3sjd/U=; b=wAhW53O7riyFjBWzTA0XG3pXPrXy9c02pflKCX2IOu2r5ayrm/fB1o6tVdiz4Zl6ot W0mCvl84m0ZlFCNQTEpq3tg3uxym7LUQYzGScOm1vLi+ZNYlFS1fBzEUx39LrY6mU6Db dGXdsDqhg4plmXopI8AsB8l3ssoFwCRfS+JwsrgwY9+SMYy2sz9PT+CFZG4SRC0BxeMA ITVD5WomLkjxNVRr4v38GdM/l0Sk3wnr9U1maoSy9XNriFsEnoirryNFIDgMFTeLh091 nSdUFxCKymbeEuqrollzCDUEhnzmiZhLKd+RvhywOjqDhPbXek+3cpDDlECKnYBU4MNa qM4w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1737472546; x=1738077346; 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=SvugShSkE19vxPC4Zwoh2Wo9A7je9UiUf85WI3sjd/U=; b=R58ajEuZXUWTKA2BSawmUip6AckYe3bdtbj0k6duircT4+K9yCvZtwWvkW5pOGyD13 Zabim8AJTZuAtf1PmpyWeB+HCDa54IprJ3sIPUEcnIT+K7gONyxjug8f5tpisdThZEPT 7XY1qNDEA72666ca6gv9//5mZgH0k4yg6hx5NTPht3/1/nSq/5yxXXxb1e56O9F9gBBN 6rh+4qatibK8S/vYGNoswzFayn0ipuOxMRGY86yRSEtHOP3+b4sx28TFgp+Hn4Wwp5xX MIvXQ9pL/GXjURG+obFoNWOPFGiUnomHZz2TGX4d5zOyCQS7M/032HOWO+nKYNySVzhN i5+w== X-Forwarded-Encrypted: i=1; AJvYcCVVkdhVOWsUP3q/HFL8/5FRuyUj4OrkB1/Adi2Sv+DvgzVggBwZETGzLwMfjBDBqpz3b01L8N4XPINFbv8=@vger.kernel.org X-Gm-Message-State: AOJu0Yz94TOou+mxd5XXkNIXmu04qfHrM5qFDUBCWzyFGaJFwTzQl+iO gYcp82bAwlOI5JnPUQvQi7+6O/LsI2iMDfvYDDMs9o7cGvTw9qEucG8EKybcsnw= X-Gm-Gg: ASbGnct8No18bfdRDH48+/f9bNPac9jJlJaVH5jpfSd2HTdJ84Yg7v8YxPuVR5yrski 43q9lAYbUuc+P3fV7onIadYpyyqkzgMgoUPmZP+crsV6h/2ady/8GfE1MJzeWLkGn4B8PpDb3Dm FGmuxyMPBDbiY8kMkJ0ZeC0jJmlvyQn5g2Mhgwnmp0DGolCvRnUnir+NWDBblQGlhhd+52XojT3 ybqUb7UDthRiqxwZ1Fokqbk8TJG3DjdBKT/bMzCSgSARfHv6zA3cfqlwq8fMUze4VZ+2OuDgCE= X-Google-Smtp-Source: AGHT+IGmWQgLP0oShFNos3lRgIKGS81KPZyyyfYAl5Ap+K5h/aBJR8EY+XcHucmV4GzqhIiM28AjlQ== X-Received: by 2002:a05:600c:524c:b0:435:306:e5dd with SMTP id 5b1f17b1804b1-4389143a66dmr152559035e9.22.1737472546502; Tue, 21 Jan 2025 07:15:46 -0800 (PST) Received: from localhost ([196.207.164.177]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-43890367b48sm182318395e9.0.2025.01.21.07.15.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 21 Jan 2025 07:15:46 -0800 (PST) Date: Tue, 21 Jan 2025 18:15:43 +0300 From: Dan Carpenter To: Peng Fan Cc: "Peng Fan (OSS)" , 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" 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: On Tue, Jan 21, 2025 at 02:35:59PM +0000, Peng Fan wrote: > Hi Dan, > > > Subject: Re: [PATCH 3/4] rtc: Introduce devm_rtc_allocate_device_priv > > > > 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..e0e1a488b795645d > > 7c9453490d6c > > > dba510cc5db5 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. > > Thanks for help improving the code. I will include this in V2 and post > out after we reach a goal on how to support the 2nd RTC on i.MX95. Don't do what I said actually... Let's find a better way. I don't know why rtc_class_ops function pointers take a device pointer instead of an rtc_device pointer. Or if they did take a device pointer why not the &rtc->dev like you suggested? But let's not do both like this. Migrating all the function pointers is a lot of work but not impossible. regards, dan carpenter