Messages in this thread | | | Date | Wed, 25 Nov 2015 14:39:44 +0000 | From | Vladimir Murzin <> | Subject | Re: [RFC PATCH 00/10] Support for Cortex-M Prototyping System |
| |
On 25/11/15 11:22, Arnd Bergmann wrote: >> I'd be happy to hear any feedback/comments on this series! > Looks pretty good overall, I didn't have any specific concerns.
Thanks for your time!
> >> > Remain questions: >> > >> > - Application notes 399/400 have PSRAM located at address different to what >> > we have for AN385/AN386, so I'm wondering what is the best practice to handle >> > CONFIG_DRAM_BASE? Different defconfig or there is better place? > I would not want to add more than one defconfig for a relatively simple > platform. Most actual users would have to modify their config anyway, so > I think it's best to just have one config file for the most common machine > here and let users change the dram base if they have the other one. >
I think the same, just wanted to check that I didn't misunderstand something.
> We can also look at using the Kconfig fragments infrastructure more here. > >> > - I'm not sure about naming of dts files: Application Notes (mps2-an*) vs Cortex-M (mps2-cm*); >> > any preference? > I don't mind either way.
Ok, I'll leave things as they are till anybody has different opinion on that.
Cheers Vladimir
> > Arnd > > >
| |