From: gregory.clement@free-electrons.com (Gregory CLEMENT)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: mvebu: GPIO reset on GlobalScale Mirabox
Date: Thu, 26 Mar 2015 18:16:19 +0100 [thread overview]
Message-ID: <55143EE3.4040201@free-electrons.com> (raw)
In-Reply-To: <55134082.7010404@free-electrons.com>
On 26/03/2015 00:10, Gregory CLEMENT wrote:
> Hi,
>
> On 26/03/2015 00:16, Leigh Brown wrote:
>> Hi Andrew,
>>
>> On 2015-03-25 22:36, Andrew Lunn wrote:
>>>> Thanks for your contribution, however the reset button is not always
>>>> connected to this GPIO. At least on the Mirabox I have the reset
>>>> button
>>>> triggers an hardware reset. My concern is that for some board the dts
>>>> representation would be wrong.
>>>
>>> When looking at WiFi issues, it became clear there are two different
>>> wifi designs. I would not be too surprised if this reset button
>>> changed at the same time.
>>>
>>> Gregory, Leigh, is your wifi on the SDIO bus, or the USB bus?
>>>
>>> The mirabox i have has an sdio wifi device. But i cannot easily test
>>> the reset button, i blew the 5v power rail, so all USB is dead, and my
>>> rootfs was on mmc, which is implemented via USB :-(
>>
>> The WiFi on my Mirabox is on the SDIO bus.
>>
>> [ 29.408420] mwifiex_sdio mmc0:0001:1: WLAN FW already running! Skip
>> FW dnld
>> [ 29.408430] mwifiex_sdio mmc0:0001:1: WLAN FW is active
>> [ 29.511297] mwifiex_sdio mmc0:0001:1: driver_version = mwifiex 1.0
>> (14.66.35.p52)
>
> I think it is the same for me. I brought my Mirabox at ELC, but it is in
> my room and just after ELC I will go to the airport. If I find a plug
> there, I will boot it and I will confirm you, else we will have to wait
> until Friday morning CET.
So I checked and I confirm that the Wifi is on the SDIO bus.
Going back to how representing the reset button. We have several option:
1. Modifying the file armada-370-mirabox.dts as done in this patch.
2. Introduce a armada-370-mirabox.dtsi file and adding two new dts file using this
one for each variation of the Mirabox
3. Using the Transaction Device Tree + Overlays framwork and creating a device tree
fragment.
I already commented about the 1.
For 2., it seems overkill for me and we need to be able to make difference with the
different flavor of the board, which doesn't seem possible now.
The 3. could be a good solution, but there is some pending question as where to store
this DT fragment and as for 2., how to recognize the board.
Thanks,
Gregory
>
>
> Thanks,
>
> Gregory
>
>
>>
>> Regards,
>>
>> Leigh.
>>
>
>
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2015-03-26 17:16 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-25 21:33 [PATCH] ARM: mvebu: GPIO reset on GlobalScale Mirabox Leigh Brown
2015-03-25 22:16 ` Gregory CLEMENT
2015-03-25 22:36 ` Andrew Lunn
2015-03-25 23:16 ` Leigh Brown
2015-03-25 23:10 ` Gregory CLEMENT
2015-03-26 17:16 ` Gregory CLEMENT [this message]
2015-03-25 23:00 ` Leigh Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55143EE3.4040201@free-electrons.com \
--to=gregory.clement@free-electrons.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox