From: Nicolas Boichat <drinkcat@chromium.org>
To: Mark Brown <broonie@kernel.org>
Cc: Rob Herring <robh+dt@kernel.org>, David Airlie <airlied@linux.ie>,
Daniel Vetter <daniel@ffwll.ch>,
Mark Rutland <mark.rutland@arm.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
Tomeu Vizoso <tomeu.vizoso@collabora.com>,
Steven Price <steven.price@arm.com>,
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>,
Liam Girdwood <lgirdwood@gmail.com>,
dri-devel <dri-devel@lists.freedesktop.org>,
Devicetree List <devicetree@vger.kernel.org>,
lkml <linux-kernel@vger.kernel.org>,
linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>,
"moderated list:ARM/Mediatek SoC support"
<linux-mediatek@lists.infradead.org>,
Hsin-Yi Wang <hsinyi@chromium.org>
Subject: Re: [PATCH v3 4/7] drm/panfrost: Add support for multiple regulators
Date: Tue, 21 Jan 2020 11:37:14 +0700 [thread overview]
Message-ID: <CANMq1KC_-g45wdGgGiBmEyVXAJMkKwsJBJXGBHOMJk_=NyfpYw@mail.gmail.com> (raw)
In-Reply-To: <20200114151643.GW3897@sirena.org.uk>
On Tue, Jan 14, 2020 at 10:16 PM Mark Brown <broonie@kernel.org> wrote:
>
> On Tue, Jan 14, 2020 at 03:15:59PM +0800, Nicolas Boichat wrote:
>
> > - I couldn't find a way to detect the number of regulators in the
> > device tree, if we wanted to refuse to probe the device if there
> > are too many regulators, which might be required for safety, see
> > the thread on v2 [1].
>
> You'd need to enumerate all the properties of the device and look
> for things matching *-supply.
I see ,-) I was hoping for something slightly cleaner, or maybe an
existing function in the core.
Steven: How strongly do you feel about this? If so I can add that
check in the next revision.
Also, just a heads-up, I'm out for the next 2 weeks, I'll send v4 after that.
>
> Reviewed-by: Mark Brown <broonie@kernel.org>
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Boichat <drinkcat@chromium.org>
To: Mark Brown <broonie@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
Devicetree List <devicetree@vger.kernel.org>,
Tomeu Vizoso <tomeu.vizoso@collabora.com>,
David Airlie <airlied@linux.ie>,
lkml <linux-kernel@vger.kernel.org>,
Liam Girdwood <lgirdwood@gmail.com>,
dri-devel <dri-devel@lists.freedesktop.org>,
Steven Price <steven.price@arm.com>,
Rob Herring <robh+dt@kernel.org>,
"moderated list:ARM/Mediatek SoC support"
<linux-mediatek@lists.infradead.org>,
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>,
Daniel Vetter <daniel@ffwll.ch>,
Hsin-Yi Wang <hsinyi@chromium.org>,
Matthias Brugger <matthias.bgg@gmail.com>,
linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v3 4/7] drm/panfrost: Add support for multiple regulators
Date: Tue, 21 Jan 2020 11:37:14 +0700 [thread overview]
Message-ID: <CANMq1KC_-g45wdGgGiBmEyVXAJMkKwsJBJXGBHOMJk_=NyfpYw@mail.gmail.com> (raw)
In-Reply-To: <20200114151643.GW3897@sirena.org.uk>
On Tue, Jan 14, 2020 at 10:16 PM Mark Brown <broonie@kernel.org> wrote:
>
> On Tue, Jan 14, 2020 at 03:15:59PM +0800, Nicolas Boichat wrote:
>
> > - I couldn't find a way to detect the number of regulators in the
> > device tree, if we wanted to refuse to probe the device if there
> > are too many regulators, which might be required for safety, see
> > the thread on v2 [1].
>
> You'd need to enumerate all the properties of the device and look
> for things matching *-supply.
I see ,-) I was hoping for something slightly cleaner, or maybe an
existing function in the core.
Steven: How strongly do you feel about this? If so I can add that
check in the next revision.
Also, just a heads-up, I'm out for the next 2 weeks, I'll send v4 after that.
>
> Reviewed-by: Mark Brown <broonie@kernel.org>
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Boichat <drinkcat@chromium.org>
To: Mark Brown <broonie@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
Devicetree List <devicetree@vger.kernel.org>,
Tomeu Vizoso <tomeu.vizoso@collabora.com>,
David Airlie <airlied@linux.ie>,
lkml <linux-kernel@vger.kernel.org>,
Liam Girdwood <lgirdwood@gmail.com>,
dri-devel <dri-devel@lists.freedesktop.org>,
Steven Price <steven.price@arm.com>,
Rob Herring <robh+dt@kernel.org>,
"moderated list:ARM/Mediatek SoC support"
<linux-mediatek@lists.infradead.org>,
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>,
Daniel Vetter <daniel@ffwll.ch>,
Hsin-Yi Wang <hsinyi@chromium.org>,
Matthias Brugger <matthias.bgg@gmail.com>,
linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v3 4/7] drm/panfrost: Add support for multiple regulators
Date: Tue, 21 Jan 2020 11:37:14 +0700 [thread overview]
Message-ID: <CANMq1KC_-g45wdGgGiBmEyVXAJMkKwsJBJXGBHOMJk_=NyfpYw@mail.gmail.com> (raw)
In-Reply-To: <20200114151643.GW3897@sirena.org.uk>
On Tue, Jan 14, 2020 at 10:16 PM Mark Brown <broonie@kernel.org> wrote:
>
> On Tue, Jan 14, 2020 at 03:15:59PM +0800, Nicolas Boichat wrote:
>
> > - I couldn't find a way to detect the number of regulators in the
> > device tree, if we wanted to refuse to probe the device if there
> > are too many regulators, which might be required for safety, see
> > the thread on v2 [1].
>
> You'd need to enumerate all the properties of the device and look
> for things matching *-supply.
I see ,-) I was hoping for something slightly cleaner, or maybe an
existing function in the core.
Steven: How strongly do you feel about this? If so I can add that
check in the next revision.
Also, just a heads-up, I'm out for the next 2 weeks, I'll send v4 after that.
>
> Reviewed-by: Mark Brown <broonie@kernel.org>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Nicolas Boichat <drinkcat@chromium.org>
To: Mark Brown <broonie@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>,
Devicetree List <devicetree@vger.kernel.org>,
Tomeu Vizoso <tomeu.vizoso@collabora.com>,
David Airlie <airlied@linux.ie>,
lkml <linux-kernel@vger.kernel.org>,
Liam Girdwood <lgirdwood@gmail.com>,
dri-devel <dri-devel@lists.freedesktop.org>,
Steven Price <steven.price@arm.com>,
Rob Herring <robh+dt@kernel.org>,
"moderated list:ARM/Mediatek SoC support"
<linux-mediatek@lists.infradead.org>,
Alyssa Rosenzweig <alyssa.rosenzweig@collabora.com>,
Hsin-Yi Wang <hsinyi@chromium.org>,
Matthias Brugger <matthias.bgg@gmail.com>,
linux-arm Mailing List <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v3 4/7] drm/panfrost: Add support for multiple regulators
Date: Tue, 21 Jan 2020 11:37:14 +0700 [thread overview]
Message-ID: <CANMq1KC_-g45wdGgGiBmEyVXAJMkKwsJBJXGBHOMJk_=NyfpYw@mail.gmail.com> (raw)
In-Reply-To: <20200114151643.GW3897@sirena.org.uk>
On Tue, Jan 14, 2020 at 10:16 PM Mark Brown <broonie@kernel.org> wrote:
>
> On Tue, Jan 14, 2020 at 03:15:59PM +0800, Nicolas Boichat wrote:
>
> > - I couldn't find a way to detect the number of regulators in the
> > device tree, if we wanted to refuse to probe the device if there
> > are too many regulators, which might be required for safety, see
> > the thread on v2 [1].
>
> You'd need to enumerate all the properties of the device and look
> for things matching *-supply.
I see ,-) I was hoping for something slightly cleaner, or maybe an
existing function in the core.
Steven: How strongly do you feel about this? If so I can add that
check in the next revision.
Also, just a heads-up, I'm out for the next 2 weeks, I'll send v4 after that.
>
> Reviewed-by: Mark Brown <broonie@kernel.org>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
next prev parent reply other threads:[~2020-01-21 4:37 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-01-14 7:15 [PATCH v3 0/7] Add dts for mt8183 GPU (and misc panfrost patches) Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` [PATCH v3 1/7] dt-bindings: gpu: mali-bifrost: Add Mediatek MT8183 Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` [PATCH v3 2/7] arm64: dts: mt8183: Add node for the Mali GPU Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` [PATCH v3 3/7] drm/panfrost: Improve error reporting in panfrost_gpu_power_on Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` [PATCH v3 4/7] drm/panfrost: Add support for multiple regulators Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 7:15 ` Nicolas Boichat
2020-01-14 15:16 ` Mark Brown
2020-01-14 15:16 ` Mark Brown
2020-01-14 15:16 ` Mark Brown
2020-01-14 15:16 ` Mark Brown
2020-01-21 4:37 ` Nicolas Boichat [this message]
2020-01-21 4:37 ` Nicolas Boichat
2020-01-21 4:37 ` Nicolas Boichat
2020-01-21 4:37 ` Nicolas Boichat
2020-01-22 13:40 ` Steven Price
2020-01-22 13:40 ` Steven Price
2020-01-22 13:40 ` Steven Price
2020-01-22 13:40 ` Steven Price
2020-01-20 14:43 ` Steven Price
2020-01-20 14:43 ` Steven Price
2020-01-20 14:43 ` Steven Price
2020-01-20 14:43 ` Steven Price
2020-01-20 17:03 ` Mark Brown
2020-01-20 17:03 ` Mark Brown
2020-01-20 17:03 ` Mark Brown
2020-01-20 17:03 ` Mark Brown
2020-01-20 17:09 ` Steven Price
2020-01-20 17:09 ` Steven Price
2020-01-20 17:09 ` Steven Price
2020-01-20 17:09 ` Steven Price
2020-01-14 7:16 ` [PATCH v3 5/7] drm/panfrost: Add support for multiple power domains Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
2020-01-20 14:53 ` Steven Price
2020-01-20 14:53 ` Steven Price
2020-01-20 14:53 ` Steven Price
2020-01-20 14:53 ` Steven Price
2020-02-07 3:06 ` Nicolas Boichat
2020-02-07 3:06 ` Nicolas Boichat
2020-02-07 3:06 ` Nicolas Boichat
2020-02-07 3:06 ` Nicolas Boichat
2020-01-14 7:16 ` [PATCH v3 6/7, RFC] drm/panfrost: Add mt8183-mali compatible string Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
2020-01-14 7:16 ` [PATCH v3 7/7, RFC] drm/panfrost: devfreq: Add support for 2 regulators Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
2020-01-14 7:16 ` Nicolas Boichat
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='CANMq1KC_-g45wdGgGiBmEyVXAJMkKwsJBJXGBHOMJk_=NyfpYw@mail.gmail.com' \
--to=drinkcat@chromium.org \
--cc=airlied@linux.ie \
--cc=alyssa.rosenzweig@collabora.com \
--cc=broonie@kernel.org \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=dri-devel@lists.freedesktop.org \
--cc=hsinyi@chromium.org \
--cc=lgirdwood@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=mark.rutland@arm.com \
--cc=matthias.bgg@gmail.com \
--cc=robh+dt@kernel.org \
--cc=steven.price@arm.com \
--cc=tomeu.vizoso@collabora.com \
/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.