Confira !!
Para : Vinod Koul <vkoul @ xxxxxxxxxx>, Christian Lamparter <chunkeey @ xxxxxxxxx>, Greg Kroah-Hartman <gregkh @ xxxxxxxxxxxxxxxxxxx>
Assunto : Re: [PATCH v6 0/5] usb: xhci: adicione suporte para controladores USB Renesas
De : Mathias Nyman <mathias.nyman@xxxxxxxxxxxxxxxxx>
Data : quinta-feira, 30 jan 2020 19:07:25 +0200
Cc : John Stultz <john.stultz@xxxxxxxxxx>, Mathias Nyman <mathias.nyman@xxxxxxxxx>, linux-arm-msm @ xxxxxxxxxxxxxxxxx, Bjorn Andersson <bjorn.andersson@xxxxxxxxxxxx>, Yoshihiro Shimoda <yoshihiro.shimoda.u >, Lista USB <linux-usb @ xxxxxxxxxxxxxxx>, lista de distribuição do kernel Linux <linux-kernel @ xxxxxxxxxxxxxxx>, Alan Stern <popa @ xxxxxxxxxxxxxxxxxxx>, Heikki Krogerus <heikki.krogerus@xxxxxxxxxxxxxxxx>
Em resposta a : <20200125053237.GG2841@vkoul-mobl>
Referências : < 20200113084005.849071-1-vkoul@kernel.org > <20200121064608.GA2841@vkoul-mobl> < CAAd0S9Dd7Ygx7TgV3E_A6z29efG7jsE1- xy48_cHotroWuk_ZA@mailV50> 32807Z32>
Agente do usuário : Mozilla / 5.0 (X11; Linux x86_64; rv: 60.0) Gecko / 20100101 Thunderbird / 60.9.0
Em 25.1.2020 7.32, Vinod Koul escreveu:
Em Seg, 13 de janeiro de 2020 às 12:42 Vinod Koul <vkoul @ xxxxxxxxxx> escreveu:
Esta série adiciona suporte aos controladores USB Renesas uPD720201 e uPD720202.
Isso exige que o firmware seja carregado e, caso os dispositivos tenham ROM, esses
também será programado se estiver vazio. Se a ROM estiver programada, ela também será executada.
Isso inclui dois patches da Christian que apoiavam esses controladores
sem ROM e, mais tarde, meus patches para suporte à ROM e várias versões de firmware,
gancho debugfs para exclusão de rom e exportação de funções xhci-pci.
...
Mathias, algum comentário sobre esta série ..?
Oi Vinod
Desculpe a demora.
Talvez um driver de carregamento de firmware como este que envolva o driver xhci pci possa
trabalhos.
Um benefício é que podemos pular a busca pelo nome certo do firmware com base
no PCI ID. Cada um desses controladores Renesas agora possui seu próprio pci_device_id
entrada na tabela pci_ids [] e poderia ter os nomes de firmware suportados
em .driver_data. Dessa forma, não precisaríamos adicionar o renesas_fw_table [] ou
talvez até a função renesas_needs_fw_dl () nesta série.
Sei que isso não pode ser facilmente alterado porque usb_hcd_pci_probe () assume o
ponteiro pci_device_id como argumento e espera que id.driver_data seja um
Ponteiro de driver HC.
Portanto, isso acaba sendo uma pergunta para Greg e Alan:
Faria sentido alterar usb_hcd_pci_probe () para pegar um ponteiro de driver HC
como um argumento em vez de um ponteiro para pci_device_id?
O ponteiro pci_device_id é usado apenas para extrair o identificador do driver HC.
Dessa forma, os dados do driver podem ser usados para, bem, dados do driver.
Heikki realmente sugeriu isso há algum tempo para mim, naquela época a idéia era
aprimore o código de xhci quirks usando driver_data para sinalizadores de quirk em vez de
localizando e configurando-os mais tarde.
Existem algumas outras aberturas relacionadas a esta série. Principalmente porque eu não sou (ainda)
familiarizado com todos os detalhes, então vou apenas listá-los aqui.
- É realmente suficiente adicionar o driver Renesas ao Makefile antes do xhci-pci
driver para garantir que ele seja correspondido e analisado com base no ID do fornecedor / dispositivo
antes que o driver xhci-pci seja correspondido e analisado com base na classe pci?
E se o driver Renesas for um módulo e o xhci-pci compilado?
- Anteriormente, o probe não voltava antes da adição do hcd e de tudo configurado.
Agora, com o probe request_firmware_nowait (), o retorno é antecipado com êxito e o
xhci_pci_probe () antigo que configura tudo é chamado mais tarde pela solicitação
retorno de chamada do firmware. Portanto, poderia haver um novo conjunto de corridas possível.
Por exemplo, remover pci pode ser chamado de carregamento de firmware médio ou quando o antigo
O xhci_pci_probe ainda está configurando as coisas.
Entendi que um request_firmware () síncrono no probe tem seu próprio
problemas, não tenho certeza se existe uma boa solução para isso.
- Antes de o firmware ser gravado no controlador, a versão do firmware é
comparado com um número codificado nos drivers renesas_fw_table [].
Isso significa que novas versões de firmware não podem ser suportadas sem o patch do driver.
Isso é intencional?
- Mathias
Referências :
[PATCH v6 0/5] usb: xhci: adicione suporte para controladores USB Renesas
De: Vinod Koul
Re: [PATCH v6 0/5] usb: xhci: Adicione suporte para controladoras USB Renesas
De: Vinod Koul
Re: [PATCH v6 0/5] usb: xhci: Adicione suporte para controladoras USB Renesas
De: Christian Lamparter
Re: [PATCH v6 0/5] usb: xhci: Adicione suporte para controladoras USB Renesas
De: Christian Lamparter
Re: [PATCH v6 0/5] usb: xhci: Adicione suporte para controladoras USB Renesas
De: Vinod Koul
Anterior à data: [PATCH] venus: vdec: use suspensão automática do pmruntime
Next by Date: Re: [PATCH v2] dt-bindings: clk: qcom: corrige a auto-validação, dividir e limpar a sujeira
Previous by thread: Re: [PATCH v6 0/5] usb: xhci: Adicione suporte para controladores USB Renesas
Próximo por tópico: Re: [PATCH v6 0/5] usb: xhci: Adicione suporte para controladores USB Renesas
Índice (es):
Date
Thread
Fonte
Até a próxima !!
Nenhum comentário:
Postar um comentário