Hi Svyatoslav, On Sat, 15 Feb 2025 at 11:12, Svyatoslav Ryhel <[email protected]> wrote: > > сб, 15 лют. 2025 р. о 20:07 Simon Glass <[email protected]> пише: > > > > Hi Svyatoslav, > > > > On Sat, 15 Feb 2025 at 10:42, Svyatoslav Ryhel <[email protected]> wrote: > > > > > > сб, 15 лют. 2025 р. о 19:24 Simon Glass <[email protected]> пише: > > > > > > > > Hi Svyatoslav, > > > > > > > > On Sat, 15 Feb 2025 at 08:52, Svyatoslav Ryhel <[email protected]> > > > > wrote: > > > > > > > > > > Add get_display_timing ops for internal bridges linked to > > > > > panels that do not support EDID (MIPI-DSI panels for example) > > > > > or have EDID not routed. > > > > > > > > > > Signed-off-by: Svyatoslav Ryhel <[email protected]> > > > > > --- > > > > > drivers/video/bridge/video-bridge-uclass.c | 11 +++++++++++ > > > > > include/video_bridge.h | 18 ++++++++++++++++++ > > > > > 2 files changed, 29 insertions(+) > > > > > > > > This should have a test, e.g. in test/dm/video.c or similar. > > > > > > > > > > I totally agree, but it seems that there is no tests for > > > UCLASS_VIDEO_BRIDGE unless I am missing smth. May you pls point me to > > > it? Maybe I can add simple bridge analog of linux driver as sandbox > > > test case? > > > > Oh dear, that's my fault. Yes, please add something simple. > > > > Surely, but I would need ofnode graph merged in first. Then I can add > simple bridge and lvds encoder in linux style. And convert simple > panel to be able to work with graph. I assume that would be correct > course.
Ultimately yes, but you could add a very simple video-bridge test to start, that just gets a device and calls a driver method? Regards, Simon
