Lee, Doug, I've bisected a boot failure on Tegra Dalmore (which has a tps65090) down to this patch. It started in -next 0501, so I guess Lee might have pushed some patches out now even though the commit date is a little while back?
The commit is: commit 60e91b51b515b20f85697fcd397911fdb97bbdca Author: Doug Anderson <[email protected]> AuthorDate: Wed Apr 16 16:12:28 2014 -0700 Commit: Lee Jones <[email protected]> CommitDate: Wed Apr 23 12:34:01 2014 +0100 regulator: tps65090: Allow setting the overcurrent wait time The tps65090 regulator allows you to specify how long you want it to wait before detecting an overcurrent condition. Allow specifying that through the device tree (or through platform data). Signed-off-by: Doug Anderson <[email protected]> Acked-by: Simon Glass <[email protected]> Acked-by: Michael Spang <[email protected]> Acked-by: Sean Paul <[email protected]> Acked-by: Mark Brown <[email protected]> Signed-off-by: Lee Jones <[email protected]> Panic is line 309: 303 for (num = 0; num < TPS65090_REGULATOR_MAX; num++) { 304 tps_pdata = tps65090_pdata->reg_pdata[num]; 305 306 ri = &pmic[num]; 307 ri->dev = &pdev->dev; 308 ri->desc = &tps65090_regulator_desc[num]; 309 ri->overcurrent_wait_valid = tps_pdata->overcurrent_wait_valid; 310 ri->overcurrent_wait = tps_pdata->overcurrent_wait; so it looks like tps_pdata is NULL. Should likely be a check for it? -Olof On Wed, Apr 23, 2014 at 8:48 AM, Doug Anderson <[email protected]> wrote: > Lee, > > On Wed, Apr 23, 2014 at 4:51 AM, Lee Jones <[email protected]> wrote: >>> The tps65090 regulator allows you to specify how long you want it to >>> wait before detecting an overcurrent condition. Allow specifying that >>> through the device tree (or through platform data). >>> >>> Signed-off-by: Doug Anderson <[email protected]> >>> Signed-off-by: Simon Glass <[email protected]> >>> Signed-off-by: Michael Spang <[email protected]> >>> Signed-off-by: Sean Paul <[email protected]> >>> --- >>> Changes in v3: >>> - Fixed kernel-doc notation for return >>> >>> Changes in v2: >>> - Separated the overcurrent and retries changes into two patches. >>> - Now set overcurrent at probe time since it doesn't change. >>> >>> .../devicetree/bindings/regulator/tps65090.txt | 4 ++ >>> drivers/regulator/tps65090-regulator.c | 56 >>> ++++++++++++++++++++++ >>> include/linux/mfd/tps65090.h | 5 ++ >>> 3 files changed, 65 insertions(+) >> >> Applied, thanks. > > Ummmm, Mark said that he had already applied this patch to his tree (I > mentioned it in my recent summary and you can see it in this thread > too). I don't see it on git.kernel.org though > <https://git.kernel.org/cgit/linux/kernel/git/broonie/regulator.git/log/?h=for-next> > > I'm worried this will cause a merge conflict if you both apply it. > > -Doug -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
