Hi,
thanks for the response and for the PR on GitHub.
Ok. If possible, could you explain the issue with github?
Well, it's a personal thing. Aside from the usual gripe about big
corporation there is also a clause in their Terms of Service that
requires the user to "agree to submit to the exclusive jurisdiction and
venue of the courts located in the City and County of San Francisco,
California." And that's certainly something I don't agree with.
I think in this case we could add _bm and _bp as exceptions as well.
The _bm case is the most prevalent in the current source so that's a
great help. There are currently two cases of _gp and around 10 cases of
_gc so that should be manageable to redefine in the architecture code.
It also seems like I missed a 5th suffix - _gm - but that also has
around 10 uses currently.
Even with the bm exception though, the nxstyle tools reports mixed case
errors. Apparently, it cannot apply the exception for the line of code
which looks like this:
ret = (VPORTA.IN & (PIN2_bm | PIN3_bm)) >> 2;
Splitting it into two lines works though and it should yield the same
result in the machine code.
ret = (VPORTA.IN & (PIN2_bm | PIN3_bm));
ret >>= 2;
I'll incorporate these changes to the next revision of the patch series
and also submit a patch for the nxstyle.c
2. These patches depend on "nuttx/clock: make NSEC_PER_USEC and others
long" (ac42add946) being applied. It is only present in NuttX master
branch and needs to be cherry-picked for testing on the branch based
on
release 12.9
Could you please give more details here?
Since the fixes were applied to the master branch, I expect the
remaining patches in the series to go there as well - in which case
everything is sorted and nothing else needs to be done, the patch
ac42add946 is present there. The issue why that patch is needed is
explained in
https://lists.apache.org/thread/xzor3tjy9jozlmox82frd27v969b37l4 . In
short - without this patch, avr-gcc will produce malfunctioning code
when converting between ticks and time.
Hope I didn't miss any questions, please point it out if I did. Other
than that, I will try to submit second version of the patch series
during the next week.
Thanks for the review.