Hi Dmitry, On Tue, Jan 7, 2025 at 12:31 PM Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> wrote: > On Sat, 14 Dec 2024 15:37:04 +0200, Dmitry Baryshkov wrote: > > While working on the generic mode_valid() implementation for the HDMI > > Connector framework I noticed that unlike other DRM objects > > drm_connector accepts non-const pointer to struct drm_display_mode, > > while obviously mode_valid() isn't expected to modify the argument. > > > > Mass-change the DRM framework code to pass const argument to that > > callback. > > > > [...] > > Applied to drm-misc-next, thanks! > > [1/5] drm/encoder_slave: make mode_valid accept const struct drm_display_mode > commit: 7a5cd45fab0a2671aa4ea6d8fb80cea268387176 > [2/5] drm/amdgpu: don't change mode in amdgpu_dm_connector_mode_valid() > commit: b255ce4388e09f14311e7912d0ccd45a14a08d66 > [3/5] drm/sti: hda: pass const struct drm_display_mode* to hda_get_mode_idx() > commit: 5f011b442006ccb29044263df10843de80fc0b14 > [4/5] drm/connector: make mode_valid_ctx take a const struct drm_display_mode > commit: 66df9debcb29d14802912ed79a9cf9ba721b51a4 > [5/5] drm/connector: make mode_valid take a const struct drm_display_mode > commit: 26d6fd81916e62d2b0568d9756e5f9c33f0f9b7a I cannot find these in drm-misc or drm-next, but they are in drm-tip? The last one due to commit 2bdc721917cf141f ("Merge remote-tracking branch 'drm-misc/drm-misc-next' into drm-tip"). What am I missing? Thanks! P.S. Sima: noticed while resolving a merge conflict using drm-tip. Thx! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds