Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1437935 > unrolled thread

Re: [PATCH -next] ARM: dts: sun8i: Add NFC node to Allwinner A23/A33 SoC

Started byMaxime Ripard <maxime.ripard@free-electrons.com>
First post2016-07-06 22:40 +0200
Last post2016-07-07 09:30 +0200
Articles 2 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH -next] ARM: dts: sun8i: Add NFC node to Allwinner A23/A33  SoC Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-07-06 22:40 +0200
    Re: [PATCH -next] ARM: dts: sun8i: Add NFC node to Allwinner A23/A33  SoC Maxime Ripard <maxime.ripard@free-electrons.com> - 2016-07-07 09:30 +0200

#1437935 — Re: [PATCH -next] ARM: dts: sun8i: Add NFC node to Allwinner A23/A33 SoC

FromMaxime Ripard <maxime.ripard@free-electrons.com>
Date2016-07-06 22:40 +0200
SubjectRe: [PATCH -next] ARM: dts: sun8i: Add NFC node to Allwinner A23/A33 SoC
Message-ID<rS1LY-5xe-15@gated-at.bofh.it>

[Multipart message — attachments visible in raw view] — view raw

Hi,

On Wed, Jul 06, 2016 at 08:33:52AM +0800, Icenowy Zheng wrote:
> Add NAND Flash controller node definition to the A23/A33 SoC.
> 
> Signed-off-by: Icenowy Zheng <icenowy@aosc.xyz>

Has this ever been tested? On what board, with what NAND chip?

> ---
>  arch/arm/boot/dts/sun8i-a23-a33.dtsi | 23 +++++++++++++++++++++++
>  1 file changed, 23 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sun8i-a23-a33.dtsi b/arch/arm/boot/dts/sun8i-a23-a33.dtsi
> index 7e05e09..270c38a 100644
> --- a/arch/arm/boot/dts/sun8i-a23-a33.dtsi
> +++ b/arch/arm/boot/dts/sun8i-a23-a33.dtsi
> @@ -209,6 +209,14 @@
>  					"apb2_uart3", "apb2_uart4";
>  		};
>  
> +		nand_clk: clk@01c20080 {
> +			#clock-cells = <0>;
> +			compatible = "allwinner,sun4i-a10-mod0-clk";
> +			reg = <0x01c20080 0x4>;
> +			clocks = <&osc24M>, <&pll6 1>;
> +			clock-output-names = "nand";
> +		};
> +

This should be in a separate patch.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[toc] | [next] | [standalone]


#1438221

FromMaxime Ripard <maxime.ripard@free-electrons.com>
Date2016-07-07 09:30 +0200
Message-ID<rSbUZ-3OW-9@gated-at.bofh.it>
In reply to#1437935

[Multipart message — attachments visible in raw view] — view raw

On Thu, Jul 07, 2016 at 08:04:41AM +0800, Icenowy Zheng wrote:
> with a http://linux-sunxi.org/Aoson_M751s tablet, (SKhynix
> H27UCG8T2ETR-BC NAND chip).
> 
> I used the patchset at
> http://lists.infradead.org/pipermail/linux-mtd/2016-June/067848.html
> to ensure the NAND chip to work properly (otherwise the chip id
> cannot be correctly identified)

(Please reply in-line and wrap your mails).

Ok. That needs to be in the commit log then.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web