Hi Maarten,
Thanks for the patch.
Acked-by: Vincent Abriou
On 04/26/2016 04:11 PM, Maarten Lankhorst wrote:
> The async name is deprecated and should be changed to nonblocking.
>
> Cc: Benjamin Gaignard
> Cc: Vincent Abriou
> Signed-off-by: Maarten Lankhorst
> ---
>
Acked-by: Vincent Abriou
On 08/16/2016 09:06 AM, Shawn Guo wrote:
> Since commit 4984979b9b90 ("drm/irq: simplify irq checks in
> drm_wait_vblank"), the drm driver feature flag DRIVER_HAVE_IRQ is only
> required for drivers that have an IRQ handler managed by the DRM core.
>
Hi Laurent,
For the sti driver:
Acked-by: Vincent Abriou
Vincent
On 11/29/2016 09:56 PM, Laurent Pinchart wrote:
> Many drivers (21 to be exact) create connectors that are always
> connected (for instance to an LVDS or DSI panel). Instead of forcing
> them to implement a dumm
72,8 @@ static int sti_dvo_bind(struct device *dev, struct device
> *master, void *data)
> bridge->driver_private = dvo;
> bridge->funcs = &sti_dvo_bridge_funcs;
> bridge->of_node = dvo->dev.of_node;
> + bridge->encoder_type = DRM_MODE_ENCODER_LVDS;
> +
> err = drm_bridge_add(bridge);
> if (err) {
> DRM_ERROR("Failed to add bridge\n");
For the sti dvo part:
Acked-by: Vincent Abriou
Vincent
>
vers/gpu/drm/sti/sti_hdmi.c | 3 +-
Hi Laurent,
For sti dvo, hda and hdmi:
Acked-by: Vincent Abriou
Vincent
> drivers/gpu/drm/sun4i/sun4i_rgb.c | 13 +++---
> include/drm/drm_bridge.h | 3 +-
> 23 files changed, 83 ins
drm/sti driver is now part of drm-misc as a small driver.
Signed-off-by: Vincent Abriou
---
MAINTAINERS | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index 4ea82b2..66b424a 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -4417,7 +4417,7 @@ DRM
gt; +interacting with community members on mailing lists, IRC, or bug
> +trackers. The community represents the project as a whole, and abusive
> +or bullying behaviour is not tolerated by the project.
>
Acked-by: Vincent Abriou
___
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel
Hi Chris,
Thank for the patch.
Acked-by: Vincent Abriou
Vincent
On 01/25/2017 11:10 AM, Chris Wilson wrote:
> drivers/gpu/drm/sti/sti_plane.c:76:33: error: ‘struct drm_framebuffer’
> has no member named ‘pixel_format’; did you mean ‘format’?
>
> I didn't look to hard at the
Thank for this patch.
It is working fine with sti driver.
Acked-by: Vincent Abriou
Tested-by: Vincent Abriou
Vincent
On 01/26/2017 11:56 PM, Noralf Trønnes wrote:
> drm_debugfs_cleanup() now removes all minor->debugfs_list entries
> automatically, so it's not nec
Acked-by: Vincent Abriou
On 01/12/2017 05:27 PM, Fabien Dessenne wrote:
> Since nonblocking atomic commits are now supported, the driver can
> now use drm_atomic_helper_commit().
>
> Signed-off-by: Fabien Dessenne
> ---
> drivers/gpu/drm/
Acked-by: Vincent Abriou
On 01/12/2017 05:27 PM, Fabien Dessenne wrote:
> Use drm-core to handle event.
> This is required to be able to use the nonblocking helpers.
>
> Signed-off-by: Fabien Dessenne
> ---
> drivers/gpu/drm/st
Acked-by: Vincent Abriou
On 01/12/2017 05:27 PM, Fabien Dessenne wrote:
> Fix a division by 0 case : in some cases, when the HQVDP plane is being
> disabled atomic_check() is called with "mode->clock = 0".
> In that case, do not check for scaling capabilities.
>
> Si
Remove stih416 chip deprecated register and functionnalities since stih416
chip is no more supported from Linux v4.9.
Vincent Abriou (2):
drm/sti: remove deprecated sink_term config
drm/sti: remove deprecated legacy vtg slave
drivers/gpu/drm/sti/sti_hdmi.c | 7 -
drivers/gpu/drm/sti
stih416 chip family is no more supported in Linux v4.9.
It is then useless to keep vtg slave field since it not used at all for
the stih407/10 chip family supported by sti driver.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_vtg.c | 58 +++
1
stih416 chip family is no more supported in Linux v4.9.
Then sink_term config becomes useless. The field of the
register it was referring to is tag as reserved for stih410
chip family.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hdmi.c | 7 ---
1 file changed, 7 deletions
This patch series fix issues met around gpd plane:
- fix gdp clock selection and enabling
- fix div by zero when GDP plane is being disabled
- do not process gdp update requests with unmodified parameters
Vincent Abriou (3):
drm/sti: enable gdp pixel clock in atomic_update
drm/sti: do not
Set gdp pix clock rate and parent in atomic_check function and enable
it in the atomic_update only the first time.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_gdp.c | 70 ---
1 file changed, 32 insertions(+), 38 deletions(-)
diff --git a
Do not process update requests with unmodified parameters.
This typically happens when the driver is called with legacy
(non-atomic) IOCTL : in that case atomic_update() is called multiple
times with the same parameters.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_gdp.c | 15
Fix a division by 0 case : in some cases, when the GDP plane is being
disabled atomic_check() is called with "mode->clock = 0".
In that case, do not set parent and pixel clock rate.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_gdp.c | 2 +-
1 file changed, 1 in
Remove the hdmi_mode_property and use drm_detect_hdmi_monitor function
to check whether the connected monitor is HDMI capable or DVI only.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hdmi.c | 31 ++-
drivers/gpu/drm/sti/sti_hdmi.h | 17
Unbind all the components when the STI driver is unregistered.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_drv.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/sti/sti_drv.c b/drivers/gpu/drm/sti/sti_drv.c
index e433853..60a41cc 100644
--- a/drivers/gpu/drm
From: Fabien Dessenne
Reset the fps debug information when 'fps_show' is updated.
Signed-off-by: Fabien Dessenne
---
drivers/gpu/drm/sti/sti_drv.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/gpu/drm/sti/sti_drv.c b/drivers/gpu/drm/sti/sti_drv.c
index 60a41cc..2b8cf59 100644
-
aling if mode is not set
Vincent Abriou (8):
drm/sti: remove deprecated sink_term config
drm/sti: remove deprecated legacy vtg slave
drm/sti: enable gdp pixel clock in atomic_update
drm/sti: do not set gdp pixel clock rate if mode is not set
drm/sti: do not post GDP
Fix compilation warning introduced by:
commit 0c7ff84f7f9d ("drm/sti: remove deprecated legacy vtg slave")
commit 5e60f595d6ca ("drm/sti: use atomic_helper for commit")
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_drv.c | 9 -
drivers/gpu/drm/sti/sti_
ers/gpu/drm/qxl/qxl_display.c | 5 ++--
> drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 5 ++--
> drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 5 ++--
> drivers/gpu/drm/sti/sti_crtc.c | 5 ++--
For sti:
Acked-by: Vincent Abriou
__
++--
> drivers/gpu/drm/qxl/qxl_display.c | 5 +++--
> drivers/gpu/drm/rcar-du/rcar_du_crtc.c | 5 +++--
> drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 5 +++--
> drivers/gpu/drm/sti/sti_crtc.c | 5 +++--
For sti:
Acked-by: Vincent Abriou
On stih407-410 chip family the GDP layers are able to support up to UHD
resolution (3840 x 2160).
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_gdp.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu/drm/sti
Add Benjamin Gaignard and Vincent Abriou as STM maintainers:
Benjamin Gaignard
Vincent Abriou
Signed-off-by: Vincent Abriou
---
MAINTAINERS | 2 ++
1 file changed, 2 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index 81cdd03..757d487 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
On 05/24/2017 04:52 PM, Daniel Vetter wrote:
> Seems entirely cargo-culted.
>
> Cc: Benjamin Gaignard
> Cc: Vincent Abriou
> Signed-off-by: Daniel Vetter
> ---
> drivers/gpu/drm/sti/sti_drv.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/g
igned-off-by: Laurent Pinchart
> Acked-by: Vincent Abriou
> ---
> drivers/gpu/drm/sti/sti_gdp.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> Cc: Benjamin Gaignard
> Cc: Vincent Abriou
>
> diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hdmi.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/gpu/drm/sti/sti_hdmi.c b/drivers/gpu/drm/sti/sti_hdmi.c
index c6aa291..d850dda 100644
--- a/drivers/gpu/drm/sti/sti_hdmi.c
+++ b/drivers/gpu/drm/sti/sti_hdmi.c
@@ -1054,6
nment when the plane is in use (gdp is
running).
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_gdp.c | 15 +++
drivers/gpu/drm/sti/sti_plane.h | 2 ++
2 files changed, 17 insertions(+)
diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu/drm/sti/sti_gdp.c
index 3f
Fix typo and issue while creating the vid and mixer debugfs entries.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_compositor.c | 8 ++--
drivers/gpu/drm/sti/sti_compositor.h | 4 ++--
drivers/gpu/drm/sti/sti_crtc.c | 2 +-
3 files changed, 9 insertions(+), 5 deletions
On 09/15/2016 04:27 PM, Ville Syrjälä wrote:
> On Wed, Sep 14, 2016 at 01:40:02PM +0200, Vincent Abriou wrote:
>> When a plane is going to be enabled we re-evaluate the possible crtcs
>> for the associated drm plane. Only the crtc on which the plane should be
>> di
Fix typo and issue while creating the vid and mixer debugfs entries.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_compositor.c | 22 +-
drivers/gpu/drm/sti/sti_compositor.h | 4 ++--
drivers/gpu/drm/sti/sti_crtc.c | 2 +-
3 files changed, 12 insertions
Acked-by: Vincent Abriou
On 09/15/2016 01:39 PM, Baoyou Xie wrote:
> We get 4 warnings when building kernel with W=1:
> drivers/gpu/drm/sti/sti_gdp.c:476:5: warning: no previous prototype for
> 'sti_gdp_field_cb' [-Wmissing-prototypes]
> drivers/gpu/drm/sti/sti_hqvd
Acked-by: Vincent Abriou
On 09/15/2016 03:41 PM, Fabien Dessenne wrote:
> Add some missing \n in logs.
>
> Signed-off-by: Fabien Dessenne
> ---
> drivers/gpu/drm/sti/sti_gdp.c | 2 +-
> drivers/gpu/drm/sti/sti_hda.c | 4 ++--
> drivers/gpu/drm/sti/sti_hdmi.c | 2 +-
Acked-by: Vincent Abriou
On 09/15/2016 03:41 PM, Fabien Dessenne wrote:
> Do not rely on plane->status to define whether this is the first update
> but rather check for hqvdp->xp70_initialized bit status.
> This avoids multiple and unwanted calls to sti_vtg_register_client()
>
Acked-by: Vincent Abriou
On 09/15/2016 03:41 PM, Fabien Dessenne wrote:
> Do not rely on plane->status to define whether this is the first update
> but rather check for gdp->vtg.
> This avoids multiple and unwanted calls to sti_vtg_register_client()
> which breaks th
Acked-by: Vincent Abriou
On 09/15/2016 03:41 PM, Fabien Dessenne wrote:
> When a drm_plane is being disabled, its ->crtc member is set to NULL
> before the .atomic_disable() func is called.
> To get the crtc of the plane, read old_state->crtc instead of
> drm_plane->c
Acked-by: Vincent Abriou
On 09/15/2016 03:41 PM, Fabien Dessenne wrote:
> Each pipe shall have its own notifier block to manage the vblank event.
> This fixes issues where a client registered on given pipe is later
> abusively notified of events on the other pipe.
>
> Signed
Acked-by: Vincent Abriou
On 09/15/2016 03:41 PM, Fabien Dessenne wrote:
> This is more generic and more consistent with the other members of the
> sti_compositor struct.
>
> Signed-off-by: Fabien Dessenne
> ---
> drivers/gpu/drm/sti/sti_compositor.c | 4 ++--
>
Acked-by: Vincent Abriou
On 09/15/2016 03:41 PM, Fabien Dessenne wrote:
> crtc_atomic_flush performs some additional processing, like plane
> enable at mixer level.
> Enable only the planes attached to the CRTC.
>
> Signed-off-by: Fabien Dessenne
> ---
> drivers/gpu/d
Acked-by: Vincent Abriou
On 09/15/2016 03:41 PM, Fabien Dessenne wrote:
> In atomic mode the crtc_xxx (eg crtc_hdisplay) members of the mode
> structure may be unset before calling atomic_check/commit for planes.
> Instead of, use xxx members which are actually set.
>
> Signed
On 09/15/2016 05:57 PM, Ville Syrjälä wrote:
> On Thu, Sep 15, 2016 at 04:59:55PM +0200, Vincent ABRIOU wrote:
>>
>>
>> On 09/15/2016 04:27 PM, Ville Syrjälä wrote:
>>> On Wed, Sep 14, 2016 at 01:40:02PM +0200, Vincent Abriou wrote:
>>>> When a plane
Acked-by: Vincent Abriou
On 09/19/2016 03:33 PM, ville.syrjala at linux.intel.com wrote:
> From: Ville Syrjälä
>
> drm/sti/sti_mixer.c:361:6: warning: symbol 'sti_mixer_set_matrix' was not
> declared. Should it be static?
> drm/sti/sti_gdp.c:476:5: warning: symbol &
stih415 and stih416 platform are obsolete and no more supported.
Only stih407 and stih410 platform are maintained.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/Kconfig| 6 +-
drivers/gpu/drm/sti/sti_compositor.c | 19 --
drivers/gpu/drm/sti/sti_hda.c
stih415 and stih416 platform are obsolete and no more supported.
Only stih407 and stih410 platform are maintained.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/Kconfig| 6 +-
drivers/gpu/drm/sti/Makefile | 1 -
drivers/gpu/drm/sti/sti_compositor.c
crtc_atomic_flush, enable only planes of this crtc
drm/sti: use valid video mode
Ville Syrjälä (1):
drm/sti: Fix sparse warnings
Vincent Abriou (3):
drm/sti: dpms function missing for HDMI connector
drm/sti: fix compositor debugfs creation
drm/sti: remove
Yes true, patch from Ville
https://lists.freedesktop.org/archives/dri-devel/2016-September/118631.html";
already fix patches sent by Baoyou.
Vincent
On 09/27/2016 09:07 PM, Benjamin Gaignard wrote:
> I think that create conflicts with what is already in Vincent pull
> request where we have fix th
Hi Laurent,
Ok for the sti driver.
Acked-by: Vincent Abriou
On 12/17/2016 11:39 PM, Laurent Pinchart wrote:
> The field contains a pointer to the parent platform device of the DRM
> device. As struct drm_device also contains a dev pointer to the struct
> device embedded in the platfo
This update allows to have a clearer fps log trace by adding information
about plane size and plane pixel format.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_plane.c | 17 +
drivers/gpu/drm/sti/sti_plane.h | 2 +-
2 files changed, 14 insertions(+), 5 deletions
While binding the sti drm driver the initial connectors status are set and the
fbdev is created.
Vincent Abriou (2):
drm/sti: set initial connectors status at binding
drm/sti: create fbdev at binding
drivers/gpu/drm/sti/sti_drv.c | 27 ++-
drivers/gpu/drm/sti
This avoid to have an "unknown" connector status after the sti driver is
registered.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_dvo.c | 3 +++
drivers/gpu/drm/sti/sti_hdmi.c | 3 +++
2 files changed, 6 insertions(+)
diff --git a/drivers/gpu/drm/sti/sti_dvo.c b/drive
Do not wait for a hot plug event to create fbdev.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_drv.c | 27 ++-
1 file changed, 14 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_drv.c b/drivers/gpu/drm/sti/sti_drv.c
index ff71e25
sti_vtac.c file is deprecated since stih416 chip family is not
supported anymore.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/Makefile | 1 -
drivers/gpu/drm/sti/sti_drv.c | 1 -
drivers/gpu/drm/sti/sti_drv.h | 1 -
drivers/gpu/drm/sti/sti_vtac.c | 223
Thanks for the patch.
Acked-by: Vincent Abriou
I will take it for pull request that should be issued by the end of the
week.
Vincent
On 12/21/2016 06:30 AM, Arvind Yadav wrote:
> Here, If devm_ioremap_nocache will fail. It will return NULL.
> Kernel can run into a NULL-pointer deref
Hi Arvind,
Thank for this patch.
Since sti_vtac.c file is deprecated, I rather suppress the sti_vtac.c file.
I just push a new patch to suppress it.
Thank for pointing this out.
Br
Vincent
On 12/21/2016 06:22 AM, Arvind Yadav wrote:
> Here, If devm_ioremap_nocache will fail. It will return NULL.
On 01/04/2017 10:40 AM, Daniel Vetter wrote:
> On Wed, Jan 04, 2017 at 10:26:46AM +0100, Vincent Abriou wrote:
>> sti_vtac.c file is deprecated since stih416 chip family is not
>> supported anymore.
>>
>> Signed-off-by: Vincent Abriou
>
> Ehrm, removing existing
stih416 chip family is no more supported in Linux v4.9.
It is then useless to keep sti_vtac.c file since it not used at all for
the stih407/10 chip family supported by sti driver.
Signed-off-by: Vincent Abriou
---
v2:
Take into account Daniel's comment => add a more explicit commit
On 01/04/2017 10:39 AM, Daniel Vetter wrote:
> On Wed, Jan 04, 2017 at 10:15:45AM +0100, Vincent Abriou wrote:
>> This avoid to have an "unknown" connector status after the sti driver is
>> registered.
>>
>> Signed-off-by: Vincent Abriou
>
> Why i
Do not wait for a hot plug event to create fbdev.
Cc: Tomi Valkeinen
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_drv.c | 27 ++-
1 file changed, 14 insertions(+), 13 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_drv.c b/drivers/gpu/drm/sti/sti_drv.c
Acked-by: Vincent Abriou
On 09/30/2016 05:17 PM, Arnaud Pouliquen wrote:
> This fix allows to play audio while HDMI is disconnected.
> When HDMI is disable, audio configuration is stored and samples
> are dropped (by HDMI IP).
> When HDMI is enabled, audio HDMI configuration is
mware only once
drm/sti: do not post HQVDP command if no update
Vincent Abriou (3):
drm/sti: update fps debugfs entries
drm/sti: create fbdev at binding
drm/sti: remove deprecated sti_vtac.c file
drivers/gpu/drm/sti/Makefile| 1 -
drivers/gpu/drm/sti/sti_drv
your connector cable but you want
your fbdev to have the size you have defined in the command line.
Then when you plug your connector cable, the fbdev will match your display
size.
In case no command line is specified, the 1024x768 fbdev resolution is keept
as default fbdev resolution.
Vincent
: Vincent Abriou
---
drivers/gpu/drm/drm_fb_helper.c | 32 +++-
1 file changed, 27 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm/drm_fb_helper.c
index 77d9ac6..115c265 100644
--- a/drivers/gpu/drm/drm_fb_helper.c
+++ b/drivers/gpu
drm_pick_cmdline_mode width and height parameters are useless.
Just remove them.
Cc: Daniel Vetter
Cc: Jani Nikula
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/drm_fb_helper.c| 7 +++
drivers/gpu/drm/i915/intel_fbdev.c | 2 +-
include/drm/drm_fb_helper.h| 3 +--
3 files
(1):
drm/sti: allow audio playback on HDMI even if disabled.
Arvind Yadav (1):
drm/sti: sti_vtg: Handle return NULL error from devm_ioremap_nocache
Fabien DESSENNE (2):
drm/sti: load XP70 firmware only once
drm/sti: do not post HQVDP command if no update
Vincent Abriou (3)
: Vincent Abriou
---
Patch v2:
add a break in the connector for loop when a first cmdline mode is found
drivers/gpu/drm/drm_fb_helper.c | 34 +-
1 file changed, 29 insertions(+), 5 deletions(-)
diff --git a/drivers/gpu/drm/drm_fb_helper.c b/drivers/gpu/drm
On 01/10/2017 12:39 PM, Daniel Vetter wrote:
> On Tue, Jan 10, 2017 at 12:21:09PM +0100, Vincent Abriou wrote:
>> In case no connector is found while creating the fbdev, gives the
>> possibility to specify the default fbdev size by firstly checking if the
>> command line is
On 01/11/2017 08:48 AM, Daniel Vetter wrote:
> On Tue, Jan 10, 2017 at 10:06:44PM +0200, Laurent Pinchart wrote:
>> Hi Vincent,
>>
>> On Tuesday 10 Jan 2017 13:33:29 Vincent ABRIOU wrote:
>>> On 01/10/2017 12:39 PM, Daniel Vetter wrote:
>>>> On Tue,
Hi Arnd,
Your patch will be part of the next pull request for the STI driver that
will be done by the end of next week.
Reviewed-by: Vincent Abriou
On 02/25/2016 10:11 PM, Arnd Bergmann wrote:
> The STi drm driver correctly warns about invalid format strings
> when built with
Hi,
Have you any comment for this proposal?
BR
Vincent
On 02/12/2016 10:26 AM, Vincent Abriou wrote:
> From: Fabien Dessenne
>
> If a buffer is interlaced, this "Bottom Field First" flag specifies
> which of the top or the bottom field shall be displayed first.
> W
On 02/29/2016 04:32 PM, Daniel Vetter wrote:
> On Fri, Feb 26, 2016 at 11:33:08AM +0100, Vincent ABRIOU wrote:
>> Hi,
>>
>> Have you any comment for this proposal?
>
> I guess since we don't really have userspace that uses interlaced modes,
> much less actuall
; Emil Velikov wrote:
>>
>> On 29 October 2015 at 14:21, Vincent ABRIOU wrote:
>>> Hi Takashi,
>>>
>>> Removing FW_LOADER_USER_HELPER_FALLBACK leads to a failure in our HQVDP
>>> firmware execution.
>>> Indeed, our firmware is not built-in. It is
system is
available so request_firmware function will match.
CONFIG_FW_LOADER_USER_HELPER_FALLBACK is then no more mandatory.
Cc: Takashi Iwai
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/Kconfig | 1 -
drivers/gpu/drm/sti/sti_hqvdp.c | 302
2
Thierry Reding (5):
drm/sti: Select FW_LOADER
drm/sti: Store correct CRTC index in events
drm/sti: Use drm_crtc_vblank_*() API
drm/sti: Build monolithic driver
drm/sti: Do not export symbols
Ville Syrjälä (1):
drm/sti: Constify function pointer structs
Hi Emil,
On 11/03/2015 03:55 PM, Emil Velikov wrote:
> Hi Vincent,
>
> On 2 November 2015 at 10:05, Vincent Abriou wrote:
>> @@ -754,6 +901,9 @@ static void sti_hqvdp_atomic_update(struct drm_plane
>> *drm_plane,
>> sti_hqvdp_update_hvsrc(HVSRC
compare_of, child_np);
> - of_node_put(child_np);
> + component_match_add_of(dev, &match, child_np);
> child_np = of_get_next_available_child(node, child_np);
> }
>
Hi Russel,
Ok for the sti driver.
Acked-by: Vincent Abriou
Hi Boris,
Thanks for the patch.
Acked-by: Vincent Abriou
Vincent
On 06/07/2016 01:48 PM, Boris Brezillon wrote:
> All outputs have a 1:1 relationship between connectors and encoders
> and the driver is relying on the atomic helpers: we can drop the custom
> ->best_encoder() imp
changed, 3 insertions(+), 3 deletions(-)
>
> Cc: Benjamin Gaignard
> Cc: Vincent Abriou
>
> diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu/drm/sti/sti_gdp.c
> index ff33c38da197..be7e80535083 100644
> --- a/drivers/gpu/drm/sti/sti_gdp.c
> +++ b/drivers/gpu/drm/st
Hi Laurent,
On 06/09/2016 11:17 AM, Laurent Pinchart wrote:
> Hi Vincent,
>
> On Thursday 09 Jun 2016 09:52:05 Vincent ABRIOU wrote:
>> On 06/09/2016 01:32 AM, Laurent Pinchart wrote:
>>> The driver needs the number of bytes per pixel, not the bpp and depth
>>> i
Vincent Abriou (2):
drm/sti: GDP cropping fails when we remove 2 pixels horizontally
drm/sti: implement atomic_check for the planes
drivers/gpu/drm/sti/sti_cursor.c | 101 +++---
drivers/gpu/drm/sti/sti_gdp.c| 216 +--
drivers/gpu/drm/sti
From: Bich Hemon
When a GDP gets a scale request (which it does not support), it accepts it
but crops or clamps and outputs a warning message.
Signed-off-by: Bich Hemon
---
drivers/gpu/drm/sti/sti_gdp.c | 36 +++-
1 file changed, 31 insertions(+), 5 deletions(-)
GDP source width should be equal to the destination width to get
rid of this issue.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_gdp.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_gdp.c b/drivers/gpu/drm/sti/sti_gdp.c
index
From: Fabien Dessenne
It may happen that the cursor is displayed with wrong colors which can
be explained by a CLUT wrongly fetched at the first display.
Fetching the CLUT at each commit (=move) ensures that the right colors
are used, at least from the first cursor move.
Signed-off-by: Fabien De
Atomic update should never fail. Thus all checks must be done in
the atomic_check function for each plane (gdp, hqvdp and cursor).
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_cursor.c | 90 +---
drivers/gpu/drm/sti/sti_gdp.c| 173
From: Bich Hemon
Use BT601 for SD/ED resolution and BT709 for HD resolution
Signed-off-by: Bich Hemon
---
drivers/gpu/drm/sti/sti_tvout.c | 61 ++---
drivers/gpu/drm/sti/sti_vid.c | 31 +
2 files changed, 64 insertions(+), 28 deletions(
at start up.
Bich Hemon (2):
drm/sti: adjust delay for DVO
drm/sti: fix dvo data_enable signal
Vincent Abriou (9):
drm/sti: do not clip RGB/YUV component value at connector side
drm/sti: fix panel detection for DVO connector
drm/sti: add missing encoder cleanup for DVO connector
drm
The DVO connector is tag as disconnect because of a wrong management
of the panel detection.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_dvo.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_dvo.c b/drivers/gpu/drm/sti/sti_dvo.c
Disable the clipping mode for hdmi, dvo and hda connectors.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_tvout.c | 14 ++
1 file changed, 6 insertions(+), 8 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_tvout.c b/drivers/gpu/drm/sti/sti_tvout.c
index 1c27a2c
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_tvout.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/gpu/drm/sti/sti_tvout.c b/drivers/gpu/drm/sti/sti_tvout.c
index 784a3e2..0b171fd 100644
--- a/drivers/gpu/drm/sti/sti_tvout.c
+++ b/drivers/gpu/drm/sti/sti_tvout.c
From: Bich Hemon
Modify delay to display last pixel column on DVO
Signed-off-by: Bich Hemon
---
drivers/gpu/drm/sti/sti_vtg.c | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/sti/sti_vtg.c b/drivers/gpu/drm/sti/sti_vtg.c
index 313d703..32c7986 100644
---
From: Bich Hemon
Modify AWG algorithm in order to handle more than 1023 lines
Signed-off-by: Bich Hemon
---
drivers/gpu/drm/sti/sti_awg_utils.c | 51 +
1 file changed, 35 insertions(+), 16 deletions(-)
diff --git a/drivers/gpu/drm/sti/sti_awg_utils.c
b/dri
Make sure the HD DACS are disabled when the HDA connector
is created.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hda.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/drivers/gpu/drm/sti/sti_hda.c b/drivers/gpu/drm/sti/sti_hda.c
index 49cce83..293a133 100644
--- a/drivers
Set the infoframe transmission mode according to the type of
the infoframe.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hdmi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/gpu/drm/sti/sti_hdmi.c b/drivers/gpu/drm/sti/sti_hdmi.c
index cd50156..8537852
Vendor specific infoframe is mandatory for 4K2K resolution.
Without this, the HDMI protocol compliance fails.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hdmi.c | 82 +-
1 file changed, 73 insertions(+), 9 deletions(-)
diff --git a/drivers
Clear all infoframe registers when the HDMI link is stopped.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hdmi.c | 43 ++
1 file changed, 43 insertions(+)
diff --git a/drivers/gpu/drm/sti/sti_hdmi.c b/drivers/gpu/drm/sti/sti_hdmi.c
index
Make the value of the colorspace of the HDMI infoframe configurable.
HDMI colorspace could be: RGB, YUV422 or YUV444
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hdmi.c | 68 +-
drivers/gpu/drm/sti/sti_hdmi.h | 12
2 files changed
Configures the framer of the HDMI connection.
By default starts in HDMI mode and can be swtich to DVI.
Signed-off-by: Vincent Abriou
---
drivers/gpu/drm/sti/sti_hdmi.c | 30 --
drivers/gpu/drm/sti/sti_hdmi.h | 15 +++
2 files changed, 43 insertions(+), 2
1 - 100 of 176 matches
Mail list logo
