[PATCH v4 00/14] Add Andes AE350 fdt driver support

[Yu-Chien Peter Lin <peterlin@andestech.com>]

On Thu, Oct 27, 2022 at 06:13:48PM +0530, Anup Patel wrote:
> On Thu, Oct 27, 2022 at 5:54 PM Yu-Chien Peter Lin
> <peterlin@andestech.com> wrote:
> >
> > > > > Applied this series to the riscv/opensbi repo.
> > > > >
> > > > > Any thoughts on using the generic platform ?
> > > > >
> > > > > Thanks,
> > > > > Anup
> > > >
> > > > Hi Anup,
> > > >
> > > > The FW_TEXT_START on AE350 is different from generic platform.
> > > > Will it become a configuration?
> > >
> > > The generic platform has FW_PIC enabled so you can load and
> > > run generic firmwares from any address.
> > >
> > > > I think we would keep the platform specific files in andes/ae350
> > > > until we have to refactor the duplicate code.
> > >
> > > You can add platform specific overrides to generic platforms which
> > > are selected based on compatible string in the root DT node.
> > >
> > > Regards,
> > > Anup
> >
> > Hi Anup,
> >
> > We're considering using generic platform, but I don't find any
> > vendor_ext_provider is implemented as a platform_override hook,
> > does it mean we need to drop it if we move to generic platform?
> 
> vendor_ext_provider() is available in platform_override of generic
> platform.
> 
> Please check the latest OpenSBI sources.
> 
> Regards,
> Anup
> 

Hi Anup,

Sorry, I know there is vendor_ext_provider in platform_override,
except AE350, no other platforms actually have such hook in their
platform specific files, so I was wondering if the current acceptance
policy won't let us keep that function even in platform/andes/ae350.

Thanks,
Peter Lin