>>>>> "Sagar" == Sagar Kadam <sagar.kadam@xxxxxxxxxxxx> writes: > Hello Peter, >> Are both affected by this issue? if not, we will need to extend the code >> to handle them differently. >> > No, this issue is present in FU540-C000 SoC i.e SoC- specific, and so I can check > for the soc-compatible string as follows: > - match = of_match_node(ocores_i2c_match, pdev->dev.of_node); > - if (match && (long)match->data == TYPE_SIFIVE_REV0) > + if (of_device_is_compatible(pdev->dev.of_node, > + "sifive,fu540-c000-i2c")) > Please let me know if this is okay. Yes, that sounds sensible. Thanks. -- Bye, Peter Korsgaard