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

名前変更したいもの一覧! #454

Open
Tracked by #478
meltingrabbit opened this issue Oct 6, 2022 · 22 comments
Open
Tracked by #478

名前変更したいもの一覧! #454

meltingrabbit opened this issue Oct 6, 2022 · 22 comments
Assignees
Labels
enhancement New feature or request priority::high priorityg high

Comments

@meltingrabbit
Copy link
Collaborator

meltingrabbit commented Oct 6, 2022

概要

名前変更したいもの一覧!

詳細

ひとまず集めよう

リストアップ後やるべきこと

  • 変更
  • ドキュメント整備
@meltingrabbit
Copy link
Collaborator Author

If Wrapper -> Driver

じゃあ今の Driver は?

@sksat
Copy link
Collaborator

sksat commented Oct 6, 2022

単体の Driver という語を無くし, IfWrapperDevice Driver, DriverComponent Driver とするなどの案

@meltingrabbit
Copy link
Collaborator Author

文脈

  • if wrapper はたしかにデバイスドライバだけど,Driverもドライバなだよなぁという
  • Driver,プロトコル・スタックと,コンポーネントの抽象化,の2つの役割がありそう.

@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Oct 6, 2022

  • IF Wrapper -> Device Driver: DD
  • Driver -> Component Driver: CD
  • Driver Super -> Component Driver Super: CDS
  • Driver Inctance -> Component Inctance: CI
  • IF List: Device List: DL

@meltingrabbit
Copy link
Collaborator Author

device は, SW 的な device の原義に寄せていき,いわゆるコンポは componet として区別していこう(お気持ち表明)

@sksat
Copy link
Collaborator

sksat commented Oct 6, 2022

  • 現状のC2A語のIFDriverは単純に曖昧だし過剰
  • IfWrapperがデバドラなのはかなり明らかなのでdriverと名付けたいが,現状のC2A語のDriverがconflictする
  • 一旦現状に即した名前としてはやっぱりIfWrapper -> Device Driver, Driver -> Component Driverがよさそう
    • Component Driverについてはもうちょっと責任を見つめて整理したい気はするけど,それは後からでもいい
    • 実際C2A userのsrc_user/Drivers/見たらコンポの名前のディレクトリが並んでいるので気持ちからしてそうでしょう

@meltingrabbit
Copy link
Collaborator Author

src_user/Drivers/ ,今雑に srx.c とかあるけど,本当は型番とかつけないとだめそうだね. component driver なら(また別で話が進んでる,driver 分離の文脈でも)

@meltingrabbit
Copy link
Collaborator Author

meltingrabbit commented Oct 6, 2022

slack からの引用

KOBA789
「コンポドライバは新規で書かないとですね」「でもこのコンポも物理層は UART ですよね」「じゃあデバドラは使いまわせますね」という会話ができる。素晴らしい

@meltingrabbit meltingrabbit added the priority::high priorityg high label Oct 6, 2022
@sksat
Copy link
Collaborator

sksat commented Oct 6, 2022

現C2A語のIFってIfWrapperにあるやつ全部の抽象化インターフェースではないですよね(たぶん)?Device Listだけは不適当だと思う

@meltingrabbit
Copy link
Collaborator Author

ですね.正確には Drivrer Super に載せる通信インターフェースです.

フレーム解析やってほしいやつ.

UART, CCSDS, SPI, I2C とか.

@meltingrabbit
Copy link
Collaborator Author

GPIO とか memory (バス直結の.SPI とかで繋がってるのではなく)とかは含んでない

@meltingrabbit
Copy link
Collaborator Author

Device Listだけは不適当

たしかにな.

@meltingrabbit
Copy link
Collaborator Author

Communication Interface List とかになってしまうが,,,

@sksat
Copy link
Collaborator

sksat commented Oct 6, 2022

どちらかというと,フレーム解析やってほしい,という特徴の方に即して命名すべきかも

@meltingrabbit
Copy link
Collaborator Author

てか,これ別に if wrapper にいる必要はなくて, driver super の設定パラメタにしてしまったほうがきれいだな.

@sksat
Copy link
Collaborator

sksat commented Oct 6, 2022

フレーム解析やってほしいタイプのデバイス,とそれを共通化して扱わなければならないためのインターフェース(Cなので関数ポインタテーブル),そういうインターフェースを使って他のコンポとお話するComponent Driver,という整理にすべきかな

@meltingrabbit
Copy link
Collaborator Author

Drivre inctance,真の driver instance を配列で持ってる時点で instance ではない.

component serviceでは?

@meltingrabbit
Copy link
Collaborator Author

マイクロカーネルの用語をこっからツモってきた方がいい説

https://seiya.me/microkernel-book.pdf

@sksat
Copy link
Collaborator

sksat commented Oct 6, 2022

Device DriverとComponentの違いって何よ,という話があり,「テレコマとして抽象できるもの」をComponentと呼ぶことにして,テレコマ(の型)を引き出す部分をComponent Driverとするのがよさそう,ということに.
あと,Driver InstanceもかなりC2A語で,これは↑のテレコマの型に実際のコンフィグを注入する太郎であり,かつ真のdriver instanceを配列で持ってたりする(同じものが複数載ってる場合など)ので,Component Serviceとでも呼ぶのがよさそう.

@meltingrabbit meltingrabbit self-assigned this Nov 4, 2022
@meltingrabbit meltingrabbit added the enhancement New feature or request label Nov 4, 2022
@sksat
Copy link
Collaborator

sksat commented Mar 13, 2023

if_list したい device を StreamDevice とか FrameDevice みたいに名付けて,DriverSuper はそう呼ぶとよさそうと思うなどした

@sksat
Copy link
Collaborator

sksat commented May 1, 2023

今の IfWrapper って device driver というよりは HAL なんだよな

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request priority::high priorityg high
Projects
None yet
Development

No branches or pull requests

2 participants