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 X-Spam-Level: X-Spam-Status: No, score=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham 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 F0734C4360F for ; Thu, 4 Apr 2019 18:59:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id CE8E02075E for ; Thu, 4 Apr 2019 18:59:20 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729887AbfDDS7U (ORCPT ); Thu, 4 Apr 2019 14:59:20 -0400 Received: from atrey.karlin.mff.cuni.cz ([195.113.26.193]:59826 "EHLO atrey.karlin.mff.cuni.cz" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729789AbfDDS7U (ORCPT ); Thu, 4 Apr 2019 14:59:20 -0400 Received: by atrey.karlin.mff.cuni.cz (Postfix, from userid 512) id A7DD8802D8; Thu, 4 Apr 2019 20:59:08 +0200 (CEST) Date: Thu, 4 Apr 2019 20:59:19 +0200 From: Pavel Machek To: Dmitry Torokhov Cc: Nick Crews , Guenter Roeck , Enric Balletbo i Serra , Benson Leung , linux-leds@vger.kernel.org, Jacek Anaszewski , Alexandre Belloni , Alessandro Zummo , linux-rtc@vger.kernel.org, linux-kernel , Duncan Laurie , Simon Glass Subject: Re: [PATCH v5 3/3] platform/chrome: Standardize Chrome OS keyboard backlight name Message-ID: <20190404185919.GB27340@amd> References: <20190404171007.160878-1-ncrews@chromium.org> <20190404171007.160878-3-ncrews@chromium.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="DKU6Jbt7q3WqK7+M" Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-rtc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-rtc@vger.kernel.org --DKU6Jbt7q3WqK7+M Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu 2019-04-04 11:55:27, Dmitry Torokhov wrote: > On Thu, Apr 4, 2019 at 11:41 AM Nick Crews wrote: > > > > On Thu, Apr 4, 2019 at 11:43 AM Dmitry Torokhov wrote: > > > > > > On Thu, Apr 4, 2019 at 10:36 AM Guenter Roeck wro= te: > > > > > > > > On Thu, Apr 4, 2019 at 10:11 AM Nick Crews wr= ote: > > > > > > > > > > We want all backlights for the system keyboard to > > > > > use a common name, so the name "platform::kbd_backlight" > > > > > would be better than the current "chromeos::kbd_backlight" > > > > > name. Normally this wouldn't be worth changing, but the new > > > > > Wilco keyboard backlight driver uses the "platform" name. > > > > > We want to make it so all Chrome OS devices are consistent, > > > > > so we'll change the name here too. > > > > > > > > > > > > > Wondering - who is the "we" you are talking about ? > > > > You're right, I should have been more precise. > > I was referring to Pavel, Enric, and myself. Pavel had this opinion her= e: > > https://lkml.org/lkml/2019/4/4/1040. I don't know what Pavel meant by "= we" > > in that comment, but I would guess that could mean the other LED mainta= iners > > as well? I talked with Enric 1:1 and he didn't see the problem with cha= nging it, > > though perhaps he was only considering our use of the LED via powerd, > > and not users in general. I'm guessing Pavel's and Enric's meanings tho= ugh, > > excuse me if I am misinterpreting. > > > > > > > > This also has a potential of breaking existing setups if somebody did > > > happen to match on entire name instead of suffix. Such changes have to > > > be considered very carefully; at this point I am against of doing > > > this. > > > > Would it make sense to keep the old name as is, and only make the new > > Wilco name begin with "platform:"? What would you think is best? >=20 > Given that we do not have a single instance of platform::kbd_backlight > in kernel at this time I have no idea why Pavel is trying to push this > for Wilco driver. See the documentation in the email I sent few seconds ago. I hope it explains my reasoning, if not, I'll explain it. Pavel --=20 (english) http://www.livejournal.com/~pavelmachek (cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blo= g.html --DKU6Jbt7q3WqK7+M Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iEYEARECAAYFAlymVAcACgkQMOfwapXb+vKuIgCfQ4vAKLf3CJRwuVbZyvRR1gjE /FEAoIV1bUD24tCqtjsj6OhV3dG0cvrF =SDUB -----END PGP SIGNATURE----- --DKU6Jbt7q3WqK7+M--