[PATCH v2 2/2] drm/msm/dpu: Issue MDSS reset during initialization

2022-03-01 Thread Bjorn Andersson
It's typical for the bootloader to bring up the display for showing a boot splash or efi framebuffer. But in some cases the kernel driver ends up only partially configuring (in particular) the DPU, which might result in e.g. that two different data paths attempts to push data to the interface - wit

Re: linux-next: build warning after merge of the drm-misc tree

2022-03-01 Thread Stephen Rothwell
Hi all, On Thu, 20 Jan 2022 14:26:39 +1100 Stephen Rothwell wrote: > > On Wed, 17 Nov 2021 13:49:26 +1100 Stephen Rothwell > wrote: > > > > After merging the drm-misc tree, today's linux-next build (htmldocs) > > produced this warning: > > > > include/drm/gpu_scheduler.h:316: warning: Functi

Re: linux-next: build warning after merge of the drm tree

2022-03-01 Thread Stephen Rothwell
Hi all, On Wed, 2 Feb 2022 15:10:45 +1100 Stephen Rothwell wrote: > > After merging the drm tree, today's linux-next build (htmldocs) produced > this warning: > > include/drm/drm_connector.h:637: warning: Function parameter or member > 'edid_hdmi_rgb444_dc_modes' not described in 'drm_display

Re: linux-next: build warning after merge of the drm tree

2022-03-01 Thread Stephen Rothwell
Hi all, On Wed, 2 Feb 2022 09:38:37 +0100 Hans de Goede wrote: > > On 2/2/22 05:03, Stephen Rothwell wrote: > > > > On Wed, 2 Feb 2022 15:02:01 +1100 Stephen Rothwell > > wrote: > >> > >> After merging the drm tree, today's linux-next build (htmldocs) produced > >> this warning: > >> > >> d

Re: [PATCH 1/5] arm64: dts: qcom: msm8996: Drop flags for mdss irqs

2022-03-01 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-03-01 16:52:06) > The number of interrupt cells for the mdss interrupt controller is 1, > meaning there should only be one cell for the interrupt number, not two. > Drop the second cell containing (unused) irq flags. > > Signed-off-by: Dmitry Baryshkov > --- Reviewe

Re: [PATCH 2/5] arm64: dts: qcom: sdm630: Drop flags for mdss irqs

2022-03-01 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-03-01 16:52:07) > The number of interrupt cells for the mdss interrupt controller is 1, > meaning there should only be one cell for the interrupt number, not two. > Drop the second cell containing (unused) irq flags. > > Signed-off-by: Dmitry Baryshkov > --- Reviewe

Re: [PATCH 3/5] arm64: dts: qcom: sdm660: Drop flags for mdss irqs

2022-03-01 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-03-01 16:52:08) > The number of interrupt cells for the mdss interrupt controller is 1, > meaning there should only be one cell for the interrupt number, not two. > Drop the second cell containing (unused) irq flags. > > Signed-off-by: Dmitry Baryshkov > --- Reviewe

Re: [PATCH 4/5] arm64: dts: qcom: sdm845: Drop flags for mdss irqs

2022-03-01 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-03-01 16:52:09) > The number of interrupt cells for the mdss interrupt controller is 1, > meaning there should only be one cell for the interrupt number, not two. > Drop the second cell containing (unused) irq flags. > > Signed-off-by: Dmitry Baryshkov > --- Reviewe

Re: [PATCH 5/5] arm64: dts: qcom: sm8250: Drop flags for mdss irqs

2022-03-01 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-03-01 16:52:10) > The number of interrupt cells for the mdss interrupt controller is 1, > meaning there should only be one cell for the interrupt number, not two. > Drop the second cell containing (unused) irq flags. > > Signed-off-by: Dmitry Baryshkov > --- Reviewe

Re: [RESEND PATCH] dt-bindings: display/msm: add missing brace in dpu-qcm2290.yaml

2022-03-01 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-03-01 16:14:10) > Add missing brace in dpu-qcm2290.yaml. While we are at it, also fix > indentation for another brace, so it matches the corresponding line. > > Reported-by: Rob Herring > Cc: Loic Poulain > Reviewed-by: Bjorn Andersson > Signed-off-by: Dmitry Barys

Re: [PATCH v2 2/2] drm/msm/dpu: Issue MDSS reset during initialization

