Re: [PATCH 1/4] fbdev: imsttfb: Fix error handling in init_imstt()

2023-05-24 Thread Markus Elfring
>> The return value was overlooked from a call of >> the function “fb_alloc_cmap”. >> >> * Thus use a corresponding error check. >> >> * Add two jump targets so that a bit of exception handling >>    can be better reused at the end of this function. … >> +++ b/drivers/video/fbdev/imsttfb.c … >> @@

Re: [PATCH 3/6] drm/i915/uc/gsc: extract release and security versions from the gsc binary

2023-05-24 Thread Teres Alexis, Alan Previn
On Fri, 2023-05-05 at 09:04 -0700, Ceraolo Spurio, Daniele wrote: alan: snip > +int intel_gsc_fw_get_binary_info(struct intel_uc_fw *gsc_fw, const void > *data, size_t size) > +{ alan:snip > + /* > + * The GSC binary starts with the pointer layout, which contains the > + *

Re: [PATCH 3/4] fbdev: imsttfb: Move a variable assignment for an error code in imsttfb_probe()

2023-05-24 Thread Markus Elfring
>> The value “-ENOMEM” was assigned to the variable “ret” >> at the beginning. >> Move this statement directly before the first ioremap() call. > > Please do not move such variables without real need. Is there a need to explain desirable effects better? > It makes backporting (of this and maybe

Re: [PATCH 1/2] fbdev: Move two variable assignments in fb_alloc_cmap_gfp()

2023-05-24 Thread Markus Elfring
>> Move the assignment for the local variables “size” and “flags” >> because the computed values were only used in a single if branch. > > Please do not move such variables without real need. Is there a need to explain desirable effects better? > It makes backporting (of this and maybe

Re: [PATCH] arm64: dts: qcom: enable dual ("bonded") DSI mode for DB845c

2023-05-24 Thread Bjorn Andersson
On Thu, 4 May 2023 19:04:30 +0300, Dmitry Baryshkov wrote: > Now as both lt9611 and drm/msm drivers were updated to handle the 4k > modes over DSI, enable "bonded" DSI mode on DB845c. This way the board > utilizes both DSI links and thus can support 4k on the HDMI output. > > Applied, thanks!

Re: [PATCH v14 1/2] drm: add kms driver for loongson display controller

2023-05-24 Thread Sui Jingfeng
On 2023/5/25 12:09, Sui Jingfeng wrote: Hi, On 2023/5/23 00:40, WANG Xuerui wrote: On 5/22/23 21:13, Sui Jingfeng wrote: Hi, On 2023/5/22 18:25, WANG Xuerui wrote: On 2023/5/22 18:17, Sui Jingfeng wrote: Hi, On 2023/5/22 18:05, WANG Xuerui wrote: On 2023/5/22 17:49, Sui Jingfeng wrote:

Re: [PATCH v14 1/2] drm: add kms driver for loongson display controller

2023-05-24 Thread Sui Jingfeng
Hi, On 2023/5/23 00:40, WANG Xuerui wrote: On 5/22/23 21:13, Sui Jingfeng wrote: Hi, On 2023/5/22 18:25, WANG Xuerui wrote: On 2023/5/22 18:17, Sui Jingfeng wrote: Hi, On 2023/5/22 18:05, WANG Xuerui wrote: On 2023/5/22 17:49, Sui Jingfeng wrote: Hi, On 2023/5/22 17:28, WANG Xuerui

[linux-next:master] BUILD REGRESSION cf09e328589a2ed7f6c8d90f2edb697fb4f8a96b

2023-05-24 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git master branch HEAD: cf09e328589a2ed7f6c8d90f2edb697fb4f8a96b Add linux-next specific files for 20230524 Error/Warning reports: https://lore.kernel.org/oe-kbuild-all/202305240732.wucsrnaj-...@intel.com https

[v3 4/4] drm/panel: Support for Starry-ili9882t TDDI MIPI-DSI panel

2023-05-24 Thread Cong Yang
The Starry-ili9882 is a 10.51" WUXGA TFT panel. which fits in nicely with the existing panel-boe-tv101wum-nl6 driver. From the datasheet,MIPI need to keep the LP11 state before the lcm_reset pin is pulled high. So add lp11_before_reset flag. Signed-off-by: Cong Yang Reviewed-by: Douglas Anderson

[v3 3/4] dt-bindings: display: panel: Add compatible for Starry ili9882t

2023-05-24 Thread Cong Yang
The STARRY ili9882t is a 10.51" WUXGA TFT LCD panel, which fits in nicely with the existing panel-boe-tv101wum-nl6 driver. Hence, we add a new compatible with panel specific config. Signed-off-by: Cong Yang Reviewed-by: Douglas Anderson Acked-by: Conor Dooley ---

[v3 2/4] drm/panel: Support for Starry-himax83102-j02 TDDI MIPI-DSI panel

2023-05-24 Thread Cong Yang
The Starry-himax83102-j02 is a 10.51" WUXGA TFT panel. which fits in nicely with the existing panel-boe-tv101wum-nl6 driver. From the datasheet[1], MIPI needs to keep the LP11 state before the lcm_reset pin is pulled high, so increase lp11_before_reset flag. [1]:

[v3 1/4] dt-bindings: display: panel: Add compatible for Starry himax83102-j02

2023-05-24 Thread Cong Yang
The STARRY himax83102-j02 is a 10.51" WUXGA TFT LCD panel, which fits in nicely with the existing panel-boe-tv101wum-nl6 driver. Hence, we add a new compatible with panel specific config. Signed-off-by: Cong Yang Reviewed-by: Douglas Anderson Acked-by: Conor Dooley ---

[v3 0/4] Support Starry-himax83102-j02 and Starry-ili9882t TDDI MIPI-DSI panel

2023-05-24 Thread Cong Yang
Compare V2: order of the tables match the order they're referenced. Cong Yang (4): dt-bindings: display: panel: Add compatible for Starry himax83102-j02 drm/panel: Support for Starry-himax83102-j02 TDDI MIPI-DSI panel dt-bindings: display: panel: Add compatible for Starry ili9882t

Re: [v4,12/13] drm/fbdev-generic: Implement dedicated fbdev I/O helpers

2023-05-24 Thread Sui Jingfeng
Reviewed-by: Sui Jingfeng On 2023/5/24 17:21, Thomas Zimmermann wrote: Implement dedicated fbdev helpers for framebuffer I/O instead of using DRM's helpers. Use an fbdev generator macro for deferred I/O to create the callbacks. Fbdev-generic was the only caller of the DRM helpers, so remove

Re: [RFC PATCH v2 11/13] drm/msm/dpu: add a field describing inline rotation to dpu_caps

2023-05-24 Thread Dmitry Baryshkov
On Thu, 25 May 2023 at 02:20, Abhinav Kumar wrote: > > > > On 3/20/2023 6:18 PM, Dmitry Baryshkov wrote: > > We need to know if the platform supports inline rotation on any of the > > SSPP blocks or not. Add this information to struct dpu_caps in a form of > > the boolean field has_inline_rot. >

Re: [RFC PATCH v2 10/13] drm/msm/dpu: add list of supported formats to the DPU caps

2023-05-24 Thread Dmitry Baryshkov
On Thu, 25 May 2023 at 02:16, Abhinav Kumar wrote: > > > > On 3/20/2023 6:18 PM, Dmitry Baryshkov wrote: > > As we are going to add virtual planes, add the list of supported formats > > to the hw catalog entry. It will be used to setup universal planes, with > > later selecting a pipe depending

Re: [PATCH v6 5/6] drm: lcdif: Add multiple encoders and first bridges support

2023-05-24 Thread Ying Liu
Hi Marek, On Thu, May 11, 2023 at 12:24 AM Marek Vasut wrote: > On 5/10/23 11:24, Liu Ying wrote: > > The single LCDIF embedded in i.MX93 SoC may drive multiple displays > > simultaneously. Look at LCDIF output port's remote port parents to > > find all enabled first bridges. Add an encoder

Re: [Freedreno] [RFC PATCH v2 09/13] drm/msm/dpu: move pstate->pipe initialization to dpu_plane_atomic_check

2023-05-24 Thread Dmitry Baryshkov
On Thu, 25 May 2023 at 02:04, Abhinav Kumar wrote: > > > > On 5/24/2023 3:46 PM, Abhinav Kumar wrote: > > > > > > On 3/20/2023 6:18 PM, Dmitry Baryshkov wrote: > >> In preparation to virtualized planes support, move pstate->pipe > >> initialization from dpu_plane_reset() to

Re: [PATCH] drm/ast: Fix modeset failed on DisplayPort

2023-05-24 Thread Jammy Huang
Hi Thomas, Thanks, I will modify the patch accordingly. On 2023/5/24 下午 06:47, Thomas Zimmermann wrote: Hi, this patch also fell through the cracks. Apologies. Am 24.05.23 um 04:29 schrieb Jammy Huang: Hi Thomas, Do you have other suggestion for this patch?? The main issue was that

[PATCH v2] drm/ast: Fix modeset failed on DisplayPort

2023-05-24 Thread Jammy Huang
If we switch display and update cursor together, it could lead to modeset failed because of concurrent access to IO registers. Add lock protection in DP's edid access to avoid this problem. Signed-off-by: Jammy Huang --- v2 changes: - Fix build error since new struct ast_device is used. ---

Re: [PATCH 21/36] drm/amd/display: add CRTC 3D LUT support

2023-05-24 Thread kernel test robot
Hi Melissa, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-misc/drm-misc-next] [also build test WARNING on linus/master v6.4-rc3 next-20230524] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest

Re: [PATCH v14 1/9] drm/display/dsc: Add flatness and initial scale value calculations

2023-05-24 Thread Jessica Zhang
On 5/24/2023 12:05 PM, Marijn Suijten wrote: On 2023-05-24 10:45:14, Jessica Zhang wrote: Add helpers to calculate det_thresh_flatness and initial_scale_value as these calculations are defined within the DSC spec. Reviewed-by: Marijn Suijten Reviewed-by: Dmitry Baryshkov Signed-off-by:

Re: [RFC PATCH v2 11/13] drm/msm/dpu: add a field describing inline rotation to dpu_caps

2023-05-24 Thread Abhinav Kumar
On 3/20/2023 6:18 PM, Dmitry Baryshkov wrote: We need to know if the platform supports inline rotation on any of the SSPP blocks or not. Add this information to struct dpu_caps in a form of the boolean field has_inline_rot. So even for this one, will a helper to detect it from the list of

Re: [RFC PATCH v2 10/13] drm/msm/dpu: add list of supported formats to the DPU caps

2023-05-24 Thread Abhinav Kumar
On 3/20/2023 6:18 PM, Dmitry Baryshkov wrote: As we are going to add virtual planes, add the list of supported formats to the hw catalog entry. It will be used to setup universal planes, with later selecting a pipe depending on whether the YUV format is used for the framebuffer. If your

Re: [Freedreno] [RFC PATCH v2 09/13] drm/msm/dpu: move pstate->pipe initialization to dpu_plane_atomic_check

2023-05-24 Thread Abhinav Kumar
On 5/24/2023 3:46 PM, Abhinav Kumar wrote: On 3/20/2023 6:18 PM, Dmitry Baryshkov wrote: In preparation to virtualized planes support, move pstate->pipe initialization from dpu_plane_reset() to dpu_plane_atomic_check(). In case of virtual planes the plane's pipe will not be known up to the

[PATCH net-next v5 6/6] MAINTAINERS: ASP 2.0 Ethernet driver maintainers

2023-05-24 Thread Justin Chen
Add maintainers entry for ASP 2.0 Ethernet driver. Reviewed-by: Simon Horman Signed-off-by: Florian Fainelli Signed-off-by: Justin Chen --- v3 - Change from gmail to broadcom emails MAINTAINERS | 9 + 1 file changed, 9 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS

[PATCH net-next v5 5/6] net: phy: bcm7xxx: Add EPHY entry for 74165

2023-05-24 Thread Justin Chen
From: Florian Fainelli 74165 is a 16nm process SoC with a 10/100 integrated Ethernet PHY, utilize the recently defined 16nm EPHY macro to configure that PHY. Reviewed-by: Simon Horman Reviewed-by: Andrew Lunn Signed-off-by: Florian Fainelli Signed-off-by: Justin Chen ---

[PATCH net-next v5 2/6] dt-bindings: net: Brcm ASP 2.0 Ethernet controller

2023-05-24 Thread Justin Chen
From: Florian Fainelli Add a binding document for the Broadcom ASP 2.0 Ethernet controller. Signed-off-by: Florian Fainelli Signed-off-by: Justin Chen --- v5 - Fix compatible string yaml format to properly capture what we want v4 - Adjust compatible string example to

[PATCH net-next v5 3/6] net: bcmasp: Add support for ASP2.0 Ethernet controller

2023-05-24 Thread Justin Chen
Add support for the Broadcom ASP 2.0 Ethernet controller which is first introduced with 72165. This controller features two distinct Ethernet ports that can be independently operated. This patch supports: - Wake-on-LAN using magic packets - basic ethtool operations (link, counters, message

[PATCH net-next v5 4/6] net: phy: mdio-bcm-unimac: Add asp v2.0 support

2023-05-24 Thread Justin Chen
Add mdio compat string for ASP 2.0 ethernet driver. Reviewed-by: Simon Horman Reviewed-by: Andrew Lunn Signed-off-by: Florian Fainelli Signed-off-by: Justin Chen --- drivers/net/mdio/mdio-bcm-unimac.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/net/mdio/mdio-bcm-unimac.c

[PATCH net-next v5 1/6] dt-bindings: net: brcm, unimac-mdio: Add asp-v2.0

2023-05-24 Thread Justin Chen
The ASP 2.0 Ethernet controller uses a brcm unimac. Reviewed-by: Simon Horman Acked-by: Conor Dooley Signed-off-by: Florian Fainelli Signed-off-by: Justin Chen --- Documentation/devicetree/bindings/net/brcm,unimac-mdio.yaml | 2 ++ 1 file changed, 2 insertions(+) diff --git

[PATCH net-next v5 0/6] Brcm ASP 2.0 Ethernet Controller

2023-05-24 Thread Justin Chen
Add support for the Broadcom ASP 2.0 Ethernet controller which is first introduced with 72165. Add support for 74165 10/100 integrated Ethernet PHY which also uses the ASP 2.0 Ethernet controller. Florian Fainelli (2): dt-bindings: net: Brcm ASP 2.0 Ethernet controller net: phy: bcm7xxx: Add

Re: [RFC PATCH v2 09/13] drm/msm/dpu: move pstate->pipe initialization to dpu_plane_atomic_check

2023-05-24 Thread Abhinav Kumar
On 3/20/2023 6:18 PM, Dmitry Baryshkov wrote: In preparation to virtualized planes support, move pstate->pipe initialization from dpu_plane_reset() to dpu_plane_atomic_check(). In case of virtual planes the plane's pipe will not be known up to the point of atomic_check() callback.

Re: [PATCH v14 3/9] drm/display/dsc: Add drm_dsc_get_bpp_int helper

2023-05-24 Thread Jessica Zhang
On 5/24/2023 12:14 PM, Marijn Suijten wrote: On 2023-05-24 10:45:16, Jessica Zhang wrote: Add helper to get the integer value of drm_dsc_config.bits_per_pixel Reviewed-by: Marijn Suijten Reviewed-by: Dmitry Baryshkov Signed-off-by: Jessica Zhang Nit: if there comes a v15, perhaps this

Re: [PATCH v13 02/10] drm/msm/dpu: add dsc blocks to the catalog of MSM8998 and SC8180X

2023-05-24 Thread Marijn Suijten
Title: DSC On 2023-05-22 17:00:31, Kuogee Hsieh wrote: > From: Abhinav Kumar > > Some platforms have DSC blocks which have not been declared in the catalog. > Complete DSC 1.1 support for all platforms by adding the missing blocks to > MSM8998 and SC8180X. > > Changes in v9: > -- add MSM8998

Re: [PATCH v13 01/10] drm/msm/dpu: set DSC flush bit correctly at MDP CTL flush register

2023-05-24 Thread Marijn Suijten
On 2023-05-22 17:00:30, Kuogee Hsieh wrote: > The DSC CTL_FLUSH register should be programmed with the 22th bit Sorry for botching this in v12 review, there's no DSC CTL_FLUSH register. Drop DSC from "The DSC CTL_FLUSH register". > (DSC_IDX) to flush the DSC hardware blocks, not the literal

Re: [PATCH v13 08/10] drm/msm/dpu: separate DSC flush update out of interface

2023-05-24 Thread Marijn Suijten
On 2023-05-22 17:00:37, Kuogee Hsieh wrote: > Currently DSC flushing happens during interface configuration at > dpu_hw_ctl_intf_cfg_v1(). Separate DSC flush away from > dpu_hw_ctl_intf_cfg_v1() by adding dpu_hw_ctl_update_pending_flush_dsc_v1() > to handle both per-DSC engine and DSC flush bits

Re: [PATCH v13 10/10] drm/msm/dpu: tear down DSC data path when DSC disabled

2023-05-24 Thread Marijn Suijten
Title: Tear down DSC datapath on encoder cleanup (from v10 review, this is v13...) On 2023-05-22 17:00:39, Kuogee Hsieh wrote: - Marijn

Re: [PATCH v13 07/10] drm/msm/dpu: always clear every individual pending flush mask

2023-05-24 Thread Marijn Suijten
On 2023-05-22 17:00:36, Kuogee Hsieh wrote: > There are two tiers of pending flush control, top levle and levle -> level > individual hardware block. Currently only the top level of > flush mask is reset to 0 but the individual pending flush masks > of particular hardware blocks are left at

Re: [PATCH 2/2] drm/i915/pmu: Make PMU sample array two-dimensional

2023-05-24 Thread Dixit, Ashutosh
On Wed, 24 May 2023 10:53:20 -0700, Tvrtko Ursulin wrote: > Hi Tvrtko, > On 24/05/2023 18:38, Dixit, Ashutosh wrote: > > On Wed, 24 May 2023 04:38:18 -0700, Tvrtko Ursulin wrote: > >> On 23/05/2023 16:19, Ashutosh Dixit wrote: > >>> No functional changes but we can remove some unsightly index

Re: [Intel-gfx] [PATCH 1/2] drm/i915/pmu: Turn off the timer to sample frequencies when GT is parked

2023-05-24 Thread Dixit, Ashutosh
On Wed, 24 May 2023 02:12:31 -0700, Andrzej Hajda wrote: > Hi Andrzej, > On 23.05.2023 17:19, Ashutosh Dixit wrote: > > pmu_needs_timer() keeps the timer running even when GT is parked, > > ostensibly to sample requested/actual frequencies. However > > frequency_sample() has the following: > > >

[PATCH 2/2] drm/i915/pmu: Make PMU sample array two-dimensional

2023-05-24 Thread Ashutosh Dixit
No functional changes but we can remove some unsightly index computation and read/write functions if we convert the PMU sample array from a one-dimensional to a two-dimensional array. v2: Retain read/store helpers (Tvrtko) Suggested-by: Tvrtko Ursulin Reviewed-by: Andrzej Hajda Reviewed-by:

[PATCH 1/2] drm/i915/pmu: Turn off the timer to sample frequencies when GT is parked

2023-05-24 Thread Ashutosh Dixit
pmu_needs_timer() keeps the timer running even when GT is parked, ostensibly to sample requested/actual frequencies. However frequency_sample() has the following: /* Report 0/0 (actual/requested) frequency while parked. */ if (!intel_gt_pm_get_if_awake(gt)) return;

[PATCH v2 0/2] drm/i915/pmu: couple of cleanups

2023-05-24 Thread Ashutosh Dixit
Cc: Andrzej Hajda Cc: Tvrtko Ursulin Signed-off-by: Ashutosh Dixit Ashutosh Dixit (2): drm/i915/pmu: Turn off the timer to sample frequencies when GT is parked drm/i915/pmu: Make PMU sample array two-dimensional drivers/gpu/drm/i915/i915_pmu.c | 32

Re: [PATCH net-next v4 2/6] dt-bindings: net: Brcm ASP 2.0 Ethernet controller

2023-05-24 Thread Conor Dooley
On Wed, May 24, 2023 at 02:47:59PM -0700, Justin Chen wrote: > On Tue, May 23, 2023 at 11:56 PM Conor Dooley > wrote: > Gotcha. I got something like this now. > > compatible: > oneOf: > - items: > - enum: > - brcm,bcm74165-asp > - const:

Re: [PATCH net-next v4 2/6] dt-bindings: net: Brcm ASP 2.0 Ethernet controller

2023-05-24 Thread Justin Chen
On Tue, May 23, 2023 at 11:56 PM Conor Dooley wrote: > > On Wed, May 24, 2023 at 07:51:07AM +0100, Conor Dooley wrote: > > Hey Justin, > > On Tue, May 23, 2023 at 04:27:12PM -0700, Justin Chen wrote: > > > On Tue, May 23, 2023 at 3:55 PM Conor Dooley wrote: > > > > On Tue, May 23, 2023 at

Re: [v4,13/13] drm/i915: Implement dedicated fbdev I/O helpers

2023-05-24 Thread Sui Jingfeng
Hi, I have just tested this patch on my i3-8100@3.6Ghz cpu + h110 motherboard with fbtest: Benchmarking... 10x10 squares: 26.44 Mpixels/s Benchmarking... 20x20 squares: 52.54 Mpixels/s Benchmarking... 50x50 squares: 128.13 Mpixels/s Benchmarking... 100x100 squares: 252.21 Mpixels/s

Re: [v2 4/4] dt-bindings: display: panel: Add compatible for Starry ili9882t

2023-05-24 Thread Doug Anderson
Hi, On Wed, May 24, 2023 at 12:45 AM Cong Yang wrote: > > The STARRY ili9882t is a 10.51" WUXGA TFT LCD panel, > which fits in nicely with the existing panel-boe-tv101wum-nl6 > driver. Hence, we add a new compatible with panel specific config. > > Signed-off-by: Cong Yang > --- >

Re: [v2 2/4] dt-bindings: display: panel: Add compatible for Starry himax83102-j02

2023-05-24 Thread Doug Anderson
Hi, On Wed, May 24, 2023 at 12:45 AM Cong Yang wrote: > > The STARRY himax83102-j02 is a 10.51" WUXGA TFT LCD panel, > which fits in nicely with the existing panel-boe-tv101wum-nl6 > driver. Hence, we add a new compatible with panel specific config. > > Signed-off-by: Cong Yang > --- >

Re: [v2 3/4] drm/panel: Support for Starry-ili9882t TDDI MIPI-DSI panel

2023-05-24 Thread Doug Anderson
Hi, On Wed, May 24, 2023 at 12:45 AM Cong Yang wrote: > > The Starry-ili9882 is a 10.51" WUXGA TFT panel. which fits in nicely with > the existing panel-boe-tv101wum-nl6 driver. From the datasheet,MIPI need > to keep the LP11 state before the lcm_reset pin is pulled high. So add >

[pull] amdgpu, radeon drm-fixes-6.4

2023-05-24 Thread Alex Deucher
Hi Dave, Daniel, Fixes for 6.4. The following changes since commit 79ef1c9d14c65a5c3f7eec47389d8c2a33be8e8d: Merge tag 'amd-drm-fixes-6.4-2023-05-18' of https://gitlab.freedesktop.org/agd5f/linux into drm-fixes (2023-05-19 11:26:21 +1000) are available in the Git repository at:

Re: [v2 1/4] drm/panel: Support for Starry-himax83102-j02 TDDI MIPI-DSI panel

2023-05-24 Thread Doug Anderson
Hi, On Wed, May 24, 2023 at 12:28 AM Cong Yang wrote: > > The Starry-himax83102-j02 is a 10.51" WUXGA TFT panel. which fits in nicely > with the existing panel-boe-tv101wum-nl6 driver. From the datasheet[1], MIPI > needs to keep the LP11 state before the lcm_reset pin is pulled high, so >

[PATCH] drm/tegra: dc: cap non-cursor plane zpos to 254

2023-05-24 Thread Emmanuel Gil Peyrot
Since cursor plane has the immutable zpos 255, other planes can’t take its place so we can cap them to 254 instead. Thanks emersion! Signed-off-by: Emmanuel Gil Peyrot --- drivers/gpu/drm/tegra/dc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git

Re: [v4,02/13] fbdev: Add initializer macros for struct fb_ops

2023-05-24 Thread Sui Jingfeng
Hi, we love your patch: On 2023/5/24 17:21, Thomas Zimmermann wrote: For framebuffers in I/O and system memory, add macros that set struct fb_ops to the respective callback functions. For deferred I/O, add macros that generate callback functions with damage handling. Add initializer macros

Re: [v4, 01/13] fbdev: Add Kconfig options to select different fb_ops helpers

2023-05-24 Thread Sui Jingfeng
Reviewed-by: Sui Jingfeng On 2023/5/24 17:21, Thomas Zimmermann wrote: Many fbdev drivers use the same set of fb_ops helpers. Add Kconfig options to select them at once. This will help with making DRM's fbdev emulation code more modular, but can also be used to simplify fbdev's driver

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Hamza Mahfooz
+ Kees On 5/24/23 15:50, Alex Deucher wrote: On Wed, May 24, 2023 at 3:46 PM Felix Kuehling wrote: Sure, I think we tried enabling warnings as errors before and had to revert it because of weird compiler quirks or the variety of compiler versions that need to be supported. Alex, are you

Re: [v4,12/13] drm/fbdev-generic: Implement dedicated fbdev I/O helpers

2023-05-24 Thread Sui Jingfeng
Tested-by: Sui Jingfeng This version works fine, I have tested it On LoongArch with drm/loongson(v14) as it is most handy for me. Also because it using fbdev-generic. fbdev of IGT report no error. Then I run fbtest from geert[1], Before apply your patch: Benchmarking... 10x10 squares:

Re: [v2 0/4] Support Starry-himax83102-j02 and Starry-ili9882t TDDI MIPI-DSI panel

2023-05-24 Thread Conor Dooley
On Wed, May 24, 2023 at 03:28:12PM +0800, Cong Yang wrote: > Compare V1:Add compatible for Starry himax83102-j02 and Starry-ili9882t > in dt-bindings. BTW, my mailer doesn't like how you threaded these patches, I guess you sent them as a reply to something I was not CCed on. > dt-bindings:

[PATCH v12 1/1] drm/i915: Allow user to set cache at BO creation

2023-05-24 Thread fei . yang
From: Fei Yang To comply with the design that buffer objects shall have immutable cache setting through out their life cycle, {set, get}_caching ioctl's are no longer supported from MTL onward. With that change caching policy can only be set at object creation time. The current code applies a

[PATCH v12 0/1] drm/i915: Allow user to set cache at BO creation

2023-05-24 Thread fei . yang
From: Fei Yang This series introduce a new extension for GEM_CREATE, 1. end support for set caching ioctl [PATCH 1/2] 2. add set_pat extension for gem_create [PATCH 2/2] v2: drop one patch that was merged separately commit 341ad0e8e254 ("drm/i915/mtl: Add PTE encode function") v3: rebased

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Hamza Mahfooz
On 5/24/23 15:54, Harry Wentland wrote: On 5/24/23 15:27, Hamza Mahfooz wrote: On 5/24/23 15:22, Alex Deucher wrote: On Wed, May 24, 2023 at 3:20 PM Hamza Mahfooz wrote: Currently, there are quite a number of issues that are quite easy for the CI to catch, that slip through the cracks.

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Ho, Kenny
[AMD Official Use Only - General] I ran some experiment yesterday to turn on CONFIG_WERROR and here are some results (summary: gcc 12 seems ok but gcc 13 gives a lot more error, but not necessarily in our module.) Build with gcc13 using Fedora 38:

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Harry Wentland
On 5/24/23 15:27, Hamza Mahfooz wrote: > On 5/24/23 15:22, Alex Deucher wrote: >> On Wed, May 24, 2023 at 3:20 PM Hamza Mahfooz wrote: >>> >>> Currently, there are quite a number of issues that are quite easy for >>> the CI to catch, that slip through the cracks. Among them, there are >>>

Re: [PATCH v3 09/17] drm/amd/display: Register Colorspace property for DP and HDMI

2023-05-24 Thread Harry Wentland
On 3/17/23 09:53, Joshua Ashton wrote: > > > On 3/17/23 13:35, Pekka Paalanen wrote: >> On Fri, 17 Mar 2023 14:50:40 +0200 >> Ville Syrjälä wrote: >> >>> On Fri, Mar 17, 2023 at 10:53:35AM +0200, Pekka Paalanen wrote: On Fri, 17 Mar 2023 01:01:38 +0200 Ville Syrjälä wrote:   

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Alex Deucher
On Wed, May 24, 2023 at 3:46 PM Felix Kuehling wrote: > > Sure, I think we tried enabling warnings as errors before and had to > revert it because of weird compiler quirks or the variety of compiler > versions that need to be supported. > > Alex, are you planning to upstream this, or is this only

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Felix Kuehling
Sure, I think we tried enabling warnings as errors before and had to revert it because of weird compiler quirks or the variety of compiler versions that need to be supported. Alex, are you planning to upstream this, or is this only to enforce more internal discipline about not ignoring

RE: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Russell, Kent
[AMD Official Use Only - General] (Adding Felix in CC) I’m a fan of adding it to KFD as well. Felix, can you foresee any issues here? Kent From: amd-gfx On Behalf Of Ho, Kenny Sent: Wednesday, May 24, 2023 3:23 PM To: Alex Deucher ; Mahfooz, Hamza Cc: Li, Sun peng (Leo) ; Wentland, Harry ;

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Hamza Mahfooz
On 5/24/23 15:22, Alex Deucher wrote: On Wed, May 24, 2023 at 3:20 PM Hamza Mahfooz wrote: Currently, there are quite a number of issues that are quite easy for the CI to catch, that slip through the cracks. Among them, there are unused variable and indentation issues. Also, we should

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Ho, Kenny
[AMD Official Use Only - General] (+ Felix) Should we do the same for other modules under amd (amdkfd)? I was going to enable full kernel werror in the kconfig used by my CI but this is probably better. Kenny From: Alex Deucher Sent: Wednesday, May 24, 2023

Re: [PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Alex Deucher
On Wed, May 24, 2023 at 3:20 PM Hamza Mahfooz wrote: > > Currently, there are quite a number of issues that are quite easy for > the CI to catch, that slip through the cracks. Among them, there are > unused variable and indentation issues. Also, we should consider all > warnings to be compile

Re: [PATCH 1/2] dt-bindings: display: panel-simple: Add Ampire AM-800480L1TMQW-T00H

2023-05-24 Thread Conor Dooley
On Wed, May 24, 2023 at 02:32:10PM +0200, Geert Uytterhoeven wrote: > Document support for the Ampire AM-800480L1TMQW-T00H 5" WVGA TFT LCD > panel. > > Signed-off-by: Geert Uytterhoeven Acked-by: Conor Dooley Thanks, Conor. signature.asc Description: PGP signature

[PATCH v2] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Hamza Mahfooz
Currently, there are quite a number of issues that are quite easy for the CI to catch, that slip through the cracks. Among them, there are unused variable and indentation issues. Also, we should consider all warnings to be compile errors, since the community will eventually end up complaining

Re: [PATCH 1/2] drm/msm/dpu: drop SSPP register dumpers

2023-05-24 Thread Abhinav Kumar
On 5/24/2023 2:48 AM, Marijn Suijten wrote: On 2023-05-23 13:01:13, Abhinav Kumar wrote: On 5/21/2023 10:21 AM, Dmitry Baryshkov wrote: Drop SSPP-specifig debugfs register dumps in favour of using debugfs/dri/0/kms or devcoredump. I did see another series which removes src_blk from the

[PATCH] drm/amd/display: enable more strict compile checks

2023-05-24 Thread Hamza Mahfooz
Currently, there are quite a number of issues that are quite easy for the CI to catch, that slip through the cracks. Among them, there unused variable and indentation issues. Also, we should consider all warnings to be compile errors, since the community will eventually end up complaining about

Re: [PATCH v14 3/9] drm/display/dsc: Add drm_dsc_get_bpp_int helper

2023-05-24 Thread Marijn Suijten
On 2023-05-24 10:45:16, Jessica Zhang wrote: > Add helper to get the integer value of drm_dsc_config.bits_per_pixel > > Reviewed-by: Marijn Suijten > Reviewed-by: Dmitry Baryshkov > Signed-off-by: Jessica Zhang Nit: if there comes a v15, perhaps this can be squashed with patch 1/9? I always

Re: [PATCH v14 0/9] Introduce MSM-specific DSC helpers

2023-05-24 Thread Marijn Suijten
On 2023-05-24 10:45:13, Jessica Zhang wrote: > There are some overlap in calculations for MSM-specific DSC variables > between DP and DSI. In addition, the calculations for initial_scale_value > and det_thresh_flatness that are defined within the DSC 1.2 specifications, > but aren't yet included

Re: [PATCH v14 1/9] drm/display/dsc: Add flatness and initial scale value calculations

2023-05-24 Thread Marijn Suijten
On 2023-05-24 10:45:14, Jessica Zhang wrote: > Add helpers to calculate det_thresh_flatness and initial_scale_value as > these calculations are defined within the DSC spec. > > Reviewed-by: Marijn Suijten > Reviewed-by: Dmitry Baryshkov > Signed-off-by: Jessica Zhang > --- >

Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()

2023-05-24 Thread Nick Desaulniers
On Wed, May 24, 2023 at 11:41 AM Nathan Chancellor wrote: > > On Wed, May 24, 2023 at 11:32:32AM -0700, Nick Desaulniers wrote: > > On Wed, May 24, 2023 at 8:38 AM Nathan Chancellor wrote: > > > > > > Clang warns: > > > > > > drivers/gpu/drm/i915/display/intel_display.c:6012:3: error:

Re: [PATCH] drm/amdgpu: Fix return types of certain NBIOv7.9 callbacks

2023-05-24 Thread Alex Deucher
PF0_PARTITION_COMPUTE_STATUS, tmp); > } > > -static enum amdgpu_memory_partition > -nbio_v7_9_get_memory_partition_mode(struct amdgpu_device *adev, u32 > *supp_modes) > +static u32 nbio_v7_9_get_memory_partition_mode(struct amdgpu_device *adev, > +

Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()

2023-05-24 Thread Nathan Chancellor
tel_async_flip_check_hw(struct > > intel_atomic_state *state, struct in > > plane->base.base.id, > > plane->base.name); > > return -EINVAL; > > } > > +

