Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not working with Linux 4.14.84-v7+ #2

Closed
lhchang opened this issue Dec 5, 2018 · 6 comments
Closed

Not working with Linux 4.14.84-v7+ #2

lhchang opened this issue Dec 5, 2018 · 6 comments

Comments

@lhchang
Copy link

lhchang commented Dec 5, 2018

Hello,
I was not able to get PTP to work with the following guide, and was not sure if this issue was related to the previously closed issue. Upon running ethtool, I am not able to see SOF_TIMESTAMPING_TX_SOFTWARE and also I noticed ptp4l status is "failed to create a clock". I double checked my .conf files and made sure that the smsc95xx.c file was patched. I've attached the following images to reflect this. Thank you for the help!

conf1
conf2
ethtool
kernel
patch

@kervel
Copy link

kervel commented Jan 21, 2019

i guess this is due to the newer B+ model which has a lan78xx chip that doesn't advertise tx timestamping yet.

@kervel
Copy link

kervel commented Jan 21, 2019

i tried to make lan78xx report TX timestamping, and at least it does report it now with the patch below.
lan78xx_patch.zip

i tested it with two raspberries and they actually seem to converge but i need to check if i was just lucky.

@gizero
Copy link
Contributor

gizero commented Jan 22, 2019

Hi! Sorry for late reply but notifications for this issue got totally buried by noise :-(
While I haven't had a chance to test ptp on a RPi 3B+, which I don't have available, you are right saying it uses a different ethernet driver (lan78xx instead of smsc95xx) than model 3B. I'm not an expert in ethernet drivers at all, but as far as I can see, smsc95xx does implement software timestamping on both RX and TX paths as well as smsc95xx. What's missing once again is the capability to announce such functionality via ethtool. Your patch seems to be correct, being the counterpart of commit torvalds/linux@a8f5cb9 in mainline for smsc95xx. You'd better try to submit it to mainline for review. linux-raspberry is not accepting patches suitable for mainline and their policy is mainline first. That's the way to go to make it available sooner to RPi people. While at it, would you mind sending a PR to this repo as well with a comment on RPi3B+ need for this patch?

@kervel
Copy link

kervel commented Jan 22, 2019

hmm today i see these messages in the ptp log on another node

Jan 22 11:24:43 hostname ptp4l: [3276.669] port 1: received DELAY_REQ without timestamp
so i guess its not done yet (strange that i didn't see these messages yesterday)

@kervel
Copy link

kervel commented Feb 19, 2019

in the mean time i did some more tests and the log messages above did not prevent the time sync from being okay (eg time sync is definately working fine). i'll post a message to the linuxptp-users ml to ask if the warning means broken driver or not.

@gizero
Copy link
Contributor

gizero commented May 2, 2019

I'm inclined to close this issue because of your last comment. Before that, can you please link back to the above mentioned ml discussion if you managed to start it up?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants