From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758120AbbICXzN (ORCPT ); Thu, 3 Sep 2015 19:55:13 -0400 Received: from mailout2.w1.samsung.com ([210.118.77.12]:28835 "EHLO mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752591AbbICXzL (ORCPT ); Thu, 3 Sep 2015 19:55:11 -0400 X-AuditID: cbfec7f4-f79c56d0000012ee-30-55e8dddcffbd Subject: Re: [PATCHv7] ARM: exynos_defconfig: Enable LEDS for Odroid-XU3/XU4 To: Anand Moon , Russell King , Kukjin Kim , Javier Martinez Canillas , Andreas Faerber , Lukasz Majewski , Sjoerd Simons References: <1441303910-3266-1-git-send-email-linux.amoon@gmail.com> Cc: linux-arm-kernel@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-kernel@vger.kernel.org From: Krzysztof Kozlowski Message-id: <55E8DDDD.80106@samsung.com> Date: Fri, 04 Sep 2015 08:55:09 +0900 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-version: 1.0 In-reply-to: <1441303910-3266-1-git-send-email-linux.amoon@gmail.com> Content-type: text/plain; charset=windows-1252 Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFvrLLMWRmVeSWpSXmKPExsVy+t/xa7p37r4INeg7zWPRvKnY4s3bNUwW r18YWvQ/fs1s8ebhZkaLTY+vsVpc3jWHzWLG+X1MFus23mK3uH2Z12L68besDtweLc09bB5/ n19n8dg56y67x6ZVnWwem5fUe2zpB/L6tqxi9Nh8utrj8ya5AM4oLpuU1JzMstQifbsEroy9 +3uZCrbxVhz71srWwNjP3cXIySEhYCIxZdsSZghbTOLCvfVsXYxcHEICSxklzh1YwQjhfGGU 6DjazwRSJSzgI3H40wVmkISIwBImie3PFoK1Cwm4SHzcvYC9i5GDg1kgW2LLK7B6NgFjic3L l7CB2LwCGhI/m06AlbMIqEpc7TgHFhcViJA4dfYtVI2gxI/J91hAbE4BV4n/S7ezQYzUk7h/ UQskzCwgL7F5zVvmCYwCs5B0zEKomoWkagEj8ypG0dTS5ILipPRcQ73ixNzi0rx0veT83E2M kEj5soNx8TGrQ4wCHIxKPLwTZj8PFWJNLCuuzD3EKMHBrCTCK3rgRagQb0piZVVqUX58UWlO avEhRmkOFiVx3rm73ocICaQnlqRmp6YWpBbBZJk4OKUaGCcocS5eIWOz8qbLbeuNa6S8z9/S DHWtsnR5V3xRZYrKg+8zssXiz/+UPGQ89+L6axwaG9ZrO3ty/DvdvVP698afh21ecpsq86XY x61NiTGxcPb+/mdV25F7FakmQjIzNI+ZF7U8UYnUsfMU2bR4+/O2iZN7ZR/FXz2rGtdtdPLR KvENdw8tMlViKc5INNRiLipOBADjyMANkAIAAA== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 04.09.2015 03:11, Anand Moon wrote: > Earlier design of the LED for Odroid XU3 was using gpio-leds > Now It was change to using both pwm-leds and gpio-leds. It is still not a reason for this change. gpio-leds were not enabled before. This could be a valid reason of adding LEDS_PWM to existing config LEDS_GPIO. But LEDS_GPIO were not enabled... so why the change on the board from gpio->(gpio+pwm) means that we have to enable LEDS_GPIO? Actually I gave up on this and I wanted to change the commit message on myself when applying. But discussion was brought up again so... clearly we have different understanding of the meaning of "WHY". :) Best regards, Krzysztof > > Signed-off-by: Anand Moon > Reviewed-by: Javier Martinez Canillas > > --- > Changes from last version > dropped following option. > CONFIG_LEDS_CLASS_FLASH > CONFIG_TRIGGER_ONESHOT > CONFIG_LEDS_TRIGGER_TIMER > CONFIG_TRIGGER_GPIO > fixed the From address > fixed the commit message. > --- > arch/arm/configs/exynos_defconfig | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/arch/arm/configs/exynos_defconfig b/arch/arm/configs/exynos_defconfig > index 9504e77..bd6b7f7 100644 > --- a/arch/arm/configs/exynos_defconfig > +++ b/arch/arm/configs/exynos_defconfig > @@ -163,6 +163,12 @@ CONFIG_MMC_SDHCI_S3C_DMA=y > CONFIG_MMC_DW=y > CONFIG_MMC_DW_IDMAC=y > CONFIG_MMC_DW_EXYNOS=y > +CONFIG_NEW_LEDS=y > +CONFIG_LEDS_CLASS=y > +CONFIG_LEDS_GPIO=y > +CONFIG_LEDS_PWM=y > +CONFIG_LEDS_TRIGGERS=y > +CONFIG_LEDS_TRIGGER_HEARTBEAT=y > CONFIG_RTC_CLASS=y > CONFIG_RTC_DRV_MAX77686=y > CONFIG_RTC_DRV_MAX77802=y >