Skip to content

Conversation

GustavSVJ
Copy link

I am using the STM32H7 along with Zephyr and a TJA1103 phy. The phy requires Clause 45 support which the STM32H7 has but it is not included in the hal layer and therefore not available in Zephyr.

I've tried opening an issue in the ST repo but it was denied due to no development boards requiring the functionoality. Another customer also got the same response on a PR a while ago.

So now I'm hoping for the next best solution...

ST PR
ST Issue

@erwango
Copy link
Member

erwango commented Oct 1, 2025

@GustavSVJ As you mentioned this fonctionality won't be implemented on H7 Cube for specific reasons.
So rather than updating HAL, can it be implemented directly within Zephyr driver ?
This change seems rather self contained.

@GustavSVJ
Copy link
Author

Dear @erwango,

Would you implement the full function in the zephyr driver or would you modify the specific bits and then call the existing "HAL_ETH_WritePHYRegister" read function?

Both solutions seem wrong to me. Re-implementing the full function is going in the opposite direction of the rest of the driver which relies on the ST HAL layer. Modifying the specific bits also seems dangerous as any changes to the ST HAL layer underneath could break the functionality in a way only showing up at runtime.

@erwango
Copy link
Member

erwango commented Oct 10, 2025

@GustavSVJ So this is a shame that your proposal is not accepted in HAL, but we don't plan to maintain a forked implementation of the HAL. Few integration fixes are allowed, temporary fixes as well waiting for official delivery from Cube, but we can't maintain an alternate implementation, which potentially broken at the next HAL release and which would have to be extended on other compatible series.
However, to complete the available HAL, we have a dedicated driver in which we can provide zephyr specific implementations. So if a developmenet to support this feature should be done, it should be done there.

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