Betaflight: dump hardware - understanding ask

Created on 8 Mar 2019  路  2Comments  路  Source: betaflight/betaflight

dump hardware only lists baro_hardware but not mag_hardware ... Is it wanted like that?
imo, we should use both in the list or remove baro_hardware from the list.

@mikeller You introduced this with #7721 and #7732 ... what is your opinion?

From my point of view, baro_hardware should disappear from the list and appear only in the dump list

BUG
Source
picture TheIsotopes

All 2 comments

@TheIsotopes: Good point. In general, I think that only settings that _need_ to be specified for a board to work should be included in the dump hardware list - this is meant to be the bare minimum, and is intended as a tool to make it easier for manufacturers and users to convert their custom firmware into a target configuration.

Using baro_hardware = AUTO should work fine for all boards, so this should be removed from the list.

Do you want to open a pull request for this?

mikeller picture mikeller on 8 Mar 2019
👍1

pull request was created now

TheIsotopes picture TheIsotopes on 8 Mar 2019
👍1
Was this page helpful?
0 / 5 - 0 ratings

Related issues

lukedempsey picture lukedempsey  路  4Comments
flint723 picture flint723  路  3Comments
stackola picture stackola  路  3Comments
prohelper picture prohelper  路  3Comments
legacy13 picture legacy13  路  4Comments