On Tue, 22 Sep 2026 12:12:56 +0200 Christian Marangi (Ansuel) wrote:
The remark on the fwnode patch are invalid. The driver is expected
to return those error as probe defer is not supported.
(a notifier and late attach/detach is used to align to phylink logic)
For the specific Airoha PCS driver it's really speculation from the AI
but HW doesn't work as described by AI.
The only real comment that is valid is about the alternative way
to give supported interface with the .get_capabilites OP.
This is currently not supported as there isn't any user and it will
be implemented as soon as there will be one. It's really to limit
the patch delta of this change and to progress further.
Maybe FIXME might be suitable?
FWIW no preference here, I defer to the human reviewer.
We can treat the AI review as a hint if human reviewer thinks the code
is fine as is.