Hi Jonathan, > Jonathan Cameron <jic23@xxxxxxxxxx> hat am 26. Dezember 2014 um 10:23 > geschrieben: > > [...] > > > > - lradc->over_sample_delay = 2; > > - ret = of_property_read_u32(lradc_node, "fsl,ave-delay", &adapt); > > - if (ret == 0) > > + if (of_property_read_u32(lradc_node, "fsl,ave-delay", &adapt)) { > > + lradc->over_sample_delay = 2; > > + } else { > > + if (adapt < 2 || adapt > LRADC_DELAY_DELAY_MASK+1) { > please run checkpatch.pl over these. Should be spaces around the + i'm afraid my checkpatch.pl doesn't find this issue. I'll fix it in the next version. Stefan -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html