From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751454AbdKUWCl (ORCPT ); Tue, 21 Nov 2017 17:02:41 -0500 Received: from mail-wr0-f195.google.com ([209.85.128.195]:38097 "EHLO mail-wr0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751302AbdKUWCj (ORCPT ); Tue, 21 Nov 2017 17:02:39 -0500 X-Google-Smtp-Source: AGs4zMYEmVqj7hrpM3sPm7Fr6RQUAOFhN/1hdM9LMnf7D6ca2Qb19GN3TT2mZV1BQvnGUFLN/VCCJA== Subject: Re: [PATCH v3 2/3] leds: Add driver for Qualcomm LPG To: Pavel Machek , Bjorn Andersson References: <20171115071345.24331-1-bjorn.andersson@linaro.org> <20171115071345.24331-3-bjorn.andersson@linaro.org> <19018aa2-c460-1d5e-8dd2-1cfa557be2f2@gmail.com> <20171120211033.GZ28761@minitux> <20171120232211.GC13906@amd> Cc: Richard Purdie , linux-kernel@vger.kernel.org, linux-leds@vger.kernel.org, linux-arm-msm@vger.kernel.org, Rob Herring , Mark Rutland , devicetree@vger.kernel.org, Fenglin Wu From: Jacek Anaszewski Message-ID: Date: Tue, 21 Nov 2017 23:01:40 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <20171120232211.GC13906@amd> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 11/21/2017 12:22 AM, Pavel Machek wrote: > On Mon 2017-11-20 13:10:33, Bjorn Andersson wrote: >> On Sun 19 Nov 13:36 PST 2017, Jacek Anaszewski wrote: >> >>> Hi Bjorn, >>> >>> Thanks for the patch. Please refer to my comments in the code. >>> >>> On 11/15/2017 08:13 AM, Bjorn Andersson wrote: >> [..] >>>> diff --git a/drivers/leds/Kconfig b/drivers/leds/Kconfig >>>> index 52ea34e337cd..ccc3aa4b2474 100644 >>>> --- a/drivers/leds/Kconfig >>>> +++ b/drivers/leds/Kconfig >>>> @@ -651,6 +651,13 @@ config LEDS_POWERNV >>>> To compile this driver as a module, choose 'm' here: the module >>>> will be called leds-powernv. >>>> >>>> +config LEDS_QCOM_LPG >>>> + tristate "LED support for Qualcomm LPG" >>>> + depends on LEDS_CLASS >>> >>> You were mentioning that this driver is for a MFD child block, >>> so we should probably depend on the parent MFD driver? >>> >> >> There's no build time dependency between the two, so it's not strictly >> necessary. >> >> Adding a dependency on MFD_SPMI_PMIC would indirectly add a dependency >> on ARCH_QCOM, which limit build testing and stop some static code >> checkers to check the driver. >> >> So, unless you strongly object I would prefer not to mention the >> MFD. > > OTOH, this way even users that can't have qualcom LPG are asked the > question. > > So right way is depends on LEdS_CLASS && (BUILD_TEST || MFD_SPMI_PMIC). Ack. -- Best regards, Jacek Anaszewski