From: Guillaume Gardet <Guillaume.Gardet@arm.com> To: Krzysztof Kozlowski <krzk@kernel.org> Cc: "linux-samsung-soc@vger.kernel.org" <linux-samsung-soc@vger.kernel.org>, Kukjin Kim <kgene@kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: RE: [PATCH V2 1/2] arm: dts: exynos: Add GPU/Mali T604 node to exynos5250 Date: Wed, 31 Jul 2019 14:37:33 +0000 [thread overview] Message-ID: <DBBPR08MB4678D2A9A042269A8934E84183DF0@DBBPR08MB4678.eurprd08.prod.outlook.com> (raw) In-Reply-To: <CAJKOXPevC8GN=te7ub5Voq=dfn2HgJBbb=niVdD-gtWDUsq3Ow@mail.gmail.com> > -----Original Message----- > From: Krzysztof Kozlowski <krzk@kernel.org> > Sent: 26 July 2019 12:40 > To: Guillaume Gardet <Guillaume.Gardet@arm.com> > Cc: linux-samsung-soc@vger.kernel.org; Kukjin Kim <kgene@kernel.org>; > linux-arm-kernel@lists.infradead.org > Subject: Re: [PATCH V2 1/2] arm: dts: exynos: Add GPU/Mali T604 node to > exynos5250 > > On Thu, 25 Jul 2019 at 14:37, Guillaume Gardet <Guillaume.Gardet@arm.com> > wrote: > > > No, regulator is not provided. On all Exynos boards Mali uses its > > > own dedicated regulator (and sometimes additional one). It worked in > > > your case because the regulator was set as always-on but this is a > workaround for lack of consumers. > > > Once there is a consumer, the regulator should not be always-on to > > > save the energy when not used. > > > > I do not have the required info here. Do you have it, or could I skip the > mali-supply for now? > > You can guess from regulator name :) > For the Arndale, the schematics are saying it is buck4. Thanks. This is buck4 as well for snow. On Arndale board, buck4 has a fixed voltage of 1V contrary to snow, which leads to the following error: [ 21.131930] panfrost 11800000.gpu: Cannot set voltage 925000 uV The following change fixes the previous error: - regulator-min-microvolt = <1000000>; - regulator-max-microvolt = <1000000>; + regulator-min-microvolt = <850000>; + regulator-max-microvolt = <1300000>; Should I remove arndale enablement for now until this is fixed separately, or could I add this fixup in this patch serie, as it is related? Regards, Guillaume > > Best regards, > Krzysztof IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.
WARNING: multiple messages have this Message-ID (diff)
From: Guillaume Gardet <Guillaume.Gardet@arm.com> To: Krzysztof Kozlowski <krzk@kernel.org> Cc: "linux-samsung-soc@vger.kernel.org" <linux-samsung-soc@vger.kernel.org>, Kukjin Kim <kgene@kernel.org>, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: RE: [PATCH V2 1/2] arm: dts: exynos: Add GPU/Mali T604 node to exynos5250 Date: Wed, 31 Jul 2019 14:37:33 +0000 [thread overview] Message-ID: <DBBPR08MB4678D2A9A042269A8934E84183DF0@DBBPR08MB4678.eurprd08.prod.outlook.com> (raw) In-Reply-To: <CAJKOXPevC8GN=te7ub5Voq=dfn2HgJBbb=niVdD-gtWDUsq3Ow@mail.gmail.com> > -----Original Message----- > From: Krzysztof Kozlowski <krzk@kernel.org> > Sent: 26 July 2019 12:40 > To: Guillaume Gardet <Guillaume.Gardet@arm.com> > Cc: linux-samsung-soc@vger.kernel.org; Kukjin Kim <kgene@kernel.org>; > linux-arm-kernel@lists.infradead.org > Subject: Re: [PATCH V2 1/2] arm: dts: exynos: Add GPU/Mali T604 node to > exynos5250 > > On Thu, 25 Jul 2019 at 14:37, Guillaume Gardet <Guillaume.Gardet@arm.com> > wrote: > > > No, regulator is not provided. On all Exynos boards Mali uses its > > > own dedicated regulator (and sometimes additional one). It worked in > > > your case because the regulator was set as always-on but this is a > workaround for lack of consumers. > > > Once there is a consumer, the regulator should not be always-on to > > > save the energy when not used. > > > > I do not have the required info here. Do you have it, or could I skip the > mali-supply for now? > > You can guess from regulator name :) > For the Arndale, the schematics are saying it is buck4. Thanks. This is buck4 as well for snow. On Arndale board, buck4 has a fixed voltage of 1V contrary to snow, which leads to the following error: [ 21.131930] panfrost 11800000.gpu: Cannot set voltage 925000 uV The following change fixes the previous error: - regulator-min-microvolt = <1000000>; - regulator-max-microvolt = <1000000>; + regulator-min-microvolt = <850000>; + regulator-max-microvolt = <1300000>; Should I remove arndale enablement for now until this is fixed separately, or could I add this fixup in this patch serie, as it is related? Regards, Guillaume > > Best regards, > Krzysztof IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you. _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2019-07-31 14:37 UTC|newest] Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-07-24 7:20 [PATCH] arm: dts: exynos: Add GPU/Mali T604 node to exynos5250 Guillaume GARDET 2019-07-24 7:20 ` Guillaume GARDET 2019-07-24 19:43 ` Krzysztof Kozlowski 2019-07-24 19:43 ` Krzysztof Kozlowski 2019-07-25 8:34 ` [PATCH V2 1/2] " Guillaume 2019-07-25 8:34 ` Guillaume 2019-07-25 8:34 ` [PATCH V2 2/2] dt-bindings: gpu: mali-midgard: Add samsung exynos5250 compatible Guillaume 2019-07-25 8:34 ` Guillaume 2019-07-25 11:19 ` [PATCH V2 1/2] arm: dts: exynos: Add GPU/Mali T604 node to exynos5250 Krzysztof Kozlowski 2019-07-25 11:19 ` Krzysztof Kozlowski 2019-07-25 11:35 ` Guillaume Gardet 2019-07-25 11:35 ` Guillaume Gardet 2019-07-25 12:26 ` Krzysztof Kozlowski 2019-07-25 12:26 ` Krzysztof Kozlowski 2019-07-25 12:37 ` Guillaume Gardet 2019-07-25 12:37 ` Guillaume Gardet 2019-07-26 10:40 ` Krzysztof Kozlowski 2019-07-26 10:40 ` Krzysztof Kozlowski 2019-07-31 14:37 ` Guillaume Gardet [this message] 2019-07-31 14:37 ` Guillaume Gardet 2019-08-01 9:35 ` Marek Szyprowski 2019-08-01 9:35 ` Marek Szyprowski 2019-08-30 10:44 ` [PATCH V3 0/4] Add GPU/Mali T604 node to arndale and snow (exynos5250) Guillaume Gardet 2019-08-30 10:44 ` Guillaume Gardet 2019-08-30 10:44 ` [PATCH V3 1/5] dt-bindings: gpu: mali-midgard: Add samsung exynos5250 compatible Guillaume Gardet 2019-08-30 10:44 ` Guillaume Gardet 2019-09-02 15:31 ` Krzysztof Kozlowski 2019-09-02 15:31 ` Krzysztof Kozlowski 2019-09-03 6:56 ` Krzysztof Kozlowski 2019-09-03 6:56 ` Krzysztof Kozlowski 2019-09-03 7:02 ` Guillaume Gardet 2019-09-03 7:02 ` Guillaume Gardet 2019-09-03 7:04 ` Krzysztof Kozlowski 2019-09-03 7:04 ` Krzysztof Kozlowski 2019-09-03 7:33 ` [PATCH] " Guillaume Gardet 2019-09-03 7:33 ` Guillaume Gardet 2019-09-03 7:36 ` Krzysztof Kozlowski 2019-09-03 7:36 ` Krzysztof Kozlowski 2019-09-03 9:32 ` Rob Herring 2019-09-03 9:32 ` Rob Herring 2019-08-30 10:44 ` [PATCH V3 2/5] arm: dts: exynos: Add GPU/Mali T604 node to exynos5250 Guillaume Gardet 2019-08-30 10:44 ` Guillaume Gardet 2019-08-30 10:45 ` [PATCH V3 3/5] arm: dts: exynos: snow: Enable GPU/Mali T604 node Guillaume Gardet 2019-08-30 10:45 ` Guillaume Gardet 2019-08-30 10:45 ` [PATCH V3 4/5] arm: dts: exynos: arndale: " Guillaume Gardet 2019-08-30 10:45 ` Guillaume Gardet 2019-08-30 10:45 ` [PATCH V3 5/5] arm: dts: exynos: arndale: Fix min/max for buck4 regulator for gpu Guillaume Gardet 2019-08-30 10:45 ` Guillaume Gardet
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=DBBPR08MB4678D2A9A042269A8934E84183DF0@DBBPR08MB4678.eurprd08.prod.outlook.com \ --to=guillaume.gardet@arm.com \ --cc=kgene@kernel.org \ --cc=krzk@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-samsung-soc@vger.kernel.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: linkBe 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.