Re: [d-kernel] Ядро 6.6/sisyphus с поддержкой СнК Байкал-М

[Vitaly Chikunov <vt@altlinux.org>]

Hi,

On Wed, Jan 15, 2025 at 05:37:48PM +0400, Daniil Gnusarev wrote:
> Добрый день.
> 
> Патчи для серии baikal-m для ядра un-def/p10.
> Исправлено переключение видеорежимов. Разрешение вплоть до поддерживаемого
> 2560*1440 по HDMI выходу.
> До конца применен патч (BROKEN) dwc-i2s: support Baikal-M SoC.
> Протестированы на плате TF307 версии MB-S-D с прошивкой 5.6.
> 
> The following changes since commit 293f1f92b6625101e9afa5f55f197963fe5f1cd7:
> 
>   1:6.1.124-alt1 (2025-01-09 18:32:31 +0300)
> 
> are available in the Git repository at:
> 
>   https://gitlab.basealt.space/altworkstation/kernel-image.git
> tags/baikal-m-un-def-p10-6.1.124-alt1

Вы добавляете коммит "ALT: Completely re-apply (BROKEN) dwc-i2s:
support Baikal-M SoC", с целью дополнить коммит "ALT: Re-apply
(BROKEN) dwc-i2s: support Baikal-M SoC" при этом описание коммита
одинаковое у обоих коммитов хотя новый коммит только удаляет лишний
вызов dev_err_ratelimited. Спасибо за фикс, но

1. Не могли бы вы указать описании зачем он нужен, а не просто повторить
описание того коммита, который вы исправляете? Например, о том, что удален
лишний вызов dev_err_ratelimited возникший при решении конфликта в
таком-то коммите.

Так как это новый косммит с исправлением, пожалуйста, удалите из него тэги
относящиеся к прошлому коммиту.

    [ vt: Fix merge conflict. ]
    Signed-off-by: Vitaly Chikunov <vt@altlinux.org>

Тэги должны отражать только тот коммит, который делается.

2. Если апстрим делает замену dev_dbg на dev_err_ratelimited, может и в
новом коммите надо это заменить?



> 
> for you to fetch changes up to 58cdcff579d1d282678a1b7289d15bf347487bb2:
> 
>   drm: baikal-vdu: fix mode switching (2025-01-15 10:50:34 +0400)
> 
> ----------------------------------------------------------------
> Alexey Sheplyakov (1):
>       ALT: Completely re-apply (BROKEN) dwc-i2s: support Baikal-M SoC
> 
> Daniil Gnusarev (1):
>       drm: baikal-vdu: fix mode switching
> 
>  drivers/gpu/drm/baikal/baikal_vdu_crtc.c  | 16 +++++++++++-----
>  drivers/gpu/drm/baikal/baikal_vdu_plane.c |  3 +++
>  sound/soc/dwc/dwc-i2s.c                   |  1 -
>  3 files changed, 14 insertions(+), 6 deletions(-)