From patchwork Wed Jul 31 16:44:10 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Link Mauve X-Patchwork-Id: 13748973 Received: from luna.linkmauve.fr (luna.linkmauve.fr [82.65.109.163]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id AC88D1BD515; Wed, 31 Jul 2024 16:53:08 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=82.65.109.163 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722444791; cv=none; b=oHoxlnHj1RdRIPyhF4cm8kSMXW9OhLeF8dETNbCQ93JcgZVbV1b2Kuqmqr7V0p1sNwL9IU7JXmkRRx/43gt38rgt7vMX4qRXALZLPGN9Dy58O7WMPuMHs/V4SNTwAM2W59bBbTYROLmrnfDK7cDepC/BsyQOLocztpXu1D/OUSQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722444791; c=relaxed/simple; bh=c8ptxbLdxQ5/F2reX0rnqqAZCIKgf2WjwASUQ5secZI=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version:Content-Type; b=M1az/60/rXn6u0ZOhNoYyW1QGMv4hIh6GTHO7Nm+WTYmc0mCkRTCC08s5tK4r7l6TFTe8DJneEIvBD6PbdBv8gjFQcbbZhCh9ZowVyt6UQsPqFWy0IKv1JmmYMzEXSYU6s0LS4t6mg2WwfV+pBxR2LcOeglTXwhJhkKXADee8h4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=linkmauve.fr; spf=pass smtp.mailfrom=linkmauve.fr; arc=none smtp.client-ip=82.65.109.163 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=linkmauve.fr Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linkmauve.fr Received: by luna.linkmauve.fr (Postfix, from userid 1000) id CB91714649D5; Wed, 31 Jul 2024 18:44:26 +0200 (CEST) From: Emmanuel Gil Peyrot To: linux-sunxi@lists.linux.dev Cc: Emmanuel Gil Peyrot , Maxime Ripard , Paul Kocialkowski , Mauro Carvalho Chehab , Greg Kroah-Hartman , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-staging@lists.linux.dev, linux-arm-kernel@lists.infradead.org Subject: [PATCH 0/4] media: cedrus: Add a JPEG decoder Date: Wed, 31 Jul 2024 18:44:10 +0200 Message-ID: <20240731164422.206503-1-linkmauve@linkmauve.fr> X-Mailer: git-send-email 2.46.0 Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 The Video Engine present on all (?) AllWinner SoCs includes a JPEG decoder, which shares the same engine as the MPEG2 decoder. I mostly used the linux-sunxi resources on the engine[0], as well as some reverse-engineered userland using the legacy /dev/cedar_dev API[1]. I wrote the Onix library[2] to exercise this driver. With it, I have tested this series on the A10, A20 and A64, as found in Olimex Lime, Lime 2 and PinePhone respectively, so further testing on other SoCs would be very welcome. On the A10 and A20, since only the tiled ST12 format is supported, the file dump binary is a bit unusable, but I have tested displaying to DRM and to Wayland just fine on these SoCs. They also only support downscaling from 4:4:4 to 4:2:2, unlike newer SoCs which support downscaling from either 4:4:4 or 4:2:2 to 4:2:2 or 4:2:0. This prompts the issue of which formats to advertise: currently I chose to only expose NV12, but ideally NV16 would be exposed when the user tries to decode a 4:2:2 or 4:4:4 image. By the time the user has called ENUM_FMTS on the capture queue, the driver doesn’t know the subsampling in the JPEG file, so it can’t expose only the supported formats. A possible solution could be to add a new v4l2_ext_ctrl describing the JPEG, but this is a new uAPI which should get discussed outside of this driver. [0] https://linux-sunxi.org/VE_Register_guide [1] https://github.com/jemk/cedrus [2] https://linkmauve.fr/dev/onix/ Emmanuel Gil Peyrot (1): media: cedrus: Add JPEG decoder Jernej Skrabec (3): media: cedrus: Setup secondary output formats media: cedrus: Enable JPEG decoding in various variants media: cedrus: Don't require requests for all codecs drivers/staging/media/sunxi/cedrus/Kconfig | 1 + drivers/staging/media/sunxi/cedrus/Makefile | 2 +- drivers/staging/media/sunxi/cedrus/cedrus.c | 10 + drivers/staging/media/sunxi/cedrus/cedrus.h | 5 + .../staging/media/sunxi/cedrus/cedrus_hw.c | 14 +- .../staging/media/sunxi/cedrus/cedrus_jpeg.c | 352 ++++++++++++++++++ .../staging/media/sunxi/cedrus/cedrus_regs.h | 25 +- .../staging/media/sunxi/cedrus/cedrus_video.c | 11 +- 8 files changed, 413 insertions(+), 7 deletions(-) create mode 100644 drivers/staging/media/sunxi/cedrus/cedrus_jpeg.c