[EXTERNAL]Re: [PATCH] lib: sbi_platform: Add platform specific pmp_set() and pmp_disable()

[Anup Patel]

Hi Chao-ying Fu,

On Mon, Jun 16, 2025 at 9:47 PM Chao-ying Fu <cfu at mips.com> wrote:
>
> Hi Anup,
>
> > Hi Chao-ying Fu,
>
>
> > On Sat, Jun 14, 2025 at 10:58 PM Anup Patel <apatel at ventanamicro.com> wrote:
> > >
> > > From: Chao-ying Fu <cfu at wavecomp.com>
> > >
> > > Allow platforms to implement platform specific PMP setup and
> > > PMP disable functions which are called before actual PMP CSRs
> > > are configured.
> > >
> > > Also, implement pmp_set() and pmp_disable() for MIPS P8700.
> > >
> > > Signed-off-by: Chao-ying Fu <cfu at mips.com>
> > > Signed-off-by: Anup Patel <apatel at ventanamicro.com>
>
> > Please try this patch at your end on top of the latest OpenSBI
> > and see if this helps on MIPS P8700.
>
> > Regards,
> > Anup
>
>   Yes, this patch works functionally the same as what the old version does on FPGA and QEMU.

Thanks, I will apply this patch to the riscv/opensbi repo.

> Note: I added two more patches from Vladimir to deal with MMIO regions on top of this patch.

With this patch, we don't need the PMP_MMIO flag added by Vladimir's patches.
Is there any other patch ?

Regards,
Anup