From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sebastian Reichel Date: Thu, 30 Nov 2017 22:06:33 +0000 Subject: Re: [PATCH v3 4/4] DTS: Pandora: fix panel compatibility string Message-Id: <20171130220633.imkfyeiqtftnw4j4@earth> MIME-Version: 1 Content-Type: multipart/mixed; boundary="nt4vnptky4atvscp" List-Id: References: <20171128160437.GD28152@atomide.com> <20171128161834.GE28152@atomide.com> <4EE1E298-6461-48FF-977C-958DD16AD83A@goldelico.com> <20171130152430.GM28152@atomide.com> In-Reply-To: <20171130152430.GM28152@atomide.com> To: Tony Lindgren Cc: Mark Rutland , DTML , linux-fbdev@vger.kernel.org, Discussions about the Letux Kernel , Bartlomiej Zolnierkiewicz , David Airlie , "H. Nikolaus Schaller" , dri-devel@lists.freedesktop.org, Russell King , Rob Herring , Linux Kernel Mailing List , Julia Lawall , Tomi Valkeinen , Thierry Reding , Laurent Pinchart , =?iso-8859-1?Q?Beno=EEt?= Cousson , kernel@pyra-handheld.com, linux-omap , Linux ARM --nt4vnptky4atvscp Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable Hi, On Thu, Nov 30, 2017 at 07:24:30AM -0800, Tony Lindgren wrote: > * H. Nikolaus Schaller [171128 18:35]: > > Hi, > >=20 > > > Am 28.11.2017 um 17:18 schrieb Tony Lindgren : > > >=20 > > > * H. Nikolaus Schaller [171128 16:17]: > > >> Hi Tony, > > >>=20 > > >>> Am 28.11.2017 um 17:04 schrieb Tony Lindgren : > > >>>=20 > > >>> * H. Nikolaus Schaller [171128 15:52]: > > >>>> We can remove the unnecessary "omapdss," prefix because > > >>>> the omapdrm driver takes care of it when matching with > > >>>> the driver table. > > >>>=20 > > >>> So is this needed as a fix or is this another clean-up? > > >>>=20 > > >>> So is this is really needed as a fix? > > >>=20 > > >> Hm. How do you differentiate between "fix" and "cleanup"? > > >> Maybe it is more a wording than a content issue... > > >>=20 > > >> For me it is a "fix" because it is semantically wrong to have > > >> a prefix where it is not needed. And "fixing" it changes the > > >> compiler output by 8 bytes. > > >=20 > > > How about let's call it a "typo fix" then? :) > >=20 > > Well, it is not really a typo. >=20 > Well what if the stable people pick it into earlier stable series > based on the word fix in the subject? That has happened before. >=20 > I suggest you update the dts patches to use wording like > "update compatible to use new naming" or something similar. Patch 4/4 is a Fix and should be applied to stable trees. "omapdss," prefix was never supposed to be in the DTS files, is not supposed to be in there now and will break some time in the future. Explanation: The early init of omapdss adds the prefix at runtime, so that the binding can use generic properties and the kernel can use omapdss specific drivers until the generic ones can be used. -- Sebastian --nt4vnptky4atvscp Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEE72YNB0Y/i3JqeVQT2O7X88g7+poFAloggOYACgkQ2O7X88g7 +poY8w/+LukAeMDv10ezogc+xAbg7rWzErh43W2ZI/pibY0rEKuuq0yNmYsyjzov tfDTL91fM8soDgX2+rG1E4NOdZgolNqsqbvAB4TXWrzW0ORyjdbG+D2/1rgKBfY+ h8Dn/66QNuUd6+eTx+6AANgMzjNzHLRJwoM0eISYFCU3L6EXB8YFqoXApUqY5LBg do3v5DFqO+6obHUGJ8oJTjapCIEiIMsQ4DLFbLPIiAAJY48yNt8oWzRldcIDEzBp s1Cs7M+Zi5PbZDl8Q2YC19Q4YcMONR03VK3W+98T6P9dl0+Al0UW1YQhw3ZaqVih juCgcINTbVIgxI0ISzfJfiNOmOea5lLp8o++upAhhsnP2tEZHjyGe97OAc8S+ETx a8mjNYtKsFvBVHsh/d52PIa+H7NoW3UBkQWZsxyfpzqhibJAAN5O4DJHQXpdq7uM ce4IOR/LK7RDIGhTneW2vR//fwBMYWrQspfYTvx24N6mqe/gyw3PnYQjAsJNSXjC YBA4T99apk6BuXiRNzM7mpbUjOKrgGNUtBQKAKD9YUcAm+HfxMPdBiAwQFHlLpVX +BMBxARqPy3gHGoQtUyg/TlTVHRPbaF0uhQLoXuBG7Awe5CqK0BE4+Dht8ogLIQ4 4FAn98DRUhlOEs724cOGwA4DlTdBziaGuSaE9Q4r/1IuhLGaUXw= =LhAU -----END PGP SIGNATURE----- --nt4vnptky4atvscp--