Hello Roger, On 13/01/23 15:18, Roger Quadros wrote: > Siddharth, > > On 11/01/2023 13:44, Siddharth Vadapalli wrote: >> The CPTS hardware doesn't support PPS signal generation. Using the GenFx >> (periodic signal generator) function, it is possible to model a PPS signal >> followed by routing it via the time sync router to the CPTS_HWy_TS_PUSH >> (hardware time stamp) input, in order to generate timestamps at 1 second >> intervals. >> >> This series adds driver support for enabling PPS signal generation. >> Additionally, the documentation for the am65-cpts driver is updated with >> the bindings for the "ti,pps" property, which is used to inform the >> pair [CPTS_HWy_TS_PUSH, GenFx] to the cpts driver. The PPS example is >> enabled for AM625-SK board by default, by adding the timesync_router node >> to the AM62x SoC, and configuring it for PPS in the AM625-SK board dts. >> >> Grygorii Strashko (3): >> dt-binding: net: ti: am65x-cpts: add 'ti,pps' property >> net: ethernet: ti: am65-cpts: add pps support >> net: ethernet: ti: am65-cpts: adjust pps following ptp changes >> >> Siddharth Vadapalli (2): >> arm64: dts: ti: k3-am62-main: Add timesync router node >> arm64: dts: ti: k3-am625-sk: Add cpsw3g cpts PPS support > > Device tree patches need to be sent separately. You don't need to involve > net maintainers for that. > > If you introduce a new binding then that needs to be in maintainer's > tree before you can send a related device tree patch. Thank you for letting me know. Would I need to resend the series in order for it to be reviewed? I was hoping that if I get feedback for this series, I will implement it and post just the bindings and driver patches as the v2 series, dropping the device tree patches. Please let me know. Regards, Siddharth.