Re: [PATCH 03/19] iosys-map: Add a few more helpers

2022-02-04 Thread Lucas De Marchi
On Fri, Feb 04, 2022 at 08:05:56PM +0100, Thomas Zimmermann wrote: Hi Am 04.02.22 um 18:44 schrieb Lucas De Marchi: First the simplest ones: - iosys_map_memset(): when abstracting system and I/O memory, just like the memcpy() use case, memset() also has dedicated fu

Re: [PATCH 15/21] fbcon: Consistently protect deferred_takeover with console_lock()

2022-02-04 Thread Sam Ravnborg
On Mon, Jan 31, 2022 at 10:05:46PM +0100, Daniel Vetter wrote: > This shouldn't be a problem in practice since until we've actually > taken over the console there's nothing we've registered with the > console/vt subsystem, so the exit/unbind path that check this can't > do the wrong thing. But it's

Re: [PATCH 16/21] fbcon: Move console_lock for register/unlink/unregister

2022-02-04 Thread Sam Ravnborg
Hi Daniel. On Mon, Jan 31, 2022 at 10:05:47PM +0100, Daniel Vetter wrote: > Ideally console_lock becomes an implementation detail of fbcon.c and > doesn't show up anywhere in fbmem.c. We're still pretty far from that, > but at least the register/unregister code is there now. > > With this the do_

Re: [PATCH 17/21] fbcon: Move more code into fbcon_release

2022-02-04 Thread Sam Ravnborg
On Mon, Jan 31, 2022 at 10:05:48PM +0100, Daniel Vetter wrote: > con2fb_release_oldinfo() has a bunch more kfree() calls than > fbcon_exit(), but since kfree() on NULL is harmless doing that in both > places should be ok. This is also a bit more symmetric now again with > fbcon_open also allocating

Re: [PATCH 18/21] fbcon: untangle fbcon_exit

2022-02-04 Thread Sam Ravnborg
Hi Daniel, On Mon, Jan 31, 2022 at 10:05:49PM +0100, Daniel Vetter wrote: > There's a bunch of confusions going on here: > - The deferred fbcon setup notifier should only be cleaned up from > fb_console_exit(), to be symmetric with fb_console_init() > - We also need to make sure we don't race wi

Re: [PATCH 19/21] fbcon: Maintain a private array of fb_info

2022-02-04 Thread Sam Ravnborg
Hi Daniel, On Mon, Jan 31, 2022 at 10:05:50PM +0100, Daniel Vetter wrote: > Accessing the one in fbmem.c without taking the right locks is a bad > idea. Instead maintain our own private copy, which is fully protected > by console_lock() (like everything else in fbcon.c). That copy is > serialized

Re: [git pull] drm fixes for 5.17-rc3

2022-02-04 Thread pr-tracker-bot
The pull request you sent on Fri, 4 Feb 2022 16:09:15 +1000: > git://anongit.freedesktop.org/drm/drm tags/drm-fixes-2022-02-04 has been merged into torvalds/linux.git: https://git.kernel.org/torvalds/c/31462d9e47cf6e2cb10a69c833b5e081fff7086d Thank you! -- Deet-doot-dot, I am a bot. https://ko

Re: [PATCH v2 1/4] drm/format-helper: Add drm_fb_{xrgb8888,gray8}_to_mono_reversed()

