On Wed, 9 Jan 2013, Russell King - ARM Linux wrote:
> On Wed, Jan 09, 2013 at 10:27:53AM -0500, Nicolas Pitre wrote:
> > Anyone with good coccinelle skills around to deal with the users?
>
> I'm not sure that's a solution.
Well, I was thinking that coccinelle could handle the majority of the
354 users when the "fix" is obvious enough to be automated.
That said, if we want people to fix their code, it is probably necessary
to merge your patch right away so the warnings are actually being seen,
and revert it right before the final v3.8 release if the remaining
warnings are still too numerous. Repeat with next cycle.
> For example:
>
> err = gpio_request(en_vdd_1v05, "EN_VDD_1V05");
> if (err) {
> pr_err("%s: gpio_request failed: %d\n", __func__, err);
> return err;
> }
> ...
> regulator = regulator_get(NULL, "vdd_ldo0,vddio_pex_clk");
> if (IS_ERR_OR_NULL(regulator)) {
> pr_err("%s: regulator_get failed: %d\n", __func__,
> (int)PTR_ERR(regulator));
> goto err_reg;
> }
>
> regulator_enable(regulator);
>
> err = tegra_pcie_init(true, true);
> ...
> err_reg:
> gpio_free(en_vdd_1v05);
>
> return err;
> }
>
> Now, regulator_get() returns error-pointers for real errors when it's
> configured in. When regulator support is not configured, it returns
> NULL.
>
> So, one solution here would be:
>
> if (IS_ERR(regulator)) {
> err = PTR_ERR(regulator);
> pr_err("%s: regulator_get failed: %d\n", __func__, err);
> goto err_reg;
> }
>
> but leaves us with the question: is it safe to call tegra_pcie_init()
> without regulator support?
The best approach is to assume it is not, but unlike the current code,
this should be fixed by returning an appropriate error code instead of 0.
Nicolas
--
To unsubscribe from this list: send the line "unsubscribe linux-arch" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
