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 mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 47D19C433F5 for ; Fri, 19 Nov 2021 16:29:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 3126461B27 for ; Fri, 19 Nov 2021 16:29:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236463AbhKSQcz (ORCPT ); Fri, 19 Nov 2021 11:32:55 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59074 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233713AbhKSQcy (ORCPT ); Fri, 19 Nov 2021 11:32:54 -0500 Received: from mail-il1-x132.google.com (mail-il1-x132.google.com [IPv6:2607:f8b0:4864:20::132]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C4F64C06173E for ; Fri, 19 Nov 2021 08:29:52 -0800 (PST) Received: by mail-il1-x132.google.com with SMTP id i9so5257404ilu.1 for ; Fri, 19 Nov 2021 08:29:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ndufresne-ca.20210112.gappssmtp.com; s=20210112; h=message-id:subject:from:to:cc:date:in-reply-to:references :user-agent:mime-version:content-transfer-encoding; bh=NA+2l6X3VrlJxsJamfaAblN1iQr9G5GeyeDjrzqWldE=; b=pWYGr8brYDg2I9xcuGuhurThossAPr1q87fBQPqCiM2BAja4C7uFnX65Fix38gLbMR at4/FYFXGd1o9zn1i0YRa4Hl3P62bx1s2GsXPKOGSzrVsAT1Xw8HlYNWbORdzoDIOeqe 0qslCNUJ66VzbM5OECQeydfyhNNqUWwGP6afEEwmpoOMoNIZnNaZfchouPoszwNYzV1v y1bimltU6qqxfdJSKgW6NjHgo6foggqrexAHaTR5BhNExaTs0kreVzSvKeiuvCJBGMM5 4hhlHH8MqKp2ss3LA9dpdsxQOKHOGkUr880PDTOgcILg0r5nps7M4Bb//54zUt1tJBXH UXjw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:subject:from:to:cc:date:in-reply-to :references:user-agent:mime-version:content-transfer-encoding; bh=NA+2l6X3VrlJxsJamfaAblN1iQr9G5GeyeDjrzqWldE=; b=3+3QA1rhAYY2VyJ9mfk7WAXe1lZ11JDyt8VF5/YRQb+UZo+izvt/Ds6x11dLbNQqhc ZPC6pyYt8sz96qgg/eDVm3Esb7p2P45YdsYVxg406Q4WTOHo20iLskTYG44nO2bKfjRp ZxAXpTFYPTcI/wAdLhh+kV1PMCEHE8mjGtqddm0M/9xn3G9EpxYcrQjgpPlfNYAoWfCd WJoLzzZOV2dYDoCw9CWvnxpYBb8kHcBGo/duAsICz+cW43O0X7Wwe0ZSO7cHylIT6PK5 Rw5uOx3cw/Ug70R8ibCsW++fsvMpDi4q84Ja8pHCUBT7vGUnjWRM/KX9mjUMm2X0yCSV +M8A== X-Gm-Message-State: AOAM530dGPnYuX0aGk4uEgDwwKwUptYjkria5yWRkiHoXaViKmw87le+ 9z59eH0LQVDq+nd8yLoKZbSNyQ== X-Google-Smtp-Source: ABdhPJy9+PK69iHyuVZcAK9TkCljtUM0G3YiAMHQdYjwnsW/n1HgnSVsWmM0mEP/36cqcfaj9/5ecQ== X-Received: by 2002:a92:d343:: with SMTP id a3mr5688828ilh.136.1637339392190; Fri, 19 Nov 2021 08:29:52 -0800 (PST) Received: from nicolas-tpx395.localdomain (mtl.collabora.ca. [66.171.169.34]) by smtp.gmail.com with ESMTPSA id c7sm108143iob.28.2021.11.19.08.29.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 19 Nov 2021 08:29:51 -0800 (PST) Message-ID: <7f94eaacfddb8c5434c17f1e069ea87a17657ce9.camel@ndufresne.ca> Subject: Re: [RFC 0/5] arm64: imx8mm: Enable Hantro VPUs From: Nicolas Dufresne To: Adam Ford , Tim Harvey Cc: linux-media , Schrempf Frieder , Marek Vasut , Jagan Teki , Adam Ford-BE , cstevens@beaconembedded.com, Ezequiel Garcia , Philipp Zabel , Mauro Carvalho Chehab , Rob Herring , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Greg Kroah-Hartman , Heiko Stuebner , Lucas Stach , Joakim Zhang , Alice Guo , Peng Fan , "open list:HANTRO VPU CODEC DRIVER" , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE" , open list , "open list:STAGING SUBSYSTEM" Date: Fri, 19 Nov 2021 11:29:48 -0500 In-Reply-To: References: <20211106183802.893285-1-aford173@gmail.com> <718f7f6d6cd564d031c1963f1590c62d549ae725.camel@ndufresne.ca> <8db00a4b6faa99c940d9bc86e17161eb0db5efe3.camel@ndufresne.ca> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.42.1 (3.42.1-1.fc35) MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Adam, Tim, [...] > > > > Nicolas and Adam, > > > > > > > > For the H1 patches in this series: I've been able to test the IMX8MM > > > > H1 JPEG encode using GStreamer 1.18.5: > > > > $ gst-inspect-1.0 | grep -e "v4l2.*enc" > > > > video4linux2: v4l2jpegenc: V4L2 JPEG Encoder > > > > $ gst-launch-1.0 videotestsrc ! jpegenc ! rtpjpegpay ! udpsink > > > ^ v4l2jpegenc > > > > > > This is just a transcript error ? > > > > Nicolas, > > > > No! Thanks for catching my mistake. I was testing with software encode... ooops! > > > > 'gst-launch-1.0 videotestsrc ! v4l2jpegenc ! fakesink' actually hangs > > the board so likely a power-domain issue there? > > The v4l2-compliance tests fail on the h1 decoder with a hang, but I > think we're writing to registers which are not documented in the Mini > TRM. The Mini TRM doesn't explicitly show the JPEG encoding as a > feature, but some of the registers state JPEG, but because some of the > registers written for the H1 are not documented in the TRM. If those > registers are restricted or not in this SoC, I am concerned that it > might be related. I'll try to run some more tests this weekend to > check on the status of the power-domain stuff. To verify if the HW support JPEG encoding you can read SWREG63 bit 25. This is in the TRM, just not labelled properly. To mimic the decoding side, would be "HW synthesis config register X" with the bit labelled SW_ENC_JPEG_PROF (but PROF/profile is on or off). If your board hang while reading this, you likely didn't get the power bit right. IMX8 has an undocumented control block thing that we have been fighting with in imx8q, perhaps that's your issue. Few driver was proposed, we are still pending on NXP solution to be submitted (they asked us to wait, still waiting =)). > > > > > > > > > host=192.168.1.146 port=5000 > > > > viewed on client@192.168.1.146 via: > > > > $ gst-launch-1.0 udpsrc port=5000 ! application/x-rtp,payload=96 ! > > > > rtpjpegdepay ! jpegdec ! autovideosink > > > > > > > > For the G1/G2 patches in the series I don't see any Gstreamer > > > > 'v4l2.*dec' elements. Perhaps I need a newer version of Gstreamer. > > > > > > Most likely yes, I suggest building gstreamer/ branch "main", GStreamer has now > > > a single repository. We are very close to 1.20, which will include stable API > > > support of H264, MPEG2 and VP8 decoding. > > > > > > > Ok, let me see if I can navigate through the build process and I'll > > get back to you. > > > > Thanks, > > > > Tim > > > > > > > > > > I have CSI capture and DSI display currently working on > > > > imx8mm-venice-gw73xx-0x that I can play with. The CSI sensor only > > > > supports RAW8/RAW10 (and gstreamer currently only supports RAW8) and I > > > > can't efficiently convert to something the JPEG encoder likes without > > > > bayer2rgbneon (a libneon version). > > > > > > > > I see from the IMX8MMRM that the 2D GPU supports scaling etc with a > > > > wide range of data formats but I'm not sure how to tap into this as > > > > that hardware is managed by the vivante driver. On the IMX6QDL there > > > > is a separate IPU block that Philipp Zabel wrote a nice mem2mem > > > > csc/scaler driver for but I don't see any equivalent currently for > > > > IMX8MM. > > > > > > > > Best regards, > > > > > > > > Tim > > >