Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-11 Thread Alan Cudmore
Looks good - I generated the HTML and PDF versions. Thanks, Alan On Tue, Apr 11, 2023 at 5:05 PM Joel Sherrill wrote: > Sorry. I didn't realise there were that many revisions and when I searched > my inbox I missed it > > I reverted V2 and applied V6. Hopefully ok now > > On Tue, Apr 11, 2023,

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-11 Thread Joel Sherrill
Sorry. I didn't realise there were that many revisions and when I searched my inbox I missed it I reverted V2 and applied V6. Hopefully ok now On Tue, Apr 11, 2023, 2:54 PM Alan Cudmore wrote: > Sorry, meant to reply to the list too.. I think you pushed v2 of my patch. > I was up to v6 that

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-11 Thread Alan Cudmore
Sorry, meant to reply to the list too.. I think you pushed v2 of my patch. I was up to v6 that fixed a bunch of issues. What is the best way to fix this? Rewind, or I can submit a new patch based on this. Thanks, Alan On Tue, Apr 11, 2023, 2:29 PM Joel Sherrill wrote: > Pushed. Thanks. > >

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-11 Thread Joel Sherrill
Pushed. Thanks. Please check that it looks good to you. --joel On Fri, Mar 31, 2023 at 11:15 AM Alan Cudmore wrote: > This patch adds the documentation for building and running RTEMS on the > Kendryte K210 > RISC-V SoC. The generic riscv introducion was re-arranged to list the > multilib

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-03 Thread Chris Johns
On 4/4/2023 12:55 am, Alan Cudmore wrote: > Hi Chris, > Sorry I should have read that and formatted my changes correctly. I submitted > a > v5 patch, but as I sent it, I realized that there are a couple of places where > the RTEMS version macro can be embedded rather than hard-coding "6". I

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-03 Thread Alan Cudmore
I submitted a v4 patch for the user/bsps/bsps-riscv.rst page. I improved the instructions for running on Renode by providing a .resc file that would work, rather than modifying an existing script that was used to run Linux. This should work until I have some time to figure out why the

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-03 Thread Chris Johns
Hi Alan, We have a standard for the documentation source: https://git.rtems.org/rtems-docs/tree/README.txt#n469 Please note the line length. That can be relaxed when pasting in output but we need the written text to be within the bounds. Thanks Chris On 1/4/2023 3:15 am, Alan Cudmore wrote:

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-03 Thread Alan Cudmore
Hi Chris, Sorry I should have read that and formatted my changes correctly. I submitted a v5 patch, but as I sent it, I realized that there are a couple of places where the RTEMS version macro can be embedded rather than hard-coding "6". I should have taken more time and avoided flooding the

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-03 Thread Hesham Almatary
Thanks Alan for investigating this issue and following up with the updates. It's fine to leave it for now, perhaps even report it to Renode if we think it's a problem with their simulator or something. On Sun, 2 Apr 2023 at 17:22, Alan Cudmore wrote: > > I submitted a v4 patch for the

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-01 Thread Alan Cudmore
Update on K210/ticker on Renode: If I define CLOCK_DRIVER_USE_ONLY_BOOT_PROCESSOR in the RISC-V BSP, ticker works on the unmodified .resc file. Alan On Sat, Apr 1, 2023 at 8:17 PM Alan Cudmore wrote: > Hi Hesham, > I found a difference between the .resc file I use to run single tests and > the

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-01 Thread Alan Cudmore
Hi Hesham, I found a difference between the .resc file I use to run single tests and the .resc file that the renode-test uses. The resc file where ticker.exe fails has this extra statement: $ex='machine SetSerialExecution True' If I comment that out, ticker runs on Renode. I couldn't find that in

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-01 Thread Alan Cudmore
Interesting - I get the same error when I run ticker on renode now. I just tried it on a board and it works. It also still passes the renode-test that I run, which includes ticker ticker along with a dozen other tests. I'll look into why this does not work right now. Thanks, Alan On Sat, Apr 1,

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-01 Thread Hesham Almatary
Thanks Alan! It Looks good to me. I'd appreciate your help with running on the simulator. I followed your documentation in this patch, but ticker seems to fatal error as follows: "*** FATAL *** fatal source: 6 (RTEMS_FATAL_SOURCE_BSP) CPU: 0 fatal code: 3342 (0x0d0e) RTEMS version:

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-01 Thread Alan Cudmore
Hi Hesham, I applied your suggestions and sent a v3 patch. Thanks for the review, Alan On Sat, Apr 1, 2023 at 1:43 PM Hesham Almatary wrote: > On Fri, 31 Mar 2023 at 17:15, Alan Cudmore wrote: > > > > This patch adds the documentation for building and running RTEMS on the > Kendryte K210 > >

Re: [PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-04-01 Thread Hesham Almatary
On Fri, 31 Mar 2023 at 17:15, Alan Cudmore wrote: > > This patch adds the documentation for building and running RTEMS on the > Kendryte K210 > RISC-V SoC. The generic riscv introducion was re-arranged to list the > multilib variants > then the specific hardware targets. In addition a couple of

[PATCH v2] docs/user: add docs for riscv/kendrytek210 BSP variant

2023-03-31 Thread Alan Cudmore
This patch adds the documentation for building and running RTEMS on the Kendryte K210 RISC-V SoC. The generic riscv introducion was re-arranged to list the multilib variants then the specific hardware targets. In addition a couple of errors were fixed for the generic QEMU commands. V2