New Arch For Mkimage

I am attempting to use mkimage to build a uImage for a new architecture: csky. This architecture is already in the mainline kernel.
It appears the only thing to do is to added a new enumeration after IH_ARCH_RISCV in u-boot/include/image.h and to add a corresponding entry into uimage_arch in u-boot/common/image.c. Is this correct?
Randy Yates Apex Semiconductors (USA) Company Limited 984-368-8148 (cell, work) 919-412-8994 (cell, personal) Randy Yates randyy@scc-inc.com Apex Semiconductors (USA) Company Limited http://www.apexsemi-usa.com 984-368-8148 (cell, work) 919-412-8994 (cell, personal)
This message contains confidential information. If you are not the intended recipient, notify the sender immediately and delete this e-mail from your system. You should always check e-mail for viruses. Sender does not accept liability for any viruses that may be inadvertently transmitted.

On Mon, Jul 26, 2021 at 02:08:09PM -0400, Randy Yates wrote:
I am attempting to use mkimage to build a uImage for a new architecture: csky. This architecture is already in the mainline kernel.
It appears the only thing to do is to added a new enumeration after IH_ARCH_RISCV in u-boot/include/image.h and to add a corresponding entry into uimage_arch in u-boot/common/image.c. Is this correct?
Sorry for the delay, include/image.h is also a place you'll want to update.

Hey Tom,
I did that. Did you want me to issue a pull request? I'm not sure of the proper process.
--Randy
On 8/2/21 6:01 PM, Tom Rini wrote:
ATTENTION: This Email originated outside SCC. Use caution before clicking on any links or attachments.
On Mon, Jul 26, 2021 at 02:08:09PM -0400, Randy Yates wrote:
I am attempting to use mkimage to build a uImage for a new architecture: csky. This architecture is already in the mainline kernel.
It appears the only thing to do is to added a new enumeration after IH_ARCH_RISCV in u-boot/include/image.h and to add a corresponding entry into uimage_arch in u-boot/common/image.c. Is this correct?
Sorry for the delay, include/image.h is also a place you'll want to update.
This message contains confidential information. If you are not the intended recipient, notify the sender immediately and delete this e-mail from your system. You should always check e-mail for viruses. Sender does not accept liability for any viruses that may be inadvertently transmitted.

On Mon, Aug 02, 2021 at 06:06:37PM -0400, Randy Yates wrote:
Hey Tom,
I did that. Did you want me to issue a pull request? I'm not sure of the proper process.
The general answer is that you'll want to submit a series of patches, likely with git send-email, so that nothing gets mangled, that adds csky support. And you should start with a minimal set of functionality and add to it, just like the Linux kernel support was done I assume. If there's QEMU support as well, updating .azure-pipelines.yml and .gitlab-ci.yml (and tools/docker/Dockerfile) to build and test should also be done. Thanks!
participants (2)
-
Randy Yates
-
Tom Rini