Re: [PATCH 3/6] drm: zynqmp_dp: Add locking

2024-03-18 Thread Sean Anderson
On 3/18/24 13:59, Laurent Pinchart wrote: > Hi Sean, > > On Mon, Mar 18, 2024 at 01:29:12PM -0400, Sean Anderson wrote: >> On 3/18/24 13:16, Laurent Pinchart wrote: >> > On Fri, Mar 15, 2024 at 07:09:13PM -0400, Sean Anderson wrote: >> >> Add some locking, since none is provided by the drm subsyst

Re: [PATCH 3/6] drm: zynqmp_dp: Add locking

2024-03-18 Thread Laurent Pinchart
Hi Sean, On Mon, Mar 18, 2024 at 01:29:12PM -0400, Sean Anderson wrote: > On 3/18/24 13:16, Laurent Pinchart wrote: > > On Fri, Mar 15, 2024 at 07:09:13PM -0400, Sean Anderson wrote: > >> Add some locking, since none is provided by the drm subsystem. This will > > > > That's not quite right, the

Re: [PATCH 3/6] drm: zynqmp_dp: Add locking

2024-03-18 Thread Sean Anderson
On 3/18/24 13:16, Laurent Pinchart wrote: > Hi Sean, > > Thank you for the patch. > > On Fri, Mar 15, 2024 at 07:09:13PM -0400, Sean Anderson wrote: >> Add some locking, since none is provided by the drm subsystem. This will > > That's not quite right, the DRM core doesn't call bridge operations

Re: [PATCH 3/6] drm: zynqmp_dp: Add locking

2024-03-18 Thread Laurent Pinchart
Hi Sean, Thank you for the patch. On Fri, Mar 15, 2024 at 07:09:13PM -0400, Sean Anderson wrote: > Add some locking, since none is provided by the drm subsystem. This will That's not quite right, the DRM core doesn't call bridge operations concurrently. We may need locking to protect against rac

Re: [PATCH 3/6] drm: zynqmp_dp: Add locking

2024-03-18 Thread Sean Anderson
drm-zynqmp_dp-Downgrade-log-level-for-aux-retries-message/20240316-071208 > base: v6.8 > patch link: > https://lore.kernel.org/r/20240315230916.1759060-4-sean.anderson%40linux.dev > patch subject: [PATCH 3/6] drm: zynqmp_dp: Add locking > config: alpha-allyesconfig > (htt

Re: [PATCH 3/6] drm: zynqmp_dp: Add locking

2024-03-16 Thread kernel test robot
https://lore.kernel.org/r/20240315230916.1759060-4-sean.anderson%40linux.dev patch subject: [PATCH 3/6] drm: zynqmp_dp: Add locking config: alpha-allyesconfig (https://download.01.org/0day-ci/archive/20240316/202403161747.trmfawhm-...@intel.com/config) compiler: alpha-linux-gcc (GCC) 13.2.0 reproduce (thi

[PATCH 3/6] drm: zynqmp_dp: Add locking

2024-03-15 Thread Sean Anderson
Add some locking, since none is provided by the drm subsystem. This will prevent the IRQ/workers/bridge API calls from stepping on each other's toes. Signed-off-by: Sean Anderson --- drivers/gpu/drm/xlnx/zynqmp_dp.c | 59 +++- 1 file changed, 42 insertions(+), 17 del