From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id ED64E5252 for ; Tue, 19 Sep 2023 13:28:11 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6F1D5C433C7; Tue, 19 Sep 2023 13:28:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1695130091; bh=cgpCq/1XRr4sA5/czdd3pzytIQeVY1Qabe8rnlGEetw=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=EZjTyyCK7kjl9OdIIvMyX53DBnuWzFFOhBDgIdiJtyxuhujEYoXg9ve+t2tYwYOzA EPi7jftAKCpfzMLJcCVDyD2mueiG46cV4RwmEzU5LK6wPBCW1uiWiMdQqarD5qbP5j zlYkfTCUh2JqIP7pZyFDuysIB+RFwj9obLB27hSqB8e5oduWO1gbk1q0eCQP/vq8Ak CowtUWCoslS+GrxT2gzjd8+fi8bpmgBLI/XTMd0wlyT9UZvb2cGvioLPunxwwOMMCW QGxbKkXPJDRwB38s7q+MhX6N8FMXblCRoc/okCswY/8pqHpyr8L5ez9bP8YNtfnclE PXWxnTlB9jJrA== Received: from johan by xi.lan with local (Exim 4.96) (envelope-from ) id 1qiam8-00006U-1U; Tue, 19 Sep 2023 15:28:25 +0200 Date: Tue, 19 Sep 2023 15:28:24 +0200 From: Johan Hovold To: Sasha Levin Cc: linux-kernel@vger.kernel.org, stable@vger.kernel.org, Konrad Dybcio , Bjorn Andersson , agross@kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH AUTOSEL 6.5 30/36] arm64: dts: qcom: sc8280xp-x13s: Add camera activity LED Message-ID: References: <20230908192848.3462476-1-sashal@kernel.org> <20230908192848.3462476-30-sashal@kernel.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Tue, Sep 19, 2023 at 09:06:54AM -0400, Sasha Levin wrote: > On Tue, Sep 19, 2023 at 08:15:04AM +0200, Johan Hovold wrote: > >On Mon, Sep 18, 2023 at 05:41:38PM -0400, Sasha Levin wrote: > >> On Mon, Sep 11, 2023 at 08:33:02AM +0200, Johan Hovold wrote: > >> >On Fri, Sep 08, 2023 at 03:28:41PM -0400, Sasha Levin wrote: > >> >> From: Konrad Dybcio > >> >> > >> >> [ Upstream commit 1c63dd1c5fdafa8854526d7d60d2b741c813678d ] > >> >> > >> >> Disappointigly, the camera activity LED is implemented in software. > >> >> Hook it up as a gpio-led and (until we have camera *and* a "camera on" > >> >> LED trigger) configure it as a panic indicator. > >> >> > >> >> Signed-off-by: Konrad Dybcio > >> >> Link: https://lore.kernel.org/r/20230805-topic-x13s_cam_led-v1-1-443d752158c4@linaro.org > >> >> Signed-off-by: Bjorn Andersson > >> >> Signed-off-by: Sasha Levin > >> > > >> >This is a new feature if anything, not a fix. Please drop from all > >> >autosel queues. > >> > >> Not a feature, but hardware enablement. > > > >Call it what you will, but please drop it. Otherwise by that logic you'd > >need to backport all devicetree patches (as well as most driver changes) > >since they ultimately aim at enabling hardware. > > Not all, only ones that re-use existing kernel driver but enable it for > new hardware (i.e. adding a new pci-id/usb-id/dts entries). Again, that's basically all our device-tree patches. And that can break in all sorts of ways. So again, please drop. This does not belong in stable. Johan