All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Duje Mihanović" <duje@dujemihanovic.xyz>
To: Ulf Hansson <ulf.hansson@linaro.org>
Cc: Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzk+dt@kernel.org>,
	Conor Dooley <conor+dt@kernel.org>,
	David Wronek <david@mainlining.org>,
	Karel Balej <balejk@matfyz.cz>,
	phone-devel@vger.kernel.org,
	~postmarketos/upstreaming@lists.sr.ht,
	linux-arm-kernel@lists.infradead.org, linux-clk@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-pm@vger.kernel.org
Subject: Re: [PATCH v2 3/4] clk: mmp: Add PXA1908 power domain driver
Date: Fri, 22 Aug 2025 22:55:02 +0200	[thread overview]
Message-ID: <6193521.lOV4Wx5bFT@radijator> (raw)
In-Reply-To: <CAPDyKFoHWNuSmnN0e=QR73r0Ea-XJogbB8S3K+_=VRovzXL2Sw@mail.gmail.com>

On Friday, 22 August 2025 11:36:02 Central European Summer Time Ulf Hansson 
wrote:
> By looking at the implementation of the power-domain code below, it
> seems to me that this code is better maintained within the pmdomain
> subsystem (drivers/pmdomain/pxa perhaps). May I suggest that you move
> it there.
> 
> I guess the easiest way to do this is to export the
> pxa1908_pd_register() function - but you could explore using the
> auxiliary bus too, to instantiate a power-domain driver as an
> auxiliary driver.

The auxiliary bus was exactly what I needed, thanks. As for the path, I think 
drivers/pmdomain/marvell would be a better fit and I'll go with that if you 
don't object.

Regards,
--
Duje




  reply	other threads:[~2025-08-23  8:20 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-21 11:17 [PATCH v2 0/4] Marvell PXA1908 power domains Duje Mihanović
2025-08-21 11:17 ` [PATCH v2 1/4] dt-bindings: clock: marvell,pxa1908: Add syscon compatible to apmu Duje Mihanović
2025-08-22 19:35   ` Rob Herring (Arm)
2025-08-21 11:17 ` [PATCH v2 2/4] dt-bindings: power: Add Marvell PXA1908 domains Duje Mihanović
2025-08-22 19:36   ` Rob Herring
2025-08-22 20:59     ` Duje Mihanović
2025-08-21 11:17 ` [PATCH v2 3/4] clk: mmp: Add PXA1908 power domain driver Duje Mihanović
2025-08-22  9:36   ` Ulf Hansson
2025-08-22 20:55     ` Duje Mihanović [this message]
2025-08-22 13:46   ` kernel test robot
2025-08-21 11:17 ` [PATCH v2 4/4] arm64: dts: marvell: pxa1908: Add power domains Duje Mihanović

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=6193521.lOV4Wx5bFT@radijator \
    --to=duje@dujemihanovic.xyz \
    --cc=balejk@matfyz.cz \
    --cc=conor+dt@kernel.org \
    --cc=david@mainlining.org \
    --cc=devicetree@vger.kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=phone-devel@vger.kernel.org \
    --cc=robh@kernel.org \
    --cc=sboyd@kernel.org \
    --cc=ulf.hansson@linaro.org \
    --cc=~postmarketos/upstreaming@lists.sr.ht \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.