[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