linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] wireless/atmel: remove time_t usage
@ 2017-02-23 16:14 Alexandre Belloni
  2017-03-08 15:13 ` Kalle Valo
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: Alexandre Belloni @ 2017-02-23 16:14 UTC (permalink / raw)
  To: linux-arm-kernel

last_qual never really holds a time. It only holds jiffies. Make it the
same type as jiffies.

Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
 drivers/net/wireless/atmel/atmel.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/atmel/atmel.c b/drivers/net/wireless/atmel/atmel.c
index e12f62356fd1..27b110dc8cc6 100644
--- a/drivers/net/wireless/atmel/atmel.c
+++ b/drivers/net/wireless/atmel/atmel.c
@@ -513,7 +513,7 @@ struct atmel_private {
 	} station_state;
 
 	int operating_mode, power_mode;
-	time_t last_qual;
+	unsigned long last_qual;
 	int beacons_this_sec;
 	int channel;
 	int reg_domain, config_reg_domain;
-- 
2.11.0

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* wireless/atmel: remove time_t usage
  2017-02-23 16:14 [PATCH] wireless/atmel: remove time_t usage Alexandre Belloni
@ 2017-03-08 15:13 ` Kalle Valo
       [not found] ` <20170308151340.9D6E360209@smtp.codeaurora.org>
  2017-03-20 16:43 ` atmel: " Kalle Valo
  2 siblings, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2017-03-08 15:13 UTC (permalink / raw)
  To: linux-arm-kernel

Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> last_qual never really holds a time. It only holds jiffies. Make it the
> same type as jiffies.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

The title should start with "atmel:", I can fix that during commit.

-- 
https://patchwork.kernel.org/patch/9588521/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 5+ messages in thread

* wireless/atmel: remove time_t usage
       [not found] ` <20170308151340.9D6E360209@smtp.codeaurora.org>
@ 2017-03-08 16:10   ` Alexandre Belloni
  2017-03-08 18:53     ` Kalle Valo
  0 siblings, 1 reply; 5+ messages in thread
From: Alexandre Belloni @ 2017-03-08 16:10 UTC (permalink / raw)
  To: linux-arm-kernel

Hi,

On 08/03/2017 at 15:13:40 +0000, Kalle Valo wrote:
> Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> > last_qual never really holds a time. It only holds jiffies. Make it the
> > same type as jiffies.
> > 
> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> 
> The title should start with "atmel:", I can fix that during commit.
> 

If you prefer, I can send a v2

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

^ permalink raw reply	[flat|nested] 5+ messages in thread

* wireless/atmel: remove time_t usage
  2017-03-08 16:10   ` Alexandre Belloni
@ 2017-03-08 18:53     ` Kalle Valo
  0 siblings, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2017-03-08 18:53 UTC (permalink / raw)
  To: linux-arm-kernel

Alexandre Belloni <alexandre.belloni@free-electrons.com> writes:

> Hi,
>
> On 08/03/2017 at 15:13:40 +0000, Kalle Valo wrote:
>> Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
>> > last_qual never really holds a time. It only holds jiffies. Make it the
>> > same type as jiffies.
>> > 
>> > Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>> 
>> The title should start with "atmel:", I can fix that during commit.
>> 
>
> If you prefer, I can send a v2

Thanks, but no need. I can easily edit the commit log when I commit it.

-- 
Kalle Valo

^ permalink raw reply	[flat|nested] 5+ messages in thread

* atmel: remove time_t usage
  2017-02-23 16:14 [PATCH] wireless/atmel: remove time_t usage Alexandre Belloni
  2017-03-08 15:13 ` Kalle Valo
       [not found] ` <20170308151340.9D6E360209@smtp.codeaurora.org>
@ 2017-03-20 16:43 ` Kalle Valo
  2 siblings, 0 replies; 5+ messages in thread
From: Kalle Valo @ 2017-03-20 16:43 UTC (permalink / raw)
  To: linux-arm-kernel

Alexandre Belloni <alexandre.belloni@free-electrons.com> wrote:
> last_qual never really holds a time. It only holds jiffies. Make it the
> same type as jiffies.
> 
> Signed-off-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

Patch applied to wireless-drivers-next.git, thanks.

5c5105666de5 atmel: remove time_t usage

-- 
https://patchwork.kernel.org/patch/9588521/

Documentation about submitting wireless patches and checking status
from patchwork:

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2017-03-20 16:43 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-02-23 16:14 [PATCH] wireless/atmel: remove time_t usage Alexandre Belloni
2017-03-08 15:13 ` Kalle Valo
     [not found] ` <20170308151340.9D6E360209@smtp.codeaurora.org>
2017-03-08 16:10   ` Alexandre Belloni
2017-03-08 18:53     ` Kalle Valo
2017-03-20 16:43 ` atmel: " Kalle Valo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).