You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@nuttx.apache.org by GitBox <gi...@apache.org> on 2019/12/21 11:41:39 UTC
[GitHub] [incubator-nuttx] davids5 opened a new pull request #1: imxrt fixes
davids5 opened a new pull request #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1
In bringing up the NXP RDDRONE-FMURT6 these issues were discovered and fixed.
- False detection
- No Detection I2C
- System lock up hang due to interrupt storm
Prior to this PR
```
NuttShell (NSH)
nsh> i2cdetect -b 1
Scanning I2C bus: 1
0 1 2 3 4 5 6 7 8 9 a b c d e f
00: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
10: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
20: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
30: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
40: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
50: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
60: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
70: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
nsh> i2cdetect -b 2
Scanning I2C bus: 2
0 1 2 3 4 5 6 7 8 9 a b c d e f
00: 00 01 02 03 04 05 06 07 08 09 0a 0b 0c 0d <HUNG>
```
Top Traces are the I2C IRQ, SPI IRQ and CPU IRQ,
Big Picture
Focus on Storm
The Storme rate.
With this PR
```
NuttShell (NSH)
nsh> i2cdetect -b 1
Scanning I2C bus: 1
0 1 2 3 4 5 6 7 8 9 a b c d e f
00: -- -- -- -- -- -- -- -- -- -- -- -- -- -- 0e --
10: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
20: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
30: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
40: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
50: -- -- -- -- -- 55 -- -- -- -- -- -- -- -- -- --
60: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
70: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
nsh> i2cdetect -b 2
Scanning I2C bus: 2
0 1 2 3 4 5 6 7 8 9 a b c d e f
00: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
10: -- -- -- -- -- -- -- -- -- -- -- -- -- -- 1e --
20: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
30: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
40: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
50: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
60: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
70: -- -- -- -- -- -- -- -- -- -- -- -- -- -- -- --
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] davids5 commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
davids5 commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568267090
> 'master' has never been 'dev' in this repository. You are thinking of a different repository that was managed by a single person.
Yes - I am so glad you are so exacting - I was referring to historically before NuttX joined ASF. Most recently in the repo hosted on bitbucket, with Greg as the sole committer.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] davids5 commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
davids5 commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568473865
@btashton
> @davids5 This touches some of the same IO stuff that I have been working on on the imxrt platform, some of the changes are style. Did you make these changes via one of the existing tools, or was this manual. I want to make sure my changes apply cleanly on top of this. For now I will carry this in my tree.
yeah - we really should talk this through. I had a very detailed discussion on yahoo on what we had learns from the STM32 night organic-lay grown nightmare.
The gist of it is: There are clear patterns that should be use when there is a chip family.
@mubes make some great strides in this on the RT, but took some shortcuts when the imxrt20 came in. We should **not** follow that pattern.
All the gpio tables need to broken out and the included like the clockconfig does STM32 or the dasiy does n the imxrt.
#ifddef rash has to be avoided or kept to a minimum. It use to be before the 20 commit.
Scope has to be be maintained.
Let' plan a call then bring it back to the list.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] jarivanewijk commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
jarivanewijk commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568463453
Thank you for your work, @davids5!
> We should warn all the NXP IMXRT and I think NXP S32K users the code will hang the OS.
A lot of the peripheral code for S32K seems to be taken from i.MX RT, so it is probably affected by this is as well. There are probably not many S32K users yet, though. We are working at NXP to improve the support for the S32K1xx and their evaluation boards. The progress is slow but steady. After the holidays I will apply your fixes to S32K on our internal NuttX fork. We will open a PR here eventually. (January/February?)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
Re: [GitHub] [incubator-nuttx] patacongo merged pull request #1:
imxrt fixes
Posted by Xiang Xiao <xi...@gmail.com>.
Should we move GitBox email to other list like review@nuttx.apache.org?
Thanks
Xiang
On Tue, Dec 24, 2019 at 9:02 AM GitBox <gi...@apache.org> wrote:
>
> patacongo merged pull request #1: imxrt fixes
> URL: https://github.com/apache/incubator-nuttx/pull/1
>
>
>
>
> ----------------------------------------------------------------
> This is an automated message from the Apache Git Service.
> To respond to the message, please log on to GitHub and use the
> URL above to go to the specific comment.
>
> For queries about this service, please contact Infrastructure at:
> users@infra.apache.org
>
>
> With regards,
> Apache Git Services
[GitHub] [incubator-nuttx] patacongo merged pull request #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
patacongo merged pull request #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] davids5 commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
davids5 commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568469970
> Thank you for your work, @davids5!
>
> > We should warn all the NXP IMXRT and I think NXP S32K users the code will hang the OS.
>
> A lot of the peripheral code for S32K seems to be taken from i.MX RT, so it is likely affected by this is as well. There are probably not many S32K users yet, though. We are working at NXP to improve the support for the S32K1xx and their evaluation boards. The progress is slow but steady. After the holidays I will apply your fixes to S32K on our internal NuttX repository. We will open a PR here eventually. (January/February?)
@jarivanewijk - yes exactly. It was cloned by @patacongo before I had the RDDRONE-FMURT6 HW to validate it on. It may be marked EXPERIMENTAL , but I have not followed it's progress.
I will ask @igalloway for HW and I can cross check locally as well.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] patacongo commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
patacongo commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568266571
'master' has never been 'dev' in this repository. You are thinking of a different repository that was managed by a single person.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] davids5 edited a comment on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
davids5 edited a comment on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568473865
@btashton
> @davids5 This touches some of the same IO stuff that I have been working on on the imxrt platform, some of the changes are style. Did you make these changes via one of the existing tools, or was this manual. I want to make sure my changes apply cleanly on top of this. For now I will carry this in my tree.
yeah - we really should talk this through. I had a very detailed discussion on yahoo on what we had learned from the STM32 organically grown nightmare.
The gist of it is: There are clear patterns that should be use when there is a chip family.
@mubes make some great strides in this on the RT, but took some shortcuts when the imxrt20 came in. We should **not** follow that pattern.
All the gpio tables need to broken out and the included like the clockconfig does STM32 or the dasiy does n the imxrt.
#ifddef rash has to be avoided or kept to a minimum. It use to be before the 20 commit.
Scope has to be be maintained.
Let' plan a call then bring it back to the list.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] patacongo commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
patacongo commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568268123
With a single committer, you can greatly simplify things since it is the sole reponsibility of that single committed to go from the patch to PR to its clean incorporation. It requires more discipline to work as a group.
PS: I always hated the PRs to master in the Bitbucket repsitories. But I never hated all PRs as you said. I should have set up a 'dev' branch and made sure that all PRs were against the 'dev' branch. Although it does take a little effort to keep a dev branch up to date.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] btashton commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
btashton commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568574794
@mubes No complaints on my end (beyond the tedious task of transcribing datasheets to code). This family has been handled so much better by NXP than ST did with the STM32 family.
There are a few things I have tweaked across all of the devices, but that is mostly minor and usually just using a better define for example because not all have PADMUX_WAKEUP:
```
- if (index >= IMXRT_PADMUX_WAKEUP_INDEX)
+ if (index >= IMXRT_PADMUX_SNVS_START_INDEX)
```
Also a couple typeo's like
```
-#define GPIO_LPSPI1_SDI_2 (GPIO_PERIPH | GPIO_ALT4 | GPIO_PADMUX(IMXRT_PADMUX_GPIO_SD_B0_05_INDEX))fb
+#define GPIO_LPSPI1_SDI_2 (GPIO_PERIPH | GPIO_ALT4 | GPIO_PADMUX(IMXRT_PADMUX_GPIO_SD_B0_05_INDEX))
```
There is also a GPIOMUX that they added to help with the lower pincount while still allowing high speed GPIO. Anyway I need to just get this stuff out for comment. Probably need a couple nights to get the core code done, not sure how long the board bit will take. The memory interfaces are a little different.
Anyway this is getting off topic for this PR. I'll put a DRAFT PR up before I get the board code done, either a here or on my fork.
@davids5 I have not forgotten about your offer, just will likely need to wait until the 1st with travel.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] jarivanewijk edited a comment on issue #1: imxrt
fixes
Posted by GitBox <gi...@apache.org>.
jarivanewijk edited a comment on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568463453
Thank you for your work, @davids5!
> We should warn all the NXP IMXRT and I think NXP S32K users the code will hang the OS.
A lot of the peripheral code for S32K seems to be taken from i.MX RT, so it is likely affected by this is as well. There are probably not many S32K users yet, though. We are working at NXP to improve the support for the S32K1xx and their evaluation boards. The progress is slow but steady. After the holidays I will apply your fixes to S32K on our internal NuttX fork. We will open a PR here eventually. (January/February?)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] davids5 commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
davids5 commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568198479
> So we will merge PR to master branch directly? No dev branch?
@Apache9
As it stands master has been dev. If we had release branches then master could still be dev. But all this is yet to be seen.
I will let this sit until the PPMC votes on work flow. I only needed it merged to backport to the PX4 stable version of nuttx 8.2. It was important because it cause the OS to hang.
I usually prefer to back-port after Greg has applied the patch or PR and then fixed things his subsequent commits. That way the back port == upstream.
But I can carry the commits without the [BACKPORT] marker in PX4 for now.
We should warn all the NXP IMXRT and I think NXP S32K users the code will hang the OS.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
Re: [GitHub] [incubator-nuttx] Apache9 commented on issue #1: imxrt fixes
Posted by "张铎 (Duo Zhang)" <pa...@gmail.com>.
You can see that we have already sent the discussions on the PR to the dev
list, and based on the activity of nuttx community, it will soon become
noisy. I agree that we could keep the new PR nofitication to dev list, but
I do not think we should send the comments to dev list.
Justin Mclean <ju...@classsoftware.com> 于2019年12月22日周日 下午12:43写道：
> Hi,
>
> I'd recommend having new issues sent to dev@ so people can see them
> (which I believe is the default and how It’s currently set up). If we find
> this gets too noisy then we can think about changing it or creating new
> list, but IMO there’s no need to solve an issue we don’t have yet.
>
> Thanks,
> Justin
Re: [GitHub] [incubator-nuttx] Apache9 commented on issue #1: imxrt
fixes
Posted by Justin Mclean <ju...@classsoftware.com>.
Hi,
I'd recommend having new issues sent to dev@ so people can see them (which I believe is the default and how It’s currently set up). If we find this gets too noisy then we can think about changing it or creating new list, but IMO there’s no need to solve an issue we don’t have yet.
Thanks,
Justin
Re: [GitHub] [incubator-nuttx] Apache9 commented on issue #1: imxrt fixes
Posted by "张铎 (Duo Zhang)" <pa...@gmail.com>.
I think a PR is more like an issue rather than a commit?
Justin Mclean <ju...@classsoftware.com> 于2019年12月22日周日 下午12:40写道：
> HI,
>
> > So if no objections, let's file an infra issue to create an issues@nuttx
> > mailing list and also forward the notification there?
>
> We can create one ourselves, why why have an issues@ when we already have
> a commit@?
>
> Thanks,
> Justin
Re: [GitHub] [incubator-nuttx] Apache9 commented on issue #1: imxrt
fixes
Posted by Justin Mclean <ju...@classsoftware.com>.
HI,
> So if no objections, let's file an infra issue to create an issues@nuttx
> mailing list and also forward the notification there?
We can create one ourselves, why why have an issues@ when we already have a commit@?
Thanks,
Justin
Re: [GitHub] [incubator-nuttx] Apache9 commented on issue #1: imxrt fixes
Posted by "张铎 (Duo Zhang)" <pa...@gmail.com>.
I checked HBase, we are forwarding the nofitication to issues@hbase, seems
more reasonable.
So if no objections, let's file an infra issue to create an issues@nuttx
mailing list and also forward the notification there?
Thanks.
Abdelatif Guettouche <ab...@gmail.com> 于2019年12月21日周六
下午11:28写道：
> We already have a commits@nuttx mailing list.
>
>
> On Sat, Dec 21, 2019 at 3:23 PM 张铎(Duo Zhang) <pa...@gmail.com>
> wrote:
> >
> > I think we should create a mailing list called commits@nuttx or
> something
> > else and let the forwarded github message go there...
> >
> > GitBox <gi...@apache.org> 于2019年12月21日周六 下午11:16写道：
> >
> > > Apache9 commented on issue #1: imxrt fixes
> > > URL:
> > >
> https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568188369
> > >
> > >
> > > So we will merge PR to master branch directly? No dev branch?
> > >
> > > ----------------------------------------------------------------
> > > This is an automated message from the Apache Git Service.
> > > To respond to the message, please log on to GitHub and use the
> > > URL above to go to the specific comment.
> > >
> > > For queries about this service, please contact Infrastructure at:
> > > users@infra.apache.org
> > >
> > >
> > > With regards,
> > > Apache Git Services
> > >
>
Re: [GitHub] [incubator-nuttx] Apache9 commented on issue #1: imxrt fixes
Posted by Abdelatif Guettouche <ab...@gmail.com>.
We already have a commits@nuttx mailing list.
On Sat, Dec 21, 2019 at 3:23 PM 张铎(Duo Zhang) <pa...@gmail.com> wrote:
>
> I think we should create a mailing list called commits@nuttx or something
> else and let the forwarded github message go there...
>
> GitBox <gi...@apache.org> 于2019年12月21日周六 下午11:16写道：
>
> > Apache9 commented on issue #1: imxrt fixes
> > URL:
> > https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568188369
> >
> >
> > So we will merge PR to master branch directly? No dev branch?
> >
> > ----------------------------------------------------------------
> > This is an automated message from the Apache Git Service.
> > To respond to the message, please log on to GitHub and use the
> > URL above to go to the specific comment.
> >
> > For queries about this service, please contact Infrastructure at:
> > users@infra.apache.org
> >
> >
> > With regards,
> > Apache Git Services
> >
Re: [GitHub] [incubator-nuttx] Apache9 commented on issue #1: imxrt fixes
Posted by "张铎 (Duo Zhang)" <pa...@gmail.com>.
I think we should create a mailing list called commits@nuttx or something
else and let the forwarded github message go there...
GitBox <gi...@apache.org> 于2019年12月21日周六 下午11:16写道：
> Apache9 commented on issue #1: imxrt fixes
> URL:
> https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568188369
>
>
> So we will merge PR to master branch directly? No dev branch?
>
> ----------------------------------------------------------------
> This is an automated message from the Apache Git Service.
> To respond to the message, please log on to GitHub and use the
> URL above to go to the specific comment.
>
> For queries about this service, please contact Infrastructure at:
> users@infra.apache.org
>
>
> With regards,
> Apache Git Services
>
[GitHub] [incubator-nuttx] Apache9 commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
Apache9 commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568188369
So we will merge PR to master branch directly? No dev branch?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] btashton commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
btashton commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568330736
@davids5 This touches some of the same IO stuff that I have been working on on the imxrt platform, some of the changes are style. Did you make these changes via one of the existing tools, or was this manual. I want to make sure my changes apply cleanly on top of this. For now I will carry this in my tree.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] jarivanewijk edited a comment on issue #1: imxrt
fixes
Posted by GitBox <gi...@apache.org>.
jarivanewijk edited a comment on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568463453
Thank you for your work, @davids5!
> We should warn all the NXP IMXRT and I think NXP S32K users the code will hang the OS.
A lot of the peripheral code for S32K seems to be taken from i.MX RT, so it is likely affected by this is as well. There are probably not many S32K users yet, though. We are working at NXP to improve the support for the S32K1xx and their evaluation boards. The progress is slow but steady. After the holidays I will apply your fixes to S32K on our internal NuttX repository. We will open a PR here eventually. (January/February?)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] mubes commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
mubes commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568575636
Yeah, lets move this onto another PR so we can fix up whatever needs to be done to solidify the filesystem structure for the rt's. Certainly I think moving the GPIOs, IOMUX and ClockConfig into the subdirectory will go a long way to fixing whats outstanding (well, that and fixing Mr. Hamfisteds' typos).
DAVE
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-nuttx] mubes commented on issue #1: imxrt fixes
Posted by GitBox <gi...@apache.org>.
mubes commented on issue #1: imxrt fixes
URL: https://github.com/apache/incubator-nuttx/pull/1#issuecomment-568570219
Way back when DavidS and I talked about this at length and then there was a lot of work done in imxrt to split it into the different chips - 1050, 60 and 20 to form the various hardware/rt10xx directories.
At the moment the only place there is any family member selection (other than to pull in the correct chip specific subdir) is in imxrt_clockconfig.c, imxrt_gpio.c and imxrt_iomux.c. Those can (and probably should) be refactored into the same pattern as the other files which, I think, would keep the structure clean.
Does that satisfy your concern @btashton , or are you looking for a more fundamental restructure? (BTW, this structure is a nicer way of highlighting changes and having proper discussion about them to find the best way forward).
DAVE
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services