mbox series

[0/3] ath10k: sdio: essential sdio fixes

Message ID 20180620084222.3521-1-niklas.cassel@linaro.org
Headers show
Series ath10k: sdio: essential sdio fixes | expand

Message

Niklas Cassel June 20, 2018, 8:42 a.m. UTC 
These fixes were originally embedded in a patch
"ath10k_sdio: virtual scatter gather for receive"
posted by Alagu Sankar <alagusankar@silex-india.com>
https://patchwork.kernel.org/patch/9979579/

However, while that patch implements scatter gather
(without using the proper sg types), it also
includes several other random fixes, including
but not limited to checkpatch.pl fixes.

This patch series is an attempt to extract
the most essential fixes included in that commit.

Each fix is now an individual commit, as upstream
requires that each logical change is self contained.

All of these fixes are essential to get working
sdio support for ath10k.

Other patches are needed to get working sdio support,
including the high-latency patch series from Erik Stromdahl.


Niklas Cassel (3):
  ath10k: sdio: use same endpoint id for all packets in a bundle
  ath10k: sdio: allocate correct size for RECV_1MORE_BLOCK rx packets
  ath10k: sdio: set skb len for all rx packets

 drivers/net/wireless/ath/ath10k/htc.h  | 1 +
 drivers/net/wireless/ath/ath10k/sdio.c | 9 ++++++++-
 2 files changed, 9 insertions(+), 1 deletion(-)

-- 
2.17.1

Comments

Kalle Valo June 29, 2018, 11:49 a.m. UTC | #1 
Niklas Cassel <niklas.cassel@linaro.org> writes:

> All packets in a bundle should use the same endpoint id as the

> first lookahead.

>

> This matches how things are done is ath6kl, however,

> this patch can theoretically handle several bundles

> in ath10k_sdio_mbox_rx_process_packets().

>

> Without this patch we get lots of errors about invalid endpoint id:

>

> ath10k_sdio mmc2:0001:1: invalid endpoint in look-ahead: 224

> ath10k_sdio mmc2:0001:1: failed to get pending recv messages: -12

> ath10k_sdio mmc2:0001:1: failed to process pending SDIO interrupts: -12

>

> Signed-off-by: Alagu Sankar <alagusankar@silex-india.com>

> Signed-off-by: Niklas Cassel <niklas.cassel@linaro.org>


You have Alagu's s-o-b first so that implies Alagu is the author. So
should I add From header for Alagu and add you (Niklas) as
Co-Developed-by? Or vice versa?

-- 
Kalle Valo
Kalle Valo June 29, 2018, 11:53 a.m. UTC | #2 
Kalle Valo <kvalo@codeaurora.org> writes:

> Niklas Cassel <niklas.cassel@linaro.org> writes:

>

>> All packets in a bundle should use the same endpoint id as the

>> first lookahead.

>>

>> This matches how things are done is ath6kl, however,

>> this patch can theoretically handle several bundles

>> in ath10k_sdio_mbox_rx_process_packets().

>>

>> Without this patch we get lots of errors about invalid endpoint id:

>>

>> ath10k_sdio mmc2:0001:1: invalid endpoint in look-ahead: 224

>> ath10k_sdio mmc2:0001:1: failed to get pending recv messages: -12

>> ath10k_sdio mmc2:0001:1: failed to process pending SDIO interrupts: -12

>>

>> Signed-off-by: Alagu Sankar <alagusankar@silex-india.com>

>> Signed-off-by: Niklas Cassel <niklas.cassel@linaro.org>

>

> You have Alagu's s-o-b first so that implies Alagu is the author. So

> should I add From header for Alagu and add you (Niklas) as

> Co-Developed-by? Or vice versa?


Ah, the same issue is with all three patches. So whatever we decide,
I'll do the same changes on all three.

-- 
Kalle Valo
Niklas Cassel June 29, 2018, 1:11 p.m. UTC | #3 
On Fri, Jun 29, 2018 at 02:53:44PM +0300, Kalle Valo wrote:
> Kalle Valo <kvalo@codeaurora.org> writes:

> 

> > Niklas Cassel <niklas.cassel@linaro.org> writes:

> >

> >> All packets in a bundle should use the same endpoint id as the

> >> first lookahead.

> >>

> >> This matches how things are done is ath6kl, however,

> >> this patch can theoretically handle several bundles

> >> in ath10k_sdio_mbox_rx_process_packets().

> >>

> >> Without this patch we get lots of errors about invalid endpoint id:

> >>

> >> ath10k_sdio mmc2:0001:1: invalid endpoint in look-ahead: 224

> >> ath10k_sdio mmc2:0001:1: failed to get pending recv messages: -12

> >> ath10k_sdio mmc2:0001:1: failed to process pending SDIO interrupts: -12

> >>

> >> Signed-off-by: Alagu Sankar <alagusankar@silex-india.com>

> >> Signed-off-by: Niklas Cassel <niklas.cassel@linaro.org>

> >

> > You have Alagu's s-o-b first so that implies Alagu is the author. So

> > should I add From header for Alagu and add you (Niklas) as

> > Co-Developed-by? Or vice versa?


Hello Kalle,

It is not always obvious how the combination of git-author
and Co-Developed-by should be:
http://lkml.iu.edu/hypermail/linux/kernel/1801.2/00988.html

Alagu deserves most credit, since I have simply taken parts
of a very big patch and split it into smaller pieces.
I tried to do the right thing by having his Signed-off-by first.

Let's go with your suggestion and add a From: header with
Alagu's email, and a Co-Developed-by tag with my email.
(Note that both Signed-off-bys are still needed according to
submitting-patches.rst)

