FDT support for RTL8366 on etherswitch framework

[yamori813_yahoo.co.jp]

Summary

FDT away for RTL8366. Also fixed FDT support gpioiic.c.

Boot log is this.

https://gist.github.com/yamori813/bd6ccdbd73dd02eebfdd0bd6d92411d8

Test Plan

on MZK-W300NAG(RT2880 + RTL8366SR)

[mizhka]

All is fine, one improvement is possible (gpio_pins for non-fdt case).

sys/dev/gpio/gpioiic.c

61–67 ↗

(On Diff #33929)

Is it possible to make support of SCL/SDA as "struct gpio_pin" for non-FDT case as well?

[sgalabov]

as far as if_rt changes and dts additions go, I am ok with this.

[loos]

I don't like all the duplicate code in gpioiic.c.

Can you explain which bugs are you trying to fix ?

gpioiic seems to works fine with FDT, but if you want to use the new API, maybe you should consider something like what was done with gpioled (and its FDT counterpart).

[yamori813_yahoo.co.jp]

In D12649#263981, @loos wrote:

I don't like all the duplicate code in gpioiic.c.

Can you explain which bugs are you trying to fix ?

gpioiic seems to works fine with FDT, but if you want to use the new API, maybe you should consider something like what was done with gpioled (and its FDT counterpart).

I think IVAR can't use on FDT. Current gpioiic.c is use IVAR. Tell me dts file on work current gpioiic.c.

Current code is crash in gpioiic_attach() at devi->pins[] access in at FDT build.

Other FDT base gpio code use simplebus or ofwbus by DRIVER_MODULE. But gpioiic not use this.

[gonzo]

Custom dts files shouldn't be put to sys/gnu/dts. This directory is periodically synced with upstream tree, so custom changes may be hard to maintain. The right way to new .dts is to put it to boot/fdt/dts/mips. All include operators should just work

[yamori813_yahoo.co.jp]

• review refict

[yamori813_yahoo.co.jp]

In D12649#264042, @gonzo wrote:

Custom dts files shouldn't be put to sys/gnu/dts. This directory is periodically synced with upstream tree, so custom changes may be hard to maintain. The right way to new .dts is to put it to boot/fdt/dts/mips. All include operators should just work

my dts file move to sys/boot/fdt/dts/mips

[yamori813_yahoo.co.jp]

• gpioiic is sepalate like gpioled

[yamori813_yahoo.co.jp]

loos Please show me complete dts file use gpioiic.c.

sys/dev/gpio/gpioiic.c

61–67 ↗

(On Diff #33929)

I seem other non-FDT gpio code use directly value. I think good that is not change this implement.

[yamori813_yahoo.co.jp]

I will put separate this revision.