From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Rini Date: Wed, 25 Jan 2017 07:58:10 -0500 Subject: [U-Boot] [PATCH 1/2] arm: Remove unofficial mach-type number uses In-Reply-To: <20170125082353.10e9d087.albert.aribaud@3adev.fr> References: <1485293505-22940-1-git-send-email-trini@konsulko.com> <9e0bbce3-c0f7-f7e8-2e33-88503896c0eb@mleia.com> <20170124231332.GR22316@bill-the-cat> <20170125082353.10e9d087.albert.aribaud@3adev.fr> Message-ID: <20170125125810.GF22316@bill-the-cat> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, Jan 25, 2017 at 08:23:53AM +0100, Albert ARIBAUD wrote: > Hi Vladimir, > > Le Wed, 25 Jan 2017 03:34:13 +0200, Vladimir Zapolskiy a > ?crit : > > > Hypothetically it could be possible that a machine maintainer finds that > > after rebasing private changes on top of the latest Linux release, a board > > mach type is not recognized by the Linux kernel due to an expired and > > removed entry in mach-types, then s/he updates a record in Russell's DB > > to get the old machine type ID again. However a counterpart change in > > U-boot board config won't be synced automatically, and IMHO it should be > > stated that this problem is neglected, obviously because it causes > > maintenance burden for no gain. > > I suspect there's also a case where a custom (non-mainline) kernel is > used on a given board and expects a machine ID from U-Boot, but this > machine ID is not required to be in RMK's list. Ugly, but such things > happen. > > > That said, your change is good, it may produce a discontent in particular > > cases, but it won't be problematic to remove it by partial commit > > reverting and adding an explanatory comment into the code. > > Agreed. Sigh, I guess we've got enough cases of this that I'll re-work things based on the full mach-types list instead. Thanks folks! -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: Digital signature URL: