From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 18176C6FD1D for ; Mon, 20 Mar 2023 16:34:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Subject:Cc:To:From:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=fWRvaEh+wHTry9H/u9pg9H0T3x3MFE8emVZqhFzpQpQ=; b=uc+zzskFvlPvwo WWtFbN/86SL37VFgFHofK2HWyKpzlo4E8e8Lg7rpOj9KMp1puHW0fLDKnegrmdctBRjW4AnEsId+b RU6BGs55J0mWCjV3QSOgaByITa770CPIYffe/glI67zt3yWHAK/EAPIM46M2OPRJY3LZ/kxtBbUcV HYTXnue4j4ZL7EoKpUWBZbp/GfTICGTcjR/etMhs4q+KlKiACG6sf+SFc3Xhw0KuztWphl36acT1b NazBpio65quQypxuBYQsgYw+R61KTHyVM93ibQ8IPETIyIgQH0f3ZtvtPy8KkU0DS3L16TWBgEP/P nipVFDkaVHT7Iec+k2bg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1peISS-009ncL-0U; Mon, 20 Mar 2023 16:34:04 +0000 Received: from mail-wm1-x331.google.com ([2a00:1450:4864:20::331]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1peISP-009nan-0E for linux-arm-kernel@lists.infradead.org; Mon, 20 Mar 2023 16:34:02 +0000 Received: by mail-wm1-x331.google.com with SMTP id i10-20020a05600c354a00b003ee0da1132eso1308746wmq.4 for ; Mon, 20 Mar 2023 09:34:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; t=1679330039; h=in-reply-to:content-disposition:mime-version:references:subject:cc :to:from:date:message-id:from:to:cc:subject:date:message-id:reply-to; bh=cX8KlgrWKmB3FXjHkapylAo4MCiAP3BWza2EtPBzxYU=; b=eB866SGqHanTUqtUTJuZGjwSngVrG6odH5UrI3TYammU7sWeqF8Q9qzUOsXltCwNtt kRGYYT23O0ENMTWCQr89/lHlOB7ihamXET6n91ty11GCX8/D5EQcGHv4bzfR0WaJAXhs vNt2YEEiEgfanzTqTN9lmhLBmhs0FxR1Yq5JfCjhRWRVCMmmzQ7zGCQnoEiDscieDg9A uuLGjjCeuE/hak0YLf45HdVWSFJah5uh2tjh5KjGE5zEy6zzmrhijqlTkJmDWTMfW/ts 8D9NkZ9fHHBEH/8k/GtX5YpbASkPU3kxDIQnRK4uynfbdb7yXVk4C8T28Qg37GagO8PE WUzw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1679330039; h=in-reply-to:content-disposition:mime-version:references:subject:cc :to:from:date:message-id:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=cX8KlgrWKmB3FXjHkapylAo4MCiAP3BWza2EtPBzxYU=; b=ONO/RAmqOOAWa8l3j9IKWuv4mEAi/piyq6kwvgECeEOd+5AW7nzHwVwrtNsvyKvbXi sb2TzQ1eYyL5pjgesx+Ft8ZSSsziF+f4WNpKbJC/+Egn3TTB4C8fHyu+waaZldSAi32J M32/0DR0S0++yuUILQoa6E5bUZUr0P+k7GK8FiJnd3kvPtJI1EUhpluxUGXGt8PfgI18 DdT+COfLoMH/9pAUwbPdmCJ5iyLnz+FB2LQB/hA1wn7JB/SM26Z3JsnX82ONXFP2+p4o gt+nmp31pw8Z4nWJm0tYQVhIBohJ7zsHK2JsnSALFYdynOLTfZ9xw3LgYLfSKPct0wZw nrgQ== X-Gm-Message-State: AO0yUKXxJe1WVhBwWoJXdP2yNkzX/rGhPjSSY2sL70PlzI3mv7O0e2Jy kayQuufCOup0vG+XzdTymt8= X-Google-Smtp-Source: AK7set9o+8tkdMu+iaV/b3SeymKtnfydzAmGbhL7FTYNudrs6Z0syHqO0oNJnFGDIXVqSYGwXvTwaQ== X-Received: by 2002:a05:600c:2114:b0:3ed:290b:dc68 with SMTP id u20-20020a05600c211400b003ed290bdc68mr151068wml.12.1679330038574; Mon, 20 Mar 2023 09:33:58 -0700 (PDT) Received: from Ansuel-xps. (93-34-89-197.ip49.fastwebnet.it. [93.34.89.197]) by smtp.gmail.com with ESMTPSA id p11-20020a05600c1d8b00b003daffc2ecdesm16865837wms.13.2023.03.20.09.33.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Mar 2023 09:33:58 -0700 (PDT) Message-ID: <64188af6.050a0220.c5fe1.1d96@mx.google.com> X-Google-Original-Message-ID: Date: Mon, 20 Mar 2023 17:33:56 +0100 From: Christian Marangi To: Michal Kubiak Cc: Andrew Lunn , Florian Fainelli , Vladimir Oltean , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Rob Herring , Krzysztof Kozlowski , Heiner Kallweit , Russell King , Gregory Clement , Sebastian Hesselbarth , Andy Gross , Bjorn Andersson , Konrad Dybcio , Pavel Machek , Lee Jones , John Crispin , netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-leds@vger.kernel.org Subject: Re: [net-next PATCH v5 02/15] net: dsa: qca8k: add LEDs basic support References: <20230319191814.22067-1-ansuelsmth@gmail.com> <20230319191814.22067-3-ansuelsmth@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230320_093401_128395_A00BFB55 X-CRM114-Status: GOOD ( 26.00 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Mon, Mar 20, 2023 at 05:30:05PM +0100, Michal Kubiak wrote: > On Sun, Mar 19, 2023 at 08:18:01PM +0100, Christian Marangi wrote: > > Add LEDs basic support for qca8k Switch Family by adding basic > > brightness_set() support. > > > > Since these LEDs refelect port status, the default label is set to > > ":port". DT binding should describe the color, function and number of > > the leds using standard LEDs api. > > > > These LEDs supports only blocking variant of the brightness_set() > > function since they can sleep during access of the switch leds to set > > the brightness. > > > > While at it add to the qca8k header file each mode defined by the Switch > > Documentation for future use. > > > > Signed-off-by: Christian Marangi > > --- > > Hi Christian, > > The patch looks good to me. I just found one nitpick in the comment. > > Thanks, > Michal > > > +static int > > +qca8k_parse_port_leds(struct qca8k_priv *priv, struct fwnode_handle *port, int port_num) > > +{ > > + struct fwnode_handle *led = NULL, *leds = NULL; > > + struct led_init_data init_data = { }; > > + enum led_default_state state; > > + struct qca8k_led *port_led; > > + int led_num, led_index; > > + int ret; > > + > > + leds = fwnode_get_named_child_node(port, "leds"); > > + if (!leds) { > > + dev_dbg(priv->dev, "No Leds node specified in device tree for port %d!\n", > > + port_num); > > + return 0; > > + } > > + > > + fwnode_for_each_child_node(leds, led) { > > + /* Reg represent the led number of the port. > > + * Each port can have at least 3 leds attached > > Nitpick: "at least" -> "at most" > Oh god... I added the extra comment and totally missed the other. Sorry! Btw ok for the description of the LED mapping? It's a bit complex so tried to do my best to describe them. -- Ansuel _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel