From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752628AbdLLV20 (ORCPT ); Tue, 12 Dec 2017 16:28:26 -0500 Received: from fllnx209.ext.ti.com ([198.47.19.16]:54920 "EHLO fllnx209.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752415AbdLLV2Y (ORCPT ); Tue, 12 Dec 2017 16:28:24 -0500 Subject: Re: [PATCH v3 3/6] leds: lp8860: Update the dt parsing for LED labeling To: Jacek Anaszewski , , , , CC: , , References: <20171212185809.23880-1-dmurphy@ti.com> <20171212185809.23880-5-dmurphy@ti.com> From: Dan Murphy Message-ID: <2ea30a5d-d013-1015-3882-a03e4ae540d3@ti.com> Date: Tue, 12 Dec 2017 15:27:01 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="windows-1252" Content-Language: en-US Content-Transfer-Encoding: 7bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Jacek On 12/12/2017 03:13 PM, Jacek Anaszewski wrote: > Hi Dan, > > Thanks for the patch set. > > On 12/12/2017 07:58 PM, Dan Murphy wrote: >> Update the DT parsing for the label node so that >> the label is retrieved from the device child as >> opposed to being part of the parent. >> >> This will align this driver with the LED >> binding documentation >> >> Documentation/devicetree/bindings/leds/common.txt >> >> Signed-off-by: Dan Murphy >> --- >> >> v3 - Changed the label generation to pull the name from the i2c device id >> as opposed to pulling the id from the parent dt node since that will just be >> led-controller - https://patchwork.kernel.org/patch/10093753/ >> >> v2 - no changes >> >> drivers/leds/leds-lp8860.c | 23 ++++++++++++----------- >> 1 file changed, 12 insertions(+), 11 deletions(-) >> >> diff --git a/drivers/leds/leds-lp8860.c b/drivers/leds/leds-lp8860.c >> index 3e70775a2d54..bc432764c99d 100644 >> --- a/drivers/leds/leds-lp8860.c >> +++ b/drivers/leds/leds-lp8860.c >> @@ -22,6 +22,7 @@ >> #include >> #include >> #include >> +#include >> >> #define LP8860_DISP_CL1_BRT_MSB 0x00 >> #define LP8860_DISP_CL1_BRT_LSB 0x01 >> @@ -86,8 +87,6 @@ >> >> #define LP8860_CLEAR_FAULTS 0x01 >> >> -#define LP8860_DISP_LED_NAME "display_cluster" >> - >> /** >> * struct lp8860_led - >> * @lock - Lock for reading/writing the device >> @@ -107,7 +106,7 @@ struct lp8860_led { >> struct regmap *eeprom_regmap; >> struct gpio_desc *enable_gpio; >> struct regulator *regulator; >> - const char *label; >> + char label[LED_MAX_NAME_SIZE]; >> }; >> >> struct lp8860_eeprom_reg { >> @@ -365,19 +364,21 @@ static int lp8860_probe(struct i2c_client *client, >> int ret; >> struct lp8860_led *led; >> struct device_node *np = client->dev.of_node; >> + struct device_node *child_node; >> + const char *name; >> >> led = devm_kzalloc(&client->dev, sizeof(*led), GFP_KERNEL); >> if (!led) >> return -ENOMEM; >> >> - led->label = LP8860_DISP_LED_NAME; >> - >> - if (client->dev.of_node) { >> - ret = of_property_read_string(np, "label", &led->label); >> - if (ret) { >> - dev_err(&client->dev, "Missing label in dt\n"); >> - return -EINVAL; >> - } >> + for_each_available_child_of_node(np, child_node) { >> + ret = of_property_read_string(child_node, "label", &name); >> + if (!ret) >> + snprintf(led->label, sizeof(led->label), "%s:%s", >> + id->name, name); >> + else >> + snprintf(led->label, sizeof(led->label), >> + "%s::display_cluster", id->name); > > Here we have the same indentation problem as in case of leds-lm3692x.c I will fix it up and re-send. Dan > >> } >> >> led->enable_gpio = devm_gpiod_get_optional(&client->dev, >> > -- ------------------ Dan Murphy