From: Giulio Benetti <giulio.benetti@benettiengineering.com> To: Jesse Taube <mr.bossman075@gmail.com>, Andre Przywara <andre.przywara@arm.com>, Maxime Ripard <mripard@kernel.org>, Chen-Yu Tsai <wens@csie.org>, Jernej Skrabec <jernej.skrabec@gmail.com>, Samuel Holland <samuel@sholland.org> Cc: Rob Herring <robh+dt@kernel.org>, Mesih Kilinc <mesihkilinc@gmail.com>, Icenowy Zheng <icenowy@aosc.io>, George Hilliard <thirtythreeforty@gmail.com>, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev Subject: Re: [PATCH 00/14] ARM: suniv: dts: update Allwinner F1C100 Date: Mon, 7 Mar 2022 19:22:37 +0100 [thread overview] Message-ID: <94974188-9687-6ea0-50a8-0c7eb41cd5bf@benettiengineering.com> (raw) In-Reply-To: <7de7a907-249c-8bdd-b364-689436e3b285@gmail.com> Hi Jesse, On 07/03/22 19:03, Jesse Taube wrote: > > > On 3/7/22 09:34, Andre Przywara wrote: >> The Allwinner F1C100 SoC didn't see much love since its initial merge in >> 2018: the originally submitted .dts files were very basic, and didn't >> cover such simple peripherals as MMC and SPI. >> On top of that the watchdog compatible string was wrong, leading to a >> non-functional watchdog and reset functionality. >> >> This series aims to fix that, after the series MMC and SPI work, and >> make dtbs_check comes back clean. >> This was tested with mounting a filesystem on /dev/mmcblk0 on a >> LicheePi Nano, also with accessing the SPI flash through /dev/mtdblock >> and mtd_debug. Reboot and watchdog now also work. >> >> Mainline U-Boot recently gained F1C100 support, and those DT updates are >> needed there as well to get full MMC and SPI access. > Thank you so much for doing this Giulio and I really appreciate this. > I'm sorry I haven't been available lately i have had other activities > lately. I didn't do it, Andre did it :-) I'm full of other things to do too. Anyway it's great having patches here to be tested and reviewed. Hope to find some time soon to test them. -- Giulio Benetti Benetti Engineering sas >> The series is structured as follows: >> - Patches 01/14 and 02/14 fix the watchdog, which allows to properly >> reboot the system. >> - Patches 03-06 fix some shortcomings of the existing DT files, to make >> them DT binding compliant. >> - Patches 07-09 are Jesse's recent MMC patches, with the comments from >> the last version addressed [1]. >> - Patches 10-12 add SPI support, to enable access to the SPI flash on >> the LicheePi Nano board. >> - The final two patches (13/14 and 14/14) add the F1C100 platform to >> the multi_v5_defconfig, since it was not covered by any other >> defconfig before, and an ARMv5 compliant kernel is not commonly >> offered by distributions. >> >> I saw George's series from two years ago to add USB support[2], that >> looks good on the first glance, I will comment on that once I did some >> testing on that. > There are patches for USB host as well. There are working patches > floating around for the LCD controller, as well as patches for audio, > but audio only works in 5.2 even though none of the files the patches > touched have changed since. To test the LCD controller I made a DAC to > output to VGA unfortunately the blanking regions aren't set to 0v so its > not to VGA spec but if there is a black boarder it works fine. >> Cheers, >> Andre >> >> Changelog for the MMC patches [1]: >> - bindings doc: extend commit message >> - .dtsi: extend commit message, re-order mmc0_pins node, add >> drive-strength >> - .dts: extend commit message, add alias, regulator and disable-wp >> >> [1] https://lore.kernel.org/linux-arm-kernel/20220130220325.1983918-1-Mr.Bossman075@gmail.com/ >> [2] https://lore.kernel.org/linux-usb/20200331170219.267732-1-thirtythreeforty@gmail.com/ >> >> Andre Przywara (10): >> dt-bindings: watchdog: sunxi: fix F1C100s compatible >> ARM: dts: suniv: F1C100: fix watchdog compatible >> dt-bindings: arm: sunxi: document LicheePi Nano name >> ARM: dts: suniv: F1C100: fix CPU node >> ARM: dts: suniv: F1C100: fix timer node >> dt-bindings: spi: sunxi: document F1C100 controllers >> ARM: dts: suniv: F1C100: add SPI support >> ARM: dts: suniv: licheepi-nano: add SPI flash >> ARM: configs: sync multi_v5_defconfig from savedefconfig >> ARM: configs: multi_v5: Enable Allwinner F1C100 >> >> Jesse Taube (4): >> ARM: dts: suniv: F1C100: add clock and reset macros >> dt-bindings: mmc: sunxi: add Allwinner F1c100s compatible >> ARM: dts: suniv: F1C100: add MMC controllers >> ARM: dts: suniv: licheepi-nano: add microSD card >> >> .../devicetree/bindings/arm/sunxi.yaml | 5 + >> .../bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 3 + >> .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + >> .../watchdog/allwinner,sun4i-a10-wdt.yaml | 2 +- >> .../boot/dts/suniv-f1c100s-licheepi-nano.dts | 31 ++++++ >> arch/arm/boot/dts/suniv-f1c100s.dtsi | 102 ++++++++++++++++-- >> arch/arm/configs/multi_v5_defconfig | 25 ++--- >> 7 files changed, 140 insertions(+), 29 deletions(-) >>
WARNING: multiple messages have this Message-ID (diff)
From: Giulio Benetti <giulio.benetti@benettiengineering.com> To: Jesse Taube <mr.bossman075@gmail.com>, Andre Przywara <andre.przywara@arm.com>, Maxime Ripard <mripard@kernel.org>, Chen-Yu Tsai <wens@csie.org>, Jernej Skrabec <jernej.skrabec@gmail.com>, Samuel Holland <samuel@sholland.org> Cc: Rob Herring <robh+dt@kernel.org>, Mesih Kilinc <mesihkilinc@gmail.com>, Icenowy Zheng <icenowy@aosc.io>, George Hilliard <thirtythreeforty@gmail.com>, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev Subject: Re: [PATCH 00/14] ARM: suniv: dts: update Allwinner F1C100 Date: Mon, 7 Mar 2022 19:22:37 +0100 [thread overview] Message-ID: <94974188-9687-6ea0-50a8-0c7eb41cd5bf@benettiengineering.com> (raw) In-Reply-To: <7de7a907-249c-8bdd-b364-689436e3b285@gmail.com> Hi Jesse, On 07/03/22 19:03, Jesse Taube wrote: > > > On 3/7/22 09:34, Andre Przywara wrote: >> The Allwinner F1C100 SoC didn't see much love since its initial merge in >> 2018: the originally submitted .dts files were very basic, and didn't >> cover such simple peripherals as MMC and SPI. >> On top of that the watchdog compatible string was wrong, leading to a >> non-functional watchdog and reset functionality. >> >> This series aims to fix that, after the series MMC and SPI work, and >> make dtbs_check comes back clean. >> This was tested with mounting a filesystem on /dev/mmcblk0 on a >> LicheePi Nano, also with accessing the SPI flash through /dev/mtdblock >> and mtd_debug. Reboot and watchdog now also work. >> >> Mainline U-Boot recently gained F1C100 support, and those DT updates are >> needed there as well to get full MMC and SPI access. > Thank you so much for doing this Giulio and I really appreciate this. > I'm sorry I haven't been available lately i have had other activities > lately. I didn't do it, Andre did it :-) I'm full of other things to do too. Anyway it's great having patches here to be tested and reviewed. Hope to find some time soon to test them. -- Giulio Benetti Benetti Engineering sas >> The series is structured as follows: >> - Patches 01/14 and 02/14 fix the watchdog, which allows to properly >> reboot the system. >> - Patches 03-06 fix some shortcomings of the existing DT files, to make >> them DT binding compliant. >> - Patches 07-09 are Jesse's recent MMC patches, with the comments from >> the last version addressed [1]. >> - Patches 10-12 add SPI support, to enable access to the SPI flash on >> the LicheePi Nano board. >> - The final two patches (13/14 and 14/14) add the F1C100 platform to >> the multi_v5_defconfig, since it was not covered by any other >> defconfig before, and an ARMv5 compliant kernel is not commonly >> offered by distributions. >> >> I saw George's series from two years ago to add USB support[2], that >> looks good on the first glance, I will comment on that once I did some >> testing on that. > There are patches for USB host as well. There are working patches > floating around for the LCD controller, as well as patches for audio, > but audio only works in 5.2 even though none of the files the patches > touched have changed since. To test the LCD controller I made a DAC to > output to VGA unfortunately the blanking regions aren't set to 0v so its > not to VGA spec but if there is a black boarder it works fine. >> Cheers, >> Andre >> >> Changelog for the MMC patches [1]: >> - bindings doc: extend commit message >> - .dtsi: extend commit message, re-order mmc0_pins node, add >> drive-strength >> - .dts: extend commit message, add alias, regulator and disable-wp >> >> [1] https://lore.kernel.org/linux-arm-kernel/20220130220325.1983918-1-Mr.Bossman075@gmail.com/ >> [2] https://lore.kernel.org/linux-usb/20200331170219.267732-1-thirtythreeforty@gmail.com/ >> >> Andre Przywara (10): >> dt-bindings: watchdog: sunxi: fix F1C100s compatible >> ARM: dts: suniv: F1C100: fix watchdog compatible >> dt-bindings: arm: sunxi: document LicheePi Nano name >> ARM: dts: suniv: F1C100: fix CPU node >> ARM: dts: suniv: F1C100: fix timer node >> dt-bindings: spi: sunxi: document F1C100 controllers >> ARM: dts: suniv: F1C100: add SPI support >> ARM: dts: suniv: licheepi-nano: add SPI flash >> ARM: configs: sync multi_v5_defconfig from savedefconfig >> ARM: configs: multi_v5: Enable Allwinner F1C100 >> >> Jesse Taube (4): >> ARM: dts: suniv: F1C100: add clock and reset macros >> dt-bindings: mmc: sunxi: add Allwinner F1c100s compatible >> ARM: dts: suniv: F1C100: add MMC controllers >> ARM: dts: suniv: licheepi-nano: add microSD card >> >> .../devicetree/bindings/arm/sunxi.yaml | 5 + >> .../bindings/mmc/allwinner,sun4i-a10-mmc.yaml | 3 + >> .../bindings/spi/allwinner,sun6i-a31-spi.yaml | 1 + >> .../watchdog/allwinner,sun4i-a10-wdt.yaml | 2 +- >> .../boot/dts/suniv-f1c100s-licheepi-nano.dts | 31 ++++++ >> arch/arm/boot/dts/suniv-f1c100s.dtsi | 102 ++++++++++++++++-- >> arch/arm/configs/multi_v5_defconfig | 25 ++--- >> 7 files changed, 140 insertions(+), 29 deletions(-) >> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-03-07 18:37 UTC|newest] Thread overview: 94+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-03-07 14:34 [PATCH 00/14] ARM: suniv: dts: update Allwinner F1C100 Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-07 14:34 ` [PATCH 01/14] dt-bindings: watchdog: sunxi: fix F1C100s compatible Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-08 16:08 ` Rob Herring 2022-03-08 16:08 ` Rob Herring 2022-03-09 23:02 ` Guenter Roeck 2022-03-09 23:02 ` Guenter Roeck 2022-03-10 0:46 ` Samuel Holland 2022-03-10 0:46 ` Samuel Holland 2022-03-14 17:39 ` Andre Przywara 2022-03-14 17:39 ` Andre Przywara 2022-03-07 14:34 ` [PATCH 02/14] ARM: dts: suniv: F1C100: fix watchdog compatible Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-09 23:03 ` Guenter Roeck 2022-03-09 23:03 ` Guenter Roeck 2022-03-07 14:34 ` [PATCH 03/14] dt-bindings: arm: sunxi: document LicheePi Nano name Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-08 16:09 ` Rob Herring 2022-03-08 16:09 ` Rob Herring 2022-03-11 1:27 ` Samuel Holland 2022-03-11 1:27 ` Samuel Holland 2022-03-07 14:34 ` [PATCH 04/14] ARM: dts: suniv: F1C100: add clock and reset macros Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-11 1:30 ` Samuel Holland 2022-03-11 1:30 ` Samuel Holland 2022-03-07 14:34 ` [PATCH 05/14] ARM: dts: suniv: F1C100: fix CPU node Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-08 2:44 ` Jesse Taube 2022-03-08 2:44 ` Jesse Taube 2022-03-08 4:23 ` Icenowy Zheng 2022-03-08 4:23 ` Icenowy Zheng 2022-03-08 10:42 ` Andre Przywara 2022-03-08 10:42 ` Andre Przywara 2022-03-07 14:34 ` [PATCH 06/14] ARM: dts: suniv: F1C100: fix timer node Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-11 2:19 ` Samuel Holland 2022-03-11 2:19 ` Samuel Holland 2022-03-07 14:34 ` [PATCH 07/14] dt-bindings: mmc: sunxi: add Allwinner F1c100s compatible Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-08 16:10 ` Rob Herring 2022-03-08 16:10 ` Rob Herring 2022-03-11 2:19 ` Samuel Holland 2022-03-11 2:19 ` Samuel Holland 2022-03-11 15:41 ` Ulf Hansson 2022-03-11 15:41 ` Ulf Hansson 2022-03-07 14:34 ` [PATCH 08/14] ARM: dts: suniv: F1C100: add MMC controllers Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-11 2:19 ` Samuel Holland 2022-03-11 2:19 ` Samuel Holland 2022-03-07 14:34 ` [PATCH 09/14] ARM: dts: suniv: licheepi-nano: add microSD card Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-11 2:19 ` Samuel Holland 2022-03-11 2:19 ` Samuel Holland 2022-03-07 14:34 ` [PATCH 10/14] dt-bindings: spi: sunxi: document F1C100 controllers Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-08 16:10 ` Rob Herring 2022-03-08 16:10 ` Rob Herring 2022-03-11 2:19 ` Samuel Holland 2022-03-11 2:19 ` Samuel Holland 2022-03-07 14:34 ` [PATCH 11/14] ARM: dts: suniv: F1C100: add SPI support Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-11 2:19 ` Samuel Holland 2022-03-11 2:19 ` Samuel Holland 2022-03-11 13:33 ` Andre Przywara 2022-03-11 13:33 ` Andre Przywara 2022-03-07 14:34 ` [PATCH 12/14] ARM: dts: suniv: licheepi-nano: add SPI flash Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-11 2:20 ` Samuel Holland 2022-03-11 2:20 ` Samuel Holland 2022-03-07 14:34 ` [PATCH 13/14] ARM: configs: sync multi_v5_defconfig from savedefconfig Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-08 9:38 ` Arnd Bergmann 2022-03-08 9:38 ` Arnd Bergmann 2022-03-08 12:07 ` Andre Przywara 2022-03-08 12:07 ` Andre Przywara 2022-03-08 13:33 ` Arnd Bergmann 2022-03-08 13:33 ` Arnd Bergmann 2022-03-08 13:40 ` Arnd Bergmann 2022-03-08 13:40 ` Arnd Bergmann 2022-03-08 14:30 ` Nicolas Ferre 2022-03-08 14:30 ` Nicolas Ferre 2022-03-08 15:17 ` Arnd Bergmann 2022-03-08 15:17 ` Arnd Bergmann 2022-03-10 10:33 ` Andre Przywara 2022-03-10 10:33 ` Andre Przywara 2022-03-07 14:34 ` [PATCH 14/14] ARM: configs: multi_v5: Enable Allwinner F1C100 Andre Przywara 2022-03-07 14:34 ` Andre Przywara 2022-03-07 18:03 ` [PATCH 00/14] ARM: suniv: dts: update " Jesse Taube 2022-03-07 18:03 ` Jesse Taube 2022-03-07 18:22 ` Giulio Benetti [this message] 2022-03-07 18:22 ` Giulio Benetti 2022-03-11 1:38 ` Jesse Taube 2022-03-11 1:38 ` Jesse Taube
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=94974188-9687-6ea0-50a8-0c7eb41cd5bf@benettiengineering.com \ --to=giulio.benetti@benettiengineering.com \ --cc=andre.przywara@arm.com \ --cc=devicetree@vger.kernel.org \ --cc=icenowy@aosc.io \ --cc=jernej.skrabec@gmail.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-sunxi@lists.linux.dev \ --cc=mesihkilinc@gmail.com \ --cc=mr.bossman075@gmail.com \ --cc=mripard@kernel.org \ --cc=robh+dt@kernel.org \ --cc=samuel@sholland.org \ --cc=thirtythreeforty@gmail.com \ --cc=wens@csie.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.