rename & refactor: IF list -> DS handler registry

[sksat]

#21 の rename によって src_user/IfWrapper/if_list.c の歪さが表出する．そもそもこの存在をどうにかしたいというのはありつつ，実態に即した場所と名前は src_user/Settings/DriverSuper/handler_registry.h などなので，これも変更する必要がある．

[sksat]

各ハンドラは HAL_init(), HAL_rx(), HAL_tx(), HAL_reopen() とかかなあ．場所は一旦 src/src_user/settings/component_driver_super/hal_handler.h かな

[sksat]

IF_init とかは実際は関数ポインタの array なので，名前で明示したい（そもそも関数ポインタを集約する構造体を作ってそれの array にすべきではある）

[meltingrabbit]

handlerだと，EHとかぶるので
hal_handler_registry.h
かなぁ？

[meltingrabbit]

HAL_init_list
HAL_rx_list

..

でよさそう

[meltingrabbit]

extern  int     (*IF_TX[]    )(void* my_if, void* data_v, int data_size);

↓

extern  int     (*HAL_tx_list[]    )(void* config, void* data, int data_size);

rx は data→buffer

[meltingrabbit]

typedef enum
{
  CCSDS,
  UART,
  IF_LIST_MAX
} IF_LIST_ENUM;

↓

typedef enum
{
  HAL_HANDLER_LIST_CCSDS,
  HAL_HANDLER_LIST_UART,
  HAL_HANDLER_LIST_MAX
} HAL_HANDLER_LIST;

[meltingrabbit]

typedef enum
{
  IF_REOPEN_TLM_DISRUPTION = 100
} IF_REOPEN_REASON;

↓

typedef enum
{
  HAL_HANDLER_REOPEN_REASON_TLM_DISRUPTION = 100
} HAL_HANDLER_REOPEN_REASON;

[sksat]

CDS_IF_RX_BUFFER_SIZE も CDS_HAL_RX_HANDLER_BUFFER_SIZE かな

[meltingrabbit]

あ～，そこまで手を出すと rx_buffer_size_in_if_rx_ なども変わるので，issueきって後回し，はいかがだろうか？（だいぶ大変になっちゃう）

[meltingrabbit]

CDS_IF_RX_BUFFER_SIZE , rx_buffer_size_in_if_rx_ だけだからやっちゃうか

[sksat]

そう．これだけだし CDS_IF_RX_BUFFER_SIZE は core でデフォルト設定提供して（例によって undef して）user 側で定義することもあるやつですよね（つまり core -> user のインターフェースですよね）．なのでこのタイミングで変えたい．

[meltingrabbit]

OK

[meltingrabbit]

ret_from_if_rx も
ret_from_if_tx も

if_config → hal_config
CDS_if_rx_buffer_

[meltingrabbit]

大文字 IF で grep すると結構出てくるね．コメントが

[meltingrabbit]

p_super->interface を p_super->hal_handler とかにします？

struct ComponentDriverSuper
{
  // 【継承先まで公開】
  HAL_HANDLER_LIST hal_handler_id;                             //!< 継承先の機器の使用 IF
  void*            hal_config;                                 //!< IF 設定

[sksat]

if_list に意識が引っ張られてたけど，hal_handlers だな