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

2020-07-19 Thread Bin Meng
> > 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 > > wrote: > > > > > > >-Ori

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

2020-07-19 Thread Rick Chen
ragnesh Patel ; Sagar Kadam > > >; U-Boot Mailing List > > >Cc: Bin Meng > > >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

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

2020-07-17 Thread Bin Meng
eng > >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 > >

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

2020-06-25 Thread Pragnesh Patel
>-Original Message- >From: Bin Meng >Sent: 23 June 2020 11:00 >To: Rick Chen ; Simon Glass ; >Pragnesh Patel ; Sagar Kadam >; U-Boot Mailing List >Cc: Bin Meng >Subject: [PATCH 1/5] sysreset: syscon: Don't assume default value for offset >and mask prop

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

2020-06-25 Thread Simon Glass
On Mon, 22 Jun 2020 at 23:30, Bin Meng wrote: > > From: Bin Meng > > Per the DT binding, is a required property. Let's abort > the probe if it is missing. For the property, current codes > assume a default value of zero, which is not correct either. > > Signed-off-by: Bin Meng > --- > >

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

2020-06-22 Thread Bin Meng
From: Bin Meng Per the DT binding, is a required property. Let's abort the probe if it is missing. For the property, current codes assume a default value of zero, which is not correct either. Signed-off-by: Bin Meng --- drivers/sysreset/sysreset_syscon.c | 14 -- 1 file