Re: [PATCH] drm/i915: Fix clang -Wimplicit-fallthrough in intel_async_flip_check_hw()

2023-05-24 Thread Nick Desaulniers
t; plane->base.base.id, > plane->base.name); > return -EINVAL; > } > + break; > > case I915_FORMAT_MOD_X_TILED: >

Re: [PATCH v2] amdgpu: validate drm_amdgpu_gem_va addrs

2023-05-24 Thread Christian König
Am 24.05.23 um 00:53 schrieb Chia-I Wu: Validate drm_amdgpu_gem_va addrs in amdgpu_gem_va_ioctl. amdgpu_vm_bo_replace_map no longer needs to validate (and its validations were insufficient either). amdgpu_vm_bo_map has internal users and its validations are kept. No, please keep all

Re: [PATCH 1/4] fbdev: imsttfb: Fix error handling in init_imstt()

2023-05-24 Thread Helge Deller
On 5/23/23 19:38, Markus Elfring wrote: From: Markus Elfring Date: Tue, 23 May 2023 14:32:39 +0200 The return value was overlooked from a call of the function “fb_alloc_cmap”. * Thus use a corresponding error check. * Add two jump targets so that a bit of exception handling can be better

Re: [PATCH v3 09/17] drm/amd/display: Register Colorspace property for DP and HDMI

