Re: [PATCH v1 1/2] drm/panel: simple: add Multi-Innotechnology MI1010AIT-1CP1

2021-08-05 Thread Ahmad Fatoum
Hello Oleksij, On 05.08.21 06:37, Oleksij Rempel wrote: > From: Sam Ravnborg > > The Multi Innotechnology is a 10.1" 1280x800 panel. > > The datasheet did not specify specific values for sync, back, front porch. > The values are a best guess based on values for similar panels. > > Co-Developed

Re: [PATCH v1 2/2] drm/panel: simple: add LOGIC Technologies LTTD800480070-L6WH-RT

2021-08-05 Thread Ahmad Fatoum
Hello Oleksi, On 05.08.21 06:37, Oleksij Rempel wrote: > From: Søren Andersen > > Add support for the LOGIC Technologies, Inc LTTD800480070-L6WH-RT > > Co-Developed-by: Søren Andersen > Co-Developed-by: Sam Ravnborg > Signed-off-by: Søren Andersen > Signed-off-by: Sam Ravnborg > Signed-off-

[PATCH v1 1/1] drm/bridge: anx7625: Tune K value for IVO panel

2021-08-05 Thread Xin Ji
IVO panel require less input video clock variation than video clock variation in DP CTS spec. This patch decreases the K value of ANX7625 which will shrink eDP Tx video clock variation to meet IVO panel's requirement. Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 17

Re: [PATCH v10 1/4] dt-bindings:drm/bridge:anx7625:add vendor define flags

2021-08-05 Thread Xin Ji
On Thu, Jul 29, 2021 at 01:08:24PM -0600, Rob Herring wrote: > On Mon, Jul 19, 2021 at 06:10:09PM +0800, Xin Ji wrote: > > Add 'bus-type' and 'data-lanes' define for port0. Define DP tx lane0, > > lane1 swing register array define, and audio enable flag. > > > > The device which cannot pass DP tx

Re: [PATCH v4 2/3] dt-bindings: Add DT bindings for QiShenglong Gopher 2b panel

2021-08-05 Thread Paul Cercueil
Hi Sam, Le mer., août 4 2021 at 20:25:23 +0200, Sam Ravnborg a écrit : On Wed, Aug 04, 2021 at 03:23:52AM +0300, Artjom Vejsel wrote: Add DT bindings for QiShenglong Gopher 2b 4.3" 480(RGB)x272 TFT LCD panel. Signed-off-by: Artjom Vejsel Reviewed-by: Sam Ravnborg Paul, I assume you wi

Re: [PATCH -next v2] drm/msm: Use list_move_tail instead of list_del/list_add_tail in msm_gem.c

2021-08-05 Thread Dmitry Baryshkov
On 09/06/2021 10:28, Baokun Li wrote: Using list_move_tail() instead of list_del() + list_add_tail() in msm_gem.c. Reported-by: Hulk Robot Signed-off-by: Baokun Li Reviewed-by: Dmitry Baryshkov --- V1->V2: CC mailist drivers/gpu/drm/msm/msm_gem.c | 3 +-- 1 file changed, 1 ins

Re: [PATCH 03/46] drm/i915/guc: Don't return -EAGAIN to user when guc_ids exhausted

2021-08-05 Thread Daniel Vetter
On Tue, Aug 03, 2021 at 03:29:00PM -0700, Matthew Brost wrote: > Rather than returning -EAGAIN to the user when no guc_ids are available, > implement a fair sharing algorithm in the kernel which blocks submissons > until guc_ids become available. Submissions are released one at a time, > based on p

Re: [Intel-gfx] [PATCH 04/46] drm/i915/guc: Don't allow requests not ready to consume all guc_ids

2021-08-05 Thread Daniel Vetter
On Tue, Aug 03, 2021 at 03:29:01PM -0700, Matthew Brost wrote: > Add a heuristic which checks if over half of the available guc_ids are > currently consumed by requests not ready to be submitted. If this > heuristic is true at request creation time (normal guc_id allocation > location) force all su

Re: [PATCH v4 0/3] Add ilitek ili9341 panel driver

2021-08-05 Thread Dillon Min
Hi All Just a gentle ping, thanks. Best regards. Dillon On Sat, 24 Jul 2021 at 11:44, wrote: > > From: Dillon Min > > Since the st,sf-tc240t-9370-t dts binding already exist in stm32f429-disco.dts > but, the panel driver didn't get accepted from mainline. it's time to submit > patch fot it. >

Re: [PATCH v4 1/3] dt-bindings: display: panel: Add ilitek ili9341 panel bindings

2021-08-05 Thread Linus Walleij
On Sat, Jul 24, 2021 at 5:44 AM wrote: > From: Dillon Min > > Add documentation for "ilitek,ili9341" panel. > > Signed-off-by: Dillon Min > Reviewed-by: Linus Walleij > Reviewed-by: Rob Herring > Link: > https://lore.kernel.org/lkml/1626853288-31223-2-git-send-email-dillon.min...@gmail.com/

Re: [PATCH v4 3/3] drm/panel: Add ilitek ili9341 panel driver

2021-08-05 Thread Linus Walleij
On Sat, Jul 24, 2021 at 5:44 AM wrote: > From: Dillon Min > > This driver combines tiny/ili9341.c mipi_dbi_interface driver > with mipi_dpi_interface driver, can support ili9341 with serial > mode and parallel rgb interface mode by different dts bindings. > > Signed-off-by: Dillon Min > Reporte

Re: [PATCH v4 2/3] ARM: dts: stm32: fix dtbs_check warning on ili9341 dts binding

