Bug 40269 - Неиспользуемые функции в старой (?) версии baikal-hdmi.c
Summary: Неиспользуемые функции в старой (?) версии baikal-hdmi.c
Status: CLOSED NOTABUG
Alias: None
Product: Sisyphus
Classification: Development
Component: kernel-image-std-def (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: Vitaly Chikunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-06-22 16:34 MSK by mikhailnov
Modified: 2021-06-23 17:09 MSK (History)
4 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description mikhailnov 2021-06-22 16:34:03 MSK 
При сборке ядра 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...
Comment 1 Alexey Sheplyakov 2021-06-23 16:43:27 MSK 
(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...

Нет, не стоит.
Comment 2 mikhailnov 2021-06-23 17:09:28 MSK 
Спасибо за информацию. Баг завел, разумеется, не ради свежих чисел, а чтобы сообщить о недоработке в коде. Мое дело сообщить о замеченном.