From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from shelob.surriel.com (shelob.surriel.com [96.67.55.147]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 52F6BC001DF for ; Tue, 15 Aug 2023 22:02:41 +0000 (UTC) Received: from localhost ([::1] helo=shelob.surriel.com) by shelob.surriel.com with esmtp (Exim 4.96) (envelope-from ) id 1qW26V-0004pT-2u; Tue, 15 Aug 2023 18:01:31 -0400 Received: from mail-oi1-x233.google.com ([2607:f8b0:4864:20::233]) by shelob.surriel.com with esmtps (TLS1.2) tls TLS_ECDHE_ECDSA_WITH_AES_128_GCM_SHA256 (Exim 4.96) (envelope-from ) id 1qW26P-0004pG-2p for kernelnewbies@kernelnewbies.org; Tue, 15 Aug 2023 18:01:29 -0400 Received: by mail-oi1-x233.google.com with SMTP id 5614622812f47-3a80e665410so1463702b6e.2 for ; Tue, 15 Aug 2023 15:01:19 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1692136874; x=1692741674; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=EanoirrAIIqmjEmFxPEGErGDDAw5E8I+jQbh8PxdYrU=; b=Do4jZ7jAN02ACPuBm4mLKSg776UDwsiuLUSd1Ycj272fRJZY3EE5MoXos4WsV1joNi 57Ulp5WaY8fX/iIYsHYIcyCMcuv3RFJUncIKXP6bBXTSxvvngtJ2pyT/obrFXRxDpazs +/ZicRfAD95lJvnHtN3SkJBpZt2yhLnTUN4lP8TLfEDzTIdLzirBYy1NoBT1tFg5DJ9U kGCPnL7mxHkE6PE5F81uddpQnwElnVHattSZ0lJ+3viaoX0CKNNH1jqLH1RMR3Ex4aD6 1Sv6rYF4DVIJEEIZSc/7BTKGDfm6f5Lv202eF+G6mzuSH3Te22PZFu7Rjz3uWSut44TG qAjA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692136874; x=1692741674; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=EanoirrAIIqmjEmFxPEGErGDDAw5E8I+jQbh8PxdYrU=; b=aOGyrwglo/JjqkeuX698HGdIahqLty8sxr53ZfOEdqzj/ZKF0tVkN3ByHnNIpbLxOK 9WQD5d/R47bZYh2v8KoVfuxFkznwQGgLQkFWQmZ6i0pSNCwZAnyF4v3rkUAz7IcdbY39 9fTglOK5TnK00gxSQ0mf9VtIrnHtPOZD9zFCDV3SM5h1kIGtJD0gUxnQmsOUEhPC0Ncu BUFMOVanf6k8SKZ+3QBivO4mcqxnY/pfNqp0N+BH0Omo6wD48GfmCqgzHyOp335BfoVX esnXpM/0DOMo6o+HAGJQb5LIwUGuQ2vFxCAwjUNADtZCWDWFQpM7VtxEqPX/OKOTfiKc d3Tw== X-Gm-Message-State: AOJu0Yz7ha6mOseJUXdjwI+1zQrsFAu+cfbMOUboS7LaKmDq0uCjxAAU 3PyvUmu0SUuh12e0fvNtVvRNEkX9NgCBVaLRvIw= X-Google-Smtp-Source: AGHT+IHLzfVjBlYs+ef8iyZs5oJ0i4x0j9rwMup2L5KJLyWOFpr9WOyieiHvc8Q9/WUJDjtrMM0hsJz7LqjBzfVuO6Y= X-Received: by 2002:a05:6808:1391:b0:3a4:8e9b:e5cb with SMTP id c17-20020a056808139100b003a48e9be5cbmr19892632oiw.1.1692136874441; Tue, 15 Aug 2023 15:01:14 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Fawad Lateef Date: Tue, 15 Aug 2023 23:01:02 +0100 Message-ID: Subject: Re: Question regarding Linux-6.3.5 and Sunxi A20 NAND/NFC support To: Linux Kernel Mailing List , Linux sunXi Cc: boris.brezillon@collabora.com, kernelnewbies X-BeenThere: kernelnewbies@kernelnewbies.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Learn about the Linux kernel List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: kernelnewbies-bounces@kernelnewbies.org Hi everyone, Can someone guide me? I am not sure what I am missing here. I am trying to debug the code but I assume that the SUNXI NAND driver is supposed to be working and that something is missing from my changes and additions to the dts/dtsi for A20 SOM. Thanks in advance On Mon, 14 Aug 2023 at 11:16, Fawad Lateef wrote: > > Hi Bagas, > > On Mon, 14 Aug 2023 at 06:07, Bagas Sanjaya wrote: > > > > On Mon, Aug 14, 2023 at 12:40:26AM +0100, Fawad Lateef wrote: > > > Hi, > > > > > > I am trying to make Linux-6.3.5 working on Olimex/Sunxi A20 SOM with NAND. > > > > > > Found that the Sunxi A20 NAND is supported in mainline kernel > > > (drivers/mtd/nand/raw/sunxi_nand.c) and following the guidelines here > > > (https://linux-sunxi.org/Mainline_NAND_Howto); I added the NFC chip > > > pin information in sunxi-a20.dtsi and defined/enabled the nfc > > > controller in the sunxi-a20-olinuxino-micro.dts files. > > > > > > The changes I made are below. Problem is when I define the nfc node > > > entry in dts file "without" pinctrl entry (pinctrl-0 = <&nand_pins_a>, > > > <&nand_cs0_pins_a>, <&nand_rb0_pins_a>;), the sunxi_nfc_probe function > > > is called (though the No NAND chip is detected). > > > > > > And if I define the pinctrl-0 in the nfc node, all I see related to > > > NAND/sunxi in the kernel boot log is this line: platform > > > 1c03000.nand-controller: deferred probe pending. > > > > > > Can someone please guide me why the deferred probe is pending but > > > never call the sunxi_nfc_probe function? And without pinctrl-0 entry > > > why probe function is being called? > > > > > > Seems like I am missing something basic here and any pointers to debug > > > this will be appreciated. > > > > > > > > > --- linux-6.3.5.orig/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts > > > 2023-05-30 14:17:29.000000000 +0100 > > > +++ linux-6.3.5/arch/arm/boot/dts/sun7i-a20-olinuxino-micro.dts > > > 2023-08-13 23:27:22.405058025 +0100 > > > @@ -56,7 +56,7 @@ > > > > > > aliases { > > > serial0 = &uart0; > > > - serial1 = &uart6; > > > + serial1 = &uart2; > > > serial2 = &uart7; > > > spi0 = &spi1; > > > spi1 = &spi2; > > > @@ -66,17 +66,6 @@ > > > stdout-path = "serial0:115200n8"; > > > }; > > > > > > - hdmi-connector { > > > - compatible = "hdmi-connector"; > > > - type = "a"; > > > - > > > - port { > > > - hdmi_con_in: endpoint { > > > - remote-endpoint = <&hdmi_out_con>; > > > - }; > > > - }; > > > - }; > > > - > > > leds { > > > compatible = "gpio-leds"; > > > pinctrl-names = "default"; > > > @@ -90,16 +79,55 @@ > > > }; > > > }; > > > > > > -&ahci { > > > - target-supply = <®_ahci_5v>; > > > +&nfc { > > > + pinctrl-names = "default"; > > > + pinctrl-0 = <&nand_pins_a>, <&nand_cs0_pins_a>, <&nand_rb0_pins_a>; > > > status = "okay"; > > > + > > > + nand@0 { > > > + #address-cells = <2>; > > > + #size-cells = <2>; > > > + reg = <0>; > > > + allwinner,rb = <0>; > > > + > > > + nand-ecc-mode = "hw"; > > > + nand-on-flash-bbt; > > > + }; > > > + > > > + boot0@0 { > > > + label = "boot0"; > > > + reg = /bits/ 64 <0x0 0x200000>; > > > + }; > > > + > > > + boot0-rescue@200000 { > > > + label = "boot0-rescue"; > > > + reg = /bits/ 64 <0x200000 0x200000>; > > > + }; > > > + > > > + uboot@400000 { > > > + label = "uboot"; > > > + reg = /bits/ 64 <0x400000 0x200000>; > > > + }; > > > + > > > + uboot-rescue@600000 { > > > + label = "uboot-rescue"; > > > + reg = /bits/ 64 <0x600000 0x200000>; > > > + }; > > > + > > > + main@800000 { > > > + label = "main"; > > > + reg = /bits/ 64 <0x800000 0xff800000>; > > > + }; > > > + > > > }; > > > > > > -&codec { > > > +&ahci { > > > + target-supply = <®_ahci_5v>; > > > status = "okay"; > > > }; > > > > > > &cpu0 { > > > + clock-frequency = <900000000>; > > > cpu-supply = <®_dcdc2>; > > > }; > > > > > > @@ -123,34 +151,24 @@ > > > status = "okay"; > > > }; > > > > > > -&hdmi { > > > - status = "okay"; > > > -}; > > > - > > > -&hdmi_out { > > > - hdmi_out_con: endpoint { > > > - remote-endpoint = <&hdmi_con_in>; > > > - }; > > > -}; > > > - > > > &i2c0 { > > > - status = "okay"; > > > + status = "okay"; > > > > > > - axp209: pmic@34 { > > > - reg = <0x34>; > > > - interrupt-parent = <&nmi_intc>; > > > - interrupts = <0 IRQ_TYPE_LEVEL_LOW>; > > > - }; > > > + axp209: pmic@34 { > > > + reg = <0x34>; > > > + interrupt-parent = <&nmi_intc>; > > > + interrupts = <0 IRQ_TYPE_LEVEL_LOW>; > > > + }; > > > }; > > > > > > &i2c1 { > > > - status = "okay"; > > > + status = "okay"; > > > > > > - eeprom: eeprom@50 { > > > - compatible = "atmel,24c16"; > > > - reg = <0x50>; > > > - pagesize = <16>; > > > - }; > > > + eeprom: eeprom@50 { > > > + compatible = "atmel,24c16"; > > > + reg = <0x50>; > > > + pagesize = <16>; > > > + }; > > > }; > > > > > > &i2c2 { > > > @@ -160,55 +178,6 @@ > > > &lradc { > > > vref-supply = <®_vcc3v0>; > > > status = "okay"; > > > - > > > - button-191 { > > > - label = "Volume Up"; > > > - linux,code = ; > > > - channel = <0>; > > > - voltage = <191274>; > > > - }; > > > - > > > - button-392 { > > > - label = "Volume Down"; > > > - linux,code = ; > > > - channel = <0>; > > > - voltage = <392644>; > > > - }; > > > - > > > - button-601 { > > > - label = "Menu"; > > > - linux,code = ; > > > - channel = <0>; > > > - voltage = <601151>; > > > - }; > > > - > > > - button-795 { > > > - label = "Search"; > > > - linux,code = ; > > > - channel = <0>; > > > - voltage = <795090>; > > > - }; > > > - > > > - button-987 { > > > - label = "Home"; > > > - linux,code = ; > > > - channel = <0>; > > > - voltage = <987387>; > > > - }; > > > - > > > - button-1184 { > > > - label = "Esc"; > > > - linux,code = ; > > > - channel = <0>; > > > - voltage = <1184678>; > > > - }; > > > - > > > - button-1398 { > > > - label = "Enter"; > > > - linux,code = ; > > > - channel = <0>; > > > - voltage = <1398804>; > > > - }; > > > }; > > > > > > &gmac_mdio { > > > @@ -239,10 +208,6 @@ > > > status = "okay"; > > > }; > > > > > > -&otg_sram { > > > - status = "okay"; > > > -}; > > > - > > > &pio { > > > gmac_txerr: gmac-txerr-pin { > > > pins = "PA17"; > > > @@ -323,9 +288,9 @@ > > > status = "okay"; > > > }; > > > > > > -&uart6 { > > > +&uart2 { > > > pinctrl-names = "default"; > > > - pinctrl-0 = <&uart6_pi_pins>; > > > + pinctrl-0 = <&uart2_pi_pins>; > > > status = "okay"; > > > }; > > > > > > @@ -352,3 +317,28 @@ > > > usb2_vbus-supply = <®_usb2_vbus>; > > > status = "okay"; > > > }; > > > + > > > +&fe0 { > > > + status = "disabled"; > > > +}; > > > + > > > +&fe1 { > > > + status = "disabled"; > > > +}; > > > + > > > +&be0 { > > > + status = "disabled"; > > > +}; > > > + > > > +&be1 { > > > + status = "disabled"; > > > +}; > > > + > > > +&tcon0 { > > > + status = "disabled"; > > > +}; > > > + > > > +&tcon1 { > > > + status = "disabled"; > > > +}; > > > + > > > > > > > > > --- linux-6.3.5.orig/arch/arm/boot/dts/sun7i-a20.dtsi 2023-05-30 > > > 14:17:29.000000000 +0100 > > > +++ linux-6.3.5/arch/arm/boot/dts/sun7i-a20.dtsi 2023-08-08 > > > 00:30:04.689288733 +0100 > > > @@ -345,6 +345,58 @@ > > > #size-cells = <0>; > > > }; > > > > > > + nand_pins_a: nand_base0@0 { > > > + allwinner,pins = "PC0", "PC1", "PC2", > > > + "PC5", "PC8", "PC9", "PC10", > > > + "PC11", "PC12", "PC13", "PC14", > > > + "PC15", "PC16"; > > > + allwinner,function = "nand0"; > > > + allwinner,drive = ; > > > + allwinner,pull = ; > > > + }; > > > + > > > + nand_cs0_pins_a: nand_cs@0 { > > > + allwinner,pins = "PC4"; > > > + allwinner,function = "nand0"; > > > + allwinner,drive = ; > > > + allwinner,pull = ; > > > + }; > > > + > > > + nand_cs1_pins_a: nand_cs@1 { > > > + allwinner,pins = "PC3"; > > > + allwinner,function = "nand0"; > > > + allwinner,drive = ; > > > + allwinner,pull = ; > > > + }; > > > + > > > + nand_cs2_pins_a: nand_cs@2 { > > > + allwinner,pins = "PC17"; > > > + allwinner,function = "nand0"; > > > + allwinner,drive = ; > > > + allwinner,pull = ; > > > + }; > > > + > > > + nand_cs3_pins_a: nand_cs@3 { > > > + allwinner,pins = "PC18"; > > > + allwinner,function = "nand0"; > > > + allwinner,drive = ; > > > + allwinner,pull = ; > > > + }; > > > + > > > + nand_rb0_pins_a: nand_rb@0 { > > > + allwinner,pins = "PC6"; > > > + allwinner,function = "nand0"; > > > + allwinner,drive = ; > > > + allwinner,pull = ; > > > + }; > > > + > > > + nand_rb1_pins_a: nand_rb@1 { > > > + allwinner,pins = "PC7"; > > > + allwinner,function = "nand0"; > > > + allwinner,drive = ; > > > + allwinner,pull = ; > > > + }; > > > + > > > spi0: spi@1c05000 { > > > compatible = "allwinner,sun4i-a10-spi"; > > > reg = <0x01c05000 0x1000>; > > > > > > > > > > Thanks for the suggestion diff. Can you submit it as formal patch against > > current sunxi/for-next tree [1]? See > > Documentation/process/submitting-patches.rst for how to properly submit > > patches. > > > > Sorry for my unclear email. I am currently facing issues when trying > to enable and use the MTD driver for A20 SOM (implemented and likely > maintained by Boris Brezillion); I am confused why the probe is not > called in deferred probe case and overall NAND on A20 SOM is not > detected. > > I hope to get help in this regard, and of-course as soon as I got it > working, I will submit patches to be included in the kernel and DTS > files. > > I am not sure why there are DTS files for A20 SOMs (especially one > sunxi-a20-olinuxino-micro.dts which supposed to have NAND on the SOM) > without any NAND pinctrl and nfc node defined. When the kernel have > driver for it and I assume it should be working. > > Added Boris Brezellion in the CC list, hoping to get some pointers and > suggestions from him too. > > Thanks > > Fawad Lateef > > > > Thanks. > > > > [1]: https://git.kernel.org/pub/scm/linux/kernel/git/sunxi/linux.git/log/?h=sunxi/for-next > > > > -- > > An old man doll... just what I always wanted! - Clara _______________________________________________ Kernelnewbies mailing list Kernelnewbies@kernelnewbies.org https://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies