<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text --><style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style>
</head>
<body>
<meta content="text/html; charset=UTF-8">
<style type="text/css" style="">
<!--
p
{margin-top:0;
margin-bottom:0}
-->
</style>
<div dir="ltr">
<div id="x_divtagdefaultwrapper" dir="ltr" style="font-size:12pt; color:#000000; font-family:Calibri,Helvetica,sans-serif">
<p>Hi Otavio</p>
<p><br>
</p>
<p>I found a way by which I can do what I need without having that include file.</p>
<p><br>
</p>
<p>Will drop the patch as proposed in v2.</p>
<p><br>
</p>
<p>Thanks</p>
<p>Max<br>
</p>
</div>
<hr tabindex="-1" style="display:inline-block; width:98%">
<div id="x_divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" color="#000000" style="font-size:11pt"><b>Von:</b> Max Krummenacher<br>
<b>Gesendet:</b> Donnerstag, 7. September 2017 19:50:37<br>
<b>An:</b> otavio.salvador@ossystems.com.br<br>
<b>Cc:</b> meta-freescale@yoctoproject.org; Marcel Ziswiler; Stefan Agner<br>
<b>Betreff:</b> Re: [meta-freescale] [3rdparty][PATCH 07/15] apalis-imx6.conf: add an include file hook</font>
<div> </div>
</div>
</div>
<font size="2"><span style="font-size:10pt;">
<div class="PlainText">Hi Otavio<br>
<br>
Thanks for reviewing the patchset.<br>
<br>
On Wed, 2017-09-06 at 16:26 -0300, Otavio Salvador wrote:<br>
> On Wed, Sep 6, 2017 at 11:12 AM, Max Krummenacher <max.oss.09@gmail.com> wrote:<br>
> > <br>
> > Add a hook to allow to append to the machine configuration from an optional<br>
> > include file.<br>
> > <br>
> > Signed-off-by: Max Krummenacher <max.krummenacher@toradex.com><br>
> <br>
> This is not a good practice. All those can be set on the DISTRO our<br>
> customer layer so please drop this patch for v2.<br>
> <br>
<br>
I really like this hook present. It helps a lot in our downstream BSP where<br>
we combine the common features used for Nvidia Tegra and NXP based computer<br>
modules.<br>
<br>
We for instance build a non SPL U-Boot in addition usable for recovery mode.<br>
<br>
Also as Stefan mentions combining a module with a different carrier board<br>
often only requires a new device-tree file.<br>
<br>
Both are really machine and not distro related, so they logically belong to<br>
the machine configuration.<br>
<br>
Max</div>
</span></font>
</body>
</html>