Re: [PATCH] drm/gma500: Fix comment typo

2022-08-29 Thread Patrik Jakobsson
On Thu, Aug 4, 2022 at 1:48 PM Jason Wang wrote: > > The double `the' is duplicated in the comment, remove one. > > Signed-off-by: Jason Wang Pushed to drm-misc-next Thanks Patrik > --- > drivers/gpu/drm/gma500/cdv_intel_dp.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --

Re: [PATCH] tty/vt: Remove printable variable

2022-08-29 Thread Jiri Slaby
On 26. 08. 22, 22:24, Daniel Vetter wrote: Every since the 0.99.7A release when console_register() was introduced it's become impossible to call vt_console_print (called console_print() back then still) directly. Which means the initialization issue this variable protected against is no more. Gi

Re: [PATCH v1 05/35] drm/connector: Add TV standard property

2022-08-29 Thread Maxime Ripard
On Thu, Aug 25, 2022 at 05:13:29PM +0200, Noralf Trønnes wrote: > > > Den 25.08.2022 15.44, skrev Maxime Ripard: > > Hi, > > > > On Sat, Aug 20, 2022 at 10:12:46PM +0200, Noralf Trønnes wrote: > >>> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h > >>> index 1e9996b33cc8..

Re: [Intel-gfx] [PATCH 03/11] drm/edid: s/monitor_rage/vrr_range/

2022-08-29 Thread Jani Nikula
On Sat, 27 Aug 2022, Ville Syrjala wrote: > From: Ville Syrjälä > > Rename info->monitor_range to info->vrr_range to actually > reflect its usage. > > Cc: Manasi Navare > Cc: Nicholas Kazlauskas > Cc: Harry Wentland > Cc: Leo Li > Cc: Rodrigo Siqueira > Cc: amd-...@lists.freedesktop.org > Si

[PATCH -next 1/4 RESEND] drm/amd/display: clean up one inconsistent indenting

2022-08-29 Thread Yang Li
1. The indentation of statements in the same curly bracket should be consistent. 2. Variable declarations in the same function should be aligned. Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1887 Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1888 Link: https://bugzilla.openanolis.cn

[PATCH -next 4/4 RESEND] drm/amd/display: clean up some inconsistent indentings

2022-08-29 Thread Yang Li
The indentation of statements in the same curly bracket should be consistent. Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1892 Reported-by: Abaci Robot Signed-off-by: Yang Li --- drivers/gpu/drm/amd/display/dc/dcn32/dcn32_hwseq.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions

[PATCH -next 3/4 RESEND] drm/amd/display: clean up some inconsistent indentings

2022-08-29 Thread Yang Li
The indentation of statements in the same curly bracket should be consistent. Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1890 Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1891 Reported-by: Abaci Robot Signed-off-by: Yang Li --- .../amd/display/dc/dcn10/dcn10_hw_sequencer.c | 2

[PATCH -next 2/4 RESEND] drm/amd/display: clean up some inconsistent indentings

2022-08-29 Thread Yang Li
The indentation of statements in the same curly bracket should be consistent. Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=1886 Reported-by: Abaci Robot Signed-off-by: Yang Li --- .../gpu/drm/amd/display/dc/dml/dcn321/dcn321_fpu.c | 13 - 1 file changed, 8 insertions(+), 5

Re: [Intel-gfx] [PATCH 04/11] drm/edid: Define more flags

2022-08-29 Thread Jani Nikula
On Sat, 27 Aug 2022, Ville Syrjala wrote: > From: Ville Syrjälä > > Replace a bunch of hex constants with proper definitions. > > Signed-off-by: Ville Syrjälä Reviewed-by: Jani Nikula > --- > drivers/gpu/drm/drm_edid.c | 18 +- > include/drm/drm_edid.h | 14 +-

Re: [Intel-gfx] [PATCH 06/11] drm/edid: Extract drm_gtf2_mode()

2022-08-29 Thread Jani Nikula
On Sat, 27 Aug 2022, Ville Syrjala wrote: > From: Ville Syrjälä > > Extract the GTF vs. GTF2 logic into a separate function. > We'll have a second user soon. > > Signed-off-by: Ville Syrjälä Reviewed-by: Jani Nikula > --- > drivers/gpu/drm/drm_edid.c | 47

Re: [Intel-gfx] [PATCH 11/11] drm/i915: Infer vrefresh range for eDP if the EDID omits it

2022-08-29 Thread Jani Nikula
On Sat, 27 Aug 2022, Ville Syrjala wrote: > From: Ville Syrjälä > > A bunch of machines seem to have eDP panels where the EDID > indicates continuous frequency support but fails to actually > include the range descirptor. This violates the EDID 1.4 > spec, but looks like the Windows driver just h

Re: [PATCH 05/11] drm/edid: Only parse VRR range for continuous frequency displays

2022-08-29 Thread Jani Nikula
On Sat, 27 Aug 2022, Ville Syrjala wrote: > From: Ville Syrjälä > > Since we only use the parsed vrefresh range to determine > if VRR should be supported we should only accept continuous > frequency displays here. > > Cc: Manasi Navare > Cc: Nicholas Kazlauskas > Cc: Harry Wentland > Cc: Leo L

[PATCH v6 0/2] Add RZ/G2L DSI driver

2022-08-29 Thread Biju Das
This patch series aims to support the MIPI DSI encoder found in the RZ/G2L SoC. It currently supports DSI video mode only. This unit supports MIPI Alliance Specification for Display Serial Interface (DSI) Specification. This unit provides a solution for transmitting MIPI DSI compliant digital vid

[PATCH v6 1/2] dt-bindings: display: bridge: Document RZ/G2L MIPI DSI TX bindings

2022-08-29 Thread Biju Das
The RZ/G2L MIPI DSI TX is embedded in the Renesas RZ/G2L family SoC's. It can operate in DSI mode, with up to four data lanes. Signed-off-by: Biju Das Reviewed-by: Rob Herring Reviewed-by: Laurent Pinchart Reviewed-by: Geert Uytterhoeven --- v5->v6: * No change. v4->v5: * No change. v3->v4:

[PATCH v6 2/2] drm: rcar-du: Add RZ/G2L DSI driver

2022-08-29 Thread Biju Das
This driver supports the MIPI DSI encoder found in the RZ/G2L SoC. It currently supports DSI video mode only. Signed-off-by: Biju Das Acked-by: Sam Ravnborg --- v5->v6: * Updated commit description * Moved handling of arst and prst from rzg2l_mipi_dsi_startup->runtime PM suspend/resume hand

[PATCH v7] drm: Add initial ci/ subdirectory

2022-08-29 Thread Tomeu Vizoso
And use it to store expectations about what the DRM drivers are supposed to pass in the IGT test suite. Also include a configuration file that points to the out-of-tree CI scripts. By storing the test expectations along the code we can make sure both stay in sync with each other, and so we can kn

Re: [PATCH v4,1/2] soc: mediatek: Add mmsys func to adapt to dpi output for MT8186

2022-08-29 Thread AngeloGioacchino Del Regno
Il 29/08/22 05:21, xinlei@mediatek.com ha scritto: From: Xinlei Lee Add mmsys func to manipulate dpi output format config for MT8186. Co-developed-by: Jitao Shi Signed-off-by: Jitao Shi Signed-off-by: Xinlei Lee Reviewed-by: Nícolas F. R. A. Prado --- drivers/soc/mediatek/mt8186-mmsy

Re: [PATCH v4,2/2] drm: mediatek: Adjust the dpi output format to MT8186

2022-08-29 Thread AngeloGioacchino Del Regno
Il 29/08/22 05:21, xinlei@mediatek.com ha scritto: From: Xinlei Lee Dpi output needs to adjust the output format to dual edge for MT8186. Co-developed-by: Jitao Shi Signed-off-by: Jitao Shi Signed-off-by: Xinlei Lee --- drivers/gpu/drm/mediatek/mtk_dpi.c | 32 +++

[PATCH 0/4] Add DP MST DSC support to i915

2022-08-29 Thread Stanislav Lisovskiy
Currently we have only DSC support for DP SST. Stanislav Lisovskiy (4): drm: Add missing DP DSC extended capability definitions. drm/i915: Fix intel_dp_mst_compute_link_config drm/i915: Add DSC support to MST path drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate functi

[PATCH 3/4] drm/i915: Add DSC support to MST path

2022-08-29 Thread Stanislav Lisovskiy
Whenever we are not able to get enough timeslots for required PBN, let's try to allocate those using DSC, just same way as we do for SST. v2: Removed intel_dp_mst_dsc_compute_config and refactored intel_dp_dsc_compute_config to support timeslots as a parameter(Ville Syrjälä) v3: - Rebased

[PATCH 1/4] drm: Add missing DP DSC extended capability definitions.

2022-08-29 Thread Stanislav Lisovskiy
Adding DP DSC register definitions, we might need for further DSC implementation, supporting MST and DP branch pass-through mode. v2: - Fixed checkpatch comment warning v3: - Removed function which is not yet used(Jani Nikula) Reviewed-by: Vinod Govindapillai Signed-off-by: Stanislav Lisovskiy

[PATCH 2/4] drm/i915: Fix intel_dp_mst_compute_link_config

2022-08-29 Thread Stanislav Lisovskiy
We currently always exit that bpp loop because drm_dp_atomic_find_vcpi_slots doesn't care if we actually can fit those or not. I think that wasn't the initial intention here, especially when we keep trying with lower bpps, we are supposed to keep trying until we actually find some _working_ configu

[PATCH 4/4] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-08-29 Thread Stanislav Lisovskiy
We are using almost same code to loop through bpps while calling drm_dp_atomic_find_vcpi_slots - lets remove this duplication by introducing a new function intel_dp_mst_find_vcpi_slots_for_bpp Signed-off-by: Stanislav Lisovskiy --- drivers/gpu/drm/i915/display/intel_dp_mst.c | 88 +++

[PATCH v3] drm/ast: add dmabuf/prime buffer sharing support

2022-08-29 Thread oushixiong
This patch adds ast specific codes for DRM prime feature, this is to allow for offloading of rending in one direction and outputs in other. This patch is designed to solve the problem that the AST is not displayed when the server plug in a discrete graphics card at the same time. We call the dirt

Re: [PATCH v5 17/31] ACPI: video: Add Nvidia WMI EC brightness control detection (v3)

2022-08-29 Thread Hans de Goede
Hi, On 8/26/22 00:21, Daniel Dadap wrote: > On 8/25/22 9:37 AM, Hans de Goede wrote: >> On some new laptop designs a new Nvidia specific WMI interface is present >> which gives info about panel brightness control and may allow controlling >> the brightness through this interface when the embedded

Aw: [PATCH 1/2] drm/etnaviv: add HWDB entry for GC7000 r6203

2022-08-29 Thread Stefan Eschenbacher
Product_id, customer_id and eco_id should not be zero. The different versions of model 0x7000, revision 0x6203 can differ in stream_count, minor_feature5 and minor_features7. { .model = 0x7000, .revision = 0x6203,

[PATCH v2 11/11] drm/i915: Infer vrefresh range for eDP if the EDID omits it

2022-08-29 Thread Ville Syrjala
From: Ville Syrjälä A bunch of machines seem to have eDP panels where the EDID indicates continuous frequency support but fails to actually include the range descirptor. This violates the EDID 1.4 spec, but looks like the Windows driver just hacks around this by just assuming that the panel suppo

[PATCH] drm/i915/ttm: Abort suspend on i915_ttm_backup failure

2022-08-29 Thread Nirmoy Das
On system suspend when system memory is low then i915_gem_obj_copy_ttm() could fail trying to backup a lmem obj. GEM_WARN_ON() is not enough, suspend shouldn't continue if i915_ttm_backup() throws an error. Closes: https://gitlab.freedesktop.org/drm/intel/-/issues/6529 Suggested-by: Chris P Wilson

[PATCH linux-next] drm/amd/display: remove redundant vertical_line_start variable

2022-08-29 Thread cgel . zte
From: Jinpeng Cui Return value from expression directly instead of taking this in another redundant variable. Reported-by: Zeal Robot Signed-off-by: Jinpeng Cui --- drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hw_sequencer.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

Re: [PATCH v1 23/35] drm/vc4: vec: Convert to the new TV mode property

2022-08-29 Thread Maxime Ripard
Hi Noralf, On Thu, Aug 25, 2022 at 03:14:01PM +0200, Noralf Trønnes wrote: > Den 24.08.2022 17.26, skrev Maxime Ripard: > > On Sat, Aug 20, 2022 at 07:22:48PM +0200, Noralf Trønnes wrote: > >> Den 29.07.2022 18.35, skrev Maxime Ripard: > >>> Now that the core can deal fine with analog TV modes, le

Re: [PATCH 4/4] vfio/pci: Allow MMIO regions to be exported through dma-buf

2022-08-29 Thread Jason Gunthorpe
On Mon, Aug 29, 2022 at 01:04:05PM +0800, Yan Zhao wrote: > > +static struct sg_table * > > +vfio_pci_dma_buf_map(struct dma_buf_attachment *attachment, > > +enum dma_data_direction dir) > > +{ > > + size_t sgl_size = dma_get_max_seg_size(attachment->dev); > > + struct vfio_pci

[PATCH linux-next] drm/amdkfd: remove redundant variables err and ret

2022-08-29 Thread cgel . zte
From: Jinpeng Cui Return value from kfd_wait_on_events() and io_remap_pfn_range() directly instead of taking this in another redundant variable. Reported-by: Zeal Robot Signed-off-by: Jinpeng Cui --- drivers/gpu/drm/amd/amdkfd/kfd_chardev.c | 9 ++--- 1 file changed, 2 insertions(+), 7 de

[PATCH v2 00/41] drm: Analog TV Improvements

2022-08-29 Thread Maxime Ripard
Hi, Here's a series aiming at improving the command line named modes support, and more importantly how we deal with all the analog TV variants. The named modes support were initially introduced to allow to specify the analog TV mode to be used. However, this was causing multiple issues: * The

[PATCH v2 01/41] drm/tests: Order Kunit tests in Makefile

2022-08-29 Thread Maxime Ripard
Since we've recently added a ton of tests, the list starts to be a bit of a mess and creates unneeded conflicts. Let's order it alphabetically. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/tests/Makefile b/drivers/gpu/drm/tests/Makefile index 91b70f7d2769..2d9f49b62ecb 100644 --- a

[PATCH v2 02/41] drm/tests: Add Kunit Helpers

2022-08-29 Thread Maxime Ripard
As the number of kunit tests in KMS grows further, we start to have multiple test suites that, for example, need to register a mock DRM driver to interact with the KMS function they are supposed to test. Let's add a file meant to provide those kind of helpers to avoid duplication. Signed-off-by:

[PATCH v2 04/41] drm/connector: Rename subconnector state variable

2022-08-29 Thread Maxime Ripard
There is two TV subconnector related properties registered by drm_mode_create_tv_properties(): subconnector and select subconnector. While the select subconnector property is stored in the kernel by the drm_tv_connector_state structure, the subconnector property isn't stored anywhere. Worse, the

[PATCH v2 05/41] drm/atomic: Add TV subconnector property to get/set_property

2022-08-29 Thread Maxime Ripard
The subconnector property was created by drm_mode_create_tv_properties(), but wasn't exposed to the userspace through the generic atomic_get/set_property implementation, and wasn't stored in any generic state structure. Let's solve this. Signed-off-by: Maxime Ripard Reviewed-by: Noralf Trønnes

[PATCH v2 03/41] drm/atomic-helper: Rename drm_atomic_helper_connector_tv_reset to avoid ambiguity

2022-08-29 Thread Maxime Ripard
We currently have two sets of TV properties. The first one is there to deal with analog TV properties, creating properties such as the TV mode, subconnectors, saturation, hue and so on. It's created by calling the drm_mode_create_tv_properties() function. The second one is there to deal with prop

[PATCH v2 07/41] drm/connector: Only register TV mode property if present

2022-08-29 Thread Maxime Ripard
The drm_create_tv_properties() will create the TV mode property unconditionally. However, since we'll gradually phase it out, let's register it only if we have a list passed as an argument. This will make the transition easier. Signed-off-by: Maxime Ripard Acked-by: Noralf Trønnes diff --git a

[PATCH v2 08/41] drm/connector: Rename drm_mode_create_tv_properties

2022-08-29 Thread Maxime Ripard
drm_mode_create_tv_properties(), among other things, will create the "mode" property that stores the analog TV mode that connector is supposed to output. However, that property is getting deprecated, so let's rename that function to mention it's deprecated. We'll introduce a new variant of that fu

[PATCH v2 10/41] drm/modes: Add a function to generate analog display modes

2022-08-29 Thread Maxime Ripard
Multiple drivers (meson, vc4, sun4i) define analog TV 525-lines and 625-lines modes in their drivers. Since those modes are fairly standard, and that we'll need to use them in more places in the future, it makes sense to move their definition into the core framework. However, analog display usual

[PATCH v2 11/41] drm/modes: Only consider bpp and refresh before options

2022-08-29 Thread Maxime Ripard
Some video= options might have a value that contains a dash. However, the command line parsing mode considers all dashes as the separator between the mode and the bpp count. Let's rework the parsing code a bit to only consider a dash as the bpp separator if it before a comma, the options separator

[PATCH v2 06/41] drm/connector: Rename legacy TV property

2022-08-29 Thread Maxime Ripard
The current tv_mode has driver-specific values that don't allow to easily share code using it, either at the userspace or kernel level. Since we're going to introduce a new, generic, property that fit the same purpose, let's rename this one to legacy_tv_mode to make it obvious we should move away

[PATCH v2 13/41] drm/client: Add some tests for drm_connector_pick_cmdline_mode()

2022-08-29 Thread Maxime Ripard
drm_connector_pick_cmdline_mode() is in charge of finding a proper drm_display_mode from the definition we got in the video= command line argument. Let's add some unit tests to make sure we're not getting any regressions there. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/drm_clien

[PATCH v2 12/41] drm/modes: parse_cmdline: Add support for named modes containing dashes

2022-08-29 Thread Maxime Ripard
From: Geert Uytterhoeven It is fairly common for named video modes to contain dashes (e.g. "tt-mid" on Atari, "dblntsc-ff" on Amiga). Currently such mode names are not recognized, as the dash is considered to be a separator between mode name and bpp. Fix this by skipping any dashes that are not

[PATCH v2 09/41] drm/connector: Add TV standard property

2022-08-29 Thread Maxime Ripard
The TV mode property has been around for a while now to select and get the current TV mode output on an analog TV connector. Despite that property name being generic, its content isn't and has been driver-specific which makes it hard to build any generic behaviour on top of it, both in kernel and

[PATCH v2 14/41] drm/modes: Move named modes parsing to a separate function

2022-08-29 Thread Maxime Ripard
The current construction of the named mode parsing doesn't allow to extend it easily. Let's move it to a separate function so we can add more parameters and modes. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index 99a21e5cd00d..0636cb707544

[PATCH v2 15/41] drm/modes: Switch to named mode descriptors

2022-08-29 Thread Maxime Ripard
The current named mode parsing relies only the mode name, and doesn't allow to specify any other parameter. Let's convert that string list to an array of a custom structure that will hold the name and some additional parameters in the future. Signed-off-by: Maxime Ripard diff --git a/drivers/gp

[PATCH v2 21/41] drm/modes: Introduce more named modes

2022-08-29 Thread Maxime Ripard
Now that we can easily extend the named modes list, let's add a few more analog TV modes that were used in the wild, and some unit tests to make sure it works as intended. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index 065dbfbd815e..7d76

[PATCH v2 18/41] drm/connector: Add a function to lookup a TV mode by its name

2022-08-29 Thread Maxime Ripard
As part of the command line parsing rework coming in the next patches, we'll need to lookup drm_connector_tv_mode values by their name, already defined in drm_tv_mode_enum_list. In order to avoid any code duplication, let's do a function that will perform a lookup of a TV mode name and return its

[PATCH v2 19/41] drm/modes: Introduce the tv_mode property as a command-line option

2022-08-29 Thread Maxime Ripard
Our new tv mode option allows to specify the TV mode from a property. However, it can still be useful, for example to avoid any boot time artifact, to set that property directly from the kernel command line. Let's add some code to allow it, and some unit tests to exercise that code. Signed-off-by

[PATCH v2 16/41] drm/modes: Fill drm_cmdline mode from named modes

2022-08-29 Thread Maxime Ripard
The current code to deal with named modes will only set the mode name, and then it's up to drivers to try to match that name to whatever mode or configuration they see fit. The plan is to remove that need and move the named mode handling out of drivers and into the core, and only rely on modes and

[PATCH v2 22/41] drm/atomic-helper: Add a TV properties reset helper

2022-08-29 Thread Maxime Ripard
The drm_tv_create_properties() function will create a bunch of properties, but it's up to each and every driver using that function to properly reset the state of these properties leading to inconsistent behaviours. Let's create a helper that will take care of it. Signed-off-by: Maxime Ripard d

[PATCH v2 26/41] drm/vc4: vec: Refactor VEC TV mode setting

2022-08-29 Thread Maxime Ripard
From: Mateusz Kwiatkowski Change the mode_set function pointer logic to declarative config0, config1 and custom_freq fields, to make TV mode setting logic more concise and uniform. Signed-off-by: Mateusz Kwiatkowski Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/vc4/vc4_vec.c b/dri

[PATCH v2 28/41] drm/vc4: vec: Fix timings for VEC modes

2022-08-29 Thread Maxime Ripard
From: Mateusz Kwiatkowski This commit fixes vertical timings of the VEC (composite output) modes to accurately represent the 525-line ("NTSC") and 625-line ("PAL") ITU-R standards. Previous timings were actually defined as 502 and 601 lines, resulting in non-standard 62.69 Hz and 52 Hz signals b

[PATCH v2 30/41] drm/vc4: vec: Fix definition of PAL-M mode

2022-08-29 Thread Maxime Ripard
From: Mateusz Kwiatkowski PAL-M is a Brazilian analog TV standard that uses a PAL-style chroma subcarrier at 3.575611[888111] MHz on top of 525-line (480i60) timings. This commit makes the driver actually use the proper VEC preset for this mode instead of just changing PAL subcarrier frequency.

[PATCH v2 25/41] drm/vc4: vec: Convert to atomic helpers

2022-08-29 Thread Maxime Ripard
The VC4 VEC driver still uses legacy enable and disable hook implementation. Let's convert to the atomic variants. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/vc4/vc4_vec.c b/drivers/gpu/drm/vc4/vc4_vec.c index d521ffd8d75c..72eee0cbb615 100644 --- a/drivers/gpu/drm/vc4/vc4_vec.c +

[PATCH v2 23/41] drm/atomic-helper: Add an analog TV atomic_check implementation

2022-08-29 Thread Maxime Ripard
The analog TV connector drivers share some atomic_check logic, and the new TV standard property have created a bunch of new constraints that needs to be shared across drivers too. Let's create an atomic_check helper for those use cases. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/

[PATCH v2 24/41] drm/vc4: vec: Remove empty mode_fixup

2022-08-29 Thread Maxime Ripard
The mode_fixup hooks are deprecated, and the behaviour we implement is the default one anyway. Let's remove it. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/vc4/vc4_vec.c b/drivers/gpu/drm/vc4/vc4_vec.c index d5140fe0be4f..d521ffd8d75c 100644 --- a/drivers/gpu/drm/vc4/vc4_vec.c +++

[PATCH v2 20/41] drm/modes: Properly generate a drm_display_mode from a named mode

2022-08-29 Thread Maxime Ripard
The framework will get the drm_display_mode from the drm_cmdline_mode it got by parsing the video command line argument by calling drm_connector_pick_cmdline_mode(). The heavy lifting will then be done by the drm_mode_create_from_cmdline_mode() function. In the case of the named modes though, the

[PATCH v2 27/41] drm/vc4: vec: Remove redundant atomic_mode_set

2022-08-29 Thread Maxime Ripard
From: Mateusz Kwiatkowski Let's remove the superfluous tv_mode field, which was redundant with the mode field in struct drm_tv_connector_state. Signed-off-by: Mateusz Kwiatkowski Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/vc4/vc4_vec.c b/drivers/gpu/drm/vc4/vc4_vec.c index 9a37

[PATCH v2 29/41] drm/vc4: vec: Switch for common modes

2022-08-29 Thread Maxime Ripard
Now that the core has a definition for the 525 and 625 lines analog TV modes, let's switch to it for vc4. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/vc4/vc4_vec.c b/drivers/gpu/drm/vc4/vc4_vec.c index d1d40b69279e..63e4e617e321 100644 --- a/drivers/gpu/drm/vc4/vc4_vec.c +++ b/driv

[PATCH v2 17/41] drm/connector: Add pixel clock to cmdline mode

2022-08-29 Thread Maxime Ripard
We'll need to get the pixel clock to generate proper display modes for all the current named modes. Let's add it to struct drm_cmdline_mode and fill it when parsing the named mode. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index 5e898699b

[PATCH v2 41/41] drm/sun4i: tv: Convert to the new TV mode property

2022-08-29 Thread Maxime Ripard
Now that the core can deal fine with analog TV modes, let's convert the sun4i TV driver to leverage those new features. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/sun4i/sun4i_tv.c b/drivers/gpu/drm/sun4i/sun4i_tv.c index 74ff5ad6a8b9..10c0d727d700 100644 --- a/drivers/gpu/drm/sun4

[PATCH v2 31/41] drm/vc4: vec: Use TV Reset implementation

2022-08-29 Thread Maxime Ripard
The analog TV properties created by the drm_mode_create_tv_properties() are not properly initialised at reset. Let's switch our implementation to call drm_atomic_helper_connector_tv_reset(). Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/vc4/vc4_vec.c b/drivers/gpu/drm/vc4/vc4_vec.c i

[PATCH v2 37/41] drm/sun4i: tv: Remove useless function

2022-08-29 Thread Maxime Ripard
The drm_connector_to_sun4i_tv() function isn't used anywhere in the driver, so let's remove it. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/sun4i/sun4i_tv.c b/drivers/gpu/drm/sun4i/sun4i_tv.c index 3944da9a3c34..52bbba8f19dc 100644 --- a/drivers/gpu/drm/sun4i/sun4i_tv.c +++ b/drive

[PATCH v2 36/41] drm/sun4i: tv: Merge mode_set into atomic_enable

2022-08-29 Thread Maxime Ripard
Our mode_set implementation can be merged into our atomic_enable implementation to simplify things, so let's do this. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/sun4i/sun4i_tv.c b/drivers/gpu/drm/sun4i/sun4i_tv.c index f7aad995ab5b..3944da9a3c34 100644 --- a/drivers/gpu/drm/sun4i/

[PATCH v2 32/41] drm/vc4: vec: Convert to the new TV mode property

2022-08-29 Thread Maxime Ripard
Now that the core can deal fine with analog TV modes, let's convert the vc4 VEC driver to leverage those new features. We've added some backward compatibility to support the old TV mode property and translate it into the new TV norm property. Signed-off-by: Maxime Ripard diff --git a/drivers/gp

[PATCH v2 39/41] drm/sun4i: tv: Rename error label

2022-08-29 Thread Maxime Ripard
The other error labels in sun4i_tv_bind() are named after the task they perform (err_disable_clk to call clk_disable_unprepare for example). However, the err_cleanup_connector is named after the calling site (drm_connector_init failing) and will actually cleanup the encoder. Let's rename it to err

[PATCH v2 34/41] drm/sun4i: tv: Remove unused mode_valid

2022-08-29 Thread Maxime Ripard
The mode_valid implementation is pretty much a nop, let's remove it. Signed-off-by: Maxime Ripard Reviewed-by: Jernej Skrabec diff --git a/drivers/gpu/drm/sun4i/sun4i_tv.c b/drivers/gpu/drm/sun4i/sun4i_tv.c index 94883abe0dfd..53152d77c392 100644 --- a/drivers/gpu/drm/sun4i/sun4i_tv.c +++ b/dri

[PATCH v2 40/41] drm/sun4i: tv: Add missing reset assertion

2022-08-29 Thread Maxime Ripard
The reset line is deasserted at bind, and asserted if we ever encounter an error there. However, it's never asserted in unbind which will lead to a resource unbalance. Signed-off-by: Maxime Ripard Reviewed-by: Jernej Skrabec diff --git a/drivers/gpu/drm/sun4i/sun4i_tv.c b/drivers/gpu/drm/sun4i/

[PATCH v2 33/41] drm/vc4: vec: Add support for more analog TV standards

2022-08-29 Thread Maxime Ripard
From: Mateusz Kwiatkowski Add support for the following composite output modes (all of them are somewhat more obscure than the previously defined ones): - NTSC_443 - NTSC-style signal with the chroma subcarrier shifted to 4.43361875 MHz (the PAL subcarrier frequency). Never used for broadcas

[PATCH v2 38/41] drm/sun4i: tv: Remove useless destroy function

2022-08-29 Thread Maxime Ripard
Our destroy implementation is just calling the generic helper, so let's just remove our function and directly use the helper. Signed-off-by: Maxime Ripard Reviewed-by: Jernej Skrabec diff --git a/drivers/gpu/drm/sun4i/sun4i_tv.c b/drivers/gpu/drm/sun4i/sun4i_tv.c index 52bbba8f19dc..6d7e1d51569

[PATCH v2 35/41] drm/sun4i: tv: Convert to atomic hooks

2022-08-29 Thread Maxime Ripard
The sun4i TV driver still uses legacy enable and disable hook implementation. Let's convert to the atomic variants. Signed-off-by: Maxime Ripard diff --git a/drivers/gpu/drm/sun4i/sun4i_tv.c b/drivers/gpu/drm/sun4i/sun4i_tv.c index 53152d77c392..f7aad995ab5b 100644 --- a/drivers/gpu/drm/sun4i/su

[PULL] drm-intel-next

2022-08-29 Thread Jani Nikula
Hi Dave & Daniel - drm-intel-next-2022-08-29: drm/i915 feature pull for v6.1: Features and functionality: - Early Meteorlake (MTL) enabling (José, Radhakrishna, Clint, Imre, Vandita, Ville, Jani) - Support more HDMI pixel clock frequencies on DG2 (Clint) - Sanity check PCI BARs (Piotr Piórkows

Re: [PATCH 1/2] drm/etnaviv: add HWDB entry for GC7000 r6203

2022-08-29 Thread Lucas Stach
Hi Stefan, please keep the CC list intact when replying. I've almost missed this. Am Montag, dem 29.08.2022 um 13:41 +0200 schrieb Stefan Eschenbacher: > > Product_id, customer_id and eco_id should not be zero. The different > versions of model 0x7000, revision 0x6203 can differ in stream_count,

Re: [PATCH 3/3] drm: omapdrm: Do no allocate non-scanout GEMs through DMM/TILER

2022-08-29 Thread Ivaylo Dimitrov
Hi, On 29.08.22 г. 5:51 ч., Yongqin Liu wrote: Hi, Ivaylo Sorry for the late response, and Thanks very much for the detailed explanations! On Thu, 18 Aug 2022 at 18:23, Ivaylo Dimitrov wrote: Hi, On 17.08.22 г. 7:52 ч., Yongqin Liu wrote: Hi, Ivaylo On Mon, 15 Aug 2022 at 14:23, Ivaylo

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-29 Thread Maxime Ripard
Hi Mateusz On Wed, Aug 24, 2022 at 06:42:18PM +0200, Mateusz Kwiatkowski wrote: > Hi Maxime, > > W dniu 18.08.2022 o 17:56, Geert Uytterhoeven pisze: > > Hi Maxime, > > > > On Thu, Aug 18, 2022 at 5:46 PM Maxime Ripard wrote: > >> On Thu, Aug 18, 2022 at 05:34:30PM +0200, Geert Uytterhoeven wrot

Re: [PATCH] drm/bridge: sii902x: add support for DRM_BRIDGE_ATTACH_NO_CONNECTOR

2022-08-29 Thread Neil Armstrong
On 25/08/2022 14:48, Linus Walleij wrote: On Wed, Aug 17, 2022 at 3:31 PM Neil Armstrong wrote: On 15/08/2022 02:15, Dmitry Osipenko wrote: 08.08.2022 12:51, Neil Armstrong пишет: On 08/08/2022 11:15, Neil Armstrong wrote: This patch broke ARM/QEMU vexpress display because of_drm_find_brid

[PATCH v4 0/8] drm/vc4: Reset HDMI link at hotplug

2022-08-29 Thread Maxime Ripard
Hi, This is a follow-up of the work to support the interactions between the hotplug and the scrambling support for vc4: https://lore.kernel.org/dri-devel/20210507150515.257424-11-max...@cerno.tech/ https://lore.kernel.org/dri-devel/20211025152903.1088803-10-max...@cerno.tech/ https://lore.kernel.

[PATCH v4 1/8] drm/vc4: hdmi: Constify drm_display_mode

2022-08-29 Thread Maxime Ripard
We don't modify the drm_display_mode pointer we have in the driver in most places, so let's make them const. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 16 drivers/gpu/drm/vc4/vc4_hdmi.h | 2 +- 2 files changed, 9 insertions(+), 9 deletions(-) diff --git

[PATCH v4 2/8] drm/vc4: hdmi: Remove unused argument in vc4_hdmi_supports_scrambling

2022-08-29 Thread Maxime Ripard
Even though vc4_hdmi_supports_scrambling takes a mode as an argument, it never uses it. Let's remove it. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/v

Re: [PATCH] drm/amd/amdgpu: Add modeset module parameter support

2022-08-29 Thread Alex Deucher
On Mon, Aug 29, 2022 at 9:43 AM Zhen Ni wrote: > > Nomodeset kernel parameter is for all graphics cards. Amdgpu cannot > be set separately in some scenarios, such as hybrid graphics(i + a). > Add modeset module parameter for amdgpu to set kernel mode separately. > > Signed-off-by: Zhen Ni amdgpu

[PATCH v4 4/8] drm/vc4: hdmi: Simplify the hotplug handling

2022-08-29 Thread Maxime Ripard
Our detect callback has a bunch of operations to perform depending on the current and last status of the connector, such a setting the CEC physical address or enabling the scrambling again. This is currently dealt with a bunch of if / else statetements that make it fairly difficult to read and ext

[PATCH v4 3/8] drm/vc4: hdmi: Remove mutex in detect

2022-08-29 Thread Maxime Ripard
We recently introduced a new mutex to protect concurrent execution of ALSA and KMS hooks, and the concurrent access to some of vc4_hdmi fields. However, using it in the detect hook was creating a reentrency issue with CEC code. Indeed, calling cec_s_phys_addr_from_edid from detect might call the C

[PATCH v4 5/8] drm/vc4: hdmi: Switch to detect_ctx

2022-08-29 Thread Maxime Ripard
We'll need the locking context in future patch, so let's convert .detect to .detect_ctx. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdmi.c b/drivers/gpu/drm/vc4/vc4_hdmi.c inde

[PATCH v4 6/8] drm/vc4: hdmi: Move vc4_hdmi_supports_scrambling() around

2022-08-29 Thread Maxime Ripard
We'll need it earlier in the driver, so let's move it next to the other scrambling-related helpers. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/vc4/vc4_hdmi.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/drivers/gpu/drm/vc4/vc4_hdm

[PATCH v4 7/8] drm/vc4: hdmi: Reset link on hotplug

2022-08-29 Thread Maxime Ripard
During a hotplug cycle (such as a TV going out of suspend, or when the cable is disconnected and reconnected), the expectation is that the same state used before the disconnection is reused until the next commit. However, the HDMI scrambling requires that some flags are set in the monitor, and tho

[PATCH v4 8/8] drm/scdc: Document hotplug gotchas

2022-08-29 Thread Maxime Ripard
There's some interactions between the SCDC setup and the disconnection / reconnection of displays. Let's document it and a solution. Signed-off-by: Maxime Ripard --- drivers/gpu/drm/display/drm_scdc_helper.c | 13 + 1 file changed, 13 insertions(+) diff --git a/drivers/gpu/drm/displ

Re: [PATCH v4,2/2] drm: mediatek: Adjust the dpi output format to MT8186

2022-08-29 Thread Nícolas F . R . A . Prado
On Mon, Aug 29, 2022 at 11:21:48AM +0800, xinlei@mediatek.com wrote: > From: Xinlei Lee > > Dpi output needs to adjust the output format to dual edge for MT8186. > > Co-developed-by: Jitao Shi > Signed-off-by: Jitao Shi > Signed-off-by: Xinlei Lee Reviewed-by: Nícolas F. R. A. Prado Th

[PATCH 4/4] drm/i915: Extract drm_dp_atomic_find_vcpi_slots cycle to separate function

2022-08-29 Thread Stanislav Lisovskiy
We are using almost same code to loop through bpps while calling drm_dp_atomic_find_vcpi_slots - lets remove this duplication by introducing a new function intel_dp_mst_find_vcpi_slots_for_bpp v2: Fix pbn_div calculation - shouldn't matter if its DSC or not. Signed-off-by: Stanislav Lisovskiy --

Re: [PATCH 1/5] drm/bridge: anx7625: Drop of_gpio header

2022-08-29 Thread Robert Foss
Hi Maira, On Fri, 12 Aug 2022 at 22:58, Maíra Canal wrote: > > This driver includes the deprecated OF GPIO header > yet fail to use symbols from it, so drop the include. > > Cc: Andrzej Hajda > Cc: Neil Armstrong > Cc: Robert Foss > Cc: Laurent Pinchart > Cc: Jonas Karlman > Cc: Jernej Skra

Re: [PATCH] drm/bridge: anx7625: Support HDMI_I2S audio format

2022-08-29 Thread Robert Foss
On Wed, 10 Aug 2022 at 10:58, Hsin-Yi Wang wrote: > > On Tue, Jul 26, 2022 at 5:16 PM Jiaxin Yu wrote: > > > > On Tue, 2022-07-26 at 11:30 +0800, Xin Ji wrote: > > > 1. Support HDMI_I2S audio format. > > > 2. Return 0 if there is no sink connection in .hw_param callback. > > > > > > Signed-off-by

Re: [PATCH] drm/i915: fix repeated words in comments

2022-08-29 Thread Jani Nikula
On Tue, 23 Aug 2022, wangjianli wrote: > Delete the redundant word 'the'. > > Signed-off-by: wangjianli Already fixed by commit 78f48aa6f50b ("drm/i915/irq: Fix a "the the" typo"). What is this sudden influx of patches fixing repeated words everywhere? BR, Jani. > --- > drivers/gpu/drm/i91

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-29 Thread Geert Uytterhoeven
Hi Maxime, On Mon, Aug 29, 2022 at 3:30 PM Maxime Ripard wrote: > On Wed, Aug 24, 2022 at 06:42:18PM +0200, Mateusz Kwiatkowski wrote: > > - Speaking of closed captioning... a lot of different stuff were put in the > > blanking interval over the years. Like teletext in Europe. There are > > pr

Re: [PATCH v3 03/14] gpu: drm: simplify drivers using devm_regulator_*get_enable*()

2022-08-29 Thread Robert Foss
Hi Matti, On Fri, 19 Aug 2022 at 21:18, Matti Vaittinen wrote: > > Simplify drivers using managed "regulator get and enable". > > meson: > Use the devm_regulator_get_enable_optional(). Also drop the seemingly > unused struct member 'hdmi_supply'. > > sii902x: > Simplify using devm_regulator_bulk_

Re: [PATCH] drm/i915: fix repeated words in comments

2022-08-29 Thread Jani Nikula
On Tue, 23 Aug 2022, Jilin Yuan wrote: > Delete the redundant word 'other'. > Delete the redundant word 'the'. > Delete the redundant word 'will'. > > Signed-off-by: Jilin Yuan Doesn't apply because some of the hunks were already fixed by other commits. BR, Jani. > --- > drivers/gpu/drm/i

Re: [PATCH 2/4] drm/i915: Fix intel_dp_mst_compute_link_config

2022-08-29 Thread Govindapillai, Vinod
Reviewed-by: Vinod Govindapillai On Mon, 2022-08-29 at 12:58 +0300, Stanislav Lisovskiy wrote: > We currently always exit that bpp loop because drm_dp_atomic_find_vcpi_slots > doesn't care if we actually can fit those or not. > I think that wasn't the initial intention here, especially when > we

Re: [PATCH v1 04/35] drm/modes: Introduce 480i and 576i modes

2022-08-29 Thread Maxime Ripard
On Mon, Aug 29, 2022 at 04:14:54PM +0200, Geert Uytterhoeven wrote: > Hi Maxime, > > On Mon, Aug 29, 2022 at 3:30 PM Maxime Ripard wrote: > > On Wed, Aug 24, 2022 at 06:42:18PM +0200, Mateusz Kwiatkowski wrote: > > > - Speaking of closed captioning... a lot of different stuff were put in > > > t

[PATCH] drm/amd/amdgpu: Add modeset module parameter support

2022-08-29 Thread Zhen Ni
Nomodeset kernel parameter is for all graphics cards. Amdgpu cannot be set separately in some scenarios, such as hybrid graphics(i + a). Add modeset module parameter for amdgpu to set kernel mode separately. Signed-off-by: Zhen Ni --- drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 13 - 1

  1   2   >