> dev->synclko_125 = of_property_read_bool(dev->dev->of_node, > "microchip,synclko-125"); > + dev->synclko_disable = of_property_read_bool(dev->dev->of_node, > + "microchip,synclko-disable"); It seems like microchip,synclko-125 and microchip,synclko-disable are mutually exclusive? Please add an -EINVAL if both a present. This should also be mentioned in the binding document. Thanks Andrew
