[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <93490647-A0F1-41A0-8B3E-41A0C915B372@goldelico.com>
Date: Thu, 15 Jun 2023 09:00:04 +0200
From: "H. Nikolaus Schaller" <hns@...delico.com>
To: Paul Cercueil <paul@...pouillou.net>
Cc: Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
Rob Herring <robh+dt@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Conor Dooley <conor+dt@...nel.org>, linux-mips@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
list@...ndingux.net
Subject: Re: [PATCH 0/9] MIPS: CI20: Add WiFi / Bluetooth support
Hi Paul,
I was in holidays and could not review this series earlier.
> Am 04.06.2023 um 16:56 schrieb Paul Cercueil <paul@...pouillou.net>:
>
> Hi,
>
> Here's a set of patches to add support for the WiFi / Bluetooth chip on
> the CI20.
>
> WiFi works pretty well, provided it is used with the latest firmware
> provided by linux-firmware. Bluetooth does not work very well here, as
> I cannot get my wireless keyboard to pair; but it does detect it, and it
> does see they key presses when I type the pairing code.
>
> I only tested with a somewhat recent (~2022) Buildroot-based userspace.
> I enabled WEXT compatibility because the CI20 is typically used with a
> very old userspace, but I did not try to use it with old tools like
> ifconfig/iwconfig.
^^^ great since not everyone is using memory hungry latest user-space and
ifconfig/iwconfig is also available on some other OS so users like me
can share scripts.
But I had quite some issues with this series.
1. I could not boot my CI20 V2a board after applying the full series to v6.4-rc6
2. bisecting failed because vcc_33v is used in a patch preceding its definition
leading to DTC compile abort
3. after fixing I could bisect that "MIPS: DTS: CI20: Fix ACT8600 regulator node names"
is the first bad commit - although I don't see immediately why
So this series seems to be severely broken and I could not even come to
a test of WiFi and/or Bluetooth which the series claims to support.
Comments to some individual patches follow.
Best regards and looking forward to a v2 for testing,
Nikolaus
>
> Cheers,
> -Paul
>
> Paul Cercueil (9):
> MIPS: DTS: CI20: Fix regulators
> MIPS: DTS: CI20: Fix ACT8600 regulator node names
> MIPS: DTS: CI20: Add parent supplies to ACT8600 regulators
^^^ should IMHO be a separate series since it is not directly related to WiFi / Bluetooth
> MIPS: DTS: CI20: Do not force-enable CIM and WiFi regulators
> MIPS: DTS: CI20: Misc. cleanups
^^^ these two do not compile
The Misc. cleanups do not belong to this topic.
> MIPS: DTS: CI20: Parent MSCMUX clock to MPLL
^^^ this is only loosely related to Wifi / Bluetooth
> MIPS: DTS: CI20: Enable support for WiFi / Bluetooth
> MIPS: configs: CI20: Regenerate defconfig
> MIPS: configs: CI20: Enable WiFi / Bluetooth
>
> arch/mips/boot/dts/ingenic/ci20.dts | 148 +++++++++++++++++++---------
> arch/mips/configs/ci20_defconfig | 47 ++++++---
> 2 files changed, 133 insertions(+), 62 deletions(-)
>
> --
> 2.39.2
>
Powered by blists - more mailing lists