On 30/03/2024 23:04, Marc Gonzalez wrote: > On 30/03/2024 19:23, Krzysztof Kozlowski wrote: > >> On 30/03/2024 19:20, Krzysztof Kozlowski wrote: >> >>> On 28/03/2024 18:36, Marc Gonzalez wrote: >>> >>>> The ath10k driver waits for an "MSA_READY" indicator >>>> to complete initialization. If the indicator is not >>>> received, then the device remains unusable. >>>> >>>> cf. ath10k_qmi_driver_event_work() >>>> >>>> Several msm8998-based devices are affected by this issue. >>>> Oddly, it seems safe to NOT wait for the indicator, and >>>> proceed immediately when QMI_EVENT_SERVER_ARRIVE. >>> >>> This is v2, so where is the changelog? >> >> Expecting reviewer to dig previous discussions will not help your case. >> It helps reviewers if you provide necessary information, like resolution >> of previous discussion in the changelog. >> >> I dig the previous discussion, since you did not mention it here, and it >> seems you entirely ignored its outcome. That's not a DT property. >> >> NAK, sorry. Please go back to v1 and read the comments you got there. > > My apologies for omitting the changelog. > > And I don't blame you for missing the thread's resolution, > since I made a bit of a mess of it with my various messages. > > The firmware-5.bin approach was deemed DOA since these files > are parsed too late with respect to the required work-around. > Thus, we went back to either DT or a to-be-written system used > in the vendor driver.
Then please mention it briefly in the commit msg. Explain there why such implementation is the correct way to solve your problem. Best regards, Krzysztof