Re: [PATCH v5] wlcore: spi: add wl18xx support

From: Kalle Valo
Date: Mon Jul 18 2016 - 13:04:36 EST


"Reizer, Eyal" <eyalr@xxxxxx> writes:

> Add support for using with both wl12xx and wl18xx.
>
> - all wilink family needs special init command for entering wspi mode.
> extra clock cycles should be sent after the spi init command while the
> cs pin is high.
> - Use inverted chip select for sending a dummy 4 bytes command that
> completes the init stage.
>
> Signed-off-by: Eyal Reizer <eyalr@xxxxxx>
> Acked-by: Rob Herring <robh@xxxxxxxxxx>
> ---
> v1->v2:update device tree bindings configuration
> v2->v3:revert from manual gpio manipulation. use inverted chip select
> instead for sending the extra init cycle which, achieves the same hardware
> purpose.
> update device tree bindings docucmentation accordingly
> v3->v4: Remove redundant data form binding documentation and fix chip
> select number mismatch in wl1271 example
> v4->v5: Rebase on top of head of wireless-drivers-next

You now sent me two patches with v5 which is just confusing. To make the
life of maintainers easier ALWAYS increase the revision number, no
matter how small the change is.

But the "latter" v5 is corrupted somehow in patchwork:

https://patchwork.kernel.org/patch/9233597/

The "former" v5 again looks ok:

https://patchwork.kernel.org/patch/9222403/

I can't immeaditely see why that happened, but please resend.

--
Kalle Valo