40269 2021-06-22 16:34:03 +0300 Неиспользуемые функции в старой (?) версии baikal-hdmi.c 2021-06-23 17:09:28 +0300 1 1 4 Development Sisyphus kernel-image-std-def unstable x86_64 Linux CLOSED NOTABUG P5 normal --- 1 m vt asheplyakov asheplyakov kernelbot placeholder vt qa-sisyphus oldest_to_newest 199399 0 m 2021-06-22 16:34:03 +0300 При сборке ядра 5.10.45 компилятор выдает следующие предупреждения: DEBUG: drivers/gpu/drm/baikal/baikal-hdmi.c:123:13: warning: 'baikal_dw_hdmi_unbind' defined but not used [-Wunused-function] DEBUG: 123 | static void baikal_dw_hdmi_unbind(struct device *dev, struct device *master, void *data) DEBUG: | ^~~~~~~~~~~~~~~~~~~~~ DEBUG: drivers/gpu/drm/baikal/baikal-hdmi.c:112:12: warning: 'baikal_dw_hdmi_bind' defined but not used [-Wunused-function] DEBUG: 112 | static int baikal_dw_hdmi_bind(struct device *dev, struct device *master, void *data) DEBUG: | ^~~~~~~~~~~~~~~~~~~ DEBUG: drivers/hwmon/bt1-pvt.c:978:13: warning: 'pvt_disable_clks' defined but not used [-Wunused-function] DEBUG: 978 | static void pvt_disable_clks(void *data) DEBUG: | ^~~~~~~~~~~~~~~~ Например, функции baikal_dw_hdmi_bind нет в linux-5.4.114 из SDK 5.2 нет. Сделал diff drivers/gpu/drm/baikal/baikal-hdmi.c из SDK 5.2 и kernel-image-std-def 5.10.45, в SDK указан год 2021, а в kernel-image-std-def - 2019, есть и другие отличия. Возможно, стоит обновить драйвер из SDK... 199417 1 asheplyakov 2021-06-23 16:43:27 +0300 (In reply to mikhailnov from comment #0) > При сборке ядра 5.10.45 компилятор выдает следующие предупреждения: > > DEBUG: drivers/gpu/drm/baikal/baikal-hdmi.c:123:13: warning: > 'baikal_dw_hdmi_unbind' defined but not used [-Wunused-function] > DEBUG: 123 | static void baikal_dw_hdmi_unbind(struct device *dev, struct > device *master, void *data) > DEBUG: | ^~~~~~~~~~~~~~~~~~~~~ > DEBUG: drivers/gpu/drm/baikal/baikal-hdmi.c:112:12: warning: > 'baikal_dw_hdmi_bind' defined but not used [-Wunused-function] > DEBUG: 112 | static int baikal_dw_hdmi_bind(struct device *dev, struct > device *master, void *data) > DEBUG: | ^~~~~~~~~~~~~~~~~~~ > DEBUG: drivers/hwmon/bt1-pvt.c:978:13: warning: 'pvt_disable_clks' defined > but not used [-Wunused-function] > DEBUG: 978 | static void pvt_disable_clks(void *data) > DEBUG: | ^~~~~~~~~~~~~~~~ > > Например, функции baikal_dw_hdmi_bind нет в linux-5.4.114 из SDK 5.2 нет. > Сделал diff drivers/gpu/drm/baikal/baikal-hdmi.c из SDK 5.2 и > kernel-image-std-def 5.10.45, в SDK указан год 2021, а в > kernel-image-std-def - 2019, Давайте я тоже нарисую 2021, если Вам жить легче станет. Или, чего уж мелочиться - сразу 2121. > есть и другие отличия. Конечно. 1. Драйвер из SDK поддерживает прошивку только из SDK той же версии. Мы не можем забивать на совместимость, как это делает Байкал Электроникс. Поэтому наш драйвер работает с прошивками из SDK-M 4.4, 5.1, 5.2 2. В драйвере из SDK есть, например, use-after-free [1]. У нас - нет (по крайней мере в этом месте). [1] https://github.com/baikalelectronics/kernel/blob/v5.4_BE-aarch64_stable/drivers/gpu/drm/baikal/baikal_vdu_drv.c#L97-L106 > Возможно, стоит обновить драйвер из SDK... Нет, не стоит. 199419 2 m 2021-06-23 17:09:28 +0300 Спасибо за информацию. Баг завел, разумеется, не ради свежих чисел, а чтобы сообщить о недоработке в коде. Мое дело сообщить о замеченном.