signal11
Support for the PIC18F25K50, PIC18F45K50, and the P-Star 25K50 Micro.
As we discussed earlier, this pull request adds support for the PIC18F25K50, PIC18F45K50, and our P-Star 25K50 Micro (a breakout board for the PIC18F25K50).
I tested all of the apps on the P-Star, and I also ran the unit tests, and everything seemed to work.
Thanks!
Any news on this? I'd like to see this chip supported as well but it seems pull requests don't get much attention.
As far as the code is concerned, I believe references to "P-Star" should be removed. That just another product that happens to be using the Microchip chip. The code should not be contaminated with such indications. This applies to the names "P-Star_25K50_Micro" in the config files. Some kind of contribution file would be more appropriate to make such a reference.
Yep, it's been two years now since I made this pull request. I thought it would get merged faster given that I discussed it with @signal11 before making it.
On the other hand, the Microchip USB stack is Apache-licensed and it supports the PIC18F25K50 and XC8 out of the box, so is there any clear advantage to using m-stack? I am not saying that the Microchip USB stack is perfect, but if you are using the PIC18F25K50 then it would probably be easier to get started if you use the Microchip USB stack.