Hi Ulf, [...] >> + dev_warn(&pdev->dev, "conf-sdio3 register not found\n"); >> + dev_warn(&pdev->dev, "disabling SDR50 and DDR50 modes\n"); >> + dev_warn(&pdev->dev, "consider updating your dtb\n"); > > One dev_warn() should be enough. Also I don't think checkpatch > complains about long lines for dev_warn(). Right. Once you will have told if you sill want a change or not in patch 1, I will send a new version with this change. Thanks, Gregory -- Gregory Clement, Free Electrons Kernel, drivers, real-time and embedded Linux development, consulting, training and support. http://free-electrons.com -- 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