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

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

[Vitaly Chikunov]

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(-)

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

[Vitaly Chikunov]

Daniil,

On Thu, Jan 16, 2025 at 12:26:24AM +0300, Vitaly Chikunov wrote:
> 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, может и в
> новом коммите надо это заменить?

Хотя, я сам это испрвил (раз уж я допустил ту ошибку) и приложил ваш pull request.

All applied to un-def/p10 and std-def/p10, thanks.


> 
> 
> 
> > 
> > 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(-)
> _______________________________________________
> devel-kernel mailing list
> devel-kernel@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-kernel