Re: [PATCH 04/23] ARM: dts: tegra20-tamonten: add label to avdd_vdac regulator

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: luca.ceresoli@xxxxxxxxxxx, David Airlie <airlied@xxxxxxxxx>, Daniel Vetter <daniel@xxxxxxxx>, Rob Herring <robh+dt@xxxxxxxxxx>, Krzysztof Kozlowski <krzysztof.kozlowski+dt@xxxxxxxxxx>, Thierry Reding <thierry.reding@xxxxxxxxx>, Jonathan Hunter <jonathanh@xxxxxxxxxx>, Sowjanya Komatineni <skomatineni@xxxxxxxxxx>, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>, Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>, Hans Verkuil <hverkuil-cisco@xxxxxxxxx>, Dmitry Osipenko <digetx@xxxxxxxxx>
Subject: Re: [PATCH 04/23] ARM: dts: tegra20-tamonten: add label to avdd_vdac regulator
From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
Date: Tue, 15 Nov 2022 13:16:08 +0100
Cc: linux-media@xxxxxxxxxxxxxxx, linux-tegra@xxxxxxxxxxxxxxx, dri-devel@xxxxxxxxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, linux-staging@xxxxxxxxxxxxxxx, Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx>, Richard Leitner <richard.leitner@xxxxxxxxxxx>
In-reply-to: <20221109141852.729246-5-luca.ceresoli@bootlin.com>
References: <20221109141852.729246-1-luca.ceresoli@bootlin.com> <20221109141852.729246-5-luca.ceresoli@bootlin.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2

On 09/11/2022 15:18, luca.ceresoli@xxxxxxxxxxx wrote:
> From: Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx>
> 
> Useful to enable it from a board DTS.
> 

No, it does not make sense. Adding unused labels is not useful. This
must come with an user.

Best regards,
Krzysztof

References:
- [PATCH 00/23] Add Tegra20 parallel video input capture
  - From: luca . ceresoli
- [PATCH 04/23] ARM: dts: tegra20-tamonten: add label to avdd_vdac regulator
  - From: luca . ceresoli

Prev by Date: Re: [PATCH 02/23] dt-bindings: display: tegra: vi: add 'vip' property and example
Next by Date: Re: [PATCH 03/23] ARM: dts: tegra20: add label to nvidia,tegra20-vi node
Previous by thread: [PATCH 04/23] ARM: dts: tegra20-tamonten: add label to avdd_vdac regulator
Next by thread: [PATCH 05/23] staging: media: tegra-video: fix .vidioc_enum_fmt_vid_cap to return all formats
Index(es):
- Date
- Thread

[Index of Archives] [Linux Driver Development] [Linux Driver Backports] [DMA Engine] [Linux GPIO] [Linux SPI] [Video for Linux] [Linux USB Devel] [Linux Coverity] [Linux Audio Users] [Linux Kernel] [Linux SCSI] [Yosemite Backpacking]