2023-05-24 Thread Harry Wentland
On 3/8/23 04:24, Pekka Paalanen wrote: > On Tue, 7 Mar 2023 10:10:59 -0500 > Harry Wentland wrote: > >> We want compositors to be able to set the output >> colorspace on DP and HDMI outputs, based on the >> caps reported from the receiver via EDID. >> >> Signed-off-by: Harry Wentland >> Cc:

Re: [PATCH 3/4] fbdev: imsttfb: Move a variable assignment for an error code in imsttfb_probe()

2023-05-24 Thread Helge Deller
On 5/23/23 19:42, Markus Elfring wrote: From: Markus Elfring Date: Tue, 23 May 2023 18:30:26 +0200 The value “-ENOMEM” was assigned to the variable “ret” at the beginning. Move this statement directly before the first ioremap() call. Please do not move such variables without real need. It

Re: [PATCH 1/2] fbdev: Move two variable assignments in fb_alloc_cmap_gfp()

2023-05-24 Thread Helge Deller
On 5/23/23 22:15, Markus Elfring wrote: From: Markus Elfring Date: Tue, 23 May 2023 21:30:29 +0200 Move the assignment for the local variables “size” and “flags” because the computed values were only used in a single if branch. Please do not move such variables without real need. It makes

[PULL] drm-intel-gt-next

2023-05-24 Thread Tvrtko Ursulin
Hi Dave, Daniel, Here goes the first pull request for 6.5. In terms of headline features probably the fact GuC platforms can now show per client engine utilisation sticks out the most. Then there is a bunch of fixes where those in the area of GuC error capture, SLPS and firmware loading

Re: [PATCH 2/2] drm/i915/pmu: Make PMU sample array two-dimensional

2023-05-24 Thread Tvrtko Ursulin
On 24/05/2023 18:38, Dixit, Ashutosh wrote: On Wed, 24 May 2023 04:38:18 -0700, Tvrtko Ursulin wrote: Hi Tvrtko, On 23/05/2023 16:19, Ashutosh Dixit wrote: No functional changes but we can remove some unsightly index computation and read/write functions if we convert the PMU sample

[PATCH v14 8/9] drm/msm/dsi: Use MSM and DRM DSC helper methods

2023-05-24 Thread Jessica Zhang
Use MSM and DRM DSC helper methods to configure DSC for DSI. Reviewed-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten Signed-off-by: Jessica Zhang --- drivers/gpu/drm/msm/dsi/dsi_host.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

[PATCH v14 6/9] drm/msm/dpu: Use fixed DRM DSC helper for det_thresh_flatness

2023-05-24 Thread Jessica Zhang
The current dpu_hw_dsc calculation for det_thresh_flatness does not match the downstream calculation or the DSC spec. Use the DRM DSC helper for det_thresh_flatness to match downstream implementation and the DSC spec. Fixes: c110cfd1753e ("drm/msm/disp/dpu1: Add support for DSC") Reviewed-by:

[PATCH v14 4/9] drm/msm/dsi: use DRM DSC helpers for DSC setup

2023-05-24 Thread Jessica Zhang
From: Dmitry Baryshkov Use new DRM DSC helpers to setup DSI DSC configuration. The initial_scale_value needs to be adjusted according to the standard, but this is a separate change. Signed-off-by: Dmitry Baryshkov Reviewed-by: Abhinav Kumar Reviewed-by: Marijn Suijten Signed-off-by: Jessica

[PATCH v14 1/9] drm/display/dsc: Add flatness and initial scale value calculations

2023-05-24 Thread Jessica Zhang
Add helpers to calculate det_thresh_flatness and initial_scale_value as these calculations are defined within the DSC spec. Reviewed-by: Marijn Suijten Reviewed-by: Dmitry Baryshkov Signed-off-by: Jessica Zhang --- drivers/gpu/drm/display/drm_dsc_helper.c | 24

[PATCH v14 7/9] drm/msm/dpu: Fix slice_last_group_size calculation

2023-05-24 Thread Jessica Zhang
Correct the math for slice_last_group_size so that it matches the calculations downstream. Fixes: c110cfd1753e ("drm/msm/disp/dpu1: Add support for DSC") Reviewed-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten Signed-off-by: Jessica Zhang --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_dsc.c | 5

