rust-embedded / rust-embedded/svd

reject duplicate fields

Open
#248 2 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Rust
Stars
84
Forks
64
PR merge metrics
No merged PRs in 30d

Description

In https://github.com/adamgreig/stm32-rs/pull/83#issuecomment-435731760 we found that duplicate fields in a register aren't rejected by svd2rust. (There are probably other duplicate conditions that could be caught, such as duplicate register names?)

I'll make the required changes, but I need to know if you want to handle this in svd-parser already, while reading the SVD, or in svd2rust, while generating the output?

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the linked svd2rust pull request discussion, then compare how svd-parser reads SVD input with how svd2rust generates output. Determine where duplicate fields and any related duplicate conditions should be rejected, and document the expected behavior for both layers before implementing it.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
tooling
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.