[OpenWrt-Devel] [PATCH v6] ramips: add support for JS76x8 series DEV boards
mail at adrianschmutzler.de
mail at adrianschmutzler.de
Tue Jul 14 04:57:27 EDT 2020
> 2.About "missing reg property".
> Adrian Schmutzler wrote a letter to me about that, then I tested as he said, it was ok. Here is the letter from him.
@Petr: The reg property is in the DTS files, while everything else is in the DTSI. It's also a case of DRY :-)
> 5.About "Be DRY, use some common Device and just change IMAGE_SIZE and DEVICE_VARIANT".
> Here are three real type of boards by my hand, each has different flash, I don't think it's good way to change board types by modifying the DTS file, it's better to do that by selecting in menuconfig.
@Robinson Wu: That's a misunderstanding from your side. What Petr meant was just to rearrange the definitions inside mt76x8.mk like this, so you define a common blueprint and then use that for the actual device definitions (note that MTK_SOC is not required anymore, as it's defined globally at the top of the file):
+define Device/jotale_js76x8
+ DEVICE_VENDOR := Jotale
+ DEVICE_MODEL := JS76x8
+ DEVICE_PACKAGES := kmod-usb2 kmod-usb-ohci
+endef
+define Device/jotale_js76x8-16m
+ $(Device/jotale_js76x8)
+ IMAGE_SIZE := 16064k
+ DEVICE_VARIANT := 16M
+endef
+TARGET_DEVICES += jotale_js76x8-16m
and so on for the other two devices.
I just didn't request that as I personally prefer individual statement if it's "just" three lines for the common definition.
Apart from that, please note these additional comments:
7. In master, gpio setup has changed, so you only have one &gpio now. So you will need to convert e.g.
<&gpio1 5 GPIO_ACTIVE_LOW>;
to
<&gpio 37 GPIO_ACTIVE_LOW>;
8. pinctrl property rename:
Convert
+ ralink,group = "p1led_an";
+ ralink,function = "p1led_an";
to
+ groups = "p1led_an";
+ ralink,function = "p1led_an";
Sorry for not having taken care earlier, but I can review only 80 % of your PR, and that's just not enough to merge it.
Best
Adrian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: openpgp-digital-signature.asc
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.openwrt.org/pipermail/openwrt-devel/attachments/20200714/1743dd1c/attachment.sig>
More information about the openwrt-devel
mailing list