Re: [PATCH v2] drm/bridge: ti-sn65dsi86: Check bridge connection failure

2025-03-17 Thread Wolfram Sang
> where leaving the dangling devices isn't really a problem. For > instance: if somehow you fail to link up to a MIPI source then it > feels OK that the GPIO / PWM and DP-AUX bus parts of the bridge > continue to dangle. You can still use them, right? Ok, convinced. Thanks for the input. Will sen

Re: [PATCH v2] drm/bridge: ti-sn65dsi86: Check bridge connection failure

2025-03-17 Thread Doug Anderson
Hi, On Sat, Mar 15, 2025 at 2:00 AM Wolfram Sang wrote: > > Hi Doug, > > > Seems reasonable to me. I would probably put an error message in this > > case, though? I don't think regmap_read() necessarily prints an error > > so it would just be a mysterious failure for why things didn't probe, > >

Re: [PATCH v2] drm/bridge: ti-sn65dsi86: Check bridge connection failure

2025-03-15 Thread Wolfram Sang
Hi Doug, > Seems reasonable to me. I would probably put an error message in this > case, though? I don't think regmap_read() necessarily prints an error > so it would just be a mysterious failure for why things didn't probe, > right? OK, can do that. > This also only solves the problems for one

Re: [PATCH v2] drm/bridge: ti-sn65dsi86: Check bridge connection failure

2025-03-14 Thread Doug Anderson
Hi, On Fri, Mar 14, 2025 at 3:42 PM Wolfram Sang wrote: > > From: Phong Hoang > > Add a check to the very first register access function when attaching a > bridge device, so we can bail out if I2C communication does not work. > > Signed-off-by: Phong Hoang > Signed-off-by: Jacopo Mondi > Revie

[PATCH v2] drm/bridge: ti-sn65dsi86: Check bridge connection failure

2025-03-14 Thread Wolfram Sang
From: Phong Hoang Add a check to the very first register access function when attaching a bridge device, so we can bail out if I2C communication does not work. Signed-off-by: Phong Hoang Signed-off-by: Jacopo Mondi Reviewed-by: Laurent Pinchart Signed-off-by: Wolfram Sang --- Build tested o