Skip to content

fwtools/spidev: use the ID function from pkg spidev#2853

Merged
rminnich merged 2 commits into
u-root:mainfrom
rminnich:update-spidev-command-to-use-spidev.ID
Jan 13, 2024
Merged

fwtools/spidev: use the ID function from pkg spidev#2853
rminnich merged 2 commits into
u-root:mainfrom
rminnich:update-spidev-command-to-use-spidev.ID

Conversation

@rminnich
Copy link
Copy Markdown
Member

Remove the code in the command, use ID from the package. tested today on rpi4

Remove the code in the command, use ID from the package.

Signed-off-by: Ronald G. Minnich <rminnich@gmail.com>
@rminnich rminnich added the Awaiting reviewer Waiting for a reviewer. label Jan 13, 2024
@codecov
Copy link
Copy Markdown

codecov Bot commented Jan 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (1540d3a) 75.68% compared to head (69d976e) 75.68%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2853   +/-   ##
=======================================
  Coverage   75.68%   75.68%           
=======================================
  Files         425      425           
  Lines       42853    42846    -7     
=======================================
- Hits        32435    32430    -5     
+ Misses      10418    10416    -2     
Flag Coverage Δ
.-amd64 67.87% <ø> (ø)
cmds/...-amd64 71.69% <100.00%> (-0.03%) ⬇️
integration/generic-tests/...-amd64 0.00% <ø> (ø)
integration/generic-tests/...-arm 0.00% <ø> (ø)
integration/generic-tests/...-arm64 0.00% <ø> (ø)
integration/gotests/...-amd64 73.95% <ø> (ø)
integration/gotests/...-arm64 74.96% <ø> (+0.07%) ⬆️
pkg/...-amd64 76.09% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rminnich rminnich merged commit a660b43 into u-root:main Jan 13, 2024
@rminnich rminnich deleted the update-spidev-command-to-use-spidev.ID branch January 13, 2024 21:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Awaiting reviewer Waiting for a reviewer.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants