From: Bryan O'Donoghue <bod@kernel.org>
To: Bryan O'Donoghue <bryan.odonoghue@linaro.org>,
Saikiran B <bjsaikiran@gmail.com>
Cc: linux-media@vger.kernel.org, linux-arm-msm@vger.kernel.org,
rfoss@kernel.org, todor.too@gmail.com,
vladimir.zapolskiy@linaro.org, Hans de Goede <hansg@kernel.org>,
sakari.ailus@linux.intel.com, mchehab@kernel.org,
stable@vger.kernel.org
Subject: Re: [PATCH v3 3/3] media: i2c: ov02c10: Use runtime PM autosuspend to avoid brownouts
Date: Tue, 27 Jan 2026 11:11:45 +0000 [thread overview]
Message-ID: <cb902c4e-f7b9-4d70-a997-1e84c90ea18b@kernel.org> (raw)
In-Reply-To: <571cd869-847f-4697-ace3-503f123e8486@linaro.org>
On 27/01/2026 11:06, Bryan O'Donoghue wrote:
> So, SPMI is not exported in /sys/kernel/debug/regmap - however
>
> drivers/regulator/qcom-rpmh-regulator.c
>
> Lets add this to probe
>
> unsigned int val, i;
> u16 bases[] = {0x4000, 0x4300, 0x4600}; // LDO1, LDO4, LDO7
> const char *names[] = {"LDO1(1.2V)", "LDO4(1.8V)", "LDO7(2.8V)"};
> struct regmap *p_regmap = dev_get_regmap(dev->parent, NULL);
>
> if (p_regmap) {
> pr_info("--- OV02C10 PMIC RAIL DUMP START ---\n");
> for (i = 0; i < 3; i++) {
> // Check Config (Active Discharge)
> regmap_read(p_regmap, bases[i] + 0x41, &val);
> pr_info("!!! %s SEC_CTRL (0x%04x) = 0x%02x (Bit7: Active
> Discharge)\n",
> names[i], bases[i] + 0x41, val);
>
> // Check Status (Is it actually on?)
> regmap_read(p_regmap, bases[i] + 0x08, &val);
> pr_info("!!! %s STATUS (0x%04x) = 0x%02x (Bit7: VREG_OK,
> Bit0: VREG_ON)\n",
> names[i], bases[i] + 0x08, val);
>
> // Check Pull-down config (Secondary check)
> regmap_read(p_regmap, bases[i] + 0x42, &val);
> pr_info("!!! %s PD_CTRL (0x%04x) = 0x%02x\n",
> names[i], bases[i] + 0x42, val);
> }
> pr_info("--- OV02C10 PMIC RAIL DUMP END ---\n");
> }
Obviously only do this for PM8010 for the other RPMh which may not have
this offset.
---
bod
next prev parent reply other threads:[~2026-01-27 11:11 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-26 17:34 [PATCH v3 0/3] media: i2c: ov02c10: Fix brownouts and power sequence Saikiran
2026-01-26 17:34 ` [PATCH v3 1/3] media: i2c: ov02c10: Fix use-after-free in remove function Saikiran
2026-01-27 10:30 ` Hans de Goede
2026-01-26 17:34 ` [PATCH v3 2/3] media: i2c: ov02c10: Correct power-on sequence and timing Saikiran
2026-01-27 10:40 ` Hans de Goede
2026-01-27 10:47 ` Bryan O'Donoghue
2026-01-27 10:50 ` Hans de Goede
2026-01-26 17:34 ` [PATCH v3 3/3] media: i2c: ov02c10: Use runtime PM autosuspend to avoid brownouts Saikiran
2026-01-27 9:46 ` Bryan O'Donoghue
2026-01-27 10:43 ` Hans de Goede
2026-01-27 10:44 ` Hans de Goede
[not found] ` <CAAFDt1tsyvtAa84bFK2Hq5yG_F15SUUseBd5Xi-DB8GnUj7+7A@mail.gmail.com>
2026-01-27 10:50 ` Bryan O'Donoghue
[not found] ` <CAAFDt1vKn5ssoTQZduGKb5eOeN74P=FVk9f01go1d-JS71Zt0A@mail.gmail.com>
2026-01-27 11:06 ` Bryan O'Donoghue
2026-01-27 11:11 ` Bryan O'Donoghue [this message]
2026-01-27 16:20 ` Saikiran B
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cb902c4e-f7b9-4d70-a997-1e84c90ea18b@kernel.org \
--to=bod@kernel.org \
--cc=bjsaikiran@gmail.com \
--cc=bryan.odonoghue@linaro.org \
--cc=hansg@kernel.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=rfoss@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=stable@vger.kernel.org \
--cc=todor.too@gmail.com \
--cc=vladimir.zapolskiy@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox