change reset line transition

[chabala]

Resolves #51

After testing, it seems no timing change is needed, only the order of the transition needed to change.

My test code revises the earlier gist to remove all external reset logic, and use GP20 as the reset line.

https://gist.github.com/chabala/e8cc48355c03a34c209dba43b03a5526

Also, would it be worthwhile to have this example code in-project rather than in gists?

[FoamyGuy]

This change looks good and the rationale in the #51 makes sense to me.

I don't have the RP2040 Hat to test with. But I did do a quick check with the Ethernet Feather wing and the examples scripts in this repo, they all still run successfully and have no change in behavior as expected since the Ethernet Featherwing doesn't need to use reset pin during normal operation.

[FoamyGuy]

I do think it would be worthwhile to add an example for that device to this repo, just be sure there are comments that explain the specific hardware it's made to work with.

if you want to make a follow up PR to add that example inside of the exampels dir that would be awesome.

Thanks for looking into this and working on the fix @chabala

[anecdata]

Confirmed working on W5100S-EVB-Pico with the gist code.py. Also now using the change (and the correct pin) in my regular application. Thanks, @chabala!

[chabala]

Thank you both. @anecdata as you are the original author of the example code, would you like the opportunity to add it into the examples?