2014-08-13 5:55 GMT+08:00 Mark Brown <broo...@kernel.org>: > On Tue, Aug 12, 2014 at 09:37:09AM +0800, Axel Lin wrote: >> Fix below build error when !CONFIG_REGULATOR. >> >> CC drivers/gpu/drm/msm/hdmi/hdmi.o >> drivers/gpu/drm/msm/hdmi/hdmi.c: In function 'hdmi_init': >> drivers/gpu/drm/msm/hdmi/hdmi.c:126:3: error: implicit declaration of >> function 'devm_regulator_get_exclusive' >> [-Werror=implicit-function-declaration] > > So, I just looked at that driver - it errors out on probe without the > regualtor. Are you sure the fix isn't to add a dependency on the > regulator API or convert it back to using normal regulator gets (it was > converted without comment as part of a DT conversion), or indeed to > using the bulk regulator API. It looks awfully like the driver is just > open coding devm_regulator_bulk_get() and friends. > > There is no explanation at all in either the changelog or the code for > why the driver is asking for exclusive regulators. Hi Rob, Any feedback about Mark's comment?
Hi Mark, Even though not for this build fix, I think the patch is still good. Should I remove the build error message and resend the patch to you? Regards, Axel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/