Re: [PATCH v2] net: ethernet: ti: Remove TI_CPTS_MOD workaround

2020-05-07 Thread Arnd Bergmann
On Wed, May 6, 2020 at 10:57 PM Grygorii Strashko wrote: > On 06/05/2020 09:51, Clay McClure wrote: > > On Tue, May 05, 2020 at 10:41:26AM +0300, Grygorii Strashko wrote: > > > > Ok. After some thinking and hence you commit b6d49cab44b5 ("net: Make > PTP-specific drivers depend on PTP_1588_CLOCK

Re: [PATCH v2] net: ethernet: ti: Remove TI_CPTS_MOD workaround

2020-05-06 Thread Grygorii Strashko
On 06/05/2020 09:51, Clay McClure wrote: On Tue, May 05, 2020 at 10:41:26AM +0300, Grygorii Strashko wrote: It's better if you send v2 not as reply to v1. Noted, thank you, and thank you for taking the time to review my patch. just to clarify. After these two patches - the PTP_1588_CLO

Re: [PATCH v2] net: ethernet: ti: Remove TI_CPTS_MOD workaround

2020-05-05 Thread Clay McClure
On Tue, May 05, 2020 at 10:41:26AM +0300, Grygorii Strashko wrote: > It's better if you send v2 not as reply to v1. Noted, thank you, and thank you for taking the time to review my patch. > just to clarify. After these two patches > - the PTP_1588_CLOCK can still be set to "M" > - which will c

Re: [PATCH v2] net: ethernet: ti: Remove TI_CPTS_MOD workaround

2020-05-05 Thread Grygorii Strashko
On 04/05/2020 19:57, Clay McClure wrote: My recent commit b6d49cab44b5 ("net: Make PTP-specific drivers depend on PTP_1588_CLOCK") exposes a missing dependency in defconfigs that select TI_CPTS without selecting PTP_1588_CLOCK, leading to linker errors of the form: drivers/net/ethernet/ti/cp

[PATCH v2] net: ethernet: ti: Remove TI_CPTS_MOD workaround

2020-05-04 Thread Clay McClure
My recent commit b6d49cab44b5 ("net: Make PTP-specific drivers depend on PTP_1588_CLOCK") exposes a missing dependency in defconfigs that select TI_CPTS without selecting PTP_1588_CLOCK, leading to linker errors of the form: drivers/net/ethernet/ti/cpsw.o: in function `cpsw_ndo_stop': cpsw.c:(.te