From a5fb7823ce381dd7723b20a1e558297c6ef5719b Mon Sep 17 00:00:00 2001 From: David Holdeman Date: Mon, 11 Apr 2022 13:20:55 -0500 Subject: [PATCH] Try to make docs more clear --- Dev/Dev-Tips.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/Dev/Dev-Tips.md b/Dev/Dev-Tips.md index 6f05a2f2..eff1f319 100644 --- a/Dev/Dev-Tips.md +++ b/Dev/Dev-Tips.md @@ -12,12 +12,12 @@ console/binary/tunerstudio_outputs.h See the [documentation main page](https://rusefi.com/docs/html/#config) for more information and tips on how to use these files. -Q: Do new outputs need to be added at the end and their space taken out of mainUnusedEnd[]? -A: Look at the diff. if the diff is not crazy you are doing it right. If the diff touches too much stuff you are doing it wrong. +Q: Do new fields need to be added at the end and their space taken out of mainUnusedEnd[]? +A: Look at the diff after generating config files (i.e. using `firmware/gen_config.sh`). If the diff is not crazy you are doing it right. If the diff touches too much stuff you are doing it wrong. In practice I have found this to mean that they should be added to the end. Usually the tradition is not to move other existing fields for no reason. Starting an array at an odd offset is also unusual. Let's work around existing fields without shifting them please. -There may be unused "padding" variables spread throughout that can be replaced with your new variables, particularly of the 'bit' type, as we do bit packing for better communication efficiency. +There may be unused padding variables spread throughout that can be replaced with your new variables, particularly of the 'bit' type, as we do bit packing for better communication efficiency. You will likely be able to find a few uint8_t or uint16_t padding fields to replace with your new variables as well. The tradition is to name them "unusedSomething", so you can find them by searching for "unused" in rusefi_config.txt. Check config/boards/kinetis/config/controllers/algo/engine_configuration_generated_structures.h after generating to ensure the total size is 20000