2022-02-04 Thread Thomas Zimmermann
Hi Am 04.02.22 um 20:31 schrieb Javier Martinez Canillas: Hello Thomas, Thanks a lot for your feedback. On 2/4/22 16:52, Thomas Zimmermann wrote: [snip] +static void drm_fb_gray8_to_mono_reversed_line(u8 *dst, const u8 *src, size_t pixels) +{ + unsigned int xb, i; + + for (xb =

Re: [PATCH v2 1/4] drm/format-helper: Add drm_fb_{xrgb8888, gray8}_to_mono_reversed()

2022-02-04 Thread Ilia Mirkin
On Fri, Feb 4, 2022 at 10:53 AM Thomas Zimmermann wrote: > > Hi > > Am 04.02.22 um 14:43 schrieb Javier Martinez Canillas: > > Add support to convert XR24 and 8-bit grayscale to reversed monochrome for > > drivers that control monochromatic panels, that only have 1 bit per pixel. > > > > The drm_f

[PATCH 00/12] Add writeback block support for DPU

2022-02-04 Thread Abhinav Kumar
This series adds support for writeback block on DPU. Writeback block is extremely useful to validate boards having no physical displays in addition to many other use-cases where we want to get the output of the display pipeline to examine whether issue is with the display pipeline or with the panel

[PATCH 01/12] drm/msm/dpu: add writeback blocks to the sm8250 DPU catalog

2022-02-04 Thread Abhinav Kumar
Add writeback blocks to the sm8250 DPU hardware catalog. Other chipsets support writeback too but add it to sm8250 to prototype the feature so that it can be easily extended to other chipsets. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_hw_catalog.c | 73 ++

[PATCH 02/12] drm/msm/dpu: add dpu_hw_wb abstraction for writeback blocks

2022-02-04 Thread Abhinav Kumar
Add the dpu_hw_wb abstraction to program registers related to the writeback block. These will be invoked once all the configuration is set and ready to be programmed to the registers. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/Makefile | 1 + drivers/gpu/drm/msm/disp/dpu

[PATCH 03/12] drm/msm/dpu: add writeback blocks to DPU RM

2022-02-04 Thread Abhinav Kumar
Add writeback blocks to DPU resource manager so that writeback encoders can request for writeback hardware blocks through RM and their usage can be tracked. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.h | 3 ++ drivers/gpu/drm/msm/disp/dpu1/dpu_kms.h | 2 + d

[PATCH 06/12] drm/msm/dpu: make changes to dpu_encoder to support virtual encoder

2022-02-04 Thread Abhinav Kumar
Make changes to dpu_encoder to support virtual encoder needed to support writeback for dpu. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 57 + 1 file changed, 42 insertions(+), 15 deletions(-) diff --git a/drivers/gpu/drm/msm/disp/dp

[PATCH 04/12] drm/msm/dpu: add changes to support writeback in hw_ctl

2022-02-04 Thread Abhinav Kumar
Add changes to support writeback module in the dpu_hw_ctl interface. In addition inroduce a reset_intf_cfg op to reset the interface bits for the currently active interfaces in the ctl path. Signed-off-by: Abhinav Kumar --- .../gpu/drm/msm/disp/dpu1/dpu_encoder_phys_cmd.c | 3 +- .../gpu/drm/

[PATCH 05/12] drm/msm/dpu: add an API to reset the encoder related hw blocks

2022-02-04 Thread Abhinav Kumar
Add an API to reset the encoder related hw blocks to ensure a proper teardown of the pipeline. At the moment this is being used only for the writeback encoder but eventually we can start using this for all interfaces. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c

[PATCH 08/12] drm/msm/dpu: introduce the dpu_encoder_phys_* for writeback

2022-02-04 Thread Abhinav Kumar
Introduce the dpu_encoder_phys_* for the writeback interface to handle writeback specific hardware programming. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/Makefile | 1 + drivers/gpu/drm/msm/disp/dpu1/dpu_encoder_phys.h | 36 +- .../gpu/drm/msm/disp/dpu1/dpu_

[PATCH 07/12] drm/msm/dpu: add encoder operations to prepare/cleanup wb job

2022-02-04 Thread Abhinav Kumar
add dpu encoder APIs to prepare and cleanup writeback job for the writeback encoder. These shall be invoked from the prepare_wb_job/cleanup_wb_job hooks of the drm_writeback framework. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 34

[PATCH 09/12] drm/msm/dpu: add the writeback connector layer

2022-02-04 Thread Abhinav Kumar
Introduce the dpu_writeback module which serves as the interface between dpu operations and the drm_writeback. This module manages the connector related operations for dpu writeback. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/Makefile | 1 + drivers/gpu/drm/msm/disp/

[PATCH 11/12] drm/msm/dpu: gracefully handle null fb commits for writeback

2022-02-04 Thread Abhinav Kumar
kms_writeback test cases also verify with a null fb for the writeback connector job. In addition there are also other commit paths which can result in kickoffs without a valid framebuffer like while closing the fb which results in the callback to drm_atomic_helper_dirtyfb() which internally trigger

[PATCH 10/12] drm/msm/dpu: initialize dpu encoder and connector for writeback

2022-02-04 Thread Abhinav Kumar
Initialize dpu encoder and connector for writeback if the target supports it in the catalog. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 37 - drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 62 + 2 files changed, 88 in

[PATCH 12/12] drm/msm/dpu: add writeback blocks to the display snapshot

2022-02-04 Thread Abhinav Kumar
Add writeback block information while capturing the display snapshot. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c index 6327ba9

Re: [PATCH] drm/panel: simple: Assign data from panel_dpi_probe() correctly

2022-02-04 Thread Sam Ravnborg
On Tue, Feb 01, 2022 at 12:01:53PM +0100, Christoph Niedermaier wrote: > In the function panel_simple_probe() the pointer panel->desc is > assigned to the passed pointer desc. If function panel_dpi_probe() > is called panel->desc will be updated, but further on only desc > will be evaluated. So upd

Re: [PATCH v3 3/3] drm/msm/dp: enable widebus feature for display port

2022-02-04 Thread Dmitry Baryshkov
On 04/02/2022 21:36, Kuogee Hsieh wrote: Widebus feature will transmit two pixel data per pixel clock to interface. This feature now is required to be enabled to easy migrant to higher resolution applications in future. However since some legacy chipsets does not support this feature, this featur

Re: [PATCH v3 1/3] drm/msm/dp: revise timing engine programming to support widebus feature

2022-02-04 Thread Dmitry Baryshkov
On 04/02/2022 21:36, Kuogee Hsieh wrote: Widebus feature will transmit two pixel data per pixel clock to interface. Timing engine provides driving force for this purpose. This patch base on HPG (Hardware Programming Guide) to revise timing engine register setting to accommodate both widebus and n

Re: [PATCH v3 2/3] drm/msm/dp: revise timing engine programming to support compression (DSC)

2022-02-04 Thread Dmitry Baryshkov
On 04/02/2022 21:36, Kuogee Hsieh wrote: Divides horizontal width by 3 at timing engine of interface. There are major part of compression (DSC) programming have to be done at DSC controller which is not covered by this patch. Signed-off-by: Kuogee Hsieh --- drivers/gpu/drm/msm/disp/dpu1/dpu_

Re: [PATCH v3 1/3] drm/msm/dp: revise timing engine programming to support widebus feature

2022-02-04 Thread Dmitry Baryshkov
On 04/02/2022 21:36, Kuogee Hsieh wrote: Widebus feature will transmit two pixel data per pixel clock to interface. Timing engine provides driving force for this purpose. This patch base on HPG (Hardware Programming Guide) to revise timing engine register setting to accommodate both widebus and n

Re: [PATCH 04/12] drm/msm/dpu: add changes to support writeback in hw_ctl

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Add changes to support writeback module in the dpu_hw_ctl interface. In addition inroduce a reset_intf_cfg op to reset the interface bits for the currently active interfaces in the ctl path. Signed-off-by: Abhinav Kumar --- .../gpu/drm/msm/disp/dpu1/d

Re: [PATCH v6 4/4] dt-bindings: drm/bridge: anx7625: Add aux-bus node

2022-02-04 Thread Rob Herring
On Thu, 03 Feb 2022 22:10:23 +0800, Hsin-Yi Wang wrote: > List panel under aux-bus node if it's connected to anx7625's aux bus. > > Signed-off-by: Hsin-Yi Wang > --- > .../display/bridge/analogix,anx7625.yaml| 17 + > 1 file changed, 17 insertions(+) > Reviewed-by: Rob

Re: [PATCH 10/12] drm/msm/dpu: initialize dpu encoder and connector for writeback

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Initialize dpu encoder and connector for writeback if the target supports it in the catalog. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 37 - drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 62 +++

Re: [PATCH 12/12] drm/msm/dpu: add writeback blocks to the display snapshot

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Add writeback block information while capturing the display snapshot. Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm

Re: [v5 1/3] dt-bindings: msm/dsi: Add 10nm dsi phy tuning properties

2022-02-04 Thread Rob Herring
On Mon, 31 Jan 2022 01:45:17 +0530, Rajeev Nandan wrote: > In most cases, the default values of DSI PHY tuning registers should be > sufficient as they are fully optimized. However, in some cases where > extreme board parasitics cause the eye shape to degrade, the override > bits can be used to imp

Re: [PATCH 11/12] drm/msm/dpu: gracefully handle null fb commits for writeback

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: kms_writeback test cases also verify with a null fb for the writeback connector job. In addition there are also other commit paths which can result in kickoffs without a valid framebuffer like while closing the fb which results in the callback to drm_atom

Re: [PATCH 01/12] drm/msm/dpu: add writeback blocks to the sm8250 DPU catalog

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Add writeback blocks to the sm8250 DPU hardware catalog. Other chipsets support writeback too but add it to sm8250 to prototype the feature so that it can be easily extended to other chipsets. Signed-off-by: Abhinav Kumar Reviewed-by: Dmitry Baryshkov

Re: [PATCH 02/12] drm/msm/dpu: add dpu_hw_wb abstraction for writeback blocks

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Add the dpu_hw_wb abstraction to program registers related to the writeback block. These will be invoked once all the configuration is set and ready to be programmed to the registers. Reviewed-by: Dmitry Baryshkov Few minor comments bellow. Signed-

Re: [PATCH 08/12] drm/msm/dpu: introduce the dpu_encoder_phys_* for writeback

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Introduce the dpu_encoder_phys_* for the writeback interface to handle writeback specific hardware programming. Signed-off-by: Abhinav Kumar Mostly looks ok, see minor comments bellow. --- drivers/gpu/drm/msm/Makefile | 1 +

Re: [PATCH 09/12] drm/msm/dpu: add the writeback connector layer

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Introduce the dpu_writeback module which serves as the interface between dpu operations and the drm_writeback. This module manages the connector related operations for dpu writeback. Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/Makefile

Re: [PATCH v3 1/3] drm/msm/dp: revise timing engine programming to support widebus feature

2022-02-04 Thread Kuogee Hsieh
On 2/4/2022 2:05 PM, Dmitry Baryshkov wrote: On 04/02/2022 21:36, Kuogee Hsieh wrote: Widebus feature will transmit two pixel data per pixel clock to interface. Timing engine provides driving force for this purpose. This patch base on HPG (Hardware Programming Guide) to revise timing engine r

Re: [PATCH 06/12] drm/msm/dpu: make changes to dpu_encoder to support virtual encoder

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Make changes to dpu_encoder to support virtual encoder needed to support writeback for dpu. This patch will change significantly if Signed-off-by: Abhinav Kumar --- drivers/gpu/drm/msm/disp/dpu1/dpu_encoder.c | 57 + 1

Re: [PATCH v3 1/3] drm/msm/dp: revise timing engine programming to support widebus feature

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 02:26, Kuogee Hsieh wrote: On 2/4/2022 2:05 PM, Dmitry Baryshkov wrote: On 04/02/2022 21:36, Kuogee Hsieh wrote: Widebus feature will transmit two pixel data per pixel clock to interface. Timing engine provides driving force for this purpose. This patch base on HPG (Hardware Pro

Re: [PATCH 07/12] drm/msm/dpu: add encoder operations to prepare/cleanup wb job

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: add dpu encoder APIs to prepare and cleanup writeback job for the writeback encoder. These shall be invoked from the prepare_wb_job/cleanup_wb_job hooks of the drm_writeback framework. Having dpu-encoder-wide API for the single case of WB looks like an

Re: [PATCH 03/12] drm/msm/dpu: add writeback blocks to DPU RM

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Add writeback blocks to DPU resource manager so that writeback encoders can request for writeback hardware blocks through RM and their usage can be tracked. Signed-off-by: Abhinav Kumar [please excuse me for the duplicate, I've sent the email without

Re: [PATCH 05/12] drm/msm/dpu: add an API to reset the encoder related hw blocks

2022-02-04 Thread Dmitry Baryshkov
On 05/02/2022 00:17, Abhinav Kumar wrote: Add an API to reset the encoder related hw blocks to ensure a proper teardown of the pipeline. At the moment this is being used only for the writeback encoder but eventually we can start using this for all interfaces. Signed-off-by: Abhinav Kumar ---

[PATCH v2 0/3] drm/panel-edp: Debugfs for panel-edp

2022-02-04 Thread Douglas Anderson
The main goal of this series is the final patch in the series: to allow test code to reliably find out if we ended up hitting the "fallback" case of the generic edp-panel driver where we don't recognize a panel and choose to use super conservative timing. Version 1 of the patch actually landed but

[PATCH v2 1/3] drm/bridge: ti-sn65dsi86: Use drm_bridge_connector

2022-02-04 Thread Douglas Anderson
The ti-sn65dsi86 driver shouldn't hand-roll its own bridge connector. It should use the normal drm_bridge_connector. Let's switch to do that, removing all of the custom code. NOTE: this still _doesn't_ implement DRM_BRIDGE_ATTACH_NO_CONNECTOR support for ti-sn65dsi86 and that would still be a usef

[PATCH v2 2/3] drm: Plumb debugfs_init through to panels

2022-02-04 Thread Douglas Anderson
We'd like panels to be able to add things to debugfs underneath the connector's directory. Let's plumb it through. A panel will be able to put things in a "panel" directory under the connector's directory. Note that debugfs is not ABI and so it's always possible that the location that the panel get

[PATCH v2 3/3] drm/panel-edp: Allow querying the detected panel via debugfs

2022-02-04 Thread Douglas Anderson
Recently we added generic "edp-panel"s probed by EDID. To support panels in this way we look at the panel ID in the EDID and look up the panel in a table that has power sequence timings. If we find a panel that's not in the table we will still attempt to use it but we'll use conservative timings. W

Re: [Freedreno] [PATCH v4 1/2] drm/msm/dpu: simplify clocks handling

2022-02-04 Thread Jessica Zhang
On 1/31/2022 1:05 PM, Dmitry Baryshkov wrote: DPU driver contains code to parse clock items from device tree into special data struct and then enable/disable/set rate for the clocks using that data struct. However the DPU driver itself uses only parsing and enabling/disabling part (the rate se

Re: [PATCH v4 2/2] drm/msm/dp: rewrite dss_module_power to use bulk clock functions

2022-02-04 Thread Jessica Zhang
On 1/31/2022 1:05 PM, Dmitry Baryshkov wrote: In order to simplify DP code, drop hand-coded loops over clock arrays, replacing them with clk_bulk_* functions. Signed-off-by: Dmitry Baryshkov Tested-by: Jessica Zhang # RB3 (sdm845) and RB5 (qrb5165) Reviewed-by: Jessica Zhang ---

Re: [PATCH v6 3/4] drm/bridge: anx7625: Support reading edid through aux channel

2022-02-04 Thread kernel test robot
Hi Hsin-Yi, Thank you for the patch! Yet something to improve: [auto build test ERROR on drm/drm-next] [cannot apply to robh/for-next drm-intel/for-linux-next v5.17-rc2] [If your patch is applied to the wrong git tree, kindly drop us a note. And when submitting patch, we suggest to use '--base' a

Re: [PATCH v2] drm/dp: Fix off-by-one in register cache size

2022-02-04 Thread Kees Cook
Ping. This is a OOB read fix. Can something send this to Linus please? -Kees On Wed, Jan 05, 2022 at 09:33:10AM -0800, Kees Cook wrote: > The pcon_dsc_dpcd array holds 13 registers (0x92 through 0x9E). Fix the > math to calculate the max size. Found from a -Warray-bounds build: > > drivers/gpu/d

Re: [PATCH] drm/dp: Remove common Post Cursor2 register handling

2022-02-04 Thread Kees Cook
Ping, This is a OOB read fix. Can someone please pick this up? -Kees On Wed, Jan 05, 2022 at 09:35:07AM -0800, Kees Cook wrote: > The link_status array was not large enough to read the Adjust Request > Post Cursor2 register, so remove the common helper function to avoid > an OOB read, found with

Re: [BUG] gpu: drm: radeon: two possible deadlocks involving locking and waiting

2022-02-04 Thread Jia-Ju Bai
Hi Christian, Thanks for the reply :) On 2022/2/1 15:56, Christian König wrote: Hi Jia-Ju, interesting that you have found those issues with an automated tool. And yes that is a well design flaw within the radeon driver which can happen on hardware faults, e.g. when radeon_ring_backup() need

Re: [PATCH] iommu/vt-d: Remove comment reference to iommu_dev_has_feature

2022-02-04 Thread Lu Baolu
On 2/2/22 10:37 AM, Akeem G Abodunrin wrote: iommu_dev_has_feature() api has been removed by the commit 262948f8ba573 ("iommu: Delete iommu_dev_has_feature()") - So this patch removes comment about the api to avoid any confusion. Signed-off-by: Akeem G Abodunrin Cc: Lu Baolu It's not a chang

[PATCH] backlight: pwm_bl: Avoid open coded arithmetic in memory allocation

2022-02-04 Thread Christophe JAILLET
kmalloc_array()/kcalloc() should be used to avoid potential overflow when a multiplication is needed to compute the size of the requested memory. So turn a kzalloc()+explicit size computation into an equivalent kcalloc(). Signed-off-by: Christophe JAILLET --- drivers/video/backlight/pwm_bl.c |

<    1   2