Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove old HAL for DC #25

Merged
merged 1 commit into from
Jul 25, 2023
Merged

Remove old HAL for DC #25

merged 1 commit into from
Jul 25, 2023

Conversation

sksat
Copy link
Member

@sksat sksat commented Jul 24, 2023

概要

DC の HAL を消す

Issue

詳細

DC については v4 で消す.理由は以下.

  • 名前が伝わらなさすぎる
  • 既に使われてしまってはいるが,一部の C2A user に限られている
    • また,それ以外の C2A user では独自に GPIO のインターフェース定義が行われている
    • ↑ こちらの方が DriverSuper に乗っていない素直な実装なので,c2a-core に置くとしたらこれに置き換えたい
  • これを一旦 c2a-core に残してリファクタするよりも,一度消してやり直す方が整理として適切
    • GPIO は DriverSuper の仕組みに乗っているべきではないので,段階的な移行を試みてもほぼ書き換えにしかならない
    • 当然 breaking change になるが,使っている C2A user も一旦ヘッダを user 側に置けば延命可能

Originally posted by @sksat in #23 (comment)

影響範囲

src_core/hal/dc.h(元 src_core/IfWrapper/dc.h)が使えなくなる.既に使ってしまっている C2A user については,一旦もとのファイルをコピーするか,新たに core に入れる予定の GPIO のインターフェースをベースにコードを修正する必要がある.

@sksat sksat added the priority::high priorityg high label Jul 24, 2023
@sksat sksat requested a review from meltingrabbit July 24, 2023 07:23
@sksat sksat self-assigned this Jul 24, 2023
@sksat sksat merged commit a6fcc63 into develop Jul 25, 2023
24 checks passed
@sksat sksat deleted the feature/remove-hal-dc branch July 25, 2023 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority::high priorityg high
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants