[PATCH 1/5] sysreset: syscon: Don't assume default value for offset and mask property

Bin Meng bmeng.cn at gmail.com
Mon Jul 20 04:51:26 CEST 2020


Hi Rick,

On Mon, Jul 20, 2020 at 10:48 AM Rick Chen <rickchen36 at gmail.com> wrote:
>
> Hi Tom
>
> > From: Bin Meng [mailto:bmeng.cn at gmail.com]
> > Sent: Friday, July 17, 2020 2:54 PM
> > To: Pragnesh Patel
> > Cc: Bin Meng; Rick Jian-Zhi Chen(陳建志); Simon Glass; U-Boot Mailing List; Sagar Kadam
> > Subject: Re: [PATCH 1/5] sysreset: syscon: Don't assume default value for offset and mask property
> >
> > Hi Rick,
> >
> > On Fri, Jun 26, 2020 at 1:53 PM Pragnesh Patel <pragnesh.patel at sifive.com> wrote:
> > >
> > > >-----Original Message-----
> > > >From: Bin Meng <bmeng.cn at gmail.com>
> > > >Sent: 23 June 2020 11:00
> > > >To: Rick Chen <rick at andestech.com>; Simon Glass <sjg at chromium.org>;
> > > >Pragnesh Patel <pragnesh.patel at sifive.com>; Sagar Kadam
> > > ><sagar.kadam at sifive.com>; U-Boot Mailing List <u-boot at lists.denx.de>
> > > >Cc: Bin Meng <bin.meng at windriver.com>
> > > >Subject: [PATCH 1/5] sysreset: syscon: Don't assume default value for
> > > >offset and mask property
> > > >
> > > >[External Email] Do not click links or attachments unless you
> > > >recognize the sender and know the content is safe
> > > >
> > > >From: Bin Meng <bin.meng at windriver.com>
> > > >
> > > >Per the DT binding, <offset> is a required property. Let's abort the
> > > >probe if it is missing. For the <mask> property, current codes assume
> > > >a default value of zero, which is not correct either.
> > > >
> > > >Signed-off-by: Bin Meng <bin.meng at windriver.com>
> > > >---
> > > >
> > > > drivers/sysreset/sysreset_syscon.c | 14 ++++++++++++--
> > > > 1 file changed, 12 insertions(+), 2 deletions(-)
> > >
> > > Reviewed-by: Pragnesh Patel <pragnesh.patel at sifive.com>
> >
> > Would you please take the remaining 3 patches soon?
> > http://patchwork.ozlabs.org/project/uboot/list/?series=185161
>
> May I ask for your opinions about these three SYSRESET patchs. If they
> are OK to be pick up in my next PR, though they have been rejected in
> previous riscv PR ?
> https://www.mail-archive.com/u-boot@lists.denx.de/msg375465.html
>

Now the merge window for v2020.10 is open. It's OK to merge this now.

Regards,
Bin


More information about the U-Boot mailing list