Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-21 Thread Rob Herring
On Thu, Sep 14, 2017 at 04:17:36PM +0900, Masahiro Yamada wrote: > This example allocates too much for register regions. Especially, > there are only two registers in the "nand_data" interface of this > hardware (ADDR: 0x00, DATA: 0x10). > > Signed-off-by: Masahiro Yamada

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-21 Thread Rob Herring
On Thu, Sep 14, 2017 at 04:17:36PM +0900, Masahiro Yamada wrote: > This example allocates too much for register regions. Especially, > there are only two registers in the "nand_data" interface of this > hardware (ADDR: 0x00, DATA: 0x10). > > Signed-off-by: Masahiro Yamada > --- > >

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-21 Thread Masahiro Yamada
2017-09-21 15:09 GMT+09:00 Oleksij Rempel : > Hi, > > On 21.09.2017 07:26, Masahiro Yamada wrote: >> >> Hi. > > > .. > >>> Hm.. according to >>> >>> https://www.altera.com/en_US/pdfs/literature/hb/cyclone-v/cyclone5_handbook.pdf >>> Table 13-18: NAND Controller Module Data

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-21 Thread Masahiro Yamada
2017-09-21 15:09 GMT+09:00 Oleksij Rempel : > Hi, > > On 21.09.2017 07:26, Masahiro Yamada wrote: >> >> Hi. > > > .. > >>> Hm.. according to >>> >>> https://www.altera.com/en_US/pdfs/literature/hb/cyclone-v/cyclone5_handbook.pdf >>> Table 13-18: NAND Controller Module Data Space Address Range

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-21 Thread Oleksij Rempel
Hi, On 21.09.2017 07:26, Masahiro Yamada wrote: Hi. .. Hm.. according to https://www.altera.com/en_US/pdfs/literature/hb/cyclone-v/cyclone5_handbook.pdf Table 13-18: NAND Controller Module Data Space Address Range Module Instance Start AddressEnd Address NAND_DATA

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-21 Thread Oleksij Rempel
Hi, On 21.09.2017 07:26, Masahiro Yamada wrote: Hi. .. Hm.. according to https://www.altera.com/en_US/pdfs/literature/hb/cyclone-v/cyclone5_handbook.pdf Table 13-18: NAND Controller Module Data Space Address Range Module Instance Start AddressEnd Address NAND_DATA

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-20 Thread Masahiro Yamada
Hi. 2017-09-14 18:06 GMT+09:00 Oleksij Rempel : > On 14.09.2017 10:16, Masahiro Yamada wrote: >> >> Hi. >> >> >> 2017-09-14 17:04 GMT+09:00 Oleksij Rempel : >>> >>> Hi, >>> >>> i assume arch/arm/boot/dts/socfpga.dtsi should be update as well. Right? >>

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-20 Thread Masahiro Yamada
Hi. 2017-09-14 18:06 GMT+09:00 Oleksij Rempel : > On 14.09.2017 10:16, Masahiro Yamada wrote: >> >> Hi. >> >> >> 2017-09-14 17:04 GMT+09:00 Oleksij Rempel : >>> >>> Hi, >>> >>> i assume arch/arm/boot/dts/socfpga.dtsi should be update as well. Right? >> >> >> I think so. >> (also

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-14 Thread Oleksij Rempel
On 14.09.2017 10:16, Masahiro Yamada wrote: Hi. 2017-09-14 17:04 GMT+09:00 Oleksij Rempel : Hi, i assume arch/arm/boot/dts/socfpga.dtsi should be update as well. Right? I think so. (also arch/arm/boot/dts/socfpga_arria10.dtsi in the same way) Hm.. according to

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-14 Thread Oleksij Rempel
On 14.09.2017 10:16, Masahiro Yamada wrote: Hi. 2017-09-14 17:04 GMT+09:00 Oleksij Rempel : Hi, i assume arch/arm/boot/dts/socfpga.dtsi should be update as well. Right? I think so. (also arch/arm/boot/dts/socfpga_arria10.dtsi in the same way) Hm.. according to

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-14 Thread Masahiro Yamada
Hi. 2017-09-14 17:04 GMT+09:00 Oleksij Rempel : > Hi, > > i assume arch/arm/boot/dts/socfpga.dtsi should be update as well. Right? I think so. (also arch/arm/boot/dts/socfpga_arria10.dtsi in the same way) The wrong property "dma-mask" was removed by commit

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-14 Thread Masahiro Yamada
Hi. 2017-09-14 17:04 GMT+09:00 Oleksij Rempel : > Hi, > > i assume arch/arm/boot/dts/socfpga.dtsi should be update as well. Right? I think so. (also arch/arm/boot/dts/socfpga_arria10.dtsi in the same way) The wrong property "dma-mask" was removed by commit

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-14 Thread Oleksij Rempel
Hi, i assume arch/arm/boot/dts/socfpga.dtsi should be update as well. Right? On 14.09.2017 09:17, Masahiro Yamada wrote: This example allocates too much for register regions. Especially, there are only two registers in the "nand_data" interface of this hardware (ADDR: 0x00, DATA: 0x10).

Re: [PATCH] dt-bindings: nand: denali: reduce the register space in the example

2017-09-14 Thread Oleksij Rempel
Hi, i assume arch/arm/boot/dts/socfpga.dtsi should be update as well. Right? On 14.09.2017 09:17, Masahiro Yamada wrote: This example allocates too much for register regions. Especially, there are only two registers in the "nand_data" interface of this hardware (ADDR: 0x00, DATA: 0x10).