Omega2 won't reboot?

WereCatf

@Larry-Pinney The device-tree table is easy to get and I have already done it. Also, listing spidev directly inside it is not an issue. Some kernel-dev just didn't like it for aesthetic reasons and added that warning, nothing more.

Larry Pinney

@WereCatf

dmesg |grep WAR
[ 10.750853] WARNING: CPU: 0 PID: 474 at drivers/spi/spidev.c:719 init_module+0x6e33c/0x6edec spidev

WereCatf

@Larry-Pinney I just said that the warning was added in the kernel-tree for aesthetic reasons, it is not a bug or an issue. You can either ignore the warning or disable the warning in the kernel-sources, it doesn't change anything.

Larry Pinney

Really?

Let's list a few errors....
[ 0.213727] alarmtimer alarmtimer: no of_node; not parsing pinctrl DT
[ 0.275348] serial8250 serial8250: no of_node; not parsing pinctrl DT
[ 4.208491] hub 1-0:1.0: no of_node; not parsing pinctrl DT
[ 4.316169] hub 2-0:1.0: no of_node; not parsing pinctrl DT
[ 10.739534] spidev spi32766.1: buggy DT: spidev listed directly in DT

Then you can have a look at this LEDE commit to see how to fix a Device Tree Source file

commit b28e94d4bfa1f81219765fc395e63ca81badd8df
Author: L. D. Pinney ldpinney@gmail.com
Date: Thu Dec 29 01:43:08 2016 -0600

ramips: MiWiFi Nano fixes

Use the the dt-bindings macros and add the reset button.

Set the correct polarity for the LEDs and drop the default state.
Remove all trigger for the LEDs. According to the manual the LEDs are
only used to show the operation state, where blue means normal
operation.

Use the MAC-Addresses stored in EEPROM for the ethernet and the
wireless interface.

Signed-off-by: L. D. Pinney <ldpinney@gmail.com>
[use leds only for boot status indication, add proper commit message]
Signed-off-by: Mathias Kresin <dev@kresin.me>

WereCatf

@Larry-Pinney The other notices are not related to the spidev-warning.

Larry Pinney

@WereCatf

But you are wrong... all these errors come from the OMEGA2.dts and are therefore related.

Your question was.
"What's your problem with the device-tree table?"

I think it's been adequately explained.
The OMEGA2.dts file is defective and should be fixed.

WereCatf

@Larry-Pinney said in Omega2 won't reboot?:

The OMEGA2.dts file is defective and should be fixed.

Well, knock yourself out with e.g. https://github.com/WereCatf/source/blob/master/target/linux/ramips/dts/omega2p.dts I only changed a few things to better match how things are done in LEDE-proper.

Luciano S.

@WereCatf said in Omega2 won't reboot?:

Well, knock yourself out with e.g. https://github.com/WereCatf/source/blob/master/target/linux/ramips/dts/omega2p.dts I only changed a few things to better match how things are done in LEDE-proper.

Did you propose pull requests over Github too? Generally those things were not ignored.

Larry Pinney

@WereCatf

OK how about adding this line.
#include <dt-bindings/gpio/gpio.h>

and changing this
gpios = <&gpio1 12 1>;

to this
<&gpio1 12 GPIO_ACTIVE_LOW>;

and since the reset button seems non functional .
try this using another gpio.
such as 2, 11,15, or 17
<&gpio0 11 GPIO_ACTIVE_LOW>;

I would suggest 11 because it changes from high to low during testing.

Do we need to submit a ticket on the reset button too?

Larry Pinney

Do we need to submit a ticket on the reset button too?

Ticket #4088
Reset button does not work.

WereCatf

@Larry-Pinney said in Omega2 won't reboot?:

and since the reset button seems non functional .
try this using another gpio.

I changed it from KEY_WPS_something to KEY_RESTART in my dts, I haven't tried if it works in my dts or not, but it is entirely possible that is the reason why it doesn't work in the official dts. Feel free to try.

WereCatf

@Larry-Pinney said in Omega2 won't reboot?:

Ticket #4088
Reset button does not work.

Kinda useless without mentioning the repo you're opening the ticket against.

Larry Pinney

@WereCatf

Kinda doesn't matter...none of them will work

Larry Pinney

The reset button should be :

linux,code = <KEY_RESTART>;

However it needs to be mapped to the correct GPIO.

WereCatf

@Larry-Pinney It is mapped to the correct GPIO. It just seems that the pin is pulled LOW by default and it's active HIGH.

Larry Pinney

@WereCatf

Really and you tested it how?

WereCatf

@Larry-Pinney By reading the GPIO-pin's state, then connecting the pin to 3.3V and reading it again! Magic.

Larry Pinney

@WereCatf

And it's correctly mapped...yet it doesn't work?

Did you add
#include <dt-bindings/gpio/gpio.h>
NO
no...or didn't push it yet anyway

did you make the change to
GPIO_ACTIVE_LOW ? no or failed to push that change as well.

did you you read the magic with a meter?
or is it smoke and mirror magic ?

WereCatf

@Larry-Pinney said in Omega2 won't reboot?:

did you you read the magic with a meter?
or is it smoke and mirror magic ?

No, I just said, I read the GPIO-pin's state. GPIO38 is LOW by default, but when I connect 3.3V to the reset-pin it becomes HIGH, so yes, the GPIO-number is correct.

Larry Pinney

@WereCatf

It's NOT on gpio 38 otherwise known as gpio1 6...
apparently that gpio is locked by the "stock" dts file
Please try gpio 0 11

And also let's try be so hard to get along with ....
I do not need to establish a reputation (I have numerous commits to OpenWrt/LEDE)
I'm only trying to help make this toy work properly....and do not need additional BS...I actually have better things to do.