On Mon, 25 Mar 2019 at 16:55, Ben Greear <greearb@xxxxxxxxxxxxxxx> wrote: > On 3/25/19 5:14 AM, Michał Kazior wrote: > > On Sat, 23 Mar 2019 at 08:20, Arend Van Spriel > > <arend.vanspriel@xxxxxxxxxxxx> wrote: > >> > >> * resending with corrected email address from Kalle > >> -------------------------------------------------------------------- > >> + Michał > > > > Thanks! > > > > > >> On 3/22/2019 8:25 PM, Christian Lamparter wrote: > >> > On Friday, March 22, 2019 7:58:40 PM CET Tomislav Požega wrote: > >> >> When chip reset is done before the chip is checked if supported > >> >> there will be crash. Previous behaviour caused bootloops on > >> >> Archer C7 v1 units, this patch allows clean device boot without > >> >> excluding ath10k driver. > > > > Can you elaborate more a bit? What kind of crashes are you seeing? > > What does the bootloop look like? Do you have uart connected to > > diagnose? > > > > Didn't C7 v1 have the old QCA9880 hw v1 which isn't really supported > > by ath10k? I recall the v1 chip was really buggy and required > > hammering registers sometimes to get things working. > > The crash is related to the v1 chip. Is there a good way to detect > that this is the chip in question and only apply this work-around > for the problem chip? I don't know of any good way to do that. You could consider device-tree but that would be no different from having a module blacklist in the C7v1 build recipe, or to not build the module at all. That is unless you actually want to make v1 chip work with ath10k at which point there's more fun to be had before it can actually work. Michał
