<div dir="ltr">Hi Andrew,<div>Thanks for your response.</div><div><br></div><div>It's really missing from the DTS.</div><div><br></div><div>I met the issue after upgrading to Yocto 2.5.</div><div>The devtool worked well previously.<br></div><div><br></div><div>I tried other recipe, e.g. 'devtool modify u-boot-aspeed'</div><div>I could see the commits as expected.</div><div><br></div><div>-Henbin</div></div><div class="gmail_extra"><br><div class="gmail_quote">2018-08-13 8:36 GMT+08:00 Andrew Jeffery <span dir="ltr"><<a href="mailto:andrew@aj.id.au" target="_blank">andrew@aj.id.au</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Henbin,<br>
<div><div class="h5"><br>
On Fri, 10 Aug 2018, at 11:09, Henbin Chang wrote:<br>
> Thanks Joel.<br>
> <br>
> The issue could be duplicated following the steps in the romulus machine.<br>
> <br>
> 1. 'devtool modify linux-aspeed'<br>
> 2. 'cd workspace/sources/linux-<wbr>aspeed/' and edit<br>
> 'arch/arm/boot/dts/aspeed-bmc-<wbr>opp-romulus.dts'<br>
> 3. 'git add -A' and 'git commit -m 'Test modify dts' '<br>
> 4. go to build directory and 'devtool update-recipe -m patch -a<br>
> ../meta-openbmc-machines/meta-<wbr>openpower/meta-ibm/meta-<wbr>romulus/ linux-<br>
> aspeed'<br>
> 5. Confirm the below files are generated.<br>
> meta-openbmc-machines/meta-<wbr>openpower/meta-ibm/meta-<wbr>romulus/recipes-<br>
> kernel/linux/linux-aspeed/<wbr>0001-Test-modify-dts.patch<br>
> meta-openbmc-machines/meta-<wbr>openpower/meta-ibm/meta-<wbr>romulus/recipes-<br>
> kernel/linux/linux-aspeed_git.<wbr>bbappend<br>
> 6. 'devtool reset linux-aspeed' and remove ' workspace/sources/linux-<br>
> aspeed/<br>
> '<br>
> 7. 'devtool modify linux-aspeed' again<br>
> 8. 'cd workspace/sources/linux-<wbr>aspeed/ ' and the commit with the 'Test<br>
> modify dts' message not found while 'git log'<br>
> 9. 'git branch'<br>
>   dev<br>
>   dev-4.10<br>
>   dev-4.13<br>
>   dev-4.17<br>
>   dev-4.3<br>
>   dev-4.4<br>
>   dev-4.6<br>
>   dev-4.7<br>
> * devtool<br>
>   devtool-no-overrides<br>
>   devtool-override-df-obmc-ubi-<wbr>fs<br>
>   master<br>
> 10. 'git checkout dev-4.17' and 'git log'<br>
> <br>
> commit 2310638091e7b4dd97014f1b373a9b<wbr>a111ac4e58<br>
> Author: Henbin.Chang <<a href="mailto:henbin-chang@mic.com.tw">henbin-chang@mic.com.tw</a>><br>
> Date:   Thu Aug 9 16:15:48 2018 +0800<br>
> <br>
>     Test modify dts<br>
> <br>
> I had no idea why the patch was not applied in the 'devtool' branch.<br>
<br>
</div></div>My understanding of devtool isn't great, however if you patch the<br>
source this way I'm somewhat confident it _doesn't_ do a `git am`<br>
to commit the patch, rather it just uses patch(1) to modify the source<br>
in-place. As such you'll just have a dirty working tree, not modified<br>
history.<br>
<br>
Is the actual change missing from the DTS?<br>
<br>
Cheers,<br>
<br>
Andrew<br>
</blockquote></div><br></div>