linux-kernel - RE: [PATCH v7 2/2] rtc: Add NXP PCF85053 driver support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:
 <AS4PR04MB9362A4D41767614B7B81EA3DFB9EA@AS4PR04MB9362.eurprd04.prod.outlook.com>
Date: Thu, 29 Jan 2026 05:34:58 +0000
From: Lakshay Piplani <lakshay.piplani@....com>
To: "alexandre.belloni@...tlin.com" <alexandre.belloni@...tlin.com>,
	"linux-rtc@...r.kernel.org" <linux-rtc@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"robh@...nel.org" <robh@...nel.org>, "krzk+dt@...nel.org"
	<krzk+dt@...nel.org>, "conor+dt@...nel.org" <conor+dt@...nel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>
CC: Vikash Bansal <vikash.bansal@....com>, Priyanka Jain
	<priyanka.jain@....com>, Shashank Rebbapragada
	<shashank.rebbapragada@....com>, Daniel Aguirre <daniel.aguirre@....com>,
	Pankit Garg <pankit.garg@....com>
Subject: RE: [PATCH v7 2/2] rtc: Add NXP PCF85053 driver support



> -----Original Message-----
> From: Lakshay Piplani <lakshay.piplani@....com>
> Sent: Thursday, November 27, 2025 5:35 PM
> To: alexandre.belloni@...tlin.com; linux-rtc@...r.kernel.org; linux-
> kernel@...r.kernel.org; robh@...nel.org; krzk+dt@...nel.org;
> conor+dt@...nel.org; devicetree@...r.kernel.org
> Cc: Vikash Bansal <vikash.bansal@....com>; Priyanka Jain
> <priyanka.jain@....com>; Shashank Rebbapragada
> <shashank.rebbapragada@....com>; Lakshay Piplani
> <lakshay.piplani@....com>; Daniel Aguirre <daniel.aguirre@....com>;
> Pankit Garg <pankit.garg@....com>
> Subject: [PATCH v7 2/2] rtc: Add NXP PCF85053 driver support
> 
> PCF85053 is i2c based RTC which supports timer and calendar functionality.
> 
> Features supported:
> 1. Read/Write time
> 2. Get/Set Alarm
> 3. Wakeup Source
> 4. Generate up to 32768Hz clock output
> 5. Primary/Secondary i2c bus
> 
> Signed-off-by: Daniel Aguirre <daniel.aguirre@....com>
> Signed-off-by: Pankit Garg <pankit.garg@....com>
> Signed-off-by: Lakshay Piplani <lakshay.piplani@....com>
> ---
> V6 -> V7: - Addressed minor cleanups from review: use
> dev_get_drvdata()/dev_set_drvdata()
> 	    consistently, fix alarm IRQ dev_id handling, and switch to
> devm_device_init_wakeup().
> 	  - Simplified time/alarm programming by forcing 24h + binary mode
> in hardware,
> 	    dropping complex 12h/BCD handling in setters.
> 	  - Documented the 2000–2099 supported year range, explaining how
> the 00–99 year
> 	    register maps to leap-year behavior in the device.
> V5 -> V6: no changes
> V4 -> V5: no changes
> V3 -> V4: - Handle multi-host ownership explicitly using primary/secondary
> bus hadling.
>           - Probe no longer changes any CTRL bits unconditionally and do not
> clear ST/AF/OF
>             avoiding lost interrupts or silent mode changes.
>           - Read/Set time & alarm now respect HF(12/24h) and DM(BCD/BIN)
> converting
>             hour fields correctly for all combinations.
>           - Minor changes: drop noisy warnings, tidy error paths/comments.
> V2 -> V3: Add MAINTAINERS file changes to this patch
> V1 -> V2: no changes
> 
>  MAINTAINERS                |   7 +
>  drivers/rtc/Kconfig        |  10 +
>  drivers/rtc/Makefile       |   1 +
>  drivers/rtc/rtc-pcf85053.c | 717 +++++++++++++++++++++++++++++++++++++
>  4 files changed, 735 insertions(+)
>  create mode 100644 drivers/rtc/rtc-pcf85053.c
> 

Hi Alexandre,

I'm sending a gentle reminder regarding the patches that I submitted in November. 
I haven't received any review comments yet, so I’d really appreciate it if you could have a look whenever you have some time.

Best Regards
Lakshay Piplani