From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?Q?Andreas_F=c3=a4rber?= Subject: Re: [PATCH v2 8/9] ARM: dts: Add leds support to STM32F429 Discovery board Date: Tue, 27 Oct 2015 22:52:52 +0100 Message-ID: <562FF234.9010403@suse.de> References: <1445102604-11502-1-git-send-email-mcoquelin.stm32@gmail.com> <1445102604-11502-9-git-send-email-mcoquelin.stm32@gmail.com> <562FEE85.3010204@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: 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: Linus Walleij Cc: Mark Rutland , "devicetree@vger.kernel.org" , Daniel Thompson , Arnd Bergmann , Bruno Herrera , "linux-kernel@vger.kernel.org" , "linux-gpio@vger.kernel.org" , Rob Herring , Maxime Coquelin , "linux-arm-kernel@lists.infradead.org" List-Id: linux-gpio@vger.kernel.org QW0gMjcuMTAuMjAxNSB1bSAyMjo0NiBzY2hyaWViIExpbnVzIFdhbGxlaWo6Cj4gSGVhcnRiZWF0 IGlzIGF3ZXNvbWUuCgpZZXMsIGl0J3MgYXdlc29tZSBmb3IgdGVzdGluZywgYnV0IG5vdCBmb3Ig YSAuZHRzIGZpbGUgdGhhdCBlbmRzIHVwIGluIGEKZGlzdHJvIChub3QgdGhlIGNhc2UgaGVyZSwg b2YgY291cnNlKSBhbmQga2VlcHMgYmxpbmtpbmcgb24gdGhlIGRlc2suCgpJZiBzb21lb25lIHdh bnRzIExFRHMgdG8gYmxpbmssIHRoZXkgY2FuIHNldCB0aGF0IHZpYSBzeXNmcyBvciBieQptb2Rp ZnlpbmcgdGhlaXIgLmR0cyBsb2NhbGx5LgoKc3VueGkgY2hvc2UgdG8ga2VlcCBMRURzIG9mZiBt eSBkZWZhdWx0LiBGb3IgcWNvbSB3ZSBoYWQgYSBzaW1pbGFyCmRpc2N1c3Npb24gc29tZSB3ZWVr cyBhZ28uIEkgZG9uJ3QgcmVndWxhcmx5IHJlYWQgTGludXggcGF0Y2hlcywgc28gZmVlbApmcmVl IHRvIHNraW0gdGhlIGFyY2hpdmVzIHlvdXJzZWxmLgoKQ2hlZXJzLApBbmRyZWFzCgotLSAKU1VT RSBMaW51eCBHbWJILCBNYXhmZWxkc3RyLiA1LCA5MDQwOSBOw7xybmJlcmcsIEdlcm1hbnkKR0Y6 IEZlbGl4IEltZW5kw7ZyZmZlciwgSmFuZSBTbWl0aGFyZCwgR3JhaGFtIE5vcnRvbjsgSFJCIDIx Mjg0IChBRyBOw7xybmJlcmcpCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpsaW51eC1hcm0ta2VybmVsIG1haWxpbmcgbGlzdApsaW51eC1hcm0ta2VybmVs QGxpc3RzLmluZnJhZGVhZC5vcmcKaHR0cDovL2xpc3RzLmluZnJhZGVhZC5vcmcvbWFpbG1hbi9s aXN0aW5mby9saW51eC1hcm0ta2VybmVsCg== From mboxrd@z Thu Jan 1 00:00:00 1970 From: afaerber@suse.de (=?UTF-8?Q?Andreas_F=c3=a4rber?=) Date: Tue, 27 Oct 2015 22:52:52 +0100 Subject: [PATCH v2 8/9] ARM: dts: Add leds support to STM32F429 Discovery board In-Reply-To: References: <1445102604-11502-1-git-send-email-mcoquelin.stm32@gmail.com> <1445102604-11502-9-git-send-email-mcoquelin.stm32@gmail.com> <562FEE85.3010204@suse.de> Message-ID: <562FF234.9010403@suse.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Am 27.10.2015 um 22:46 schrieb Linus Walleij: > Heartbeat is awesome. Yes, it's awesome for testing, but not for a .dts file that ends up in a distro (not the case here, of course) and keeps blinking on the desk. If someone wants LEDs to blink, they can set that via sysfs or by modifying their .dts locally. sunxi chose to keep LEDs off my default. For qcom we had a similar discussion some weeks ago. I don't regularly read Linux patches, so feel free to skim the archives yourself. Cheers, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany GF: Felix Imend?rffer, Jane Smithard, Graham Norton; HRB 21284 (AG N?rnberg) From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932447AbbJ0VxF (ORCPT ); Tue, 27 Oct 2015 17:53:05 -0400 Received: from mx2.suse.de ([195.135.220.15]:43621 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754782AbbJ0Vw7 (ORCPT ); Tue, 27 Oct 2015 17:52:59 -0400 Subject: Re: [PATCH v2 8/9] ARM: dts: Add leds support to STM32F429 Discovery board To: Linus Walleij References: <1445102604-11502-1-git-send-email-mcoquelin.stm32@gmail.com> <1445102604-11502-9-git-send-email-mcoquelin.stm32@gmail.com> <562FEE85.3010204@suse.de> Cc: Maxime Coquelin , Mark Rutland , Rob Herring , "linux-gpio@vger.kernel.org" , Arnd Bergmann , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" , Daniel Thompson , Bruno Herrera From: =?UTF-8?Q?Andreas_F=c3=a4rber?= X-Enigmail-Draft-Status: N1110 Organization: SUSE Linux GmbH Message-ID: <562FF234.9010403@suse.de> Date: Tue, 27 Oct 2015 22:52:52 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Am 27.10.2015 um 22:46 schrieb Linus Walleij: > Heartbeat is awesome. Yes, it's awesome for testing, but not for a .dts file that ends up in a distro (not the case here, of course) and keeps blinking on the desk. If someone wants LEDs to blink, they can set that via sysfs or by modifying their .dts locally. sunxi chose to keep LEDs off my default. For qcom we had a similar discussion some weeks ago. I don't regularly read Linux patches, so feel free to skim the archives yourself. Cheers, Andreas -- SUSE Linux GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Felix Imendörffer, Jane Smithard, Graham Norton; HRB 21284 (AG Nürnberg)