sail-riscv icon indicating copy to clipboard operation
sail-riscv copied to clipboard

Add config schema default values for patching

Open Arielfoever opened this issue 3 months ago • 5 comments

Add patching config feather.

Arielfoever avatar Aug 12 '25 03:08 Arielfoever

According to the meeting on Augest 11th, 2025, we will consider adding default values in schema which will support patching config with the help of jsoncons after https://github.com/riscv/sail-riscv/pull/1161 being merged.

~~See also https://docs.google.com/document/d/1UNYeRJOKK-TZfYUqA1DhLTbmy6hnarMUg40L9Wm2zRY/edit?tab=t.0~~

Arielfoever avatar Aug 12 '25 07:08 Arielfoever

Can you explain what this means?

Timmmm avatar Aug 22 '25 11:08 Timmmm

Can you explain what this means?

According to the meeting on Augest 11th, 2025, we will add the default values to schema then we can generate config from that. With the help of jsoncons, the config patch can be implied.

Arielfoever avatar Aug 22 '25 12:08 Arielfoever

Can you explain what this means?

According to the meeting on Augest 11th, 2025, we will add the default values to schema then we can generate config from that. With the help of jsoncons, the config patch can be implied.

At the meeting we said we would discuss this as a separate issue from config validation, but I don't recall any agreement that we would proceed with any specific solution yet.

pmundkur avatar Aug 22 '25 14:08 pmundkur

It can be done after https://github.com/riscv/sail-riscv/pull/1291

Arielfoever avatar Sep 26 '25 08:09 Arielfoever