linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "René Kjellerup" <rk.katana.steel@gmail.com>
To: Hauke Mehrtens <hauke@hauke-m.de>
Cc: "Rafał Miłecki" <zajec5@gmail.com>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Mark Rutland" <mark.rutland@arm.com>,
	"Jon Mason" <jonmason@broadcom.com>,
	bcm-kernel-feedback-list@broadcom.com,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: Adding a Basic DT for the Linksys ea6500 v2 wireless home router
Date: Mon, 8 Oct 2018 09:34:49 -0700	[thread overview]
Message-ID: <CACE8ZKb5CxEBAdCdGAhwaQHC9_nHBjTTVwxdDOXRayqgz0FK5g@mail.gmail.com> (raw)
In-Reply-To: <86f391e5-9e38-c8b0-4753-4646c322c9bc@hauke-m.de>

Hi Hauke,

Updated the file header per your suggestion.

From e97c15ef78d2535c23f32ebbec43bced72b5eaf7 Mon Sep 17 00:00:00 2001
From: Rene Kjellerup <rk.katana.steel@gmail.com>
Date: Mon, 1 Oct 2018 14:11:56 -0700
Subject: [PATCH] ARM: dts: BCM5301X: Add basic DT for Linksys EA6500 V2

It's wireless home router based on BCM4708A0 with BCM4360 + BCM4331
wireless chipsets. The 5GHz chip currently isn't supported only due
to missing compatible firmware.

the DTS here was based on the Linksys EA6300 V1 DTS, only changing
the model strings

Signed-off-by: Rene Kjellerup <rk.katana.steel@gmail.com>
---
 arch/arm/boot/dts/Makefile                      |  1 +
 arch/arm/boot/dts/bcm4708-linksys-ea6500-v2.dts | 47 +++++++++++++++++++++++++
 2 files changed, 48 insertions(+)
 create mode 100644 arch/arm/boot/dts/bcm4708-linksys-ea6500-v2.dts

diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile
index b5bd3de87c33..adf8f28f5e44 100644
--- a/arch/arm/boot/dts/Makefile
+++ b/arch/arm/boot/dts/Makefile
@@ -88,6 +88,7 @@ dtb-$(CONFIG_ARCH_BCM_5301X) += \
     bcm4708-asus-rt-ac68u.dtb \
     bcm4708-buffalo-wzr-1750dhp.dtb \
     bcm4708-linksys-ea6300-v1.dtb \
+    bcm4708-linksys-ea6500-v2.dtb \
     bcm4708-luxul-xap-1510.dtb \
     bcm4708-luxul-xwc-1000.dtb \
     bcm4708-netgear-r6250.dtb \
diff --git a/arch/arm/boot/dts/bcm4708-linksys-ea6500-v2.dts
b/arch/arm/boot/dts/bcm4708-linksys-ea6500-v2.dts
new file mode 100644
index 000000000000..8a434d2a4b72
--- /dev/null
+++ b/arch/arm/boot/dts/bcm4708-linksys-ea6500-v2.dts
@@ -0,0 +1,47 @@
+// SPDX-License-Identifier: GPL-2.0-or-later OR MIT
+/*
+ * Copyright (C) 2017 Rafał Miłecki <rafal@milecki.pl>
+ * Copyright (C) 2018 Rene Kjellerup <rk.katana.steel@gmail.com>
+ *
+ * this DTS was originally based on bcm4708-linksys-ea6300-v1.dts
+ */
+
+/dts-v1/;
+
+#include "bcm4708.dtsi"
+#include "bcm5301x-nand-cs0-bch8.dtsi"
+
+/ {
+    compatible = "linksys,ea6500-v2", "brcm,bcm4708";
+    model = "Linksys EA6500 V2";
+
+    chosen {
+        bootargs = "console=ttyS0,115200";
+    };
+
+    memory {
+        reg = <0x00000000 0x08000000>;
+    };
+
+    gpio-keys {
+        compatible = "gpio-keys";
+        #address-cells = <1>;
+        #size-cells = <0>;
+
+        wps {
+            label = "WPS";
+            linux,code = <KEY_WPS_BUTTON>;
+            gpios = <&chipcommon 7 GPIO_ACTIVE_LOW>;
+        };
+
+        restart {
+            label = "Reset";
+            linux,code = <KEY_RESTART>;
+            gpios = <&chipcommon 11 GPIO_ACTIVE_LOW>;
+        };
+    };
+};
+
+&usb3_phy {
+    status = "okay";
+};
-- 
2.16.4

On Sun, Oct 7, 2018 at 2:08 PM Hauke Mehrtens <hauke@hauke-m.de> wrote:
>
> On 10/04/2018 11:08 PM, René Kjellerup wrote:
> > Hi Hauke,
> >
> > thank you for suggesting adding the dts patch to mainline kernel,
> > so here it is.
> > I feel Rafal technically still have the copyrights, this is pretty
> > much just a few string updates for the ea6500v2. However with this
> > dts in latest OpenWrt 18.06 loads and installs on my device.
> >
> > I will be available to do more testing should it be required.
> >
> > Regrads
> > Rene Kjellerup
> > --
> > -- as life grows older, I gain experience.
> > -- http://www.alchemiestick.net/
> >
>
> Hi Rene,
>
> Please send you patch inline in the mail and not as an attachment, I
> would suggest to use "git send-email" for sending the patch.
>
> Adding Rafal as copyright owner is ok, like you did it now. You could
> mention in the file header on which dts file this is based.
>
> Hauke
>


-- 
-- as life grows older, I gain experience.
-- http://www.alchemiestick.net/apps/

  reply	other threads:[~2018-10-08 16:35 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <CACE8ZKbUhFyWg5oK1B8+aWAa0tdqKf5Kokj80+8x1X2EUaA_DQ@mail.gmail.com>
2018-10-04 21:08 ` Adding a Basic DT for the Linksys ea6500 v2 wireless home router René Kjellerup
2018-10-07 21:08   ` Hauke Mehrtens
2018-10-08 16:34     ` René Kjellerup [this message]
2018-11-27  0:49       ` René Kjellerup
2018-11-28  0:23         ` Florian Fainelli
     [not found]           ` <CACE8ZKaMBfa5vWJMp_xVjstkuxkjO=DDwp1srA+Am1RP46gZcQ@mail.gmail.com>
2018-11-30 18:36             ` Florian Fainelli

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=CACE8ZKb5CxEBAdCdGAhwaQHC9_nHBjTTVwxdDOXRayqgz0FK5g@mail.gmail.com \
    --to=rk.katana.steel@gmail.com \
    --cc=bcm-kernel-feedback-list@broadcom.com \
    --cc=devicetree@vger.kernel.org \
    --cc=hauke@hauke-m.de \
    --cc=jonmason@broadcom.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=zajec5@gmail.com \
    /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;
as well as URLs for NNTP newsgroup(s).