Linux ARM-MSM sub-architecture
 help / color / mirror / Atom feed
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

  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