On 2026-10-02T07:31:50, Aristo Chen <[email protected]> wrote: > test: video: define WHITE and SCROLL_LINES once at file scope > > dm_test_video_text() and dm_test_video_text_12x22() each define WHITE > and SCROLL_LINES inside the function body. Macros are not scoped to a > function, so the second pair only redefines the first, and > check_vidconsole_output() further down uses the SCROLL_LINES defined > inside dm_test_video_text(). > > Define both once at file scope, before their first user, and drop the > two in-function pairs. No functional change. > > Signed-off-by: Aristo Chen <[email protected]> > > test/dm/video.c | 9 +++------ > 1 file changed, 3 insertions(+), 6 deletions(-)
Reviewed-by: Simon Glass <[email protected]>
