[OpenWrt-Devel] [PATCH] ath79: add new ar934x spi driver
Piotr Dymacz
pepe2k at gmail.com
Thu Feb 6 11:40:42 EST 2020
Hi Chuanhong,
On 06.02.2020 17:01, Chuanhong Guo wrote:
> Hi!
>
> On Thu, Feb 6, 2020 at 11:29 PM Piotr Dymacz <pepe2k at gmail.com> wrote:
>> Don't you think that ~24h for review this kind of patch wasn't enough?
>
> I've sent the patch upstream and got positive review there.[0] two
> minor comments there are already addressed in my final commit.
That was quick, great!
> And I've done flash reading and sysupgrade tests on
> ar933x/ar934x/qca955x including cs1 test on ar9344.
I remember having problems with all 3 CS in use on AR933x but can't find
now what exactly it was (IIRC, could be related with too small TSHSL_CNT
on higher clocks). But as CS1 and CS2 are muxed there with UART and it's
(almost) dead hardware I doubt we would ever see device with such
configuration.
> But I agree that I'm too impatient this time. Sorry.
No problem, maybe I'm just too slow ;) I remember some initial work on
that done by you in GH PR but... not every dev looks around there.
--
Cheers,
Piotr
>
> Regards,
> Chuanhong Guo
>
> [0] https://patchwork.kernel.org/patch/11367973/
>
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel
More information about the openwrt-devel
mailing list