* [PATCH] mwifiex: card reset: enable rescan of non-removable card
@ 2014-07-18 8:00 Andreas Fenkart
2014-07-18 8:12 ` James Cameron
2014-07-18 17:59 ` John W. Linville
0 siblings, 2 replies; 8+ messages in thread
From: Andreas Fenkart @ 2014-07-18 8:00 UTC (permalink / raw)
To: linux-wireless; +Cc: bzhao, daniel, Andreas Fenkart
mmc_rescan will scan for non-removable cards only once, hence the card
will not be rediscovered.
Signed-off-by: Andreas Fenkart <afenkart@gmail.com>
---
drivers/net/wireless/mwifiex/sdio.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/wireless/mwifiex/sdio.c b/drivers/net/wireless/mwifiex/sdio.c
index 4ce3d7b..eff3d6e 100644
--- a/drivers/net/wireless/mwifiex/sdio.c
+++ b/drivers/net/wireless/mwifiex/sdio.c
@@ -1931,6 +1931,7 @@ static void sdio_card_reset_worker(struct work_struct *work)
mmc_remove_host(target);
/* 20ms delay is based on experiment with sdhci controller */
mdelay(20);
+ reset_host->rescan_entered = 0; /* rescan non-removable cards */
mmc_add_host(target);
}
static DECLARE_WORK(card_reset_work, sdio_card_reset_worker);
--
2.0.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] mwifiex: card reset: enable rescan of non-removable card
2014-07-18 8:00 [PATCH] mwifiex: card reset: enable rescan of non-removable card Andreas Fenkart
@ 2014-07-18 8:12 ` James Cameron
2014-07-18 17:59 ` John W. Linville
1 sibling, 0 replies; 8+ messages in thread
From: James Cameron @ 2014-07-18 8:12 UTC (permalink / raw)
To: Andreas Fenkart; +Cc: linux-wireless, bzhao, daniel
On Fri, Jul 18, 2014 at 10:00:37AM +0200, Andreas Fenkart wrote:
> mmc_rescan will scan for non-removable cards only once, hence the card
> will not be rediscovered.
>
> Signed-off-by: Andreas Fenkart <afenkart@gmail.com>
> ---
> drivers/net/wireless/mwifiex/sdio.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/wireless/mwifiex/sdio.c b/drivers/net/wireless/mwifiex/sdio.c
> index 4ce3d7b..eff3d6e 100644
> --- a/drivers/net/wireless/mwifiex/sdio.c
> +++ b/drivers/net/wireless/mwifiex/sdio.c
> @@ -1931,6 +1931,7 @@ static void sdio_card_reset_worker(struct work_struct *work)
> mmc_remove_host(target);
> /* 20ms delay is based on experiment with sdhci controller */
> mdelay(20);
> + reset_host->rescan_entered = 0; /* rescan non-removable cards */
> mmc_add_host(target);
> }
> static DECLARE_WORK(card_reset_work, sdio_card_reset_worker);
Thanks! Maybe that was my problem.
--
James Cameron
http://quozl.linux.org.au/
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] mwifiex: card reset: enable rescan of non-removable card
2014-07-18 8:00 [PATCH] mwifiex: card reset: enable rescan of non-removable card Andreas Fenkart
2014-07-18 8:12 ` James Cameron
@ 2014-07-18 17:59 ` John W. Linville
2014-07-18 19:07 ` Bing Zhao
1 sibling, 1 reply; 8+ messages in thread
From: John W. Linville @ 2014-07-18 17:59 UTC (permalink / raw)
To: Andreas Fenkart; +Cc: linux-wireless, bzhao, daniel
On Fri, Jul 18, 2014 at 10:00:37AM +0200, Andreas Fenkart wrote:
> mmc_rescan will scan for non-removable cards only once, hence the card
> will not be rediscovered.
>
> Signed-off-by: Andreas Fenkart <afenkart@gmail.com>
> ---
> drivers/net/wireless/mwifiex/sdio.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/net/wireless/mwifiex/sdio.c b/drivers/net/wireless/mwifiex/sdio.c
> index 4ce3d7b..eff3d6e 100644
> --- a/drivers/net/wireless/mwifiex/sdio.c
> +++ b/drivers/net/wireless/mwifiex/sdio.c
> @@ -1931,6 +1931,7 @@ static void sdio_card_reset_worker(struct work_struct *work)
> mmc_remove_host(target);
> /* 20ms delay is based on experiment with sdhci controller */
> mdelay(20);
> + reset_host->rescan_entered = 0; /* rescan non-removable cards */
> mmc_add_host(target);
> }
> static DECLARE_WORK(card_reset_work, sdio_card_reset_worker);
Building wireless-next:
CC drivers/net/wireless/mwifiex/sdio.o
drivers/net/wireless/mwifiex/sdio.c: In function ‘mwifiex_sdio_card_reset_work’:
drivers/net/wireless/mwifiex/sdio.c:1957:2: error: ‘reset_host’ undeclared (first use in this function)
reset_host->rescan_entered = 0; /* rescan non-removable cards */
^
drivers/net/wireless/mwifiex/sdio.c:1957:2: note: each undeclared identifier is reported only once for each function it appears in
--
John W. Linville Someday the world will need a hero, and you
linville@tuxdriver.com might be all we have. Be ready.
^ permalink raw reply [flat|nested] 8+ messages in thread
* RE: [PATCH] mwifiex: card reset: enable rescan of non-removable card
2014-07-18 17:59 ` John W. Linville
@ 2014-07-18 19:07 ` Bing Zhao
2014-07-21 7:56 ` Andreas Fenkart
0 siblings, 1 reply; 8+ messages in thread
From: Bing Zhao @ 2014-07-18 19:07 UTC (permalink / raw)
To: John W. Linville, Andreas Fenkart
Cc: linux-wireless@vger.kernel.org, daniel@zonque.org
SGkgQW5kcmVhcywNCg0KPiA+IEBAIC0xOTMxLDYgKzE5MzEsNyBAQCBzdGF0aWMgdm9pZCBzZGlv
X2NhcmRfcmVzZXRfd29ya2VyKHN0cnVjdCB3b3JrX3N0cnVjdCAqd29yaykNCj4gPiAgCW1tY19y
ZW1vdmVfaG9zdCh0YXJnZXQpOw0KPiA+ICAJLyogMjBtcyBkZWxheSBpcyBiYXNlZCBvbiBleHBl
cmltZW50IHdpdGggc2RoY2kgY29udHJvbGxlciAqLw0KPiA+ICAJbWRlbGF5KDIwKTsNCj4gPiAr
CXJlc2V0X2hvc3QtPnJlc2Nhbl9lbnRlcmVkID0gMDsgLyogcmVzY2FuIG5vbi1yZW1vdmFibGUg
Y2FyZHMgKi8NCg0KWW91IG1lYW50IHRoaXM/DQoNCisJdGFyZ2V0LT5yZXNjYW5fZW50ZXJlZCA9
IDA7IC8qIHJlc2NhbiBub24tcmVtb3ZhYmxlIGNhcmRzICovDQoNClJlZ2FyZHMsDQpCaW5nDQoN
Cj4gPiAgCW1tY19hZGRfaG9zdCh0YXJnZXQpOw0KPiA+ICB9DQo+ID4gIHN0YXRpYyBERUNMQVJF
X1dPUksoY2FyZF9yZXNldF93b3JrLCBzZGlvX2NhcmRfcmVzZXRfd29ya2VyKTsNCj4gDQo+IEJ1
aWxkaW5nIHdpcmVsZXNzLW5leHQ6DQo+IA0KPiAgIENDICAgICAgZHJpdmVycy9uZXQvd2lyZWxl
c3MvbXdpZmlleC9zZGlvLm8NCj4gZHJpdmVycy9uZXQvd2lyZWxlc3MvbXdpZmlleC9zZGlvLmM6
IEluIGZ1bmN0aW9uIOKAmG13aWZpZXhfc2Rpb19jYXJkX3Jlc2V0X3dvcmvigJk6DQo+IGRyaXZl
cnMvbmV0L3dpcmVsZXNzL213aWZpZXgvc2Rpby5jOjE5NTc6MjogZXJyb3I6IOKAmHJlc2V0X2hv
c3TigJkgdW5kZWNsYXJlZCAoZmlyc3QgdXNlIGluIHRoaXMNCj4gZnVuY3Rpb24pDQo+ICAgcmVz
ZXRfaG9zdC0+cmVzY2FuX2VudGVyZWQgPSAwOyAvKiByZXNjYW4gbm9uLXJlbW92YWJsZSBjYXJk
cyAqLw0KPiAgIF4NCj4gZHJpdmVycy9uZXQvd2lyZWxlc3MvbXdpZmlleC9zZGlvLmM6MTk1Nzoy
OiBub3RlOiBlYWNoIHVuZGVjbGFyZWQgaWRlbnRpZmllciBpcyByZXBvcnRlZCBvbmx5IG9uY2UN
Cj4gZm9yIGVhY2ggZnVuY3Rpb24gaXQgYXBwZWFycyBpbg0KIA0KDQoNCg==
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] mwifiex: card reset: enable rescan of non-removable card
2014-07-18 19:07 ` Bing Zhao
@ 2014-07-21 7:56 ` Andreas Fenkart
2014-07-21 8:01 ` [PATCH v2 0/1] " Andreas Fenkart
0 siblings, 1 reply; 8+ messages in thread
From: Andreas Fenkart @ 2014-07-21 7:56 UTC (permalink / raw)
To: Bing Zhao
Cc: John W. Linville, linux-wireless@vger.kernel.org,
daniel@zonque.org
Hi
2014-07-18 21:07 GMT+02:00 Bing Zhao <bzhao@marvell.com>:
>> > @@ -1931,6 +1931,7 @@ static void sdio_card_reset_worker(struct work_struct *work)
>> > mmc_remove_host(target);
>> > /* 20ms delay is based on experiment with sdhci controller */
>> > mdelay(20);
>> > + reset_host->rescan_entered = 0; /* rescan non-removable cards */
I used the global the variable,
static struct mmc_host *reset_host;
>
> You meant this?
>
> + target->rescan_entered = 0; /* rescan non-removable cards */
of course, should be the local variable, sending out new patch.
v2
- replaced global variable by local 'target' variable
>
> Regards,
> Bing
>
>> > mmc_add_host(target);
>> > }
>> > static DECLARE_WORK(card_reset_work, sdio_card_reset_worker);
>>
>> Building wireless-next:
>>
>> CC drivers/net/wireless/mwifiex/sdio.o
>> drivers/net/wireless/mwifiex/sdio.c: In function ‘mwifiex_sdio_card_reset_work’:
>> drivers/net/wireless/mwifiex/sdio.c:1957:2: error: ‘reset_host’ undeclared (first use in this
>> function)
>> reset_host->rescan_entered = 0; /* rescan non-removable cards */
>> ^
>> drivers/net/wireless/mwifiex/sdio.c:1957:2: note: each undeclared identifier is reported only once
>> for each function it appears in
>
>
>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 0/1] mwifiex: card reset: enable rescan of non-removable card
2014-07-21 7:56 ` Andreas Fenkart
@ 2014-07-21 8:01 ` Andreas Fenkart
2014-07-21 8:01 ` [PATCH v2 1/1] " Andreas Fenkart
0 siblings, 1 reply; 8+ messages in thread
From: Andreas Fenkart @ 2014-07-21 8:01 UTC (permalink / raw)
To: linux-wireless; +Cc: bzhao, daniel, Andreas Fenkart
v2
- replaced global variable by local 'target' variable
Andreas Fenkart (1):
mwifiex: card reset: enable rescan of non-removable card
drivers/net/wireless/mwifiex/sdio.c | 1 +
1 file changed, 1 insertion(+)
--
2.0.1
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH v2 1/1] mwifiex: card reset: enable rescan of non-removable card
2014-07-21 8:01 ` [PATCH v2 0/1] " Andreas Fenkart
@ 2014-07-21 8:01 ` Andreas Fenkart
2014-07-22 3:30 ` Bing Zhao
0 siblings, 1 reply; 8+ messages in thread
From: Andreas Fenkart @ 2014-07-21 8:01 UTC (permalink / raw)
To: linux-wireless; +Cc: bzhao, daniel, Andreas Fenkart
mmc_rescan will scan for non-removable cards only once, hence the card
will not be rediscovered.
Signed-off-by: Andreas Fenkart <afenkart@gmail.com>
diff --git a/drivers/net/wireless/mwifiex/sdio.c b/drivers/net/wireless/mwifiex/sdio.c
index 4ce3d7b..ef22d59 100644
--- a/drivers/net/wireless/mwifiex/sdio.c
+++ b/drivers/net/wireless/mwifiex/sdio.c
@@ -1931,6 +1931,7 @@ static void sdio_card_reset_worker(struct work_struct *work)
mmc_remove_host(target);
/* 20ms delay is based on experiment with sdhci controller */
mdelay(20);
+ target->rescan_entered = 0; /* rescan non-removable cards */
mmc_add_host(target);
}
static DECLARE_WORK(card_reset_work, sdio_card_reset_worker);
--
2.0.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* RE: [PATCH v2 1/1] mwifiex: card reset: enable rescan of non-removable card
2014-07-21 8:01 ` [PATCH v2 1/1] " Andreas Fenkart
@ 2014-07-22 3:30 ` Bing Zhao
0 siblings, 0 replies; 8+ messages in thread
From: Bing Zhao @ 2014-07-22 3:30 UTC (permalink / raw)
To: Andreas Fenkart, linux-wireless@vger.kernel.org; +Cc: daniel@zonque.org
Hi Andreas,
> mmc_rescan will scan for non-removable cards only once, hence the card
> will not be rediscovered.
>
> Signed-off-by: Andreas Fenkart <afenkart@gmail.com>
Acked-by: Bing Zhao <bzhao@marvell.com>
Thanks,
Bing
>
> diff --git a/drivers/net/wireless/mwifiex/sdio.c b/drivers/net/wireless/mwifiex/sdio.c
> index 4ce3d7b..ef22d59 100644
> --- a/drivers/net/wireless/mwifiex/sdio.c
> +++ b/drivers/net/wireless/mwifiex/sdio.c
> @@ -1931,6 +1931,7 @@ static void sdio_card_reset_worker(struct work_struct *work)
> mmc_remove_host(target);
> /* 20ms delay is based on experiment with sdhci controller */
> mdelay(20);
> + target->rescan_entered = 0; /* rescan non-removable cards */
> mmc_add_host(target);
> }
> static DECLARE_WORK(card_reset_work, sdio_card_reset_worker);
> --
> 2.0.1
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2014-07-22 3:30 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-07-18 8:00 [PATCH] mwifiex: card reset: enable rescan of non-removable card Andreas Fenkart
2014-07-18 8:12 ` James Cameron
2014-07-18 17:59 ` John W. Linville
2014-07-18 19:07 ` Bing Zhao
2014-07-21 7:56 ` Andreas Fenkart
2014-07-21 8:01 ` [PATCH v2 0/1] " Andreas Fenkart
2014-07-21 8:01 ` [PATCH v2 1/1] " Andreas Fenkart
2014-07-22 3:30 ` Bing Zhao
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).