Message ID | 20211211175951.30763-1-luca@lucaceresoli.net |
---|---|
Headers | show |
Series | Add MAX77714 PMIC minimal driver (RTC and watchdog only) | expand |
Hi All, On 11/12/21 18:59, Luca Ceresoli wrote: > Hi, > > this series adds minimal drivers for the Maxim Semiconductor MAX77714 > (https://www.maximintegrated.com/en/products/power/power-management-ics/MAX77714.html). > Only RTC and watchdog are implemented by these patches. > > All implemented functionality is tested and working: RTC read/write, > watchdog start/stop/ping/set_timeout. > > Patches 1-3 + 6 are trivial cleanups to the max77686 drivers and Kconfig > indentation and can probably be applied easily. > > Patches 4, 5, 7, 8 and 9 add: dt bindings, mfd driver, watchdog driver and > rtc driver. A gentle ping about this series. It's at v5, all patches have at least one ack/review tag and most patches are unchanged since ~v2. It applies cleanly on current master. Is there anything I should do to help making progress? Regards,
Hi Lee, all, On 11/01/22 11:10, Luca Ceresoli wrote: > Hi All, > > On 11/12/21 18:59, Luca Ceresoli wrote: >> Hi, >> >> this series adds minimal drivers for the Maxim Semiconductor MAX77714 >> (https://www.maximintegrated.com/en/products/power/power-management-ics/MAX77714.html). >> Only RTC and watchdog are implemented by these patches. >> >> All implemented functionality is tested and working: RTC read/write, >> watchdog start/stop/ping/set_timeout. >> >> Patches 1-3 + 6 are trivial cleanups to the max77686 drivers and Kconfig >> indentation and can probably be applied easily. >> >> Patches 4, 5, 7, 8 and 9 add: dt bindings, mfd driver, watchdog driver and >> rtc driver. > > A gentle ping about this series. It's at v5, all patches have at least > one ack/review tag and most patches are unchanged since ~v2. It applies > cleanly on current master. > > Is there anything I should do to help making progress? Apologies for pinging again... but as I got no further comments about these patches I guess I can really do nothing at the moment. Lee, is this series completely in charge to you or should it be applied by the respective subsystem maintainers? Thanks.
On 1/29/22 00:40, Luca Ceresoli wrote: > Hi Lee, all, > > On 11/01/22 11:10, Luca Ceresoli wrote: >> Hi All, >> >> On 11/12/21 18:59, Luca Ceresoli wrote: >>> Hi, >>> >>> this series adds minimal drivers for the Maxim Semiconductor MAX77714 >>> (https://www.maximintegrated.com/en/products/power/power-management-ics/MAX77714.html). >>> Only RTC and watchdog are implemented by these patches. >>> >>> All implemented functionality is tested and working: RTC read/write, >>> watchdog start/stop/ping/set_timeout. >>> >>> Patches 1-3 + 6 are trivial cleanups to the max77686 drivers and Kconfig >>> indentation and can probably be applied easily. >>> >>> Patches 4, 5, 7, 8 and 9 add: dt bindings, mfd driver, watchdog driver and >>> rtc driver. >> >> A gentle ping about this series. It's at v5, all patches have at least >> one ack/review tag and most patches are unchanged since ~v2. It applies >> cleanly on current master. >> >> Is there anything I should do to help making progress? > > Apologies for pinging again... but as I got no further comments about > these patches I guess I can really do nothing at the moment. > > Lee, is this series completely in charge to you or should it be applied > by the respective subsystem maintainers? > I hesitated to take the watchdog patches because an earlier patch of the series introduces MFD_MAX77714 and the watchdog Kconfig entry lists it as dependency. I now added patch 7/9 and 8/9 to my watchdog-next tree anyway. If the mfd part doesn't make it we can still decide to take it out at some point. Note that patch 6/9 has already been applied. Guenter
Hi Guenter, On 30/01/22 02:48, Guenter Roeck wrote: > On 1/29/22 00:40, Luca Ceresoli wrote: >> Hi Lee, all, >> >> On 11/01/22 11:10, Luca Ceresoli wrote: >>> Hi All, >>> >>> On 11/12/21 18:59, Luca Ceresoli wrote: >>>> Hi, >>>> >>>> this series adds minimal drivers for the Maxim Semiconductor MAX77714 >>>> (https://www.maximintegrated.com/en/products/power/power-management-ics/MAX77714.html). >>>> >>>> Only RTC and watchdog are implemented by these patches. >>>> >>>> All implemented functionality is tested and working: RTC read/write, >>>> watchdog start/stop/ping/set_timeout. >>>> >>>> Patches 1-3 + 6 are trivial cleanups to the max77686 drivers and >>>> Kconfig >>>> indentation and can probably be applied easily. >>>> >>>> Patches 4, 5, 7, 8 and 9 add: dt bindings, mfd driver, watchdog >>>> driver and >>>> rtc driver. >>> >>> A gentle ping about this series. It's at v5, all patches have at least >>> one ack/review tag and most patches are unchanged since ~v2. It applies >>> cleanly on current master. >>> >>> Is there anything I should do to help making progress? >> >> Apologies for pinging again... but as I got no further comments about >> these patches I guess I can really do nothing at the moment. >> >> Lee, is this series completely in charge to you or should it be applied >> by the respective subsystem maintainers? >> > > I hesitated to take the watchdog patches because an earlier patch of the > series > introduces MFD_MAX77714 and the watchdog Kconfig entry lists it as > dependency. > I now added patch 7/9 and 8/9 to my watchdog-next tree anyway. If the > mfd part > doesn't make it we can still decide to take it out at some point. OK, thank you! In the meanwhile the kernel test robot also reported a build failure due to missing max77714.h file, which is added by patch 5. > Note that patch 6/9 has already been applied. Indeed, it's in Linus' master already. Regards.
On 1/30/22 04:45, Luca Ceresoli wrote: > Hi Guenter, > > On 30/01/22 02:48, Guenter Roeck wrote: >> On 1/29/22 00:40, Luca Ceresoli wrote: >>> Hi Lee, all, >>> >>> On 11/01/22 11:10, Luca Ceresoli wrote: >>>> Hi All, >>>> >>>> On 11/12/21 18:59, Luca Ceresoli wrote: >>>>> Hi, >>>>> >>>>> this series adds minimal drivers for the Maxim Semiconductor MAX77714 >>>>> (https://www.maximintegrated.com/en/products/power/power-management-ics/MAX77714.html). >>>>> >>>>> Only RTC and watchdog are implemented by these patches. >>>>> >>>>> All implemented functionality is tested and working: RTC read/write, >>>>> watchdog start/stop/ping/set_timeout. >>>>> >>>>> Patches 1-3 + 6 are trivial cleanups to the max77686 drivers and >>>>> Kconfig >>>>> indentation and can probably be applied easily. >>>>> >>>>> Patches 4, 5, 7, 8 and 9 add: dt bindings, mfd driver, watchdog >>>>> driver and >>>>> rtc driver. >>>> >>>> A gentle ping about this series. It's at v5, all patches have at least >>>> one ack/review tag and most patches are unchanged since ~v2. It applies >>>> cleanly on current master. >>>> >>>> Is there anything I should do to help making progress? >>> >>> Apologies for pinging again... but as I got no further comments about >>> these patches I guess I can really do nothing at the moment. >>> >>> Lee, is this series completely in charge to you or should it be applied >>> by the respective subsystem maintainers? >>> >> >> I hesitated to take the watchdog patches because an earlier patch of the >> series >> introduces MFD_MAX77714 and the watchdog Kconfig entry lists it as >> dependency. >> I now added patch 7/9 and 8/9 to my watchdog-next tree anyway. If the >> mfd part >> doesn't make it we can still decide to take it out at some point. > > OK, thank you! In the meanwhile the kernel test robot also reported a > build failure due to missing max77714.h file, which is added by patch 5. > Yes, I noticed. Which means I'll have to drop the patch again, and we'll have to wait for the mfd patch to land, sorry. Guenter >> Note that patch 6/9 has already been applied. > > Indeed, it's in Linus' master already. > > Regards.