Hi Sean,

> This adds an SFP binding for the processors it is present on. I have
> only tested this for the LS1046A.
> 
> Signed-off-by: Sean Anderson <sean.ander...@seco.com>

There is an upstream binding
Documentation/devicetree/bindings/nvmem/fsl,layerscape-sfp.yaml

Are you aware of that? Could you submit a patch there to add
your new compatibles? Esp. because you are using a clock which
isn't described in the upstream binding. I'm curious, do you need the
clock? I only know the LS1028A and it seems you don't have to turn on
any clock there. It might also be the case I've missed something.

-michael

Reply via email to