2022-03-01 Thread Dmitry Baryshkov
On Wed, 2 Mar 2022 at 04:27, Bjorn Andersson wrote: > > It's typical for the bootloader to bring up the display for showing a > boot splash or efi framebuffer. But in some cases the kernel driver ends > up only partially configuring (in particular) the DPU, which might > result in e.g. that two di

Re: [PATCH v3 2/2] drm/bridge: analogix_dp: Enable autosuspend

2022-03-01 Thread Brian Norris
On Tue, Feb 22, 2022 at 2:10 PM Doug Anderson wrote: > On Thu, Feb 17, 2022 at 2:42 PM Brian Norris wrote: > > --- a/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > > +++ b/drivers/gpu/drm/bridge/analogix/analogix_dp_core.c > > @@ -1121,7 +1121,7 @@ static int analogix_dp_get_modes(struct dr

Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'

2022-03-01 Thread Jakub Kicinski
On Mon, 28 Feb 2022 15:38:02 -0600 Rob Herring wrote: > Another pass at removing unnecessary use of 'allOf' with a '$ref'. > > json-schema versions draft7 and earlier have a weird behavior in that > any keywords combined with a '$ref' are ignored (silently). The correct > form was to put a '$ref'

[PATCH v4 1/2] drm/bridge: analogix_dp: Grab runtime PM reference for DP-AUX

2022-03-01 Thread Brian Norris
If the display is not enable()d, then we aren't holding a runtime PM reference here. Thus, it's easy to accidentally cause a hang, if user space is poking around at /dev/drm_dp_aux0 at the "wrong" time. Let's get a runtime PM reference, and check that we "see" the panel. Don't force any panel powe

[PATCH v4 2/2] drm/bridge: analogix_dp: Enable autosuspend

2022-03-01 Thread Brian Norris
DP AUX transactions can consist of many short operations. There's no need to power things up/down in short intervals. I pick an arbitrary 100ms; for the systems I'm testing (Rockchip RK3399), runtime-PM transitions only take a few microseconds. Signed-off-by: Brian Norris --- Changes in v4: -

Re: [PATCH v2 2/2] drm/msm/dpu: Issue MDSS reset during initialization

2022-03-01 Thread Bjorn Andersson
On Tue 01 Mar 17:47 PST 2022, Dmitry Baryshkov wrote: > On Wed, 2 Mar 2022 at 04:27, Bjorn Andersson > wrote: > > > > It's typical for the bootloader to bring up the display for showing a > > boot splash or efi framebuffer. But in some cases the kernel driver ends > > up only partially configuri

Re: [PATCH] dt-bindings: Another pass removing cases of 'allOf' containing a '$ref'

2022-03-01 Thread Bjorn Andersson
On Mon 28 Feb 13:38 PST 2022, Rob Herring wrote: > Another pass at removing unnecessary use of 'allOf' with a '$ref'. > > json-schema versions draft7 and earlier have a weird behavior in that > any keywords combined with a '$ref' are ignored (silently). The correct > form was to put a '$ref' unde

Re: [EXT] Re: [PATCH 1/9] dt-bindings: mxsfb: Add compatible for i.MX8MP

2022-03-01 Thread Marek Vasut
On 3/1/22 14:37, Robby Cai wrote: Hi, [...] I tend to agree with Marek on this one. We have an instance where the blk-ctrl and the GPC driver between 8m, mini, nano, plus are close, but different enough where each SoC has it's own set of tables and some checks. Lucas created the framework,

Re: [PATCH v9] drm/amdgpu: add drm buddy support to amdgpu

2022-03-01 Thread kernel test robot
Hi Arunpravin, Thank you for the patch! Perhaps something to improve: [auto build test WARNING on drm/drm-next] [also build test WARNING on drm-tip/drm-tip] [cannot apply to drm-intel/for-linux-next v5.17-rc6 next-20220301] [If your patch is applied to the wrong git tree, kindly drop us a note

Re: [PATCH 1/9] dt-bindings: mxsfb: Add compatible for i.MX8MP

2022-03-01 Thread Marek Vasut
On 3/1/22 14:18, Lucas Stach wrote: Am Dienstag, dem 01.03.2022 um 07:03 -0600 schrieb Adam Ford: On Tue, Mar 1, 2022 at 5:05 AM Lucas Stach wrote: Am Dienstag, dem 01.03.2022 um 11:19 +0100 schrieb Marek Vasut: On 3/1/22 11:04, Lucas Stach wrote: Hi, [...] Given the two totally differen

[PATCH v6 3/5] drm/aspeed: Update INTR_STS handling

2022-03-01 Thread Tommy Haung
Add interrupt clear register define for further chip support. Signed-off-by: Tommy Haung --- drivers/gpu/drm/aspeed/aspeed_gfx.h | 1 + drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 6 +- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx.h b/dri

[PATCH v6 2/5] ARM: dts: aspeed: ast2600-evb: Enable GFX device

2022-03-01 Thread Tommy Haung
From: Joel Stanley Enable the GFX device with a framebuffer memory region. Signed-off-by: Joel Stanley Signed-off-by: Tommy Haung --- arch/arm/boot/dts/aspeed-ast2600-evb.dts | 18 ++ 1 file changed, 18 insertions(+) diff --git a/arch/arm/boot/dts/aspeed-ast2600-evb.dts b/ar

[PATCH v6 0/5] Add Aspeed AST2600 soc display support

2022-03-01 Thread Tommy Haung
v6: Remove some unnecessary reset patch. Refine patch format. Add detail explain of SOC display reset bits. v5: Add lost reset define. v4: Add necessary reset control for ast2600. Add chip caps for futher use. These code are test on AST2500 and AST2600 by below steps. 1. Add belo

[PATCH v6 1/5] ARM: dts: aspeed: Add GFX node to AST2600

2022-03-01 Thread Tommy Haung
From: Joel Stanley The GFX device is present in the AST2600 SoC. Signed-off-by: Joel Stanley Signed-off-by: Tommy Haung --- arch/arm/boot/dts/aspeed-g6.dtsi | 11 +++ 1 file changed, 11 insertions(+) diff --git a/arch/arm/boot/dts/aspeed-g6.dtsi b/arch/arm/boot/dts/aspeed-g6.dtsi ind

[PATCH v6 4/5] drm/aspeed: Add AST2600 chip support

2022-03-01 Thread Tommy Haung
Add AST2600 chip support and setting. Signed-off-by: Tommy Haung --- drivers/gpu/drm/aspeed/aspeed_gfx_drv.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c b/drivers/gpu/drm/aspeed/aspeed_gfx_drv.c index d4b56b3c7597..d10246b1d1c2 100644 ---

[PATCH v6 5/5] ARM: dtsi: aspeed: Modified gfx reset control

2022-03-01 Thread Tommy Haung
Remove the ast2500-gfx from aspeed-g6.dtsi. In the AST2600, the ASPEED_RESET_CRT1 is replaced by ASPEED_RESET_GRAPHICS. This is no differnce between these two reset behavior but reigster location is changed. The HW controller states and FW programming resgiter will be reset by CRT reset controller

Re: [PATCH v6 5/5] ARM: dtsi: aspeed: Modified gfx reset control

2022-03-01 Thread Joel Stanley
On Wed, 2 Mar 2022 at 02:50, Tommy Haung wrote: > > Remove the ast2500-gfx from aspeed-g6.dtsi. > In the AST2600, the ASPEED_RESET_CRT1 is replaced by > ASPEED_RESET_GRAPHICS. This is no differnce between these two reset > behavior but reigster location is changed. The HW controller states > and F

Re: [PATCH v6 0/5] Add Aspeed AST2600 soc display support

2022-03-01 Thread Joel Stanley
On Wed, 2 Mar 2022 at 02:49, Tommy Haung wrote: > > v6: > Remove some unnecessary reset patch. > Refine patch format. > Add detail explain of SOC display reset bits. > > v5: > Add lost reset define. > > v4: > Add necessary reset control for ast2600. > Add chip caps for futher use. >

Re: linux-next: build warning after merge of the drm-misc tree

2022-03-01 Thread Andrey Grodzovsky
Please check you have commit c7703ce38c1e Andrey Grodzovsky   3 weeks ago    drm/amdgpu: Fix htmldoc warning Andrey On 2022-03-01 20:31, Stephen Rothwell wrote: Hi all, On Thu, 20 Jan 2022 14:26:39 +1100 Stephen Rothwell wrote: On Wed, 17 Nov 2021 13:49:26 +1100 Stephen Rothwell wrote:

Re: [PATCH 2/2] drm/i915: Remove the vm open count

2022-03-01 Thread Niranjana Vishwanathapura
On Tue, Feb 22, 2022 at 06:10:30PM +0100, Thomas Hellström wrote: vms are not getting properly closed. Rather than fixing that, Remove the vm open count and instead rely on the vm refcount. The vm open count existed solely to break the strong references the vmas had on the vms. Now instead make

Re: [PATCH v2 8/8] drm: bridge: anx7625: Switch to devm_drm_of_get_bridge

2022-03-01 Thread Jagan Teki
On Wed, Mar 2, 2022 at 4:43 AM Linus Walleij wrote: > > On Tue, Mar 1, 2022 at 3:13 PM Jagan Teki wrote: > > > devm_drm_of_get_bridge is capable of looking up the downstream > > bridge and panel and trying to add a panel bridge if the panel > > is found. > > > > Replace explicit finding calls wit

Re: [PATCH v2 8/8] drm: bridge: anx7625: Switch to devm_drm_of_get_bridge

2022-03-01 Thread Jagan Teki
On Wed, Mar 2, 2022 at 4:50 AM Linus Walleij wrote: > > On Tue, Mar 1, 2022 at 3:13 PM Jagan Teki wrote: > > > + bridge = devm_drm_of_get_bridge(dev, dev->of_node, 0, 0); > > + if (IS_ERR(bridge)) { > > + dev_err(dev, "error to get bridge\n"); > > + return

Re: [PATCH v2 2/2] drm/msm/dpu: Issue MDSS reset during initialization

2022-03-01 Thread Dmitry Baryshkov
On 02/03/2022 05:44, Bjorn Andersson wrote: On Tue 01 Mar 17:47 PST 2022, Dmitry Baryshkov wrote: On Wed, 2 Mar 2022 at 04:27, Bjorn Andersson wrote: It's typical for the bootloader to bring up the display for showing a boot splash or efi framebuffer. But in some cases the kernel driver ends

Re: linux-next: build warning after merge of the drm-misc tree

2022-03-01 Thread Stephen Rothwell
Hi Andrey, On Tue, 1 Mar 2022 22:26:12 -0500 Andrey Grodzovsky wrote: > > Please check you have commit c7703ce38c1e Andrey Grodzovsky   3 weeks ago    > drm/amdgpu: Fix htmldoc warning That has arrived in linux-next today for the first time. It is in the drm tree, but that tree has had build

[PATCH] dt-bindings: gpu: Convert aspeed-gfx bindings to yaml

2022-03-01 Thread Joel Stanley
Convert the bindings to yaml and add the ast2600 compatible string. Signed-off-by: Joel Stanley --- .../devicetree/bindings/gpu/aspeed,gfx.yaml | 69 +++ .../devicetree/bindings/gpu/aspeed-gfx.txt| 41 --- 2 files changed, 69 insertions(+), 41 deletions(-) create m

[PATCH v5 11/13] drm/i915/xehp: handle fused off CCS engines

2022-03-01 Thread Matt Roper
From: Daniele Ceraolo Spurio HW resources are divided across the active CCS engines at the compute slice level, with each CCS having priority on one of the cslices. If a compute slice has no enabled DSS, its paired compute engine is not usable in full parallel execution because the other ones alr

[PATCH] drm/i915/gt: Handle errors for i915_gem_object_trylock

2022-03-01 Thread Jiasheng Jiang
As the potential failure of the i915_gem_object_trylock(), it should be better to check it and return error if fails. Fixes: 94ce0d65076c ("drm/i915/gt: Setup a default migration context on the GT") Signed-off-by: Jiasheng Jiang --- drivers/gpu/drm/i915/gt/selftest_migrate.c | 6 +- 1 file c

Re: [PATCH 9/9] drm: mxsfb: Add support for i.MX8MP LCDIF variant

2022-03-01 Thread kernel test robot
ase: git://anongit.freedesktop.org/drm-intel for-linux-next config: mips-randconfig-c004-20220227 (https://download.01.org/0day-ci/archive/20220301/202203010150.l57eax3w-...@intel.com/config) compiler: clang version 15.0.0 (https://github.com/llvm/llvm-project d271fc04d5b97b12e6b797c6067d3c

<    1   2   3