Tom Psyborg <pozega.tomislav@xxxxxxxxx> writes: > On 17/09/2019, Kalle Valo <kvalo@xxxxxxxxxxxxxx> wrote: >> Tomislav Požega wrote: >> >>> Use ath10k_pci_soc_read32 / ath10k_pci_soc_write32 functions for >>> the rest of warm_reset functions. Until now these have been used >>> only for ath10k_pci_warm_reset_si0, but since they already exist >>> it makes sense to simplify code a bit. >>> Runtime tested with QCA9862. >>> >>> Signed-off-by: Tomislav Požega <pozega.tomislav@xxxxxxxxx> >>> Signed-off-by: Kalle Valo <kvalo@xxxxxxxxxxxxxx> > > is this a typo or should've I add your SoB line to that commit? This is just because of my pwcli patchwork script and it's stgit mode. It takes the commit log directly from the pending branch, where I need to add my s-o-b as I publish the branch. In general you should only add your own s-o-b line: https://www.kernel.org/doc/html/latest/process/submitting-patches.html#sign-your-work-the-developer-s-certificate-of-origin > btw what about this patch it's been sent out quite a several months > ago? https://patchwork.kernel.org/patch/10860301/ I have not looked at that yet but it's in my queue. -- https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches