* [PATCH v3 0/3] arm64: dts: sdm845: Add sdm845 GPU interconnect @ 2018-12-20 17:30 Jordan Crouse 2018-12-20 17:30 ` [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path Jordan Crouse ` (2 more replies) 0 siblings, 3 replies; 14+ messages in thread From: Jordan Crouse @ 2018-12-20 17:30 UTC (permalink / raw) To: freedreno Cc: linux-arm-msm, dianders, georgi.djakov, Bjorn Andersson, Arnd Bergmann, Stephen Boyd, Kees Cook, linux-soc, Sharat Masetty, dri-devel, linux-kernel, devicetree, Andy Gross, Rob Clark, David Airlie, Johan Hovold, Rob Herring, David Brown, Mark Rutland, Colin Ian King Add interconnect consumer definitions for the sdm845 GPU to device tree. This patchset includes the a6xx gpu support for interconnect as previously seen here: https://patchwork.kernel.org/patch/10718587/ It was pulled into this patchset because the interconnect-name was removed from the DT at the suggestion of Rob Herring and it was easier to combine all of these together into one set. Everything is based on the base GPU DT changes: https://patchwork.freedesktop.org/series/39308/ As well as the DT nodes from Georgi: https://patchwork.kernel.org/patch/10719483/ v3: Remove hardcoded interconnect name from driver and bindings Jordan Crouse (3): drm/msm/a6xx: Add support for an interconnect path dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU arm64: dts: sdm845: Add interconnect for GPU .../devicetree/bindings/display/msm/gpu.txt | 4 ++++ arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 ++ drivers/gpu/drm/msm/Kconfig | 1 + drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 20 +++++++++++++++++++ drivers/gpu/drm/msm/adreno/adreno_gpu.c | 9 +++++++++ drivers/gpu/drm/msm/msm_gpu.h | 3 +++ 6 files changed, 39 insertions(+) -- 2.18.0 ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path 2018-12-20 17:30 [PATCH v3 0/3] arm64: dts: sdm845: Add sdm845 GPU interconnect Jordan Crouse @ 2018-12-20 17:30 ` Jordan Crouse 2018-12-20 19:28 ` Doug Anderson 2019-01-18 18:06 ` Doug Anderson 2018-12-20 17:30 ` [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU Jordan Crouse 2018-12-20 17:30 ` [PATCH v3 3/3] arm64: dts: sdm845: Add interconnect " Jordan Crouse 2 siblings, 2 replies; 14+ messages in thread From: Jordan Crouse @ 2018-12-20 17:30 UTC (permalink / raw) To: freedreno Cc: linux-arm-msm, dianders, georgi.djakov, Bjorn Andersson, Arnd Bergmann, Stephen Boyd, Kees Cook, Sharat Masetty, dri-devel, linux-kernel, Andy Gross, Rob Clark, David Airlie, Johan Hovold, Colin Ian King Try to get the interconnect path for the GPU and vote for the maximum bandwidth to support all frequencies. This is needed for performance. Later we will want to scale the bandwidth based on the frequency to also optimize for power but that will require some device tree infrastructure that does not yet exist. v5: Remove hardcoded interconnect name and just use the default v4: Don't use a port string at all to skip the need for names in the DT v3: Use macros and change port string per Georgi Djakov Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org> --- drivers/gpu/drm/msm/Kconfig | 1 + drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 20 ++++++++++++++++++++ drivers/gpu/drm/msm/adreno/adreno_gpu.c | 9 +++++++++ drivers/gpu/drm/msm/msm_gpu.h | 3 +++ 4 files changed, 33 insertions(+) diff --git a/drivers/gpu/drm/msm/Kconfig b/drivers/gpu/drm/msm/Kconfig index 843a9d40c05e..990c4350f0c4 100644 --- a/drivers/gpu/drm/msm/Kconfig +++ b/drivers/gpu/drm/msm/Kconfig @@ -5,6 +5,7 @@ config DRM_MSM depends on ARCH_QCOM || (ARM && COMPILE_TEST) depends on OF && COMMON_CLK depends on MMU + depends on INTERCONNECT || !INTERCONNECT select QCOM_MDT_LOADER if ARCH_QCOM select REGULATOR select DRM_KMS_HELPER diff --git a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c b/drivers/gpu/drm/msm/adreno/a6xx_gmu.c index 0fb4718ef0df..781b601c6045 100644 --- a/drivers/gpu/drm/msm/adreno/a6xx_gmu.c +++ b/drivers/gpu/drm/msm/adreno/a6xx_gmu.c @@ -2,6 +2,7 @@ /* Copyright (c) 2017-2018 The Linux Foundation. All rights reserved. */ #include <linux/clk.h> +#include <linux/interconnect.h> #include <linux/pm_opp.h> #include <soc/qcom/cmd-db.h> @@ -63,6 +64,9 @@ static bool a6xx_gmu_gx_is_on(struct a6xx_gmu *gmu) static void __a6xx_gmu_set_freq(struct a6xx_gmu *gmu, int index) { + struct a6xx_gpu *a6xx_gpu = container_of(gmu, struct a6xx_gpu, gmu); + struct adreno_gpu *adreno_gpu = &a6xx_gpu->base; + struct msm_gpu *gpu = &adreno_gpu->base; int ret; gmu_write(gmu, REG_A6XX_GMU_DCVS_ACK_OPTION, 0); @@ -85,6 +89,12 @@ static void __a6xx_gmu_set_freq(struct a6xx_gmu *gmu, int index) dev_err(gmu->dev, "GMU set GPU frequency error: %d\n", ret); gmu->freq = gmu->gpu_freqs[index]; + + /* + * Eventually we will want to scale the path vote with the frequency but + * for now leave it at max so that the performance is nominal. + */ + icc_set(gpu->icc_path, 0, MBps_to_icc(7216)); } void a6xx_gmu_set_freq(struct msm_gpu *gpu, unsigned long freq) @@ -680,6 +690,8 @@ int a6xx_gmu_reset(struct a6xx_gpu *a6xx_gpu) int a6xx_gmu_resume(struct a6xx_gpu *a6xx_gpu) { + struct adreno_gpu *adreno_gpu = &a6xx_gpu->base; + struct msm_gpu *gpu = &adreno_gpu->base; struct a6xx_gmu *gmu = &a6xx_gpu->gmu; int status, ret; @@ -695,6 +707,9 @@ int a6xx_gmu_resume(struct a6xx_gpu *a6xx_gpu) if (ret) goto out; + /* Set the bus quota to a reasonable value for boot */ + icc_set(gpu->icc_path, 0, MBps_to_icc(3072)); + a6xx_gmu_irq_enable(gmu); /* Check to see if we are doing a cold or warm boot */ @@ -735,6 +750,8 @@ bool a6xx_gmu_isidle(struct a6xx_gmu *gmu) int a6xx_gmu_stop(struct a6xx_gpu *a6xx_gpu) { + struct adreno_gpu *adreno_gpu = &a6xx_gpu->base; + struct msm_gpu *gpu = &adreno_gpu->base; struct a6xx_gmu *gmu = &a6xx_gpu->gmu; u32 val; @@ -781,6 +798,9 @@ int a6xx_gmu_stop(struct a6xx_gpu *a6xx_gpu) /* Tell RPMh to power off the GPU */ a6xx_rpmh_stop(gmu); + /* Remove the bus vote */ + icc_set(gpu->icc_path, 0, 0); + clk_bulk_disable_unprepare(gmu->nr_clocks, gmu->clocks); pm_runtime_put_sync(gmu->dev); diff --git a/drivers/gpu/drm/msm/adreno/adreno_gpu.c b/drivers/gpu/drm/msm/adreno/adreno_gpu.c index bfeea50fca8a..6629dc3506eb 100644 --- a/drivers/gpu/drm/msm/adreno/adreno_gpu.c +++ b/drivers/gpu/drm/msm/adreno/adreno_gpu.c @@ -18,6 +18,7 @@ */ #include <linux/ascii85.h> +#include <linux/interconnect.h> #include <linux/kernel.h> #include <linux/pm_opp.h> #include <linux/slab.h> @@ -695,6 +696,11 @@ static int adreno_get_pwrlevels(struct device *dev, DBG("fast_rate=%u, slow_rate=27000000", gpu->fast_rate); + /* Check for an interconnect path for the bus */ + gpu->icc_path = of_icc_get(dev, NULL); + if (IS_ERR(gpu->icc_path)) + gpu->icc_path = NULL; + return 0; } @@ -732,10 +738,13 @@ int adreno_gpu_init(struct drm_device *drm, struct platform_device *pdev, void adreno_gpu_cleanup(struct adreno_gpu *adreno_gpu) { + struct msm_gpu *gpu = &adreno_gpu->base; unsigned int i; for (i = 0; i < ARRAY_SIZE(adreno_gpu->info->fw); i++) release_firmware(adreno_gpu->fw[i]); + icc_put(gpu->icc_path); + msm_gpu_cleanup(&adreno_gpu->base); } diff --git a/drivers/gpu/drm/msm/msm_gpu.h b/drivers/gpu/drm/msm/msm_gpu.h index fc4040e24a6b..66e0f28dfed8 100644 --- a/drivers/gpu/drm/msm/msm_gpu.h +++ b/drivers/gpu/drm/msm/msm_gpu.h @@ -19,6 +19,7 @@ #define __MSM_GPU_H__ #include <linux/clk.h> +#include <linux/interconnect.h> #include <linux/regulator/consumer.h> #include "msm_drv.h" @@ -118,6 +119,8 @@ struct msm_gpu { struct clk *ebi1_clk, *core_clk, *rbbmtimer_clk; uint32_t fast_rate; + struct icc_path *icc_path; + /* Hang and Inactivity Detection: */ #define DRM_MSM_INACTIVE_PERIOD 66 /* in ms (roughly four frames) */ -- 2.18.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path 2018-12-20 17:30 ` [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path Jordan Crouse @ 2018-12-20 19:28 ` Doug Anderson 2019-01-18 18:06 ` Doug Anderson 1 sibling, 0 replies; 14+ messages in thread From: Doug Anderson @ 2018-12-20 19:28 UTC (permalink / raw) To: Jordan Crouse, Georgi Djakov, Rob Clark Cc: freedreno, linux-arm-msm, Bjorn Andersson, Arnd Bergmann, Stephen Boyd, Kees Cook, Sharat Masetty, dri-devel, LKML, Andy Gross, David Airlie, Johan Hovold, Colin Ian King Hi, On Thu, Dec 20, 2018 at 9:30 AM Jordan Crouse <jcrouse@codeaurora.org> wrote: > > Try to get the interconnect path for the GPU and vote for the maximum > bandwidth to support all frequencies. This is needed for performance. > Later we will want to scale the bandwidth based on the frequency to > also optimize for power but that will require some device tree > infrastructure that does not yet exist. > > v5: Remove hardcoded interconnect name and just use the default > v4: Don't use a port string at all to skip the need for names in the DT > v3: Use macros and change port string per Georgi Djakov > > Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org> > --- > > drivers/gpu/drm/msm/Kconfig | 1 + > drivers/gpu/drm/msm/adreno/a6xx_gmu.c | 20 ++++++++++++++++++++ > drivers/gpu/drm/msm/adreno/adreno_gpu.c | 9 +++++++++ > drivers/gpu/drm/msm/msm_gpu.h | 3 +++ > 4 files changed, 33 insertions(+) There is very little difference between this an the previous version [1]. Maybe could have kept Rob Clark's Acked-by? The only change was: - gpu->icc_path = of_icc_get(dev, "gfx-mem"); + gpu->icc_path = of_icc_get(dev, NULL); Also: I assume that this is still intended to go through Georgi's tree? [1] https://lkml.kernel.org/r/20181207170656.13208-1-jcrouse@codeaurora.org -Doug ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path 2018-12-20 17:30 ` [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path Jordan Crouse 2018-12-20 19:28 ` Doug Anderson @ 2019-01-18 18:06 ` Doug Anderson 2019-01-18 19:16 ` Rob Clark 2019-01-18 21:54 ` Doug Anderson 1 sibling, 2 replies; 14+ messages in thread From: Doug Anderson @ 2019-01-18 18:06 UTC (permalink / raw) To: Jordan Crouse, Georgi Djakov, Rob Clark Cc: freedreno, linux-arm-msm, Bjorn Andersson, Arnd Bergmann, Stephen Boyd, Kees Cook, Sharat Masetty, dri-devel, LKML, Andy Gross, David Airlie, Johan Hovold, Colin Ian King, Evan Green, Sean Paul Hi, On Thu, Dec 20, 2018 at 9:30 AM Jordan Crouse <jcrouse@codeaurora.org> wrote: > > Try to get the interconnect path for the GPU and vote for the maximum > bandwidth to support all frequencies. This is needed for performance. > Later we will want to scale the bandwidth based on the frequency to > also optimize for power but that will require some device tree > infrastructure that does not yet exist. > > v5: Remove hardcoded interconnect name and just use the default nit: ${SUBJECT} says v3, but this is v5. I'll put in my usual plug for considering "patman" to help post patches. Even though it lives in the u-boot git repo it's still a gem for kernel work. <http://git.denx.de/?p=u-boot.git;a=blob;f=tools/patman/README> > @@ -85,6 +89,12 @@ static void __a6xx_gmu_set_freq(struct a6xx_gmu *gmu, int index) > dev_err(gmu->dev, "GMU set GPU frequency error: %d\n", ret); > > gmu->freq = gmu->gpu_freqs[index]; > + > + /* > + * Eventually we will want to scale the path vote with the frequency but > + * for now leave it at max so that the performance is nominal. > + */ > + icc_set(gpu->icc_path, 0, MBps_to_icc(7216)); You'll need to change icc_set() here to icc_set_bw() to match v13, AKA: - https://patchwork.kernel.org/patch/10766335/ - https://lkml.kernel.org/r/20190116161103.6937-2-georgi.djakov@linaro.org > @@ -695,6 +707,9 @@ int a6xx_gmu_resume(struct a6xx_gpu *a6xx_gpu) > if (ret) > goto out; > > + /* Set the bus quota to a reasonable value for boot */ > + icc_set(gpu->icc_path, 0, MBps_to_icc(3072)); This will also need to change to icc_set_bw() > @@ -781,6 +798,9 @@ int a6xx_gmu_stop(struct a6xx_gpu *a6xx_gpu) > /* Tell RPMh to power off the GPU */ > a6xx_rpmh_stop(gmu); > > + /* Remove the bus vote */ > + icc_set(gpu->icc_path, 0, 0); This will also need to change to icc_set_bw() I have the same questions for this series that I had in response to the email ("[v5 2/3] drm/msm/dpu: Integrate interconnect API in MDSS") <https://lkml.kernel.org/r/CAD=FV=XUeMTGH+CDwGs3PfK4igdQrCbwucw7_2ViBc4i7grvxg@mail.gmail.com> Copy / pasting here (with minor name changes) so folks don't have to follow links / search email. == I'm curious what the plan is for landing this series. Rob / Gerogi: do you have any preference? Options I'd imagine: A) Wait until interconnect lands (in 5.1?) and land this through msm-next in the version after (5.2?) B) Georgi provides an immutable branch for interconnect when his lands (assuming he's landing via pull request) and that gets pulled into the the relevant drm tree. C) Rob Acks this series and indicates that it should go in through Gerogi's tree (probably only works if Georgi plans to send a pull request). If we're going this route then (IIUC) we'd want to land this in Gerogi's tree sooner rather than later so it can get some bake time? NOTE: as per my prior reply, I believe Rob has already Acked this patch. Does anyone have a preference? It's be nice if whoever is planning to land this could indicate whether they'd prefer Jordan send a new version to handle the API change or if the relevant maintainer can just do the fixup when the patch lands. Thanks! -Doug ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path 2019-01-18 18:06 ` Doug Anderson @ 2019-01-18 19:16 ` Rob Clark 2019-01-21 17:13 ` Georgi Djakov 2019-01-18 21:54 ` Doug Anderson 1 sibling, 1 reply; 14+ messages in thread From: Rob Clark @ 2019-01-18 19:16 UTC (permalink / raw) To: Doug Anderson Cc: Jordan Crouse, Georgi Djakov, freedreno, linux-arm-msm, Bjorn Andersson, Arnd Bergmann, Stephen Boyd, Kees Cook, Sharat Masetty, dri-devel, LKML, Andy Gross, David Airlie, Johan Hovold, Colin Ian King, Evan Green, Sean Paul On Fri, Jan 18, 2019 at 1:06 PM Doug Anderson <dianders@chromium.org> wrote: > > Hi, > > On Thu, Dec 20, 2018 at 9:30 AM Jordan Crouse <jcrouse@codeaurora.org> wrote: > > > > Try to get the interconnect path for the GPU and vote for the maximum > > bandwidth to support all frequencies. This is needed for performance. > > Later we will want to scale the bandwidth based on the frequency to > > also optimize for power but that will require some device tree > > infrastructure that does not yet exist. > > > > v5: Remove hardcoded interconnect name and just use the default > > nit: ${SUBJECT} says v3, but this is v5. > > I'll put in my usual plug for considering "patman" to help post > patches. Even though it lives in the u-boot git repo it's still a gem > for kernel work. > <http://git.denx.de/?p=u-boot.git;a=blob;f=tools/patman/README> > > > > @@ -85,6 +89,12 @@ static void __a6xx_gmu_set_freq(struct a6xx_gmu *gmu, int index) > > dev_err(gmu->dev, "GMU set GPU frequency error: %d\n", ret); > > > > gmu->freq = gmu->gpu_freqs[index]; > > + > > + /* > > + * Eventually we will want to scale the path vote with the frequency but > > + * for now leave it at max so that the performance is nominal. > > + */ > > + icc_set(gpu->icc_path, 0, MBps_to_icc(7216)); > > You'll need to change icc_set() here to icc_set_bw() to match v13, AKA: > > - https://patchwork.kernel.org/patch/10766335/ > - https://lkml.kernel.org/r/20190116161103.6937-2-georgi.djakov@linaro.org > > > > @@ -695,6 +707,9 @@ int a6xx_gmu_resume(struct a6xx_gpu *a6xx_gpu) > > if (ret) > > goto out; > > > > + /* Set the bus quota to a reasonable value for boot */ > > + icc_set(gpu->icc_path, 0, MBps_to_icc(3072)); > > This will also need to change to icc_set_bw() > > > > @@ -781,6 +798,9 @@ int a6xx_gmu_stop(struct a6xx_gpu *a6xx_gpu) > > /* Tell RPMh to power off the GPU */ > > a6xx_rpmh_stop(gmu); > > > > + /* Remove the bus vote */ > > + icc_set(gpu->icc_path, 0, 0); > > This will also need to change to icc_set_bw() > > > I have the same questions for this series that I had in response to > the email ("[v5 2/3] drm/msm/dpu: Integrate interconnect API in MDSS") > <https://lkml.kernel.org/r/CAD=FV=XUeMTGH+CDwGs3PfK4igdQrCbwucw7_2ViBc4i7grvxg@mail.gmail.com> > > > Copy / pasting here (with minor name changes) so folks don't have to > follow links / search email. > > == > > I'm curious what the plan is for landing this series. Rob / Gerogi: > do you have any preference? Options I'd imagine: > > A) Wait until interconnect lands (in 5.1?) and land this through > msm-next in the version after (5.2?) > > B) Georgi provides an immutable branch for interconnect when his lands > (assuming he's landing via pull request) and that gets pulled into the > the relevant drm tree. > > C) Rob Acks this series and indicates that it should go in through > Gerogi's tree (probably only works if Georgi plans to send a pull > request). If we're going this route then (IIUC) we'd want to land > this in Gerogi's tree sooner rather than later so it can get some bake > time? NOTE: as per my prior reply, I believe Rob has already Acked > this patch. > I'm ok to ack and have it land via Georgi's tree, if Georgi wants to do that. Or otherwise, I could maybe coordinate w/ airlied to send a 2nd late msm-next pr including the gpu and display interconnect patches. BR, -R > > Does anyone have a preference? It's be nice if whoever is planning to > land this could indicate whether they'd prefer Jordan send a new > version to handle the API change or if the relevant maintainer can > just do the fixup when the patch lands. > > > Thanks! > > > -Doug ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path 2019-01-18 19:16 ` Rob Clark @ 2019-01-21 17:13 ` Georgi Djakov 2019-01-22 21:17 ` Doug Anderson 0 siblings, 1 reply; 14+ messages in thread From: Georgi Djakov @ 2019-01-21 17:13 UTC (permalink / raw) To: Rob Clark, Doug Anderson Cc: Jordan Crouse, freedreno, linux-arm-msm, Bjorn Andersson, Arnd Bergmann, Stephen Boyd, Kees Cook, Sharat Masetty, dri-devel, LKML, Andy Gross, David Airlie, Johan Hovold, Colin Ian King, Evan Green, Sean Paul Hi Rob, On 1/18/19 21:16, Rob Clark wrote: > On Fri, Jan 18, 2019 at 1:06 PM Doug Anderson <dianders@chromium.org> wrote: >> >> Hi, >> >> On Thu, Dec 20, 2018 at 9:30 AM Jordan Crouse <jcrouse@codeaurora.org> wrote: >>> >>> Try to get the interconnect path for the GPU and vote for the maximum >>> bandwidth to support all frequencies. This is needed for performance. >>> Later we will want to scale the bandwidth based on the frequency to >>> also optimize for power but that will require some device tree >>> infrastructure that does not yet exist. >>> >>> v5: Remove hardcoded interconnect name and just use the default >> >> nit: ${SUBJECT} says v3, but this is v5. >> >> I'll put in my usual plug for considering "patman" to help post >> patches. Even though it lives in the u-boot git repo it's still a gem >> for kernel work. >> <http://git.denx.de/?p=u-boot.git;a=blob;f=tools/patman/README> >> >> >>> @@ -85,6 +89,12 @@ static void __a6xx_gmu_set_freq(struct a6xx_gmu *gmu, int index) >>> dev_err(gmu->dev, "GMU set GPU frequency error: %d\n", ret); >>> >>> gmu->freq = gmu->gpu_freqs[index]; >>> + >>> + /* >>> + * Eventually we will want to scale the path vote with the frequency but >>> + * for now leave it at max so that the performance is nominal. >>> + */ >>> + icc_set(gpu->icc_path, 0, MBps_to_icc(7216)); >> >> You'll need to change icc_set() here to icc_set_bw() to match v13, AKA: >> >> - https://patchwork.kernel.org/patch/10766335/ >> - https://lkml.kernel.org/r/20190116161103.6937-2-georgi.djakov@linaro.org >> >> >>> @@ -695,6 +707,9 @@ int a6xx_gmu_resume(struct a6xx_gpu *a6xx_gpu) >>> if (ret) >>> goto out; >>> >>> + /* Set the bus quota to a reasonable value for boot */ >>> + icc_set(gpu->icc_path, 0, MBps_to_icc(3072)); >> >> This will also need to change to icc_set_bw() >> >> >>> @@ -781,6 +798,9 @@ int a6xx_gmu_stop(struct a6xx_gpu *a6xx_gpu) >>> /* Tell RPMh to power off the GPU */ >>> a6xx_rpmh_stop(gmu); >>> >>> + /* Remove the bus vote */ >>> + icc_set(gpu->icc_path, 0, 0); >> >> This will also need to change to icc_set_bw() >> >> >> I have the same questions for this series that I had in response to >> the email ("[v5 2/3] drm/msm/dpu: Integrate interconnect API in MDSS") >> <https://lkml.kernel.org/r/CAD=FV=XUeMTGH+CDwGs3PfK4igdQrCbwucw7_2ViBc4i7grvxg@mail.gmail.com> >> >> >> Copy / pasting here (with minor name changes) so folks don't have to >> follow links / search email. >> >> == >> >> I'm curious what the plan is for landing this series. Rob / Gerogi: >> do you have any preference? Options I'd imagine: >> >> A) Wait until interconnect lands (in 5.1?) and land this through >> msm-next in the version after (5.2?) >> >> B) Georgi provides an immutable branch for interconnect when his lands >> (assuming he's landing via pull request) and that gets pulled into the >> the relevant drm tree. >> >> C) Rob Acks this series and indicates that it should go in through >> Gerogi's tree (probably only works if Georgi plans to send a pull >> request). If we're going this route then (IIUC) we'd want to land >> this in Gerogi's tree sooner rather than later so it can get some bake >> time? NOTE: as per my prior reply, I believe Rob has already Acked >> this patch. >> > > I'm ok to ack and have it land via Georgi's tree, if Georgi wants to > do that. Or otherwise, I could maybe coordinate w/ airlied to send a > 2nd late msm-next pr including the gpu and display interconnect > patches. I'm fine either way. But it would be nice if both patches (this one and the dt-bindings go together. The v6 of this patch applies cleanly to my tree, but the next one (2/3) with the dt-bindings doesn't. Thanks, Georgi ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path 2019-01-21 17:13 ` Georgi Djakov @ 2019-01-22 21:17 ` Doug Anderson 0 siblings, 0 replies; 14+ messages in thread From: Doug Anderson @ 2019-01-22 21:17 UTC (permalink / raw) To: Georgi Djakov, Rob Clark Cc: Jordan Crouse, freedreno, linux-arm-msm, Bjorn Andersson, Arnd Bergmann, Stephen Boyd, Kees Cook, Sharat Masetty, dri-devel, LKML, Andy Gross, David Airlie, Johan Hovold, Colin Ian King, Evan Green, Sean Paul Hi, On Mon, Jan 21, 2019 at 9:13 AM Georgi Djakov <georgi.djakov@linaro.org> wrote: > > Hi Rob, > > On 1/18/19 21:16, Rob Clark wrote: > > On Fri, Jan 18, 2019 at 1:06 PM Doug Anderson <dianders@chromium.org> wrote: > >> > >> Hi, > >> > >> On Thu, Dec 20, 2018 at 9:30 AM Jordan Crouse <jcrouse@codeaurora.org> wrote: > >>> > >>> Try to get the interconnect path for the GPU and vote for the maximum > >>> bandwidth to support all frequencies. This is needed for performance. > >>> Later we will want to scale the bandwidth based on the frequency to > >>> also optimize for power but that will require some device tree > >>> infrastructure that does not yet exist. > >>> > >>> v5: Remove hardcoded interconnect name and just use the default > >> > >> nit: ${SUBJECT} says v3, but this is v5. > >> > >> I'll put in my usual plug for considering "patman" to help post > >> patches. Even though it lives in the u-boot git repo it's still a gem > >> for kernel work. > >> <http://git.denx.de/?p=u-boot.git;a=blob;f=tools/patman/README> > >> > >> > >>> @@ -85,6 +89,12 @@ static void __a6xx_gmu_set_freq(struct a6xx_gmu *gmu, int index) > >>> dev_err(gmu->dev, "GMU set GPU frequency error: %d\n", ret); > >>> > >>> gmu->freq = gmu->gpu_freqs[index]; > >>> + > >>> + /* > >>> + * Eventually we will want to scale the path vote with the frequency but > >>> + * for now leave it at max so that the performance is nominal. > >>> + */ > >>> + icc_set(gpu->icc_path, 0, MBps_to_icc(7216)); > >> > >> You'll need to change icc_set() here to icc_set_bw() to match v13, AKA: > >> > >> - https://patchwork.kernel.org/patch/10766335/ > >> - https://lkml.kernel.org/r/20190116161103.6937-2-georgi.djakov@linaro.org > >> > >> > >>> @@ -695,6 +707,9 @@ int a6xx_gmu_resume(struct a6xx_gpu *a6xx_gpu) > >>> if (ret) > >>> goto out; > >>> > >>> + /* Set the bus quota to a reasonable value for boot */ > >>> + icc_set(gpu->icc_path, 0, MBps_to_icc(3072)); > >> > >> This will also need to change to icc_set_bw() > >> > >> > >>> @@ -781,6 +798,9 @@ int a6xx_gmu_stop(struct a6xx_gpu *a6xx_gpu) > >>> /* Tell RPMh to power off the GPU */ > >>> a6xx_rpmh_stop(gmu); > >>> > >>> + /* Remove the bus vote */ > >>> + icc_set(gpu->icc_path, 0, 0); > >> > >> This will also need to change to icc_set_bw() > >> > >> > >> I have the same questions for this series that I had in response to > >> the email ("[v5 2/3] drm/msm/dpu: Integrate interconnect API in MDSS") > >> <https://lkml.kernel.org/r/CAD=FV=XUeMTGH+CDwGs3PfK4igdQrCbwucw7_2ViBc4i7grvxg@mail.gmail.com> > >> > >> > >> Copy / pasting here (with minor name changes) so folks don't have to > >> follow links / search email. > >> > >> == > >> > >> I'm curious what the plan is for landing this series. Rob / Gerogi: > >> do you have any preference? Options I'd imagine: > >> > >> A) Wait until interconnect lands (in 5.1?) and land this through > >> msm-next in the version after (5.2?) > >> > >> B) Georgi provides an immutable branch for interconnect when his lands > >> (assuming he's landing via pull request) and that gets pulled into the > >> the relevant drm tree. > >> > >> C) Rob Acks this series and indicates that it should go in through > >> Gerogi's tree (probably only works if Georgi plans to send a pull > >> request). If we're going this route then (IIUC) we'd want to land > >> this in Gerogi's tree sooner rather than later so it can get some bake > >> time? NOTE: as per my prior reply, I believe Rob has already Acked > >> this patch. > >> > > > > I'm ok to ack and have it land via Georgi's tree, if Georgi wants to > > do that. Or otherwise, I could maybe coordinate w/ airlied to send a > > 2nd late msm-next pr including the gpu and display interconnect > > patches. > > I'm fine either way. But it would be nice if both patches (this one and > the dt-bindings go together. The v6 of this patch applies cleanly to my > tree, but the next one (2/3) with the dt-bindings doesn't. Ah, right. You need to be based upon commit 85437cddf4e5 ("dt-bindings: drm/msm/a6xx: Document GMU and update GPU bindings") from Rob Clark's msm-next AKA <git://people.freedesktop.org/~robclark/linux> ...so I guess the easiest is to have the bindings could go through Rob Clark's tree and the code through you tree if that's what people want to do? -Doug ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path 2019-01-18 18:06 ` Doug Anderson 2019-01-18 19:16 ` Rob Clark @ 2019-01-18 21:54 ` Doug Anderson 1 sibling, 0 replies; 14+ messages in thread From: Doug Anderson @ 2019-01-18 21:54 UTC (permalink / raw) To: Jordan Crouse, Georgi Djakov, Rob Clark Cc: freedreno, linux-arm-msm, Bjorn Andersson, Arnd Bergmann, Stephen Boyd, Kees Cook, Sharat Masetty, dri-devel, LKML, Andy Gross, David Airlie, Johan Hovold, Colin Ian King, Evan Green, Sean Paul Hi, On Fri, Jan 18, 2019 at 10:06 AM Doug Anderson <dianders@chromium.org> wrote: > It's be nice if whoever is planning to > land this could indicate whether they'd prefer Jordan send a new > version to handle the API change or if the relevant maintainer can > just do the fixup when the patch lands. Breadcrumbs: Jordan went ahead and posted a new version ("[PATCH v6] drm/msm/a6xx: Add support for an interconnect path"): https://patchwork.kernel.org/patch/10771501/ -Doug ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU 2018-12-20 17:30 [PATCH v3 0/3] arm64: dts: sdm845: Add sdm845 GPU interconnect Jordan Crouse 2018-12-20 17:30 ` [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path Jordan Crouse @ 2018-12-20 17:30 ` Jordan Crouse 2018-12-20 19:30 ` Doug Anderson ` (2 more replies) 2018-12-20 17:30 ` [PATCH v3 3/3] arm64: dts: sdm845: Add interconnect " Jordan Crouse 2 siblings, 3 replies; 14+ messages in thread From: Jordan Crouse @ 2018-12-20 17:30 UTC (permalink / raw) To: freedreno Cc: linux-arm-msm, dianders, georgi.djakov, devicetree, linux-kernel, dri-devel, Rob Herring, Rob Clark, David Airlie, Mark Rutland Add documentation for the interconnect and interconnect-names bindings for the GPU node as detailed by bindings/interconnect/interconnect.txt. Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org> --- Documentation/devicetree/bindings/display/msm/gpu.txt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/display/msm/gpu.txt b/Documentation/devicetree/bindings/display/msm/gpu.txt index 9c89f4fdb8ca..5b04393dcb15 100644 --- a/Documentation/devicetree/bindings/display/msm/gpu.txt +++ b/Documentation/devicetree/bindings/display/msm/gpu.txt @@ -20,6 +20,8 @@ Required properties: - qcom,adreno-630.2 - iommus: optional phandle to an adreno iommu instance - operating-points-v2: optional phandle to the OPP operating points +- interconnect: optional phandle to a interconnect provider. See + ../interconnect/interconnect.txt for details. - qcom,gmu: For GMU attached devices a phandle to the GMU device that will control the power for the GPU. Applicable targets: - qcom,adreno-630.2 @@ -68,6 +70,8 @@ Example a6xx (with GMU): operating-points-v2 = <&gpu_opp_table>; + interconnects = <&rsc_hlos MASTER_GFX3D &rsc_hlos SLAVE_EBI1>; + qcom,gmu = <&gmu>; }; }; -- 2.18.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU 2018-12-20 17:30 ` [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU Jordan Crouse @ 2018-12-20 19:30 ` Doug Anderson 2018-12-21 18:28 ` Rob Herring 2019-02-21 12:06 ` Georgi Djakov 2 siblings, 0 replies; 14+ messages in thread From: Doug Anderson @ 2018-12-20 19:30 UTC (permalink / raw) To: Jordan Crouse, Georgi Djakov, Rob Clark Cc: freedreno, linux-arm-msm, devicetree, LKML, dri-devel, Rob Herring, David Airlie, Mark Rutland Hi, On Thu, Dec 20, 2018 at 9:30 AM Jordan Crouse <jcrouse@codeaurora.org> wrote: > > Add documentation for the interconnect and interconnect-names bindings > for the GPU node as detailed by bindings/interconnect/interconnect.txt. > > Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org> > --- > > Documentation/devicetree/bindings/display/msm/gpu.txt | 4 ++++ > 1 file changed, 4 insertions(+) I would have been fine if you had kept my tag (even though 50% of the patch changed!). ...but here it is again: Reviewed-by: Douglas Anderson <dianders@chromium.org> I assume this will be going through Georgi's tree w/ the previous patch. -Doug ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU 2018-12-20 17:30 ` [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU Jordan Crouse 2018-12-20 19:30 ` Doug Anderson @ 2018-12-21 18:28 ` Rob Herring 2019-02-21 12:06 ` Georgi Djakov 2 siblings, 0 replies; 14+ messages in thread From: Rob Herring @ 2018-12-21 18:28 UTC (permalink / raw) To: Jordan Crouse Cc: freedreno, linux-arm-msm, dianders, georgi.djakov, devicetree, linux-kernel, dri-devel, Rob Clark, David Airlie, Mark Rutland On Thu, 20 Dec 2018 10:30:25 -0700, Jordan Crouse wrote: > Add documentation for the interconnect and interconnect-names bindings > for the GPU node as detailed by bindings/interconnect/interconnect.txt. > > Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org> > --- > > Documentation/devicetree/bindings/display/msm/gpu.txt | 4 ++++ > 1 file changed, 4 insertions(+) > Reviewed-by: Rob Herring <robh@kernel.org> ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU 2018-12-20 17:30 ` [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU Jordan Crouse 2018-12-20 19:30 ` Doug Anderson 2018-12-21 18:28 ` Rob Herring @ 2019-02-21 12:06 ` Georgi Djakov 2 siblings, 0 replies; 14+ messages in thread From: Georgi Djakov @ 2019-02-21 12:06 UTC (permalink / raw) To: Rob Clark, Jordan Crouse Cc: freedreno, linux-arm-msm, dianders, devicetree, linux-kernel, dri-devel, Rob Herring, David Airlie, Mark Rutland Hi, On 12/20/18 19:30, Jordan Crouse wrote: > Add documentation for the interconnect and interconnect-names bindings > for the GPU node as detailed by bindings/interconnect/interconnect.txt. > > Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org> > --- > > Documentation/devicetree/bindings/display/msm/gpu.txt | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/msm/gpu.txt b/Documentation/devicetree/bindings/display/msm/gpu.txt > index 9c89f4fdb8ca..5b04393dcb15 100644 > --- a/Documentation/devicetree/bindings/display/msm/gpu.txt > +++ b/Documentation/devicetree/bindings/display/msm/gpu.txt > @@ -20,6 +20,8 @@ Required properties: > - qcom,adreno-630.2 > - iommus: optional phandle to an adreno iommu instance > - operating-points-v2: optional phandle to the OPP operating points > +- interconnect: optional phandle to a interconnect provider. See Nit: s/interconnect:/interconnects:/ Nit: s/a interconnect/an interconnect/ > + ../interconnect/interconnect.txt for details. > - qcom,gmu: For GMU attached devices a phandle to the GMU device that will > control the power for the GPU. Applicable targets: > - qcom,adreno-630.2 > @@ -68,6 +70,8 @@ Example a6xx (with GMU): > > operating-points-v2 = <&gpu_opp_table>; > > + interconnects = <&rsc_hlos MASTER_GFX3D &rsc_hlos SLAVE_EBI1>; > + > qcom,gmu = <&gmu>; > }; > }; > Acked-by: Georgi Djakov <georgi.djakov@linaro.org> Thanks, Georgi ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v3 3/3] arm64: dts: sdm845: Add interconnect for GPU 2018-12-20 17:30 [PATCH v3 0/3] arm64: dts: sdm845: Add sdm845 GPU interconnect Jordan Crouse 2018-12-20 17:30 ` [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path Jordan Crouse 2018-12-20 17:30 ` [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU Jordan Crouse @ 2018-12-20 17:30 ` Jordan Crouse 2018-12-20 19:31 ` Doug Anderson 2 siblings, 1 reply; 14+ messages in thread From: Jordan Crouse @ 2018-12-20 17:30 UTC (permalink / raw) To: freedreno Cc: linux-arm-msm, dianders, georgi.djakov, devicetree, linux-kernel, Andy Gross, Rob Herring, David Brown, Mark Rutland, linux-soc Define an interconnect port for the GPU to set bus capabilities. Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org> --- arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 ++ 1 file changed, 2 insertions(+) diff --git a/arch/arm64/boot/dts/qcom/sdm845.dtsi b/arch/arm64/boot/dts/qcom/sdm845.dtsi index 1005f1eb1920..155e4ba74c96 100644 --- a/arch/arm64/boot/dts/qcom/sdm845.dtsi +++ b/arch/arm64/boot/dts/qcom/sdm845.dtsi @@ -1369,6 +1369,8 @@ operating-points-v2 = <&gpu_opp_table>; + interconnects = <&rsc_hlos MASTER_GFX3D &rsc_hlos SLAVE_EBI1>; + qcom,gmu = <&gmu>; gpu_opp_table: opp-table { -- 2.18.0 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v3 3/3] arm64: dts: sdm845: Add interconnect for GPU 2018-12-20 17:30 ` [PATCH v3 3/3] arm64: dts: sdm845: Add interconnect " Jordan Crouse @ 2018-12-20 19:31 ` Doug Anderson 0 siblings, 0 replies; 14+ messages in thread From: Doug Anderson @ 2018-12-20 19:31 UTC (permalink / raw) To: Jordan Crouse, Andy Gross Cc: freedreno, linux-arm-msm, Georgi Djakov, devicetree, LKML, Rob Herring, David Brown, Mark Rutland, open list:ARM/QUALCOMM SUPPORT, Bjorn Andersson Hi, On Thu, Dec 20, 2018 at 9:30 AM Jordan Crouse <jcrouse@codeaurora.org> wrote: > > Define an interconnect port for the GPU to set bus > capabilities. > > Signed-off-by: Jordan Crouse <jcrouse@codeaurora.org> > --- > > arch/arm64/boot/dts/qcom/sdm845.dtsi | 2 ++ > 1 file changed, 2 insertions(+) For going through Andy's tree once the bindings land (presumably in Georgi's tree). Reviewed-by: Douglas Anderson <dianders@chromium.org> ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2019-02-21 12:06 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2018-12-20 17:30 [PATCH v3 0/3] arm64: dts: sdm845: Add sdm845 GPU interconnect Jordan Crouse 2018-12-20 17:30 ` [PATCH v3 1/3] drm/msm/a6xx: Add support for an interconnect path Jordan Crouse 2018-12-20 19:28 ` Doug Anderson 2019-01-18 18:06 ` Doug Anderson 2019-01-18 19:16 ` Rob Clark 2019-01-21 17:13 ` Georgi Djakov 2019-01-22 21:17 ` Doug Anderson 2019-01-18 21:54 ` Doug Anderson 2018-12-20 17:30 ` [PATCH v3 2/3] dt-bindings: drm/msm/a6xx: Document interconnect properties for GPU Jordan Crouse 2018-12-20 19:30 ` Doug Anderson 2018-12-21 18:28 ` Rob Herring 2019-02-21 12:06 ` Georgi Djakov 2018-12-20 17:30 ` [PATCH v3 3/3] arm64: dts: sdm845: Add interconnect " Jordan Crouse 2018-12-20 19:31 ` Doug Anderson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).