2021-08-05 Thread Linus Walleij
On Sat, Jul 24, 2021 at 5:44 AM wrote: > From: Dillon Min > > Since the compatible string defined from ilitek,ili9341.yaml is > "st,sf-tc240t-9370-t", "ilitek,ili9341" > > so, append "ilitek,ili9341" to avoid the below dtbs_check warning. > > arch/arm/boot/dts/stm32f429-disco.dt.yaml: display@1:

Re: [PATCH v4 2/3] ARM: dts: stm32: fix dtbs_check warning on ili9341 dts binding

2021-08-05 Thread Dillon Min
Hi Linus Thanks. On Thu, 5 Aug 2021 at 17:11, Linus Walleij wrote: > > On Sat, Jul 24, 2021 at 5:44 AM wrote: > > > From: Dillon Min > > > > Since the compatible string defined from ilitek,ili9341.yaml is > > "st,sf-tc240t-9370-t", "ilitek,ili9341" > > > > so, append "ilitek,ili9341" to avoid

Re: [Intel-gfx] [PATCH v5 07/12] drm/i915/gt: Pipelined page migration

2021-08-05 Thread Daniel Vetter
On Thu, Jun 17, 2021 at 8:30 AM Thomas Hellström wrote: > From: Chris Wilson > > If we pipeline the PTE updates and then do the copy of those pages > within a single unpreemptible command packet, we can submit the copies > and leave them to be scheduled without having to synchronously wait > unde

Re: [PATCH v1 1/2] drm/panel: simple: add Multi-Innotechnology MI1010AIT-1CP1

2021-08-05 Thread Oleksij Rempel
On Thu, Aug 05, 2021 at 07:52:24AM +0200, Ahmad Fatoum wrote: > Hello Oleksij, > > On 05.08.21 06:37, Oleksij Rempel wrote: > > From: Sam Ravnborg > > > > The Multi Innotechnology is a 10.1" 1280x800 panel. > > > > The datasheet did not specify specific values for sync, back, front porch. > > T

Re: [RESEND PATCH v2 2/2] drm: add lockdep assert to drm_is_current_master_locked

