From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Goldschmidt Date: Fri, 11 Jan 2019 11:01:54 +0100 Subject: [U-Boot] [PATCH v1 3/4] serial: add an of-platdata driver for "snps, dw-apb-uart" In-Reply-To: References: <20190107211423.10151-1-simon.k.r.goldschmidt@gmail.com> <20190107211423.10151-4-simon.k.r.goldschmidt@gmail.com> <4881796E12491D4BB15146FE0209CE64681C1E9A@DE02WEMBXB.internal.synopsys.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable To: u-boot@lists.denx.de On Fri, Jan 11, 2019 at 10:22 AM Andy Shevchenko wrote: > > On Wed, Jan 9, 2019 at 10:36 AM Alexey Brodkin > wrote: > > > > Might be a bit na=C3=AFve question but why depend on SPL_OF_PLATDATA on= ly? > > What about CONFIG_OF_EMBED? > > No driver should depend on OF_EMBED IIUC. New U-Boot spils a warning > when production boards are using it. By now, actually I even dislike the idea of depending on OF_PLATDATA myself. Maybe we should fix the OF_PLATDATA to driver matching code to better handle drivers with multiple compatible strings instead of adding dedicated driver= s. Regards, Simon