Hi Michael,
On 4/19/22 7:26 PM, Michael Walle wrote:
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.anderson@seco.com
There is an upstream binding Documentation/devicetree/bindings/nvmem/fsl,layerscape-sfp.yaml
I see it will be added in 5.18, neat.
Are you aware of that?
I am now.
Could you submit a patch there to add your new compatibles?
Sure. I didn't know there was an upstream binding, but I'll try and add these properties.
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.
You need the clock to set the programming duration properly. See the probe function for this driver. For read-only access, it is not necessary.
--Sean