2021-08-05 Thread Daniel Vetter
On Mon, Aug 02, 2021 at 06:59:57PM +0800, Desmond Cheong Zhi Xi wrote: > In drm_is_current_master_locked, accessing drm_file.master should be > protected by either drm_file.master_lookup_lock or > drm_device.master_mutex. This was previously awkward to assert with > lockdep. > > Following patch ("

Re: [Intel-gfx] [PATCH] drm/i915/userptr: Probe existence of backing struct pages upon creation

2021-08-05 Thread Maarten Lankhorst
Op 03-08-2021 om 17:57 schreef Maarten Lankhorst: > Op 2021-08-03 om 17:45 schreef Jason Ekstrand: >> On Tue, Aug 3, 2021 at 10:09 AM Daniel Vetter wrote: >>> On Wed, Jul 28, 2021 at 4:22 PM Matthew Auld >>> wrote: On Mon, 26 Jul 2021 at 17:10, Tvrtko Ursulin wrote: > On 26/07/2021

Re: [PATCH v4 0/3] Add ilitek ili9341 panel driver

2021-08-05 Thread Andy Shevchenko
On Sat, Jul 24, 2021 at 6:46 AM wrote: > > From: Dillon Min > > Since the st,sf-tc240t-9370-t dts binding already exist in stm32f429-disco.dts > but, the panel driver didn't get accepted from mainline. it's time to submit > patch fot it. > > This driver can support two different interface by diff

[PULL] drm-misc-next

2021-08-05 Thread Maarten Lankhorst
drm-misc-next-2021-08-05: drm-misc-next for v5.15: UAPI Changes: Cross-subsystem Changes: Core Changes: - Assorted docbook updates. - Unbreak damage selftests. - Define DRM_FORMAT_MAX_PLANES, maximum planes for a planar format. - Add gem fb vmap/vunmap helpers, use them in gud and vkms drivers.

Re: [PATCH v4 0/3] Add ilitek ili9341 panel driver

2021-08-05 Thread Dillon Min
Hi Andy Thanks for your question. On Thu, 5 Aug 2021 at 18:16, Andy Shevchenko wrote: > > On Sat, Jul 24, 2021 at 6:46 AM wrote: > > > > From: Dillon Min > > > > Since the st,sf-tc240t-9370-t dts binding already exist in > > stm32f429-disco.dts > > but, the panel driver didn't get accepted fr

[PATCH v3] drm/radeon: Update pitch for page flip

2021-08-05 Thread Zhenneng Li
When primary bo is updated, crtc's pitch may have not been updated, this will lead to show disorder content when user changes display mode, we update crtc's pitch in page flip to avoid this bug. This refers to amdgpu's pageflip. v1->v2: Update all of the pitch in all of the page_flip functions in

[PATCH v5 00/20] drm/sched dependency handling and implicit sync fixes

2021-08-05 Thread Daniel Vetter
Hi all, Two big changes: - bikeshed repainted in new paint, pls don't touch, it's all fresh! The functions are now called _add_dependency and _add_implicit_dependencies. - msm conversion, which includes a bugfix for the msm drm/sched conversion. I think it would be really good if the first tw

[PATCH v5 01/20] drm/sched: Split drm_sched_job_init

2021-08-05 Thread Daniel Vetter
This is a very confusingly named function, because not just does it init an object, it arms it and provides a point of no return for pushing a job into the scheduler. It would be nice if that's a bit clearer in the interface. But the real reason is that I want to push the dependency tracking helpe

[PATCH v5 04/20] drm/sched: Add dependency tracking

2021-08-05 Thread Daniel Vetter
Instead of just a callback we can just glue in the gem helpers that panfrost, v3d and lima currently use. There's really not that many ways to skin this cat. v2/3: Rebased. v4: Repaint this shed. The functions are now called _add_dependency() and _add_implicit_dependency() Reviewed-by: Boris Bre

[PATCH v5 02/20] drm/msm: Fix drm/sched point of no return rules

2021-08-05 Thread Daniel Vetter
Originally drm_sched_job_init was the point of no return, after which drivers must submit a job. I've split that up, which allows us to fix this issue pretty easily. Only thing we have to take care of is to not skip to error paths after that. Other drivers do this the same for out-fence and simila

[PATCH v5 03/20] drm/sched: Barriers are needed for entity->last_scheduled

2021-08-05 Thread Daniel Vetter
It might be good enough on x86 with just READ_ONCE, but the write side should then at least be WRITE_ONCE because x86 has total store order. It's definitely not enough on arm. Fix this proplery, which means - explain the need for the barrier in both places - point at the other side in each commen

[PATCH v5 05/20] drm/sched: drop entity parameter from drm_sched_push_job

2021-08-05 Thread Daniel Vetter
Originally a job was only bound to the queue when we pushed this, but now that's done in drm_sched_job_init, making that parameter entirely redundant. Remove it. The same applies to the context parameter in lima_sched_context_queue_task, simplify that too. v2: Rebase on top of msm adopting drm/s

[PATCH v5 08/20] drm/lima: use scheduler dependency tracking

2021-08-05 Thread Daniel Vetter
Nothing special going on here. Aside reviewing the code, it seems like drm_sched_job_arm() should be moved into lima_sched_context_queue_task and put under some mutex together with drm_sched_push_job(). See the kerneldoc for drm_sched_push_job(). v2: Rebase over renamed functions to add dependenc

[PATCH v5 06/20] drm/sched: improve docs around drm_sched_entity

2021-08-05 Thread Daniel Vetter
I found a few too many things that are tricky and not documented, so I started typing. I found a few more things that looked broken while typing, see the varios FIXME in drm_sched_entity. Also some of the usual logics: - actually include sched_entity.c declarations, that was lost in the move he

[PATCH v5 07/20] drm/panfrost: use scheduler dependency tracking

2021-08-05 Thread Daniel Vetter
Just deletes some code that's now more shared. Note that thanks to the split into drm_sched_job_init/arm we can now easily pull the _init() part from under the submission lock way ahead where we're adding the sync file in-fences as dependencies. v2: Correctly clean up the partially set up job, no

[PATCH v5 09/20] drm/v3d: Move drm_sched_job_init to v3d_job_init

2021-08-05 Thread Daniel Vetter
Prep work for using the scheduler dependency handling. We need to call drm_sched_job_init earlier so we can use the new drm_sched_job_await* functions for dependency handling here. v2: Slightly better commit message and rebase to include the drm_sched_job_arm() call (Emma). v3: Cleanup jobs under

[PATCH v5 11/20] drm/etnaviv: Use scheduler dependency handling

2021-08-05 Thread Daniel Vetter
We need to pull the drm_sched_job_init much earlier, but that's very minor surgery. v2: Actually fix up cleanup paths by calling drm_sched_job_init, which I wanted to to in the previous round (and did, for all other drivers). Spotted by Lucas. v3: Rebase over renamed functions to add dependencies

[PATCH v5 13/20] drm/gem: Delete gem array fencing helpers

2021-08-05 Thread Daniel Vetter
Integrated into the scheduler now and all users converted over. Signed-off-by: Daniel Vetter Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Thomas Zimmermann Cc: David Airlie Cc: Daniel Vetter Cc: Sumit Semwal Cc: "Christian König" Cc: linux-me...@vger.kernel.org Cc: linaro-mm-...@lists.linar

[PATCH v5 12/20] drm/msm: Use scheduler dependency handling

2021-08-05 Thread Daniel Vetter
drm_sched_job_init is already at the right place, so this boils down to deleting code. Signed-off-by: Daniel Vetter Cc: Rob Clark Cc: Sean Paul Cc: Sumit Semwal Cc: "Christian König" Cc: linux-arm-...@vger.kernel.org Cc: freedr...@lists.freedesktop.org Cc: linux-me...@vger.kernel.org Cc: lina

[PATCH v5 10/20] drm/v3d: Use scheduler dependency handling

2021-08-05 Thread Daniel Vetter
With the prep work out of the way this isn't tricky anymore. Aside: The chaining of the various jobs is a bit awkward, with the possibility of failure in bad places. I think with the drm_sched_job_init/arm split and maybe preloading the job->dependencies xarray this should be fixable. v2: Rebase

[PATCH v5 17/20] drm/etnaviv: Don't break exclusive fence ordering

2021-08-05 Thread Daniel Vetter
There's only one exclusive slot, and we must not break the ordering. Adding a new exclusive fence drops all previous fences from the dma_resv. To avoid violating the signalling order we err on the side of over-synchronizing by waiting for the existing fences, even if userspace asked us to ignore th

[PATCH v5 16/20] drm/msm: Don't break exclusive fence ordering

2021-08-05 Thread Daniel Vetter
There's only one exclusive slot, and we must not break the ordering. Adding a new exclusive fence drops all previous fences from the dma_resv. To avoid violating the signalling order we err on the side of over-synchronizing by waiting for the existing fences, even if userspace asked us to ignore t

[PATCH v5 18/20] drm/i915: delete exclude argument from i915_sw_fence_await_reservation

2021-08-05 Thread Daniel Vetter
No longer used, the last user disappeared with commit d07f0e59b2c762584478920cd2d11fba2980a94a Author: Chris Wilson Date: Fri Oct 28 13:58:44 2016 +0100 drm/i915: Move GEM activity tracking into a common struct reservation_object Signed-off-by: Daniel Vetter Cc: Maarten Lankhorst Cc: "T

[PATCH v5 15/20] drm/sched: Check locking in drm_sched_job_await_implicit

2021-08-05 Thread Daniel Vetter
You really need to hold the reservation here or all kinds of funny things can happen between grabbing the dependencies and inserting the new fences. Acked-by: Melissa Wen Signed-off-by: Daniel Vetter Cc: "Christian König" Cc: Daniel Vetter Cc: Luben Tuikov Cc: Andrey Grodzovsky Cc: Alex Deuc

[PATCH v5 14/20] drm/sched: Don't store self-dependencies

2021-08-05 Thread Daniel Vetter
This is essentially part of drm_sched_dependency_optimized(), which only amdgpu seems to make use of. Use it a bit more. This would mean that as-is amdgpu can't use the dependency helpers, at least not with the current approach amdgpu has for deciding whether a vm_flush is needed. Since amdgpu als

[PATCH v5 19/20] drm/i915: Don't break exclusive fence ordering

2021-08-05 Thread Daniel Vetter
There's only one exclusive slot, and we must not break the ordering. Adding a new exclusive fence drops all previous fences from the dma_resv. To avoid violating the signalling order we err on the side of over-synchronizing by waiting for the existing fences, even if userspace asked us to ignore th

[PATCH v5 20/20] dma-resv: Give the docs a do-over

2021-08-05 Thread Daniel Vetter
Specifically document the new/clarified rules around how the shared fences do not have any ordering requirements against the exclusive fence. But also document all the things a bit better, given how central struct dma_resv to dynamic buffer management the docs have been very inadequat. - Lots mor

[PATCH] drm/omap: Depend on CONFIG_OF

2021-08-05 Thread Laurent Pinchart
The driver accesses the drm_bridge.of_node field, which is present only if CONFIG_OF is enabled. As all platforms using omapdrm are OF-based, we can simply depend on CONFIG_OF. Signed-off-by: Laurent Pinchart --- drivers/gpu/drm/omapdrm/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(

[PATCH v11 0/4] Add MIPI rx DPI support

2021-08-05 Thread Xin Ji
Hi all, this patch series implement MIPI rx DPI feature. Please help to review. This is the v11 version, rebase all patches on the latest code. Any mistakes, please let me know, I'll fix it in the next series. Change history: v11: Fix Rob Herring comment - Move swing register description in prop

[PATCH v11 1/4] dt-bindings:drm/bridge:anx7625:add vendor define

2021-08-05 Thread Xin Ji
Add 'bus-type' and 'data-lanes' define for port0. Add DP tx lane0, lane1 swing register setting array, and audio enable flag. The device which cannot pass DP tx PHY CTS caused by long PCB trace or embedded MUX, adjusting ANX7625 PHY parameters can pass the CTS test. The adjusting type include Pre-

[PATCH v11 2/4] drm/bridge: anx7625: fix not correct return value

2021-08-05 Thread Xin Ji
At some time, the original code may return non zero value, force return 0 if operation finished. Reviewed-by: Robert Foss Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/bridge/analogix/a

[PATCH v11 3/4] drm/bridge: anx7625: add MIPI DPI input feature

2021-08-05 Thread Xin Ji
The basic anx7625 driver only support MIPI DSI rx signal input. This patch add MIPI DPI rx input configuration support, after apply this patch, the driver can support DSI rx or DPI rx by adding 'bus-type' in DT. Reviewed-by: Robert Foss Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/

[PATCH v11 4/4] drm/bridge: anx7625: add HDMI audio function

2021-08-05 Thread Xin Ji
Add audio HDMI codec function support, enable it through device true flag "analogix,audio-enable". Reviewed-by: Robert Foss Signed-off-by: Xin Ji --- drivers/gpu/drm/bridge/analogix/anx7625.c | 226 ++ drivers/gpu/drm/bridge/analogix/anx7625.h | 5 + 2 files changed, 231 i

Re: [PATCH v3 0/3] Remove all strcpy() uses

2021-08-05 Thread Greg Kroah-Hartman
On Sun, Aug 01, 2021 at 02:40:40PM +0300, Andy Shevchenko wrote: > On Sun, Aug 1, 2021 at 11:53 AM Len Baker wrote: > > > > strcpy() performs no bounds checking on the destination buffer. This > > could result in linear overflows beyond the end of the buffer, leading > > to all kinds of misbehavio

[PATCH v2 1/2] drm/panel: simple: add Multi-Innotechnology MI1010AIT-1CP1

2021-08-05 Thread Oleksij Rempel
From: Sam Ravnborg The Multi Innotechnology is a 10.1" 1280x800 panel. The datasheet did not specify specific values for sync, back, front porch. The values are a best guess based on values for similar panels. Co-Developed-by: Sam Ravnborg Co-Developed-by: Ulrich Ölmann Signed-off-by: Sam Rav

[PATCH v2 0/2] mainline panels used on Skov boards

2021-08-05 Thread Oleksij Rempel
changes v2: - set connector_type to DRM_MODE_CONNECTOR_DPI for the logictechno_lttd800480070_l6wh_rt panel Sam Ravnborg (1): drm/panel: simple: add Multi-Innotechnology MI1010AIT-1CP1 Søren Andersen (1): drm/panel: simple: add LOGIC Technologies LTTD800480070-L6WH-RT drivers/gpu/drm/panel

[PATCH v2 2/2] drm/panel: simple: add LOGIC Technologies LTTD800480070-L6WH-RT

2021-08-05 Thread Oleksij Rempel
From: Søren Andersen Add support for the LOGIC Technologies, Inc LTTD800480070-L6WH-RT Co-Developed-by: Søren Andersen Co-Developed-by: Sam Ravnborg Signed-off-by: Søren Andersen Signed-off-by: Sam Ravnborg Signed-off-by: Oleksij Rempel --- drivers/gpu/drm/panel/panel-simple.c | 35 +++

Re: [PATCH v3 0/3] Remove all strcpy() uses

2021-08-05 Thread Andy Shevchenko
On Thu, Aug 5, 2021 at 2:18 PM Greg Kroah-Hartman wrote: > On Sun, Aug 01, 2021 at 02:40:40PM +0300, Andy Shevchenko wrote: > > On Sun, Aug 1, 2021 at 11:53 AM Len Baker wrote: > > > > > > strcpy() performs no bounds checking on the destination buffer. This > > > could result in linear overflows

Re: [PATCH 58/64] powerpc: Split memset() to avoid multi-field overflow

2021-08-05 Thread Michael Ellerman
Kees Cook writes: > In preparation for FORTIFY_SOURCE performing compile-time and run-time > field bounds checking for memset(), avoid intentionally writing across > neighboring fields. > > Instead of writing across a field boundary with memset(), move the call > to just the array, and an explicit

Re: [PATCH v3 0/3] Remove all strcpy() uses

2021-08-05 Thread Greg Kroah-Hartman
On Thu, Aug 05, 2021 at 02:30:35PM +0300, Andy Shevchenko wrote: > On Thu, Aug 5, 2021 at 2:18 PM Greg Kroah-Hartman > wrote: > > On Sun, Aug 01, 2021 at 02:40:40PM +0300, Andy Shevchenko wrote: > > > On Sun, Aug 1, 2021 at 11:53 AM Len Baker wrote: > > > > > > > > strcpy() performs no bounds che

Re: [PATCH v3 09/14] vfio/pci: Change vfio_pci_try_bus_reset() to use the dev_set

2021-08-05 Thread Jason Gunthorpe
On Tue, Aug 03, 2021 at 10:52:25AM -0600, Alex Williamson wrote: > On Tue, 3 Aug 2021 13:41:52 -0300 > Jason Gunthorpe wrote: > > On Tue, Aug 03, 2021 at 10:34:06AM -0600, Alex Williamson wrote: > > > I think the vfio_pci_find_reset_target() function needs to be re-worked > > > to just tell us tru

Re: [PATCH v3 0/3] Remove all strcpy() uses

2021-08-05 Thread Andy Shevchenko
+Cc: David, Daniel, Noralf. The idea is to move fbtft under drivers/fbdev on the same terms, i.e. no acceptance of the new drivers there. The rationale is that for some of the panels it (fbtft) will be the only driver and nobody will convert it to tiny DRM. See more below. On Thu, Aug 5, 2021 at

Re: [Intel-gfx] [PATCH 24/33] drm/i915/guc: Implement banned contexts for GuC submission

2021-08-05 Thread Tvrtko Ursulin
On 27/07/2021 01:23, Matthew Brost wrote: When using GuC submission, if a context gets banned disable scheduling and mark all inflight requests as complete. Cc: John Harrison Signed-off-by: Matthew Brost Reviewed-by: John Harrison --- drivers/gpu/drm/i915/gem/i915_gem_context.c | 2 +-

[PATCH] drm/i915: Be more gentle when exiting non-persistent contexts

2021-08-05 Thread Tvrtko Ursulin
From: Tvrtko Ursulin When a non-persistent context exits we currently mark it as banned in order to trigger fast termination of any outstanding GPU jobs it may have left running. In doing so we apply a very strict 1ms limit in which the left over job has to preempt before we issues an engine res

Re: [RFC v1 0/4] drm: Add support for DRM_CAP_DEFERRED_OUT_FENCE capability

2021-08-05 Thread Daniel Vetter
On Thu, Aug 05, 2021 at 04:15:27AM +, Kasireddy, Vivek wrote: > Hi Daniel, > > > > >>> The solution: > > > >>> - To ensure full framerate, the Guest compositor has to start it's > > > >>> repaint cycle > > (including > > > >>> the 9 ms wait) when the Host compositor sends the frame callback

Re: [PATCH 04/10] drm/bridge: Document the probe issue with MIPI-DSI bridges

2021-08-05 Thread Jagan Teki
Hi Maxime, On Wed, Jul 28, 2021 at 7:05 PM Maxime Ripard wrote: > > Hi Jagan, > > On Tue, Jul 27, 2021 at 03:12:09PM +0530, Jagan Teki wrote: > > On Tue, Jul 20, 2021 at 7:15 PM Maxime Ripard wrote: > > > > > > Interactions between bridges, panels, MIPI-DSI host and the component > > > framework

Re: [PATCH] drm/omap: Depend on CONFIG_OF

2021-08-05 Thread Daniel Vetter
On Thu, Aug 05, 2021 at 01:52:54PM +0300, Laurent Pinchart wrote: > The driver accesses the drm_bridge.of_node field, which is present only > if CONFIG_OF is enabled. As all platforms using omapdrm are OF-based, we > can simply depend on CONFIG_OF. > > Signed-off-by: Laurent Pinchart Reviewed-by

Re: [PATCH v4 2/3] ARM: dts: stm32: fix dtbs_check warning on ili9341 dts binding

2021-08-05 Thread Alexandre TORGUE
Hi Dillon On 8/5/21 11:19 AM, Dillon Min wrote: Hi Linus Thanks. On Thu, 5 Aug 2021 at 17:11, Linus Walleij wrote: On Sat, Jul 24, 2021 at 5:44 AM wrote: From: Dillon Min Since the compatible string defined from ilitek,ili9341.yaml is "st,sf-tc240t-9370-t", "ilitek,ili9341" so, append

Re: [PATCH v3 07/14] vfio/platform: Use open_device() instead of open coding a refcnt scheme

2021-08-05 Thread Eric Auger
Hi Jason, On 7/29/21 2:49 AM, Jason Gunthorpe wrote: > Platform simply wants to run some code when the device is first > opened/last closed. Use the core framework and locking for this. Aside > from removing a bit of code this narrows the locking scope from a global > lock. > > Signed-off-by: Jas

Re: [PATCH v2 1/2] drm/panel: simple: add Multi-Innotechnology MI1010AIT-1CP1

2021-08-05 Thread Ahmad Fatoum
On 05.08.21 13:19, Oleksij Rempel wrote: > From: Sam Ravnborg > > The Multi Innotechnology is a 10.1" 1280x800 panel. > > The datasheet did not specify specific values for sync, back, front porch. > The values are a best guess based on values for similar panels. > > Co-Developed-by: Sam Ravnbor

Re: [PATCH v2 2/2] drm/panel: simple: add LOGIC Technologies LTTD800480070-L6WH-RT

2021-08-05 Thread Ahmad Fatoum
On 05.08.21 13:19, Oleksij Rempel wrote: > From: Søren Andersen > > Add support for the LOGIC Technologies, Inc LTTD800480070-L6WH-RT > > Co-Developed-by: Søren Andersen > Co-Developed-by: Sam Ravnborg > Signed-off-by: Søren Andersen > Signed-off-by: Sam Ravnborg > Signed-off-by: Oleksij Rem

Re: [PATCH v5 14/20] drm/sched: Don't store self-dependencies

2021-08-05 Thread Christian König
Am 05.08.21 um 12:46 schrieb Daniel Vetter: This is essentially part of drm_sched_dependency_optimized(), which only amdgpu seems to make use of. Use it a bit more. This would mean that as-is amdgpu can't use the dependency helpers, at least not with the current approach amdgpu has for decidi

Re: [PATCH v5 15/20] drm/sched: Check locking in drm_sched_job_await_implicit

2021-08-05 Thread Christian König
Am 05.08.21 um 12:47 schrieb Daniel Vetter: You really need to hold the reservation here or all kinds of funny things can happen between grabbing the dependencies and inserting the new fences. Acked-by: Melissa Wen Signed-off-by: Daniel Vetter Cc: "Christian König" Cc: Daniel Vetter Cc: Lube

Re: [PATCH v7 04/12] iommu/mediatek: Add device_link between the consumer and the larb devices

2021-08-05 Thread Dafna Hirschfeld
On 30.07.21 04:52, Yong Wu wrote: MediaTek IOMMU-SMI diagram is like below. all the consumer connect with smi-larb, then connect with smi-common. M4U | smi-common | - | |... | | larb1 larb2 | | vdec

Re: [PATCH v5 14/20] drm/sched: Don't store self-dependencies

2021-08-05 Thread Daniel Vetter
On Thu, Aug 5, 2021 at 3:18 PM Christian König wrote: > > > > Am 05.08.21 um 12:46 schrieb Daniel Vetter: > > This is essentially part of drm_sched_dependency_optimized(), which > > only amdgpu seems to make use of. Use it a bit more. > > > > This would mean that as-is amdgpu can't use the depende

Re: [PATCH v5 15/20] drm/sched: Check locking in drm_sched_job_await_implicit

2021-08-05 Thread Daniel Vetter
On Thu, Aug 5, 2021 at 3:19 PM Christian König wrote: > > Am 05.08.21 um 12:47 schrieb Daniel Vetter: > > You really need to hold the reservation here or all kinds of funny > > things can happen between grabbing the dependencies and inserting the > > new fences. > > > > Acked-by: Melissa Wen > >

[PATCH 0/2] Add support for Samsung S6D27A1 display panel

2021-08-05 Thread Markuss Broks
Add support for Samsung Mobile Displays(SMD) Samsung S6D27A1 panel. This display panel is used on Samsung mobile devices such as Samsung Galaxy Ace 2 (GT-I8160) also known as Codina. Markuss Broks (2): drm/panel: Add DT bindings for Samsung S6D27A1 display panel drm/panel: s6d27a1: Add driver

[PATCH 1/2] drm/panel: Add DT bindings for Samsung S6D27A1 display panel

2021-08-05 Thread Markuss Broks
This adds device-tree bindings for the Samsung S6D27A1 RGB DPI display panel. Signed-off-by: Markuss Broks --- .../display/panel/samsung,s6d27a1.yaml| 97 +++ 1 file changed, 97 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/panel/samsung,s6d

[PATCH 2/2] drm/panel: s6d27a1: Add driver for Samsung S6D27A1 display panel

2021-08-05 Thread Markuss Broks
This adds a driver for Samsung S6D27A1 display controller and panel. This panel is found in the Samsung GT-I8160 mobile phone, and possibly some other mobile phones. This display needs manufacturer commands to configure it; the commands used in this driver were taken from downstream driver by Gare

Re: [PATCH v5 01/20] drm/sched: Split drm_sched_job_init

2021-08-05 Thread Christian König
Am 05.08.21 um 12:46 schrieb Daniel Vetter: This is a very confusingly named function, because not just does it init an object, it arms it and provides a point of no return for pushing a job into the scheduler. It would be nice if that's a bit clearer in the interface. But the real reason is tha

Re: [PATCH v5 03/20] drm/sched: Barriers are needed for entity->last_scheduled

2021-08-05 Thread Christian König
Am 05.08.21 um 12:46 schrieb Daniel Vetter: It might be good enough on x86 with just READ_ONCE, but the write side should then at least be WRITE_ONCE because x86 has total store order. It's definitely not enough on arm. Fix this proplery, which means - explain the need for the barrier in bot

Re: [PATCH v5 04/20] drm/sched: Add dependency tracking

2021-08-05 Thread Christian König
Am 05.08.21 um 12:46 schrieb Daniel Vetter: Instead of just a callback we can just glue in the gem helpers that panfrost, v3d and lima currently use. There's really not that many ways to skin this cat. v2/3: Rebased. v4: Repaint this shed. The functions are now called _add_dependency() and _add

Re: [PATCH v5 05/20] drm/sched: drop entity parameter from drm_sched_push_job

2021-08-05 Thread Christian König
Am 05.08.21 um 12:46 schrieb Daniel Vetter: Originally a job was only bound to the queue when we pushed this, but now that's done in drm_sched_job_init, making that parameter entirely redundant. Remove it. The same applies to the context parameter in lima_sched_context_queue_task, simplify that

Re: [PATCH v5 14/20] drm/sched: Don't store self-dependencies

2021-08-05 Thread Christian König
Am 05.08.21 um 15:25 schrieb Daniel Vetter: On Thu, Aug 5, 2021 at 3:18 PM Christian König wrote: Am 05.08.21 um 12:46 schrieb Daniel Vetter: This is essentially part of drm_sched_dependency_optimized(), which only amdgpu seems to make use of. Use it a bit more. This would mean that as-is a

Re: [RESEND PATCH v6 04/14] drm/msm/dpu: Replace definitions for dpu debug macros

2021-08-05 Thread Dmitry Baryshkov
On 21/07/2021 20:55, Sean Paul wrote: From: Sean Paul The debug messages shouldn't be logged as errors when debug categories are enabled. Use the drm logging helpers to do the right thing Signed-off-by: Sean Paul Link: https://patchwork.freedesktop.org/patch/msgid/20200608210505.48519-5-s...

Re: [PATCH v5 01/20] drm/sched: Split drm_sched_job_init

2021-08-05 Thread Daniel Vetter
On Thu, Aug 5, 2021 at 3:44 PM Christian König wrote: > Am 05.08.21 um 12:46 schrieb Daniel Vetter: > > This is a very confusingly named function, because not just does it > > init an object, it arms it and provides a point of no return for > > pushing a job into the scheduler. It would be nice if

[PATCH v2 2/8] drm/ingenic: Add jz4780 Synopsys HDMI driver

2021-08-05 Thread H. Nikolaus Schaller
From: Paul Boddie A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications. Signed-off-by: Paul Boddie Signed-off-by: Ezequiel Garcia Signed-off-by: H. Nikolaus Schaller --- drivers/

[PATCH v2 3/8] drm/ingenic: Add support for JZ4780 and HDMI output

2021-08-05 Thread H. Nikolaus Schaller
From: Paul Boddie Add support for the LCD controller present on JZ4780 SoCs. This SoC uses 8-byte descriptors which extend the current 4-byte descriptors used for other Ingenic SoCs. Also, add special handling for HDMI-A connectors. For some reason, only the primary planes are working properly.

[PATCH v2 7/8] MIPS: CI20: defconfig: configure for DRM_DW_HDMI_JZ4780

2021-08-05 Thread H. Nikolaus Schaller
Enable CONFIG options as modules. Signed-off-by: Ezequiel Garcia Signed-off-by: H. Nikolaus Schaller --- arch/mips/configs/ci20_defconfig | 7 +++ 1 file changed, 7 insertions(+) diff --git a/arch/mips/configs/ci20_defconfig b/arch/mips/configs/ci20_defconfig index ab7ebb0668340..9d4716301

[PATCH v2 5/8] MIPS: DTS: jz4780: account for Synopsys HDMI driver and LCD controller

2021-08-05 Thread H. Nikolaus Schaller
From: Paul Boddie A specialisation of the generic Synopsys HDMI driver is employed for JZ4780 HDMI support. This requires a new driver, plus device tree and configuration modifications. Signed-off-by: Paul Boddie Signed-off-by: H. Nikolaus Schaller --- arch/mips/boot/dts/ingenic/jz4780.dtsi |

[PATCH v2 8/8] [RFC] drm/ingenic: convert to component framework for jz4780 hdmi

2021-08-05 Thread H. Nikolaus Schaller
This patch attempts to convert the ingenic-dw-hdmi driver into a version that uses the component framework. Unfortunately the new version does not work. Debugging shows that ingenic_dw_hdmi_bind() is never called. Suggestions for reasons and fixes are welcome. Signed-off-by: Paul Boddie Co-aut

[PATCH v2 1/8] drm/bridge: synopsis: Add mode_fixup and bridge timings support

2021-08-05 Thread H. Nikolaus Schaller
From: Paul Boddie The platform-specific configuration structure is augmented with mode_fixup and timings members so that specialisations of the Synopsys driver can introduce mode flags and bus flags. Signed-off-by: Paul Boddie Signed-off-by: Ezequiel Garcia Signed-off-by: H. Nikolaus Schaller

[PATCH v2 4/8] dt-bindings: display: Add ingenic-jz4780-hdmi DT Schema

2021-08-05 Thread H. Nikolaus Schaller
From: Sam Ravnborg Add DT bindings for the hdmi driver for the Ingenic JZ4780 SoC. Based on .txt binding from Zubair Lutfullah Kakakhel Signed-off-by: Sam Ravnborg Signed-off-by: H. Nikolaus Schaller Cc: Rob Herring Cc: devicet...@vger.kernel.org --- .../bindings/display/ingenic-jz4780-hdmi.

[PATCH v2 6/8] MIPS: DTS: CI20: add HDMI setup

2021-08-05 Thread H. Nikolaus Schaller
From: Paul Boddie We need to hook up * HDMI power regulator * HDMI connector * DDC pinmux * HDMI and LCD endpoint connections Signed-off-by: Paul Boddie Signed-off-by: H. Nikolaus Schaller --- arch/mips/boot/dts/ingenic/ci20.dts | 64 + 1 file changed, 64 insertion

[PATCH v2 0/8] MIPS: jz4780 HDMI

2021-08-05 Thread H. Nikolaus Schaller
This series adds HDMI support for JZ4780 and CI20 board V2: - code and commit messages revisited for checkpatch warnings - rebased on v5.14-rc4 - include (failed, hence RFC 8/8) attempt to convert to component framework (was suggested by Paul Cercueil a while ago) H. Nikolaus Schaller (2):

Re: [PATCH v2 1/8] drm/bridge: synopsis: Add mode_fixup and bridge timings support

2021-08-05 Thread Robert Foss
Hey Nikolaus, Thanks for submitting this series. On Thu, 5 Aug 2021 at 16:08, H. Nikolaus Schaller wrote: > > From: Paul Boddie > > The platform-specific configuration structure is augmented with > mode_fixup and timings members so that specialisations of the > Synopsys driver can introduce mod

Re: [PATCH v2 1/8] drm/bridge: synopsis: Add mode_fixup and bridge timings support

2021-08-05 Thread H. Nikolaus Schaller
Hi Robert, > Am 05.08.2021 um 16:32 schrieb Robert Foss : > > Hey Nikolaus, > > Thanks for submitting this series. > > On Thu, 5 Aug 2021 at 16:08, H. Nikolaus Schaller wrote: >> >> From: Paul Boddie >> >> + .mode_fixup = dw_hdmi_bridge_mode_fixup, > > mode_fixup() has been deprecate

Re: [PATCH v5 01/20] drm/sched: Split drm_sched_job_init

2021-08-05 Thread Christian König
Am 05.08.21 um 16:07 schrieb Daniel Vetter: On Thu, Aug 5, 2021 at 3:44 PM Christian König wrote: Am 05.08.21 um 12:46 schrieb Daniel Vetter: This is a very confusingly named function, because not just does it init an object, it arms it and provides a point of no return for pushing a job into

Re: [PATCH v2 8/8] [RFC] drm/ingenic: convert to component framework for jz4780 hdmi

2021-08-05 Thread Laurent Pinchart
Hi Nikolaus, Thank you for the patch. On Thu, Aug 05, 2021 at 04:07:57PM +0200, H. Nikolaus Schaller wrote: > This patch attempts to convert the ingenic-dw-hdmi driver > into a version that uses the component framework. Why ? What problem would this solve ? > Unfortunately the new version does

Re: [PATCH v5 14/20] drm/sched: Don't store self-dependencies

2021-08-05 Thread Daniel Vetter
On Thu, Aug 5, 2021 at 3:57 PM Christian König wrote: > Am 05.08.21 um 15:25 schrieb Daniel Vetter: > > On Thu, Aug 5, 2021 at 3:18 PM Christian König > > wrote: > >> > >> > >> Am 05.08.21 um 12:46 schrieb Daniel Vetter: > >>> This is essentially part of drm_sched_dependency_optimized(), which >

Re: [PATCH] drm/omap: Depend on CONFIG_OF

2021-08-05 Thread Peter Stuge
Laurent Pinchart wrote: > +++ b/drivers/gpu/drm/omapdrm/Kconfig > @@ -1,7 +1,7 @@ > # SPDX-License-Identifier: GPL-2.0-only > config DRM_OMAP > tristate "OMAP DRM" > - depends on DRM > + depends on DRM && OF > depends on ARCH_OMAP2PLUS || ARCH_MULTIPLATFORM > select OMAP

Re: [PATCH v5 01/20] drm/sched: Split drm_sched_job_init

2021-08-05 Thread Daniel Vetter
On Thu, Aug 5, 2021 at 4:47 PM Christian König wrote: > > Am 05.08.21 um 16:07 schrieb Daniel Vetter: > > On Thu, Aug 5, 2021 at 3:44 PM Christian König > > wrote: > >> Am 05.08.21 um 12:46 schrieb Daniel Vetter: > >>> This is a very confusingly named function, because not just does it > >>> ini

Re: [PATCH] drm/omap: Depend on CONFIG_OF

2021-08-05 Thread Daniel Vetter
On Thu, Aug 5, 2021 at 5:07 PM Peter Stuge wrote: > Laurent Pinchart wrote: > > +++ b/drivers/gpu/drm/omapdrm/Kconfig > > @@ -1,7 +1,7 @@ > > # SPDX-License-Identifier: GPL-2.0-only > > config DRM_OMAP > > tristate "OMAP DRM" > > - depends on DRM > > + depends on DRM && OF > >

Re: [PATCH v5 07/20] drm/panfrost: use scheduler dependency tracking

2021-08-05 Thread Alyssa Rosenzweig
Acked-by: Alyssa Rosenzweig On Thu, Aug 05, 2021 at 12:46:52PM +0200, Daniel Vetter wrote: > Just deletes some code that's now more shared. > > Note that thanks to the split into drm_sched_job_init/arm we can now > easily pull the _init() part from under the submission lock way ahead > where we'

Re: [PATCH v2 3/8] drm/ingenic: Add support for JZ4780 and HDMI output

2021-08-05 Thread Paul Cercueil
Hi Nikolaus & Paul, Le jeu., août 5 2021 at 16:07:52 +0200, H. Nikolaus Schaller a écrit : From: Paul Boddie Add support for the LCD controller present on JZ4780 SoCs. This SoC uses 8-byte descriptors which extend the current 4-byte descriptors used for other Ingenic SoCs. Also, add special

  1   2   >