> + if (of_property_read_u16(np, "brcm,sd-head-align", &align) == 0) > + sdio->sd_head_align = align; Hi Alexey I think you can make this: of_property_read_u16(np, "brcm,sd-head-align", &sdio->sd_head_align); of_property_read_u16() should not touch the destination variable if the properties does not exist, or if there is an error. Andrew -- 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