[PATCH v14 2/9] drm/display/dsc: add helper to set semi-const parameters

2023-05-24 Thread Jessica Zhang
From: Dmitry Baryshkov Add a helper setting config values which are typically constant across operating modes (table E-4 of the standard) and mux_word_size (which is a const according to 3.5.2). Signed-off-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten Signed-off-by: Jessica Zhang ---

[PATCH v14 9/9] drm/msm/dsi: update hdisplay calculation for dsi_timing_setup

2023-05-24 Thread Jessica Zhang
Currently, hdisplay is being divided by 3 for DSC. However, this calculation only works for cases where BPP = 8. Update hdisplay calculation to be bytes_per_line / 3, so that it accounts for cases where BPP != 8. Reviewed-by: Dmitry Baryshkov Reviewed-by: Marijn Suijten Signed-off-by: Jessica

[PATCH v14 3/9] drm/display/dsc: Add drm_dsc_get_bpp_int helper

2023-05-24 Thread Jessica Zhang
Add helper to get the integer value of drm_dsc_config.bits_per_pixel Reviewed-by: Marijn Suijten Reviewed-by: Dmitry Baryshkov Signed-off-by: Jessica Zhang --- drivers/gpu/drm/display/drm_dsc_helper.c | 13 + include/drm/display/drm_dsc_helper.h | 1 + 2 files changed, 14

