Hello,
I would like to submit two patches for AVR architecture. First patch is
a simple typo fix. Ran checkpatch on the file after changing it, no
complaints; code compiles into identical binary (compared with SHA256
checksum.)
The second patch amends commit dfd3426aa59a96af7859bd74ff4ef063caf5a122
which was a part of a series that added finer granularity of signal
support. AVR DA/DB architecture is missing in this commit - my patch
simply replicates what was done to other Make.defs files. I tested the
change by running a custom application that spawns threads and exchanges
data between them through a pipe (the threads never sleep and preemptive
multitasking is used.) To test the ability to sleep, I also used an
application that blinks two LEDs. Testing was done with the default
setting - ENABLE_PARTIAL_SIGNALS. As for DISABLE_ALL_SIGNALS, I don't
fully understand what is/is not allowed with it and it doesn't seem
documented anywhere so I wasn't able to test that.
(Does anyone know if there is documentation for this setting? I would
like to use it - from quick build tests, it looks to me that NuttX
increased in code size by 2kB over past few months and setting this
option to partial mitigates that somewhat. I was, however, not able to
find out what functionality I am forbidden from using with it. Or if -
at least - using something unsupported in particular mode will always
yield a build-time error.)
The patches are available in a git repository nuttx.git at
git.kerogit.eu accessible through HTTP/S. (Trying to prevent bot traffic
by not posting the URL in machine-readable form.) The relevant branches
are called avrdx_typo20260219 and avrdx_signals.
As previously, I would like to ask someone with GitHub account to change
this into a pull request(s).
Thanks.
