mros2-asp3-f767zi icon indicating copy to clipboard operation
mros2-asp3-f767zi copied to clipboard

Published 20 hours ago • verified publisher icon mROS-base

Consider whether should we remove `stm32f7xx_it.h`

Open takasehideki opened this issue 4 years ago • 2 comments

sample1.cfg でincludeされている stm32f7xx_it.h は不要ではないか？ 他にも application/ 以下で不要なファイルがありそう．混乱を招くもとになりうるので，削除を検討する．

https://github.com/tlk-emb/asp-stm32f767/blob/71a3c037284af45680d3351489db1cb2496083c5/workspace/comm_tests/ping/sample1.cfg#L14

takasehideki avatar Jan 03 '21 07:01 takasehideki

すみません勘違いでした．ETH_IRQHandlerの定義に必要ですね．

takasehideki avatar Jan 03 '21 08:01 takasehideki

けっきょくHAL_ETH_IRQ_Handlerを呼び出しているだけなので，やはり不要なのでは？？

https://github.com/tlk-emb/asp-stm32f767/blob/71a3c037284af45680d3351489db1cb2496083c5/application/src/stm32f7xx_it.c#L177-L189

takasehideki avatar Jan 03 '21 13:01 takasehideki