From patchwork Sat Mar 21 08:15:52 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Denis OSTERLAND-HEIM X-Patchwork-Id: 203016 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.1 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH, MAILING_LIST_MULTI, SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_2 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9ED66C43332 for ; Sat, 21 Mar 2020 08:16:01 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 75D4E20658 for ; Sat, 21 Mar 2020 08:16:01 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=diehl.com header.i=@diehl.com header.b="beeswZoq" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728086AbgCUIP6 (ORCPT ); Sat, 21 Mar 2020 04:15:58 -0400 Received: from enterprise02.smtp.diehl.com ([193.201.238.220]:35341 "EHLO enterprise02.smtp.diehl.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727961AbgCUIP6 (ORCPT ); Sat, 21 Mar 2020 04:15:58 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=diehl.com; i=@diehl.com; q=dns/txt; s=default; t=1584778556; x=1616314556; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=fNaOGd71ykhkOab84O0XrMhdHdHgb/7uZ6XyGD7xzNo=; b=beeswZoqj6Y/vCMUZ1Wp2xgU6mKcqfGiSIfv7hIpTKriyrl14C5qSgYl qV/oCp4X6SgzGy44s/NOHIYdSV1JgczvwKbbuDRGkkQHNEkciPnTx3ULa 6miJ/1s/bB20LjwwEkke9CUMZjQuloqR+U421hJnF/eMhIUp0yLGbN+gQ YSRwCuwoy2B/J51/i33mu77ttlYQrP2CLSw28+aYTpxUQu4B9kyOuRHZj yNpPJnPLFh5/gvZS+4BduS1vyVgqGOl8KZi9fTNQ5+JjH5hvZF7s3kizO xwubzXLrZJxCsN1ui/k2vH0ac0bT2nVLwBV/kGFKNAIjqjdx62DsTC8e5 A==; IronPort-SDR: QN054/gFapPR1cVtlOEJiZqDlCAwPlb/8EK4TVbBtiJo+3qyFMchAGjiDqjV2M8Kfd2CmIdaci r2DY0PjGpI4g== From: Denis Osterland-Heim To: "dmurphy@ti.com" , "pavel@ucw.cz" , "mark.rutland@arm.com" , "jacek.anaszewski@gmail.com" , "robh+dt@kernel.org" CC: "linux-kernel@vger.kernel.org" , "Denis Osterland-Heim" , "linux-leds@vger.kernel.org" , "devicetree@vger.kernel.org" , "uwe@kleine-koenig.org" Subject: [PATCH v4 3/5] leds: pwm: check result of led_pwm_set() in led_pwm_add() Thread-Topic: [PATCH v4 3/5] leds: pwm: check result of led_pwm_set() in led_pwm_add() Thread-Index: AQHV/1jvS39qjeoyFEyBmW+H8QD2bg== Date: Sat, 21 Mar 2020 08:15:52 +0000 Message-ID: <20200321081321.15614-4-Denis.Osterland@diehl.com> References: <20200321081321.15614-1-Denis.Osterland@diehl.com> In-Reply-To: <20200321081321.15614-1-Denis.Osterland@diehl.com> Accept-Language: de-DE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-mailer: Evolution 3.28.5-0ubuntu0.18.04.1 x-ms-exchange-messagesentrepresentingtype: 1 Content-ID: <7A1628E3E162584FA7567B765B51EAAE@diehl.internal> MIME-Version: 1.0 X-TrailerSkip: 1 X-GBS-PROC: byQFdw3ukCM+zy1/poiPc0+a0+k5FkEP7dU6yETXoKGIxEHL1LfZDAUVf0XEZtAp Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org led_pwm_set() now returns an error when setting the PWM fails. Cc: Uwe Kleine-König Signed-off-by: Denis Osterland-Heim --- drivers/leds/leds-pwm.c | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) -- 2.25.1 Diehl Connectivity Solutions GmbH Geschäftsführung: Horst Leonberger Sitz der Gesellschaft: Nürnberg - Registergericht: Amtsgericht Nürnberg: HRB 32315 ___________________________________________________________________________________________________ Der Inhalt der vorstehenden E-Mail ist nicht rechtlich bindend. Diese E-Mail enthaelt vertrauliche und/oder rechtlich geschuetzte Informationen. Informieren Sie uns bitte, wenn Sie diese E-Mail faelschlicherweise erhalten haben. Bitte loeschen Sie in diesem Fall die Nachricht. Jede unerlaubte Form der Reproduktion, Bekanntgabe, Aenderung, Verteilung und/oder Publikation dieser E-Mail ist strengstens untersagt. - Informationen zum Datenschutz, insbesondere zu Ihren Rechten, erhalten Sie unter https://www.diehl.com/group/de/transparenz-und-informationspflichten/ The contents of the above mentioned e-mail is not legally binding. This e-mail contains confidential and/or legally protected information. Please inform us if you have received this e-mail by mistake and delete it in such a case. Each unauthorized reproduction, disclosure, alteration, distribution and/or publication of this e-mail is strictly prohibited. - For general information on data protection and your respective rights please visit https://www.diehl.com/group/en/transparency-and-information-obligations/ diff --git a/drivers/leds/leds-pwm.c b/drivers/leds/leds-pwm.c index 6caf8bea8cd5..07eab2d8b7c7 100644 --- a/drivers/leds/leds-pwm.c +++ b/drivers/leds/leds-pwm.c @@ -91,15 +91,21 @@ static int led_pwm_add(struct device *dev, struct led_pwm_priv *priv, pwm_init_state(led_data->pwm, &led_data->pwmstate); ret = devm_led_classdev_register(dev, &led_data->cdev); - if (ret == 0) { - priv->num_leds++; - led_pwm_set(&led_data->cdev, led_data->cdev.brightness); - } else { + if (ret) { dev_err(dev, "failed to register PWM led for %s: %d\n", led->name, ret); + return ret; } - return ret; + ret = led_pwm_set(&led_data->cdev, led_data->cdev.brightness); + if (ret) { + dev_err(dev, "failed to set led PWM value for %s: %d", + led->name, ret); + return ret; + } + + priv->num_leds++; + return 0; } static int led_pwm_create_fwnode(struct device *dev, struct led_pwm_priv *priv)