Hi Rick, On Mon, Jul 20, 2020 at 10:48 AM Rick Chen <rickche...@gmail.com> wrote: > > Hi Tom > > > From: Bin Meng [mailto:bmeng...@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.pa...@sifive.com> > > wrote: > > > > > > >-----Original Message----- > > > >From: Bin Meng <bmeng...@gmail.com> > > > >Sent: 23 June 2020 11:00 > > > >To: Rick Chen <r...@andestech.com>; Simon Glass <s...@chromium.org>; > > > >Pragnesh Patel <pragnesh.pa...@sifive.com>; Sagar Kadam > > > ><sagar.ka...@sifive.com>; U-Boot Mailing List <u-boot@lists.denx.de> > > > >Cc: Bin Meng <bin.m...@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.m...@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.m...@windriver.com> > > > >--- > > > > > > > > drivers/sysreset/sysreset_syscon.c | 14 ++++++++++++-- > > > > 1 file changed, 12 insertions(+), 2 deletions(-) > > > > > > Reviewed-by: Pragnesh Patel <pragnesh.pa...@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