On Mon, 24 Jun 2024 at 23:47, Bartosz Golaszewski <brgl@xxxxxxxx> wrote:
>
> On Mon, Jun 24, 2024 at 10:17 PM Dmitry Baryshkov
> <dmitry.baryshkov@xxxxxxxxxx> wrote:
> >
> > On Mon, 24 Jun 2024 at 22:45, Bartosz Golaszewski <brgl@xxxxxxxx> wrote:
> > >
> > > From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
> > >
> > > We unnecessarily fallthrough the case for QCA6390 when initializing the
> > > device and hit the condition where - due to the lack of the enable-gpio
> > > - we disable power management despite using the power sequencer. We don't
> > > need to look for clocks on this model so it makes more sense to just
> > > register the hci device and break the switch.
> > >
> > > Reported-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
> > > Fixes: 4029dba6b6f1 ("Bluetooth: qca: use the power sequencer for QCA6390")
> > > Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
> >
> > Is this going to break the QCA6390 as present on M.2 / PCIe cards? Or
> > the older DT which didn't have pwrseq entries?
> >
>
> Neither of these has clocks that need to be driven by linux. The only
> user of QCA6390 Bluetooth in mainline is RB5. Bindings didn't exist
> before so no commitment was ever made.
This might make some laptop users unhappy. But anyway, restarting the
hci0 now gives:
[ 24.387344] Bluetooth: hci0: setting up ROME/QCA6390
[ 24.387439] qcom_geni_serial 998000.serial: serial engine reports 0
RX bytes in!
[ 24.554349] qcom_geni_serial 998000.serial: serial engine reports 0
RX bytes in!
[ 24.562056] arm-smmu 15000000.iommu: Unhandled context fault:
fsr=0x402, iova=0xfffd1080, fsynr=0x750013, cbfrsynra=0x5a3, cb=3
[ 26.914225] Bluetooth: hci0: command 0xfc00 tx timeout
[ 35.042619] Bluetooth: hci0: Reading QCA version information failed (-110)
[ 35.049721] Bluetooth: hci0: Retry BT power ON:0
[ 37.539492] Bluetooth: hci0: command 0xfc00 tx timeout
[ 45.539519] Bluetooth: hci0: Reading QCA version information failed (-110)
[ 45.546667] Bluetooth: hci0: Retry BT power ON:1
[ 48.035863] Bluetooth: hci0: command 0xfc00 tx timeout
[ 56.034783] Bluetooth: hci0: Reading QCA version information failed (-110)
[ 56.041901] Bluetooth: hci0: Retry BT power ON:2
[ 58.532174] Bluetooth: hci0: command 0xfc00 tx timeout
[ 66.531928] Bluetooth: hci0: Reading QCA version information failed (-110)
--
With best wishes
Dmitry
