On Thu, 3 Dec 2020 at 19:11, Hongwei Zhang <hongw...@ami.com> wrote: > > Thanks Joel, > > > From: Joel Stanley <j...@jms.id.au> > > Sent: Wednesday, December 2, 2020 9:17 PM > > To: Hongwei Zhang > > > > Hello Hongwei, > > > > On Wed, 2 Dec 2020 at 19:48, Hongwei Zhang <hongw...@ami.com> wrote: > > > > > > Hello Reviewer, > > > > > > Check for negative return value of fdt_noffset from calling > > > boot_get_fdt_fit(). Otherwise, when fdt subimage is corrupted, the > > > u-boot report bad hash value but continue loading kernel image and > > > get hanged later. > > > > I can see from your subject line that you intended this to go to > > Aspeed's fork of u-boot. That's fine, but you shouldn't cc the > > upstream maintainers as they don't care for these changes. > > > > If the bug exists in mainline u-boot, you should certainly send the > > fix there and ask for it to be backported to the aspeed fork. > > I think this issue existing in mainline u-boot, however, I have difficulty > to include 'master' branch of u-boot in openbmc project, to acturaly test > it. There are compiling errors. > > Also, I'm not sure for mainline u-boot, which branch should I use to test > and verification? is it 'master', or 'next' branch?
I suspect it doesn't matter much. As this is a small fix, I would chose to send it to master. (I see that's what you did) > > --Hongwei > > > > Cheers, > > > > Joel > > > > > > > > Hongwei Zhang (1): > > > Common:fdt: Check for error return value > > > > > > common/image-fdt.c | 3 +++ > > > 1 file changed, 3 insertions(+) > > > > > > -- > > > 2.17.1 > > >