Do you want me to resend the patches with these two lines
added, or can you fix them up manually?

Regards,
Niklas

> 

> Ah, the same issue is with all three patches. So whatever we decide,

> I'll do the same changes on all three.

> 

> -- 

> Kalle Valo
Kalle Valo June 29, 2018, 1:51 p.m. UTC | #4 
Niklas Cassel <niklas.cassel@linaro.org> writes:

> On Fri, Jun 29, 2018 at 02:53:44PM +0300, Kalle Valo wrote:

>> Kalle Valo <kvalo@codeaurora.org> writes:

>> 

>> > Niklas Cassel <niklas.cassel@linaro.org> writes:

>> >

>> >> All packets in a bundle should use the same endpoint id as the

>> >> first lookahead.

>> >>

>> >> This matches how things are done is ath6kl, however,

>> >> this patch can theoretically handle several bundles

>> >> in ath10k_sdio_mbox_rx_process_packets().

>> >>

>> >> Without this patch we get lots of errors about invalid endpoint id:

>> >>

>> >> ath10k_sdio mmc2:0001:1: invalid endpoint in look-ahead: 224

>> >> ath10k_sdio mmc2:0001:1: failed to get pending recv messages: -12

>> >> ath10k_sdio mmc2:0001:1: failed to process pending SDIO interrupts: -12

>> >>

>> >> Signed-off-by: Alagu Sankar <alagusankar@silex-india.com>

>> >> Signed-off-by: Niklas Cassel <niklas.cassel@linaro.org>

>> >

>> > You have Alagu's s-o-b first so that implies Alagu is the author. So

>> > should I add From header for Alagu and add you (Niklas) as

>> > Co-Developed-by? Or vice versa?

>

> Hello Kalle,

>

> It is not always obvious how the combination of git-author

> and Co-Developed-by should be:

> http://lkml.iu.edu/hypermail/linux/kernel/1801.2/00988.html


Yeah, that's true.

> Alagu deserves most credit, since I have simply taken parts

> of a very big patch and split it into smaller pieces.

> I tried to do the right thing by having his Signed-off-by first.

>

> Let's go with your suggestion and add a From: header with

> Alagu's email, and a Co-Developed-by tag with my email.

> (Note that both Signed-off-bys are still needed according to

> submitting-patches.rst)


Ok.

> Do you want me to resend the patches with these two lines

> added, or can you fix them up manually?


No need, I did that already in the pending branch. Please check:

https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=6b23bd067990df1cc1e9a4c28327393a7a3ba718

https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=bf83a5c572e0737e59e026c989b190b4e52d2ef4

https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=218f924d80b76520a4e60eead5e89f3ebd2e86c1

-- 
Kalle Valo
Niklas Cassel June 29, 2018, 1:56 p.m. UTC | #5 
On Fri, Jun 29, 2018 at 04:51:51PM +0300, Kalle Valo wrote:
> Niklas Cassel <niklas.cassel@linaro.org> writes:

> 

> > On Fri, Jun 29, 2018 at 02:53:44PM +0300, Kalle Valo wrote:

> >> Kalle Valo <kvalo@codeaurora.org> writes:

> >> 

> >> > Niklas Cassel <niklas.cassel@linaro.org> writes:

> >> >

> >> >> All packets in a bundle should use the same endpoint id as the

> >> >> first lookahead.

> >> >>

> >> >> This matches how things are done is ath6kl, however,

> >> >> this patch can theoretically handle several bundles

> >> >> in ath10k_sdio_mbox_rx_process_packets().

> >> >>

> >> >> Without this patch we get lots of errors about invalid endpoint id:

> >> >>

> >> >> ath10k_sdio mmc2:0001:1: invalid endpoint in look-ahead: 224

> >> >> ath10k_sdio mmc2:0001:1: failed to get pending recv messages: -12

> >> >> ath10k_sdio mmc2:0001:1: failed to process pending SDIO interrupts: -12

> >> >>

> >> >> Signed-off-by: Alagu Sankar <alagusankar@silex-india.com>

> >> >> Signed-off-by: Niklas Cassel <niklas.cassel@linaro.org>

> >> >

> >> > You have Alagu's s-o-b first so that implies Alagu is the author. So

> >> > should I add From header for Alagu and add you (Niklas) as

> >> > Co-Developed-by? Or vice versa?

> >

> > Hello Kalle,

> >

> > It is not always obvious how the combination of git-author

> > and Co-Developed-by should be:

> > http://lkml.iu.edu/hypermail/linux/kernel/1801.2/00988.html

> 

> Yeah, that's true.

> 

> > Alagu deserves most credit, since I have simply taken parts

> > of a very big patch and split it into smaller pieces.

> > I tried to do the right thing by having his Signed-off-by first.

> >

> > Let's go with your suggestion and add a From: header with

> > Alagu's email, and a Co-Developed-by tag with my email.

> > (Note that both Signed-off-bys are still needed according to

> > submitting-patches.rst)

> 

> Ok.

> 

> > Do you want me to resend the patches with these two lines

> > added, or can you fix them up manually?

> 

> No need, I did that already in the pending branch. Please check:

> 

> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=6b23bd067990df1cc1e9a4c28327393a7a3ba718

> 

> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=bf83a5c572e0737e59e026c989b190b4e52d2ef4

> 

> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=218f924d80b76520a4e60eead5e89f3ebd2e86c1


Hello Kalle

It looks good :)

Regards,
Niklas

> 

> -- 

> Kalle Valo