Hi,
On 11/3/21 12:03, Igor Torrente wrote:
> Hi Leandro,
>
> On 10/28/21 6:38 PM, Leandro Ribeiro wrote:
>> Hi,
>>
>> On 10/26/21 08:34, Igor Torrente wrote:
>>> Add a helper function to validate the connector configuration receive in
>>> the en
break;
> + }
> + }
> +
> + if (!format_supported) {
> + DRM_DEBUG_KMS("Invalid pixel format %p4cc\n",
> + >format->format);
> + return -EINVAL;
> + }
> +
> + retur
Add a small description and document struct fields of
drm_mode_get_plane.
Signed-off-by: Leandro Ribeiro
---
include/uapi/drm/drm_mode.h | 32
1 file changed, 32 insertions(+)
diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
index
to the
other. Suggested by Ville Syrjälä
v6: document that gamma_size field was never used. Suggested by Pekka Paalanen
and Daniel Vetter
Leandro Ribeiro (1):
drm/doc: document drm_mode_get_plane
include/uapi/drm/drm_mode.h | 32
1 file changed, 32
On 6/11/21 4:33 AM, Daniel Vetter wrote:
> On Fri, Jun 11, 2021 at 9:20 AM Pekka Paalanen wrote:
>>
>> On Thu, 10 Jun 2021 17:38:24 -0300
>> Leandro Ribeiro wrote:
>>
>>> Add a small description and document struct fields of
>>> drm_mode_get_
Add a small description and document struct fields of
drm_mode_get_plane.
Signed-off-by: Leandro Ribeiro
---
include/uapi/drm/drm_mode.h | 35 +++
1 file changed, 35 insertions(+)
diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
index
to the
other. Suggested by Ville Syrjälä
Leandro Ribeiro (1):
drm/doc: document drm_mode_get_plane
include/uapi/drm/drm_mode.h | 35 +++
1 file changed, 35 insertions(+)
--
2.31.1
On 6/9/21 8:00 PM, Leandro Ribeiro wrote:
> Add a small description and document struct fields of
> drm_mode_get_plane.
>
> Signed-off-by: Leandro Ribeiro
> ---
> include/uapi/drm/drm_mode.h | 36
> 1 file changed, 36 insertio
Add a small description and document struct fields of
drm_mode_get_plane.
Signed-off-by: Leandro Ribeiro
---
include/uapi/drm/drm_mode.h | 36
1 file changed, 36 insertions(+)
diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
index
In this patch we add a section to document what userspace should do to
find out the CRTC index. This is important as they may be many places in
the documentation that need this, so it's better to just point to this
section and avoid repetition.
Signed-off-by: Leandro Ribeiro
---
Documentation
IN and OUT fields and make the description more
concise. Suggested by Pekka Paalanen
Leandro Ribeiro (2):
drm/doc: document how userspace should find out CRTC index
drm/doc: document drm_mode_get_plane
Documentation/gpu/drm-uapi.rst| 13 +++
drivers/gpu/drm/drm_debugfs_crc.c | 8
On 5/6/21 6:10 AM, Pekka Paalanen wrote:
> On Wed, 28 Apr 2021 18:36:51 -0300
> Leandro Ribeiro wrote:
>
>> Add a small description and document struct fields of
>> drm_mode_get_plane.
>>
>> Signed-off-by: Leandro Ribeiro
>
> Hi,
>
> thanks a
On 5/12/21 10:04 AM, Pekka Paalanen wrote:
> On Wed, 12 May 2021 09:50:14 -0300
> Leandro Ribeiro wrote:
>
>> On 5/6/21 5:50 AM, Pekka Paalanen wrote:
>>> On Wed, 28 Apr 2021 18:36:50 -0300
>>> Leandro Ribeiro wrote:
>>>
>>>> In
On 5/6/21 5:50 AM, Pekka Paalanen wrote:
> On Wed, 28 Apr 2021 18:36:50 -0300
> Leandro Ribeiro wrote:
>
>> In this patch we add a section to document what userspace should do to
>> find out the CRTC index. This is important as there are multiple places
>> in
On 4/27/21 6:00 AM, Daniel Vetter wrote:
> On Tue, Apr 27, 2021 at 10:40:24AM +0300, Pekka Paalanen wrote:
>> On Mon, 26 Apr 2021 14:30:53 -0300
>> Leandro Ribeiro wrote:
>>
>>> On 4/26/21 7:58 AM, Simon Ser wrote:
>>>> On Monday, April 26th, 20
In this patch we add a section to document what userspace should do to
find out the CRTC index. This is important as there are multiple places
in the documentation that need this, so it's better to just point to
this section and avoid repetition.
Signed-off-by: Leandro Ribeiro
---
Documentation
Add a small description and document struct fields of
drm_mode_get_plane.
Signed-off-by: Leandro Ribeiro
---
include/uapi/drm/drm_mode.h | 22 ++
1 file changed, 22 insertions(+)
diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
index a5e76aa06ad5
v2: possible_crtcs field is a bitmask, not a pointer. Suggested by
Ville Syrjälä
v3: document how userspace should find out CRTC index. Also,
document that field 'gamma_size' represents the number of
entries in the lookup table. Suggested by Pekka Paalanen
and Daniel Vetter
Leandro Ribeiro (2
On 4/26/21 7:58 AM, Simon Ser wrote:
> On Monday, April 26th, 2021 at 9:36 AM, Pekka Paalanen
> wrote:
>
This should probably explain what the bits in the mask correspond to.
As in, which CRTC does bit 0 refer to, and so on.
>>>
>>> What about:
>>>
>>> "possible_crtcs: Bitmask of
On 4/23/21 8:11 AM, Pekka Paalanen wrote:
> On Thu, 22 Apr 2021 15:10:04 -0300
> Leandro Ribeiro wrote:
>
>> Add a small description and document struct fields of
>> drm_mode_get_plane.
>>
>> Signed-off-by: Leandro Ribeiro
>> ---
>> include/ua
Add a small description and document struct fields of
drm_mode_get_plane.
Signed-off-by: Leandro Ribeiro
---
include/uapi/drm/drm_mode.h | 16
1 file changed, 16 insertions(+)
diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
index a5e76aa06ad5
ugh the other commit of the series was incorrect.
So I'm pushing the first commit again.
v2: possible_crtcs field is a bitmask, not a pointer. Suggested by
Ville Syrjälä
Leandro Ribeiro (1):
drm/doc: document drm_mode_get_plane
include/uapi/drm/drm_mode.h | 16
1 file ch
On 4/8/21 8:35 AM, Daniel Vetter wrote:
> On Tue, Apr 06, 2021 at 04:21:16PM -0300, Leandro Ribeiro wrote:
>> This patch is to emphasize how userspace should use the plane format list and
>> the IN_FORMATS blob. The plane format list contains the formats that do not
>&
On 4/7/21 10:45 AM, Ville Syrjälä wrote:
> On Tue, Apr 06, 2021 at 04:21:17PM -0300, Leandro Ribeiro wrote:
>> Add a small description and document struct fields of
>> drm_mode_get_plane.
>>
>> Signed-off-by: Leandro Ribeiro
>> ---
>> include/uapi/drm/dr
Add a small description and document struct fields of
drm_mode_get_plane.
Signed-off-by: Leandro Ribeiro
---
include/uapi/drm/drm_mode.h | 19 +++
1 file changed, 19 insertions(+)
diff --git a/include/uapi/drm/drm_mode.h b/include/uapi/drm/drm_mode.h
index d1a93d2a85f9
modifiers.
Leandro Ribeiro (2):
drm/doc: document drm_mode_get_plane
drm/doc: emphasize difference between plane formats and IN_FORMATS
blob
drivers/gpu/drm/drm_plane.c | 4
include/uapi/drm/drm_mode.h | 22 ++
2 files changed, 26 insertions(+)
--
2.31.1
that these are not disjoint sets. If a format supports modifiers
but the driver can also handle it without a modifier, it should be
present in both the IN_FORMATS blob property and the plane format list.
Signed-off-by: Leandro Ribeiro
---
drivers/gpu/drm/drm_plane.c | 4
include/uapi/drm/drm_mode.h | 3
any valid mode, so
these userspace applications would not be able to use VKMS.
Call drm_helper_probe_single_connector_modes() during VKMS output
initialization in order to start with the set of all valid modes.
Signed-off-by: Leandro Ribeiro
---
drivers/gpu/drm/vkms/vkms_output.c | 6 ++
1
On 11/27/20 6:10 PM, Leandro Ribeiro wrote:
> In userspace we can use drmGetConnector() or drmGetConnectorCurrent() in
> order to retrieve connector information. The difference between both is
> that the former retrieves the complete set of modes and encoders
> associated with t
On 11/24/20 11:39 AM, Daniel Vetter wrote:
On Fri, Nov 20, 2020 at 01:19:04PM -0300, Leandro Ribeiro wrote:
Hello,
We have a patch in Weston to replace drmGetConnector() calls with
drmGetConnectorCurrent():
https://gitlab.freedesktop.org/wayland/weston/-/issues/437
https
Hello,
We have a patch in Weston to replace drmGetConnector() calls with
drmGetConnectorCurrent():
https://gitlab.freedesktop.org/wayland/weston/-/issues/437
https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/518
Unfortunately this is not working when we use VKMS (upstream
ms_crtc.c:21
vkms_vblank_simulate+0x101/0x110
I've decided to share this info with you, as it may help you somehow.
I'm also
investigating to help understand what is happening.
Thanks,
Leandro Ribeiro
On 7/31/20 1:47 PM, Melissa Wen wrote:
On 07/31, Sidong Yang wrote:
On Fri, Jul 31, 2020 a
:21
vkms_vblank_simulate+0x101/0x110
From what I've read from this thread it seems like this is not the
right fix, but I've decided to share this info with you anyway, as it
may help. I'm also trying to understand what is happening.
Thanks,
Leandro Ribeiro
On 7/31/20 1:47 PM, Melissa Wen
vkms_vblank_simulate+0x101/0x110
I've decided to share this info with you, as it may help you somehow.
I'm also
investigating to help understand what is happening.
Thanks,
Leandro Ribeiro
On 7/31/20 1:47 PM, Melissa Wen wrote:
On 07/31, Sidong Yang wrote:
On Fri, Jul 31, 2020 at 11:08:34AM
:21
vkms_vblank_simulate+0x101/0x110
From what I've read from this thread it seems like this is not the
right fix, but I've decided to share this info with you anyway, as it
may help. I'm also trying to understand what is happening.
Thanks,
Leandro Ribeiro
On 7/31/20 3:33 PM, Daniel Vetter
eir GitLab README, which should always be up to
date.
Signed-off-by: Leandro Ribeiro
---
Documentation/gpu/drm-uapi.rst | 32 ++--
1 file changed, 2 insertions(+), 30 deletions(-)
diff --git a/Documentation/gpu/drm-uapi.rst b/Documentation/gpu/drm-uapi.rst
index 94
36 matches
Mail list logo