On Tue, 5 May 2020 at 14:20, Grygorii Strashko <grygorii.strashko@xxxxxx> wrote: > > Hi Anders, Hi Grygorii, > > On 05/05/2020 14:59, Anders Roxell wrote: > > On Tue, 5 May 2020 at 13:16, Anders Roxell <anders.roxell@xxxxxxxxxx> wrote: > >> On Tue, 5 May 2020 at 13:05, Grygorii Strashko <grygorii.strashko@xxxxxx> wrote: > >>> On 05/05/2020 13:17, Anders Roxell wrote: > >>>> On Fri, 1 May 2020 at 22:50, Grygorii Strashko <grygorii.strashko@xxxxxx> wrote: > >>>>> > >>>>> The MCU CPSW Common Platform Time Sync (CPTS) provides possibility to > >>>>> timestamp TX PTP packets and all RX packets. > >>>>> > >>>>> This enables corresponding support in TI AM65x/J721E MCU CPSW driver. > >>>>> > >>>>> Signed-off-by: Grygorii Strashko <grygorii.strashko@xxxxxx> > >>>>> --- > >>>>> drivers/net/ethernet/ti/Kconfig | 1 + > >>>>> drivers/net/ethernet/ti/am65-cpsw-ethtool.c | 24 ++- > >>>>> drivers/net/ethernet/ti/am65-cpsw-nuss.c | 172 ++++++++++++++++++++ > >>>>> drivers/net/ethernet/ti/am65-cpsw-nuss.h | 6 +- > >>>>> 4 files changed, 201 insertions(+), 2 deletions(-) > >>>>> > >>>>> diff --git a/drivers/net/ethernet/ti/Kconfig b/drivers/net/ethernet/ti/Kconfig > >>>>> index 1f4e5b6dc686..2c7bd1ccaaec 100644 > >>>>> --- a/drivers/net/ethernet/ti/Kconfig > >>>>> +++ b/drivers/net/ethernet/ti/Kconfig > >>>>> @@ -100,6 +100,7 @@ config TI_K3_AM65_CPSW_NUSS > >>>>> depends on ARCH_K3 && OF && TI_K3_UDMA_GLUE_LAYER > >>>>> select TI_DAVINCI_MDIO > >>>>> imply PHY_TI_GMII_SEL > >>>>> + imply TI_AM65_CPTS > >>>> > >>>> Should this be TI_K3_AM65_CPTS ? > > > > instead of 'imply TI_K3_AM65_CPTS' don't you want to do this: > > 'depends on TI_K3_AM65_CPTS || !TI_K3_AM65_CPTS' > > > > > > Right, I'll try. It seems your defconfig is produced by randconfig as > I can't get broken cfg TI_AM65_CPTS=m and TI_K3_AM65_CPSW_NUSS=y > with neither one below: > > make ARCH=arm64 O=k3-arm64 defconfig > make ARCH=arm64 O=k3-arm64 allnoconfig > make ARCH=arm64 O=k3-arm64 allyesconfig > make ARCH=arm64 O=k3-arm64 allmodconfig > make ARCH=arm64 O=k3-arm64 alldefconfig > make ARCH=arm64 O=k3-arm64 yes2modconfig > make ARCH=arm64 O=k3-arm64 mod2yesconfig I'm so sorry, I forgot to tell you that I do my allmodconfig like this: make ARCH=arm64 KCONFIG_ALLCONFIG=arch/arm64/configs/defconfig O=k3-arm64 allmodconfig Then I'm sure I should get a bootable kernel since that uses the defconfig as a base... Cheers, Anders > > Related legacy TI CPTS threads: > https://lkml.org/lkml/2020/5/2/344 > https://lkml.org/lkml/2020/5/1/1348 > > I'd try summarize goal > TI_K3_AM65_CPSW_NUSS TI_AM65_CPTS > Y Y/N > M Y/M/N > N Y/M/N > > > -- > Best regards, > grygorii