On Tuesday 28 August 2012 12:40 PM, Stephen Warren wrote:
#include "board.h"
-#include "board-harmony.h"
#ifdef CONFIG_TEGRA_PCI
int __init harmony_pcie_init(void)
{
- struct regulator *regulator = NULL;
+ struct regulator *regulator_1v5, *regulator_pex;
int err;
- err = gpio_request(TEGRA_GPIO_EN_VDD_1V05_GPIO, "EN_VDD_1V05");
- if (err)
- return err;
+ regulator_1v5 = regulator_get(NULL, "vdd_1v5");
+ if (IS_ERR_OR_NULL(regulator_1v5)) {
+ err = PTR_ERR(regulator_1v5);
+ goto err_reg;
+ }
You need to return in case of err otherwise it will cause the crash in
regulator_disable() with NULL argument.
--
To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