[PATCH v14 0/9] Introduce MSM-specific DSC helpers

2023-05-24 Thread Jessica Zhang
There are some overlap in calculations for MSM-specific DSC variables between DP and DSI. In addition, the calculations for initial_scale_value and det_thresh_flatness that are defined within the DSC 1.2 specifications, but aren't yet included in drm_dsc_helper.c. This series moves these

[PATCH v14 5/9] drm/msm: Add MSM-specific DSC helper methods

2023-05-24 Thread Jessica Zhang
Introduce MSM-specific DSC helper methods, as some calculations are common between DP and DSC. Reviewed-by: Marijn Suijten Signed-off-by: Jessica Zhang --- drivers/gpu/drm/msm/msm_dsc_helper.h | 38 1 file changed, 38 insertions(+) diff --git

Re: [PATCH 2/2] drm/i915/pmu: Make PMU sample array two-dimensional

2023-05-24 Thread Dixit, Ashutosh
On Wed, 24 May 2023 04:38:18 -0700, Tvrtko Ursulin wrote: > Hi Tvrtko, > On 23/05/2023 16:19, Ashutosh Dixit wrote: > > No functional changes but we can remove some unsightly index computation > > and read/write functions if we convert the PMU sample array from a > > one-dimensional to a

Re: [PATCH 7/9] HID: i2c-hid: Support being a panel follower

2023-05-24 Thread Doug Anderson
Hi, On Tue, May 23, 2023 at 12:31 PM Douglas Anderson wrote: > > As talked about in the patch ("drm/panel: Add a way for other devices > to follow panel state"), we really want to keep the power states of a > touchscreen and the panel it's attached to in sync with each other. In > that spirit,

  1   2   3   >