From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@baylibre.com (Kevin Hilman) Date: Wed, 18 Jan 2017 14:29:58 -0800 Subject: [PATCH 2/2] ARM64: dts: meson-gxm-rbox-pro: Enable Bluetooth In-Reply-To: <20170117030611.23827-3-afaerber@suse.de> ("Andreas =?utf-8?Q?F=C3=A4rber=22's?= message of "Tue, 17 Jan 2017 04:06:11 +0100") References: <20170117030611.23827-1-afaerber@suse.de> <20170117030611.23827-3-afaerber@suse.de> Message-ID: To: linus-amlogic@lists.infradead.org List-Id: linus-amlogic.lists.infradead.org Andreas F?rber writes: > Add an SDIO reset GPIO and enable the serial used by the AP6255 > Bluetooth module. Based on work by Martin Blumenstingl. > > Signed-off-by: Andreas F?rber > --- > arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts | 11 ++++++++++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > index 9f04fa4e5aec..6ea225f584bd 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > @@ -58,6 +58,7 @@ > > aliases { > serial0 = &uart_AO; > + serial1 = &uart_A; > }; > > chosen { > @@ -122,7 +123,8 @@ > > sdio_pwrseq: sdio-pwrseq { > compatible = "mmc-pwrseq-simple"; > - reset-gpios = <&gpio GPIOX_6 GPIO_ACTIVE_LOW>; > + reset-gpios = <&gpio GPIOX_6 GPIO_ACTIVE_LOW>, > + <&gpio GPIOX_17 GPIO_ACTIVE_LOW>; > clocks = <&wifi32k>; > clock-names = "ext_clock"; c.f. thread on Martin's series. I don't like this (ab)use of reset-gpios for sdio-pwrseq, and there's some work coming that will make this much cleaner. So, I'll hold off on this until the other solution is ready. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: khilman@baylibre.com (Kevin Hilman) Date: Wed, 18 Jan 2017 14:29:58 -0800 Subject: [PATCH 2/2] ARM64: dts: meson-gxm-rbox-pro: Enable Bluetooth In-Reply-To: <20170117030611.23827-3-afaerber@suse.de> ("Andreas =?utf-8?Q?F=C3=A4rber=22's?= message of "Tue, 17 Jan 2017 04:06:11 +0100") References: <20170117030611.23827-1-afaerber@suse.de> <20170117030611.23827-3-afaerber@suse.de> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Andreas F?rber writes: > Add an SDIO reset GPIO and enable the serial used by the AP6255 > Bluetooth module. Based on work by Martin Blumenstingl. > > Signed-off-by: Andreas F?rber > --- > arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts | 11 ++++++++++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > index 9f04fa4e5aec..6ea225f584bd 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > @@ -58,6 +58,7 @@ > > aliases { > serial0 = &uart_AO; > + serial1 = &uart_A; > }; > > chosen { > @@ -122,7 +123,8 @@ > > sdio_pwrseq: sdio-pwrseq { > compatible = "mmc-pwrseq-simple"; > - reset-gpios = <&gpio GPIOX_6 GPIO_ACTIVE_LOW>; > + reset-gpios = <&gpio GPIOX_6 GPIO_ACTIVE_LOW>, > + <&gpio GPIOX_17 GPIO_ACTIVE_LOW>; > clocks = <&wifi32k>; > clock-names = "ext_clock"; c.f. thread on Martin's series. I don't like this (ab)use of reset-gpios for sdio-pwrseq, and there's some work coming that will make this much cleaner. So, I'll hold off on this until the other solution is ready. Kevin From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH 2/2] ARM64: dts: meson-gxm-rbox-pro: Enable Bluetooth Date: Wed, 18 Jan 2017 14:29:58 -0800 Message-ID: References: <20170117030611.23827-1-afaerber@suse.de> <20170117030611.23827-3-afaerber@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: <20170117030611.23827-3-afaerber@suse.de> ("Andreas =?utf-8?Q?F=C3=A4rber=22's?= message of "Tue, 17 Jan 2017 04:06:11 +0100") List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Andreas =?utf-8?Q?F=C3=A4rber?= Cc: Mark Rutland , devicetree@vger.kernel.org, Catalin Marinas , Will Deacon , linux-kernel@vger.kernel.org, Rob Herring , Carlo Caione , linux-amlogic@lists.infradead.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org QW5kcmVhcyBGw6RyYmVyIDxhZmFlcmJlckBzdXNlLmRlPiB3cml0ZXM6Cgo+IEFkZCBhbiBTRElP IHJlc2V0IEdQSU8gYW5kIGVuYWJsZSB0aGUgc2VyaWFsIHVzZWQgYnkgdGhlIEFQNjI1NQo+IEJs dWV0b290aCBtb2R1bGUuIEJhc2VkIG9uIHdvcmsgYnkgTWFydGluIEJsdW1lbnN0aW5nbC4KPgo+ IFNpZ25lZC1vZmYtYnk6IEFuZHJlYXMgRsOkcmJlciA8YWZhZXJiZXJAc3VzZS5kZT4KPiAtLS0K PiAgYXJjaC9hcm02NC9ib290L2R0cy9hbWxvZ2ljL21lc29uLWd4bS1yYm94LXByby5kdHMgfCAx MSArKysrKysrKysrLQo+ICAxIGZpbGUgY2hhbmdlZCwgMTAgaW5zZXJ0aW9ucygrKSwgMSBkZWxl dGlvbigtKQo+Cj4gZGlmZiAtLWdpdCBhL2FyY2gvYXJtNjQvYm9vdC9kdHMvYW1sb2dpYy9tZXNv bi1neG0tcmJveC1wcm8uZHRzIGIvYXJjaC9hcm02NC9ib290L2R0cy9hbWxvZ2ljL21lc29uLWd4 bS1yYm94LXByby5kdHMKPiBpbmRleCA5ZjA0ZmE0ZTVhZWMuLjZlYTIyNWY1ODRiZCAxMDA2NDQK PiAtLS0gYS9hcmNoL2FybTY0L2Jvb3QvZHRzL2FtbG9naWMvbWVzb24tZ3htLXJib3gtcHJvLmR0 cwo+ICsrKyBiL2FyY2gvYXJtNjQvYm9vdC9kdHMvYW1sb2dpYy9tZXNvbi1neG0tcmJveC1wcm8u ZHRzCj4gQEAgLTU4LDYgKzU4LDcgQEAKPiAgCj4gIAlhbGlhc2VzIHsKPiAgCQlzZXJpYWwwID0g JnVhcnRfQU87Cj4gKwkJc2VyaWFsMSA9ICZ1YXJ0X0E7Cj4gIAl9Owo+ICAKPiAgCWNob3NlbiB7 Cj4gQEAgLTEyMiw3ICsxMjMsOCBAQAo+ICAKPiAgCXNkaW9fcHdyc2VxOiBzZGlvLXB3cnNlcSB7 Cj4gIAkJY29tcGF0aWJsZSA9ICJtbWMtcHdyc2VxLXNpbXBsZSI7Cj4gLQkJcmVzZXQtZ3Bpb3Mg PSA8JmdwaW8gR1BJT1hfNiBHUElPX0FDVElWRV9MT1c+Owo+ICsJCXJlc2V0LWdwaW9zID0gPCZn cGlvIEdQSU9YXzYgR1BJT19BQ1RJVkVfTE9XPiwKPiArCQkJCTwmZ3BpbyBHUElPWF8xNyBHUElP X0FDVElWRV9MT1c+Owo+ICAJCWNsb2NrcyA9IDwmd2lmaTMyaz47Cj4gIAkJY2xvY2stbmFtZXMg PSAiZXh0X2Nsb2NrIjsKCmMuZi4gdGhyZWFkIG9uIE1hcnRpbidzIHNlcmllcy4gICBJIGRvbid0 IGxpa2UgdGhpcyAoYWIpdXNlIG9mCnJlc2V0LWdwaW9zIGZvciBzZGlvLXB3cnNlcSwgYW5kIHRo ZXJlJ3Mgc29tZSB3b3JrIGNvbWluZyB0aGF0IHdpbGwgbWFrZQp0aGlzIG11Y2ggY2xlYW5lci4K ClNvLCBJJ2xsIGhvbGQgb2ZmIG9uIHRoaXMgdW50aWwgdGhlIG90aGVyIHNvbHV0aW9uIGlzIHJl YWR5LgoKS2V2aW4gCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVsQGxpc3Rz LmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9saXN0aW5m by9saW51eC1hcm0ta2VybmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751171AbdARWho (ORCPT ); Wed, 18 Jan 2017 17:37:44 -0500 Received: from mail-pf0-f176.google.com ([209.85.192.176]:34419 "EHLO mail-pf0-f176.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751030AbdARWhl (ORCPT ); Wed, 18 Jan 2017 17:37:41 -0500 From: Kevin Hilman To: Andreas =?utf-8?Q?F=C3=A4rber?= Cc: linux-amlogic@lists.infradead.org, Carlo Caione , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Rob Herring , Mark Rutland , Catalin Marinas , Will Deacon , devicetree@vger.kernel.org Subject: Re: [PATCH 2/2] ARM64: dts: meson-gxm-rbox-pro: Enable Bluetooth Organization: BayLibre References: <20170117030611.23827-1-afaerber@suse.de> <20170117030611.23827-3-afaerber@suse.de> Date: Wed, 18 Jan 2017 14:29:58 -0800 In-Reply-To: <20170117030611.23827-3-afaerber@suse.de> ("Andreas =?utf-8?Q?F=C3=A4rber=22's?= message of "Tue, 17 Jan 2017 04:06:11 +0100") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1 (darwin) MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v0IMblbh025272 Andreas Färber writes: > Add an SDIO reset GPIO and enable the serial used by the AP6255 > Bluetooth module. Based on work by Martin Blumenstingl. > > Signed-off-by: Andreas Färber > --- > arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts | 11 ++++++++++- > 1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > index 9f04fa4e5aec..6ea225f584bd 100644 > --- a/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-rbox-pro.dts > @@ -58,6 +58,7 @@ > > aliases { > serial0 = &uart_AO; > + serial1 = &uart_A; > }; > > chosen { > @@ -122,7 +123,8 @@ > > sdio_pwrseq: sdio-pwrseq { > compatible = "mmc-pwrseq-simple"; > - reset-gpios = <&gpio GPIOX_6 GPIO_ACTIVE_LOW>; > + reset-gpios = <&gpio GPIOX_6 GPIO_ACTIVE_LOW>, > + <&gpio GPIOX_17 GPIO_ACTIVE_LOW>; > clocks = <&wifi32k>; > clock-names = "ext_clock"; c.f. thread on Martin's series. I don't like this (ab)use of reset-gpios for sdio-pwrseq, and there's some work coming that will make this much cleaner. So, I'll hold off on this until the other solution is ready. Kevin