Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Sven Schuchmann <schuchmann@schleissheimer.de>
To:     Pavel Machek <pavel@ucw.cz>
CC:     Dan Murphy <dmurphy@ti.com>, Rob Herring <robh+dt@kernel.org>,
        "linux-leds@vger.kernel.org" <linux-leds@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: AW: [PATCH v1] leds: lp50xx: add setting of default intensity from DT
Thread-Topic: [PATCH v1] leds: lp50xx: add setting of default intensity from
 DT
Thread-Index: AQHW7lFOn/1Jlt56EkOy+ZVk/3cx2qpGlWAAgAAP4bCAABNlgIAABpCggAAHMACAAOx+cA==
Date:   Thu, 4 Feb 2021 07:32:35 +0000
Message-ID: <DB8P190MB0634E7E9EBC92CA6DF7D32E6D9B39@DB8P190MB0634.EURP190.PROD.OUTLOOK.COM>
References: <20210119105312.2636-1-schuchmann@schleissheimer.de>
 <20210203142940.GB12369@duo.ucw.cz>
 <DB8P190MB06348FC85033135BFC3EF5C4D9B49@DB8P190MB0634.EURP190.PROD.OUTLOOK.COM>
 <20210203163555.GA23019@duo.ucw.cz>
 <DB8P190MB0634880713B530F51F95CEE0D9B49@DB8P190MB0634.EURP190.PROD.OUTLOOK.COM>
 <20210203172508.GC23019@duo.ucw.cz>
In-Reply-To: <20210203172508.GC23019@duo.ucw.cz>
Accept-Language: de-DE, en-US
Content-Language: de-DE
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: 4144fc5d-3533-40a4-8496-08d8c8df0a69
X-MS-Exchange-CrossTenant-originalarrivaltime: 04 Feb 2021 07:32:35.0237
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: ba05321a-a007-44df-8805-c7e62d5887b5
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 7rQ/Effehgq0rafZ3jq8xI13NTVPhYPd6CqbjngLNyyBClbe8QaKTBMhIT9MMyFXtQY347xq6H6HQGKf9tXzxhyjxAzbZYujURIyNJtIDMA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6P190MB0503
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Helo Pavel,

> > > Yes, sounds reasonable. Could we get default intensity of 100% on all
> > > channels if nothing else is specified?
> > >
> > > Or maybe simply "if intensity is not specified, start with 100%, and
> > > use explicit =3D0 if other color is expected".
> > >
> > Mh, if someone is already using the led driver and updates to a newer k=
ernel
> > we would then turn on all leds per default to the maximum intensity dur=
ing boot
> > until they are set the way they should be from userspace. I don't know =
if this
> > is what we want? If yes, sure, we could set them to maximum per
> > default.
>=20
> Not really. If they don't have trigger configured, nothing will happen.

Oops, you are right, my fault.

>=20
> > Also if we want to use Percentage Values (%) for setting the intensity
> > I think this should also be done for the userspace interfaces and
> > not only from DT.
>=20
> We don't want to use percentages in the API (but let me still use
> percentages in discussion).

No Problem.

Best Regards,

   Sven