Ben Greear writes:
> On 02/07/2017 01:14 AM, Valo, Kalle wrote:
>> Adrian Chadd writes:
>>
>>> Removing this method makes the diff to FreeBSD larger, as "vif" in
>>> FreeBSD is a different pointer.
>>>
>>> (Yes, I have ath10k on free
Adrian Chadd writes:
> Removing this method makes the diff to FreeBSD larger, as "vif" in
> FreeBSD is a different pointer.
>
> (Yes, I have ath10k on freebsd working and I'd like to find a way to
> reduce the diff moving forward.)
I don't like this "(void *) vif->drv_priv" style that much eithe
Joe Perches writes:
> On Tue, 2017-01-24 at 05:18 +0000, Valo, Kalle wrote:
>> Joe Perches writes:
>>
>> > On Mon, 2017-01-23 at 15:04 +, Srinivas Kandagatla wrote:
>> > > use dma_zalloc_coherent() instead of dma_alloc_coherent and memset().
>>
Joe Perches writes:
> On Mon, 2017-01-23 at 15:04 +, Srinivas Kandagatla wrote:
>> use dma_zalloc_coherent() instead of dma_alloc_coherent and memset().
> []
>> diff --git a/drivers/net/wireless/ath/ath10k/pci.c
>> b/drivers/net/wireless/ath/ath10k/pci.c
> []
>> @@ -896,7 +896,7 @@ static in
Gabriel C writes:
> On 18.12.2016 21:14, Paul E. McKenney wrote:
>> On Sun, Dec 18, 2016 at 07:57:42PM +0000, Valo, Kalle wrote:
>>> Tobias Klausmann writes:
>>>
>>>> A patch for this is already floating on the ML for a while now latest:
>>>&g
Tobias Klausmann writes:
> A patch for this is already floating on the ML for a while now latest:
> (ath9k: do not return early to fix rcu unlocking)
It's here:
https://patchwork.kernel.org/patch/9472709/
> Hopefully Kalle will include it in one of his upcoming pull requests.
Yes, I'll try to
Hi Dave,
Andy Gross writes:
> On 1 December 2016 at 04:17, Valo, Kalle wrote:
>> Kalle Valo writes:
>>
>>> It found the same problem. Interestingly I'm also building x86 with 32
>>> bit, maybe it's related?
>>>
>>> tree:
Kalle Valo writes:
> Kalle Valo writes:
>
>> "Valo, Kalle" writes:
>>
>>> Bjorn Andersson writes:
>>>
>>>> On Wed 16 Nov 10:49 PST 2016, Kalle Valo wrote:
>>>>
>>>>> Bjorn Andersson wrote:
>>>
Kalle Valo writes:
> "Valo, Kalle" writes:
>
>> Bjorn Andersson writes:
>>
>>> On Wed 16 Nov 10:49 PST 2016, Kalle Valo wrote:
>>>
>>>> Bjorn Andersson wrote:
>>>> > The correct include file for getting errno constants a
"Valo, Kalle" writes:
> Bjorn Andersson writes:
>
>> On Wed 16 Nov 10:49 PST 2016, Kalle Valo wrote:
>>
>>> Bjorn Andersson wrote:
>>> > The correct include file for getting errno constants and ERR_PTR() is
>>> > linux/err.h, rath
Bjorn Andersson writes:
> On Wed 16 Nov 10:49 PST 2016, Kalle Valo wrote:
>
>> Bjorn Andersson wrote:
>> > The correct include file for getting errno constants and ERR_PTR() is
>> > linux/err.h, rather than linux/errno.h, so fix the include.
>> >
>> > Fixes: e8b123e60084 ("soc: qcom: smem_state
Joe Perches writes:
> Correct some trivial comment typos.
> Remove unnecessary parentheses in a long line.
> Convert a return; before the end of a void function definition to just ;
>
> Signed-off-by: Joe Perches
[...]
> --- a/drivers/net/wireless/ath/ath10k/core.c
> +++ b/drivers/net/wireless
Colin King writes:
> From: Colin Ian King
>
> caldata is not being free'd on the error exit path, causing
> a memory leak. kfree it to fix the leak.
>
> Signed-off-by: Colin Ian King
> ---
> drivers/net/wireless/ath/ath10k/pci.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/
Sergei Shtylyov writes:
> On 6/21/2016 2:45 PM, Chaehyun Lim wrote:
>
>> is not used anymore, so just remove it.
>
>s/it/#include/?
Fixed that in the pending branch.
--
Kalle Valo
(Adding ath6k list)
Chaehyun Lim writes:
> It replaces struct semaphore sem with struct mutex mutex
>
> Reported-by: kbuild test robot
> Signed-off-by: Chaehyun Lim
> ---
> V2: fix build failure reported by kbuild test robot
I don't think Reported-by is correct, it should be only used when
re
David Binderman writes:
> Hello there,
>
> linux-4.7-rc2/drivers/net/wireless/ath/ath6kl/wmi.c:2547]: (style)
> Redundant condition: If 'EXPR <= 7', the comparison 'EXPR < 8' is
> always true.
>
> Source code is
>
> if (!((params->user_pri < 8) &&
> (params->user_pri <= 0x7) &&
>
>
Anton Protopopov writes:
> The ath10k_pci_hif_exchange_bmi_msg() function may return the positive
> value EIO instead of -EIO in case of error.
>
> Signed-off-by: Anton Protopopov
Applied, thanks.
--
Kalle Valo
Sudip Mukherjee writes:
> From: Sudip Mukherjee
>
> len has been initialized with a value of 0 and buf_len with 4096. There
> is no way that this condition (len > buf_len) can be true now.
>
> Signed-off-by: Sudip Mukherjee
Applied, thanks.
--
Kalle Valo
Arnd Bergmann writes:
> In some configurations, this function uses more than the warning limit
> of 1024 bytes:
>
> drivers/net/wireless/ath/ath9k/ar9003_aic.c: In function
> 'ar9003_aic_cal_post_process':
> drivers/net/wireless/ath/ath9k/ar9003_aic.c:434:1: error: the frame size of
> 1040 byte
19 matches
Mail list logo
