[lvc-project] [PATCH net] cxgb4: Remove the block with the remaining condition

[Fedor Pchelkin]

On Mon, 22. Jun 10:20, Markov Gleb wrote:
> Due to the presence of the if-else block, t6_decode support is lost.
> In previous versions, the switch operator was missing. The logic has been
> redesigned, but the problem area has not been removed.
>
> Delete the if-else block.

Попробуйте более простым языком описать, пожалуйста.  Это не научная
статья, тут в приоритете понимабельность и чёткость повествования.
Зацикливаться на _синтаксисе_ не нужно.  Человеку, впервые видящему патч,
будет сложно осознать происходящее.  Вначале рекомендуется краткое
погружение в контекст.

"the switch operator was missing" - ? как это относится к текущему
патчу?  Что за switch имеется ввиду?

"The logic has been redesigned, but the problem area has not been removed"
Сложносоставленное предложение с по сути нулевой целевой нагрузкой.  Опять
же, не статью пишете :)

Предложу такой вариант, можете его сравнить:

In t4_sge_decode_idma_state() the set of decode strings for T6 adapter is
selected inside the switch-block.  However, it is overwritten later due to
the code which existed before T6 support was implemented.

Drop the rudimentary code block.


Название патча просьба составить на более качественном уровне, не
ударяясь в синтаксис.  В чём мотивация патча?  Убрать блок чтобы его
убрать?  Здесь подходит:

  cxgb4: fix decode strings dump for T6 adapters