Pull request: u-boot-imx u-boot-imx-20210720

Peng Fan peng.fan at nxp.com
Wed Jul 21 12:23:12 CEST 2021


Sorry for non-text top posting.

The imx8ulp patch set has pin headers that would have lots warning, but we have agreement in Linux kernel that they are acceptable. Or you need we fix the line 100 warning?

Are there others blocking the patch set?

Thanks,
Peng

获取 Outlook for iOS<https://aka.ms/o0ukef>
________________________________
发件人: Stefano Babic <sbabic at denx.de>
发送时间: 星期三, 七月 21, 2021 4:17 下午
收件人: Frieder Schrempf; Tom Rini; Stefano Babic; Peng Fan
抄送: U-Boot at lists.denx.de
主题: Re: Pull request: u-boot-imx u-boot-imx-20210720

On 21.07.21 10:07, Frieder Schrempf wrote:
> Hi Tom, Stefano,
>
> On 20.07.21 19:45, Tom Rini wrote:
>> On Tue, Jul 20, 2021 at 05:17:59PM +0200, Stefano Babic wrote:
>>
>>> Hi Tom,
>>>
>>> please pull from u-boot-imx, thanks !
>>>
>>> The following changes since commit f929ce50727bf1019323d6c199dfd3a5755c5474:
>>>
>>>    Merge branch '2021-07-16-cleanup-image-support' (2021-07-17 11:39:50
>>> -0400)
>>>
>>> are available in the Git repository at:
>>>
>>>    https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.denx.de%2Fu-boot%2Fcustodians%2Fu-boot-imx.git&data=04%7C01%7Cpeng.fan%40nxp.com%7Cd5498530d41f46e0a1a008d94c1ffe53%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637624522555859574%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=dfn7EQ4vhiItrVkENd41PB184QHunZicYzRkJIx2CbQ%3D&reserved=0
>>> tags/u-boot-imx-20210720
>>>
>>> for you to fetch changes up to bd8e26606db62a0434c7f09adfa6ec666b36985b:
>>>
>>>    imx: imx8mm: Add support for Kontron Electronics SL/BL i.MX8M-Mini boards
>>> (N801x) (2021-07-19 23:30:47 +0200)
>>>
>>
>> NAK:
>>
>> ERROR: fdt or initrd relocation disabled at boot time
>> #2019: FILE: include/configs/kontron_common.h:67:
>> +       "fdt_high=0xffffffff\0" \
>>
>> And this is likely a bad idea too:
>> ERROR: fdt or initrd relocation disabled at boot time
>> #2020: FILE: include/configs/kontron_common.h:68:
>> +       "initrd_high=0xffffffff\0" \
>>
>> but I haven't looked at all of include/configs/kontron_common.h to make
>> sure it follows all of the layout stuff documented for example in:
>> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fsource.denx.de%2Fu-boot%2Fu-boot%2F-%2Fblob%2Fmaster%2Finclude%2Fconfigs%2Fti_armv7_common.h%23L32&data=04%7C01%7Cpeng.fan%40nxp.com%7Cd5498530d41f46e0a1a008d94c1ffe53%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637624522555859574%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=HrWs4G%2BxojHh7taV%2FG4Zhxzgvf3xlxC0mVEXKxI0hm0%3D&reserved=0
>> (which is also true for aarch64 best I can tell from
>> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.kernel.org%2Fdoc%2FDocumentation%2Farm64%2Fbooting.rst&data=04%7C01%7Cpeng.fan%40nxp.com%7Cd5498530d41f46e0a1a008d94c1ffe53%7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637624522555859574%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=AKcEbcYb1qRwaisd%2FwXIqX6ClVAYaTiPfWAchsk1HCg%3D&reserved=0).
>>
>> And since I'm noting things, checkpatch has a whole bunch of formatting
>> issues throughout as well (kontron and generic imx8 code both).
>
> Sorry for the mess! I have fixed these issues and just sent a v4 for the Kontron board support patches.

Thanks ! I have reverted (dropped) patches from u-boot-imx, we start again.

Best regards,
Stefano


--
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================


More information about the U-Boot mailing list