Após enviar o patch no mês passado, eu não tinha recebido nenhuma resposta dos mantenedores se o patch estava sendo revisado ou não. Isso porque o meu patch já tinha aparecido lore.

Falei com o monitor (David) sobre o que estava acontecendo e ele me recomendou esperar umas 3 semanas que é o prazo máximo que pode demorar uma revisão tendo em vista o tipo de modicação e a complexidade, ou porque eles podem ter passado o patch despercebido dentre o tanto de contribuição que eles recebem. Dado a terceira semana decidimos por fazer o RESEND, então fizemos o mesmo commit da ultima vez

kw send-patch --send -2 --cc=leonardodasigomes@gmail.com --cc=derick.william.moraes@gmail.com -- --no-cover-letter

mas mudando o título do commit para RESEND, ou seja, ao invés de:

[PATCH 2/2] drm/amd/display: Adjust set_value function with prefix to help in ftrace

ficou

[PATCH 2/2 RESEND] drm/amd/display: Adjust set_value function with prefix to help in ftrace

Após o reenvio não demorou um dia para eles me responderem para fazer uma alteração no nome da função. Eu tinha decidido por colocar o nome da função como dal_hw_gpio_set_config, mas o arquivo que eu estava modificando na verdade era o hw_hpd.c o que eles pediram para na verdade a gente colocar no nome da função como dal_hw_hpd_set_config.

Após reenvio com as alterações que eles pediram, então recebi uma mensagem do Alex Deucher com a mensagem de confirmação…

Applied the series.  Thanks!

Image

Plot twist

No meu processo de contibuição eu criei um “regex” para conseguir identificar mais problemas como esse que eu tive onde a função não está muito relacionada git grep -P '\.\b(\w+)\b = \b\1\b,', que pode ser usado em outros anos da matéria, nesse processo eu estava testando se as funções encontradas realmente faziam sentido e acabei criando o commit para as duas funções acima! Mas acabei não enviando o patch… Então o RESEND do patch na verdade é outro patch, que eu estava testando esse regex e acabei enviando sem querer (E que deu tudo certo).

Então são 2 patchs:

  • Um patch corrigindo as funções set_config e get_value no arquivo hw_hpd.c, para não possuir nome ambiguo e ter uma rastreabilidade melhor
  • Um patch corrigindo a função dcn31_apg_construct para sinalizar que essa função é do dcn31.

O primeiro patch foi enviado, revisado, atualizado e recebeu uma mensagem de approve. Já o segundo patch foi reenviado e está esperando a revisão.

Foi uma experiencia muito boa, principalmente com o kw facilitou muito o reenvio do patch 😄