New Arch For Mkimage

New Arch For Mkimage

[Randy Yates]

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.

Re: New Arch For Mkimage

[Tom Rini]

[-- Attachment #1: Type: text/plain, Size: 508 bytes --]

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.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

Re: [EXTERNAL] Re: New Arch For Mkimage

[Randy Yates]

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.

Re: [EXTERNAL] Re: New Arch For Mkimage

[Tom Rini]

[-- Attachment #1: Type: text/plain, Size: 636 bytes --]

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!

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]