From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [PATCH v2 00/15] Add basic support for Socionext Milbeaut M10V SoC Date: Mon, 18 Feb 2019 13:29:26 +0100 Message-ID: <10a0375b-0814-6b7d-97d5-ed2c2838c4b4@linaro.org> References: <1549628687-29628-1-git-send-email-sugaya.taichi@socionext.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Arnd Bergmann , Sugaya Taichi Cc: DTML , Linux Kernel Mailing List , Linux ARM , linux-clk , "open list:GPIO SUBSYSTEM" , linux-serial@vger.kernel.org, Rob Herring , Mark Rutland , Michael Turquette , Stephen Boyd , Linus Walleij , Greg Kroah-Hartman , Thomas Gleixner , Russell King , Jiri Slaby , Takao Orito , Kazuhiro Kasai , Shinji Kanematsu , Jassi List-Id: linux-gpio@vger.kernel.org On 18/02/2019 13:20, Arnd Bergmann wrote: > On Fri, Feb 8, 2019 at 1:24 PM Sugaya Taichi > wrote: >> >> Hi, >> >> Here is the series of patches the initial support for SC2000(M10V) of >> Milbeaut SoCs. "M10V" is the internal name of SC2000, so commonly used in >> source code. >> >> SC2000 is a SoC of the Milbeaut series. equipped with a DSP optimized for >> computer vision. It also features advanced functionalities such as 360-degree, >> real-time spherical stitching with multi cameras, image stabilization for >> without mechanical gimbals, and rolling shutter correction. More detail is >> below: >> https://www.socionext.com/en/products/assp/milbeaut/SC2000.html >> >> Specifications for developers are below: >> - Quad-core 32bit Cortex-A7 on ARMv7-A architecture >> - NEON support >> - DSP >> - GPU >> - MAX 3GB DDR3 >> - Cortex-M0 for power control >> - NAND Flash Interface >> - SD UHS-I >> - SD UHS-II >> - SDIO >> - USB2.0 HOST / Device >> - USB3.0 HOST / Device >> - PCI express Gen2 >> - Ethernet Engine >> - I2C >> - UART >> - SPI >> - PWM >> >> Support is quite minimal for now, since it only includes timer, clock, >> pictrl and serial controller drivers, so we can only boot to userspace >> through initramfs. Support for the other peripherals will come eventually. > > I've looked over the platform once more. Overall, it looks very good, and > I'd still like to merge this for linux-5.1, but we are running out of time > there. If you send the patches to soc@kernel.org quickly, we can try to > still merge them in, but if anything goes wrong, it will have to wait until > we start merging patches for 5.2, directly after 5.1 is out. > > I did not look at the device driver patches (clk, clocksource, pinctrl, serial) > in much detail. If you have an Ack from the maintainers, feel free to > include them in the series, otherwise let's merge the rest now and then > you can send the updated patches for inclusion through the subsystem > trees in 5.2. I'm not in copy of the clocksource patch but I acked it for the initial post. > I have sent a few comments. The only one that is really important > here is the missing platform check in the suspend options. Please > try to address most of the other commentsm, either by changing the > code, or by explaining why your version is correct. > > > > Arnd > -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog