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

PR #7148 w/ lwIP Ethernet merged #8332

Closed
wants to merge 17 commits into from

Conversation

dok-net
Copy link
Contributor

@dok-net dok-net commented Oct 9, 2021

This is for testing whether the delay_* metaphors proposed in PR #7148 also work as required for he LwIP Ethernet library from PR #8317.
@mcspr I'm posting this for you, if you should like to do some testing. I have done the same as to whether yieldUntil works for issue #7969, and I must report that sorry, no, it does not at all.

@dok-net dok-net changed the title WIP: delay0isyield # w/ lwIP Ethernet merged WIP: PR #7148 w/ lwIP Ethernet merged Oct 9, 2021
@dok-net dok-net marked this pull request as draft October 9, 2021 21:37
@dok-net dok-net changed the title WIP: PR #7148 w/ lwIP Ethernet merged PR #7148 w/ lwIP Ethernet merged Oct 9, 2021
@mcspr
Copy link
Collaborator

mcspr commented Oct 10, 2021

Thanks, but I already have 'git checkout'ed the required lib files straight from the #8317 branch

The only issue is - I can't seem to get the board w5500 to work with d1mini. Running the 'basic' esp-idf esp32(orig/s2/c3) example - it works, without a hitch. However, the lwip_ethernet lib just dumps that there's 'no hardware found' :/
Set up like it is suggested - SCLK <-> GPIO14, MOSI <-> GPIO13, MISO <-> GPIO12 and CS <-> GPIO5 (per the GPIO15 comment and b/c it is causing a boot mode change, but it also does not find the hardware with that pin anyway)

@dok-net
Copy link
Contributor Author

dok-net commented Oct 10, 2021

@mcspr Oh, I am sorry to hear that. I was obviously under the impression that pristine #8317 was running fine before. So I can only ask you to revisit my work once you have something to actually compare with. I will be happy to hear back from you once that is achieved. Thanks.

@mcspr
Copy link
Collaborator

mcspr commented Oct 13, 2021

just fyi, both PRs work, so no need for draft here

@mcspr mcspr closed this Oct 13, 2021
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

Successfully merging this pull request may close these issues.

2 participants