<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/arch/powerpc/cpu, branch v2022.07</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.</subtitle>
<id>http://cgit.235523.xyz/u-boot.git/atom/arch/powerpc/cpu?h=v2022.07</id>
<link rel='self' href='http://cgit.235523.xyz/u-boot.git/atom/arch/powerpc/cpu?h=v2022.07'/>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<updated>2022-05-20T10:36:47Z</updated>
<entry>
<title>caam: Fix crash in case caam_jr_probe failed</title>
<updated>2022-05-20T10:36:47Z</updated>
<author>
<name>Ye Li</name>
<email>ye.li@nxp.com</email>
</author>
<published>2022-05-11T08:26:20Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=cda8f87349594a1df9cc8616e25944173c9d0a0c'/>
<id>urn:sha1:cda8f87349594a1df9cc8616e25944173c9d0a0c</id>
<content type='text'>
If probing caam_jr returns failure, the variable "dev" will not be
initialized, so we can't use dev-&gt;name for the error print.
Otherwise it will cause crash.

Signed-off-by: Ye Li &lt;ye.li@nxp.com&gt;
Reviewed-by: Gaurav Jain &lt;gaurav.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Define linker sections in ascending order</title>
<updated>2022-04-26T11:48:39Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-05T13:12:35Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1245c6e2582cb67db2a84e60a61cfe5a8d77e068'/>
<id>urn:sha1:1245c6e2582cb67db2a84e60a61cfe5a8d77e068</id>
<content type='text'>
It is too confusing if sections are defined in non-ascending order.

Also linker has to go backward and then again forward when generating final
binary.

To make future changes easier, define all linker sections in ascending
order.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Reviewed-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Put bss after image when not including reset vector</title>
<updated>2022-04-26T11:48:39Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-05T13:12:34Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=1b780b654f20055f40a049da4f2ef12207757690'/>
<id>urn:sha1:1b780b654f20055f40a049da4f2ef12207757690</id>
<content type='text'>
Linker knows exact size of the image, so there is no need to use
CONFIG_SYS_MONITOR_LEN macro (which should be upper limit).

Remove usage of CONFIG_SYS_MONITOR_LEN macro to simplify setup.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Reviewed-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Remove useless SIZEOF_HEADERS and .interp from ld script</title>
<updated>2022-04-26T11:48:39Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-05T13:12:33Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=c45f000f294e096f6f5060bd528b65d14f6c6f12'/>
<id>urn:sha1:c45f000f294e096f6f5060bd528b65d14f6c6f12</id>
<content type='text'>
.interp section is not available in output ELF binary and SIZEOF_HEADERS is
needed at all.

There is no change in generated u-boot.bin binary.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Reviewed-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Remove duplicate u-boot-nand.lds</title>
<updated>2022-04-26T11:48:39Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-25T08:51:20Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ab37df9d221e12adf709c7391c001b9dd5d33606'/>
<id>urn:sha1:ab37df9d221e12adf709c7391c001b9dd5d33606</id>
<content type='text'>
Signed-off-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Remove u-boot-nand_spl.lds</title>
<updated>2022-04-26T11:48:39Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-25T04:02:03Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=0038f2362a24478a6c9a7fb186887851f2b32512'/>
<id>urn:sha1:0038f2362a24478a6c9a7fb186887851f2b32512</id>
<content type='text'>
mpc85xx SPL NAND linker script u-boot-nand_spl.lds is not used since
Jun 2014 commit 0234446fd171 ("nand_spl: remove MPC8536DS support").

Remove it.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
[Rebased]
Signed-off-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Fix CONFIG_OF_SEPARATE support</title>
<updated>2022-04-26T11:48:39Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-05T13:12:30Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=e8c0e0064c8a7b0665fd1393c2e3266f80e884a8'/>
<id>urn:sha1:e8c0e0064c8a7b0665fd1393c2e3266f80e884a8</id>
<content type='text'>
Currently CONFIG_OF_SEPARATE is completely broken and U-Boot for some
mpc85xx board (e.g. P2020) has to be compiled with CONFIG_OF_EMBED.
Otherwise it crashes during early init.

When debug console is enabled and all debug logging options are turned on
then U-Boot on P2020 with CONFIG_OF_SEPARATE prints following error:

  No valid device tree binary found at 110dc300
  initcall sequence 110d3560 failed at call 1109535c (err=-1)
  ### ERROR ### Please RESET the board ###

Problem is with appended DTB. When CONFIG_SYS_MPC85XX_NO_RESETVEC is set
U-Boot binary image without DTB ends immediately after the .u_boot_list
section. At this position is defined _end symbol at which U-Boot expects
start of the appended DTB.

Problem is that after .u_boot_list section are in linker script defined
another sections with 256 byte long padding which are completely empty.
During conversion of U-Boot ELF binary to RAW binary u-boot-nodtb.bin,
objcopy removes trailing zero padding and therefore DTB is appended at
wrong position.

Changing alignment from 256 bytes to 4 bytes fixes this issue. And appended
DTB is finally at he correct position. With this fix U-Boot on P2020 with
CONFIG_OF_SEPARATE option starts working again.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Reviewed-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Show e500 core version</title>
<updated>2022-04-26T11:48:39Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-02T22:05:10Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=efd9914ffa05dc89f7d2830267b27ba2f06ecad1'/>
<id>urn:sha1:efd9914ffa05dc89f7d2830267b27ba2f06ecad1</id>
<content type='text'>
Distinguish between e500v1 and e500v2.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Reviewed-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Rename _start_e500 symbol to _start</title>
<updated>2022-04-26T11:48:39Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-02T22:05:09Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=de14a5a95c14b0a5349c430ea35b8b07975f7ce5'/>
<id>urn:sha1:de14a5a95c14b0a5349c430ea35b8b07975f7ce5</id>
<content type='text'>
The real entry point is _start_e500. There is no _start symbol at all. So
rename _start_e500 to _start for convension that _start symbol is used as
entry point.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
Reviewed-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
<entry>
<title>powerpc: mpc85xx: Drop _start symbol</title>
<updated>2022-04-26T11:48:33Z</updated>
<author>
<name>Pali Rohár</name>
<email>pali@kernel.org</email>
</author>
<published>2022-04-25T03:59:08Z</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=2dcf776ebcf7f224f2a2e6720e863aca493a5cd9'/>
<id>urn:sha1:2dcf776ebcf7f224f2a2e6720e863aca493a5cd9</id>
<content type='text'>
_start symbol contains only 32-bit data number 0x27051956 despite it is
marked as text section. This magic number is IH_MAGIC which is used for
marking uboot image header.

mpc85xx start.S code does not define valid uboot image header, so IH_MAGIC
number in _start symbol is useless there.

Moreover this _start symbol is not used at all. Entry point is at symbol
_start_e500.

So because this _start symbol is not used for anything, completely remove
it with IH_MAGIC number. After _start symbol was _start_cont symbol, so
replace all relative address calculations by _start_cont.

Signed-off-by: Pali Rohár &lt;pali@kernel.org&gt;
[Rebased]
Signed-off-by: Priyanka Jain &lt;priyanka.jain@nxp.com&gt;
</content>
</entry>
</feed>
