Re: [PATCH] arm64: dts: ls1012a: add crypto node

2017-03-28 Thread Shawn Guo
On Tue, Mar 28, 2017 at 07:19:43AM +, Horia Geantă wrote: > For the sake of current patch, please clarify whether a v2 is needed. > IIUC: > -sec_mon node name could stay the same (existing binding) > -label names are ok, since underline is the only option allowed by DTC > -alias names are

Re: [PATCH] arm64: dts: ls1012a: add crypto node

2017-03-28 Thread Horia Geantă
On 3/24/2017 4:04 PM, Shawn Guo wrote: > On Fri, Mar 24, 2017 at 08:29:17AM +, Horia Geantă wrote: >> On 3/24/2017 9:35 AM, Shawn Guo wrote: >>> On Fri, Mar 24, 2017 at 07:17:50AM +, Horia Geantă wrote: >> +sec_mon: sec_mon@1e9 { > > Hyphen is more preferred

Re: [PATCH] arm64: dts: ls1012a: add crypto node

2017-03-24 Thread Shawn Guo
On Fri, Mar 24, 2017 at 08:29:17AM +, Horia Geantă wrote: > On 3/24/2017 9:35 AM, Shawn Guo wrote: > > On Fri, Mar 24, 2017 at 07:17:50AM +, Horia Geantă wrote: > +sec_mon: sec_mon@1e9 { > >>> > >>> Hyphen is more preferred to be used in node name than underscore.

Re: [PATCH] arm64: dts: ls1012a: add crypto node

2017-03-24 Thread Horia Geantă
On 3/24/2017 9:35 AM, Shawn Guo wrote: > On Fri, Mar 24, 2017 at 07:17:50AM +, Horia Geantă wrote: + sec_mon: sec_mon@1e9 { >>> >>> Hyphen is more preferred to be used in node name than underscore. >>> >> This would imply changing the >>

Re: [PATCH] arm64: dts: ls1012a: add crypto node

2017-03-24 Thread Shawn Guo
On Fri, Mar 24, 2017 at 07:17:50AM +, Horia Geantă wrote: > >> + sec_mon: sec_mon@1e9 { > > > > Hyphen is more preferred to be used in node name than underscore. > > > This would imply changing the > Documentation/devicetree/bindings/crypto/fsl-sec4.txt binding and > dealing

Re: [PATCH] arm64: dts: ls1012a: add crypto node

2017-03-24 Thread Horia Geantă
On 3/24/2017 3:56 AM, Shawn Guo wrote: > On Wed, Mar 22, 2017 at 02:29:39PM +0200, Horia Geantă wrote: >> LS1012A has a SEC v5.4 security engine. >> >> Signed-off-by: Horia Geantă >> --- >> arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts | 9 +++ >>

Re: [PATCH] arm64: dts: ls1012a: add crypto node

2017-03-23 Thread Shawn Guo
On Wed, Mar 22, 2017 at 02:29:39PM +0200, Horia Geantă wrote: > LS1012A has a SEC v5.4 security engine. > > Signed-off-by: Horia Geantă > --- > arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts | 9 +++ > arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dts | 9 +++ >

[PATCH] arm64: dts: ls1012a: add crypto node

2017-03-22 Thread Horia Geantă
LS1012A has a SEC v5.4 security engine. Signed-off-by: Horia Geantă --- arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dts | 9 +++ arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dts | 9 +++ arch/arm64/boot/dts/freescale/fsl-ls1012a-rdb.dts | 9 +++