From patchwork Sat Nov 27 16:41:32 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Niklas_S=C3=B6derlund?= X-Patchwork-Id: 12642415 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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 59005C433F5 for ; Sat, 27 Nov 2021 16:44:16 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230017AbhK0Qra (ORCPT ); Sat, 27 Nov 2021 11:47:30 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:37616 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232002AbhK0Qp3 (ORCPT ); Sat, 27 Nov 2021 11:45:29 -0500 Received: from mail-wr1-x42c.google.com (mail-wr1-x42c.google.com [IPv6:2a00:1450:4864:20::42c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 15A63C06173E for ; Sat, 27 Nov 2021 08:42:15 -0800 (PST) Received: by mail-wr1-x42c.google.com with SMTP id u1so25823730wru.13 for ; Sat, 27 Nov 2021 08:42:14 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ragnatech-se.20210112.gappssmtp.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=MUGQw/RbSx1j8ovUrt/ZuOt4au7QA9TK8ji9FOOrBxk=; b=HhMegenwDHWa3zeqMY6cCs4h5577RWILuB6Op9U7qQAxFirGeZePeF5AfYNNj+HDV1 90x7ylYa9LlwbL3i+fSFA2lioaSpK8wBy9Dtb1fgg8553kJhVaKeDYn0oQXiIWT5ySem k8Hig1nWVz2UgUL6z++nF5S1uT4ZLWA/vHl8VMporJE7KITAjiZR10UqSFCS9JC+erFE jhYkciV8F9eYTyyGDTid1GNGjskfZUthK8oIIDPRTglbZ+9cuvdijet/iG1hZ5XqkVYS 8MBfvft9Kd3Zv+c84ZItSJmku2/N/GwizgHzhyiNZJF5olpTfdENjT57kt3LOqS7liiN NFNQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=MUGQw/RbSx1j8ovUrt/ZuOt4au7QA9TK8ji9FOOrBxk=; b=mddqcGmlcD4aRPXnMhcx1KrcHFIdXXBmuej+AK3qRRPDFVqPikbVU2d2CtTo66zuy3 YM1Sk7GfU/rgyWtI+pyhRa2kU6FYbM1bSb7pk23iCYvhShrLGTRbWZLda526zqSFcGxp 1oTylRpDX4PLkM2W0M2Gofb3k7/YLKPEAy7xxQDP6vTXgNK/TH76+FA5DaY8+beL3L9q KK4ObxQuhGLtuLJln3Mke0vkSBTtxh7w81F8eWR/1b01uxpKU9Erkc6dYxfi1Pzw75NW tBYH2J7MRftrw7FInmtOGoCu+PIKyDuJ624c+wAE9GHdAGCqhkes4LRUAFC4SXjB+scG c9Xw== X-Gm-Message-State: AOAM531z1PbZv69FEhwxmPygqh8c7euxNxG/M0MrJKqeCPbS1d/UrPOj d0hMjp0hEkQMuN0rOh0gwqCOjA== X-Google-Smtp-Source: ABdhPJz9f8epAGt2qNO9xIHO1qqlBY5wWwmL2tBRGYp0XoaAf4FacZhagRuavoC+cFRMNbodBEeYTA== X-Received: by 2002:adf:e848:: with SMTP id d8mr22196995wrn.3.1638031333462; Sat, 27 Nov 2021 08:42:13 -0800 (PST) Received: from bismarck.berto.se (p54ac5892.dip0.t-ipconnect.de. [84.172.88.146]) by smtp.googlemail.com with ESMTPSA id u13sm2900104wmq.14.2021.11.27.08.42.12 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 27 Nov 2021 08:42:13 -0800 (PST) From: =?utf-8?q?Niklas_S=C3=B6derlund?= To: Hans Verkuil , Jacopo Mondi , Laurent Pinchart , linux-media@vger.kernel.org Cc: linux-renesas-soc@vger.kernel.org, =?utf-8?q?Niklas_S=C3=B6derlund?= Subject: [PATCH v2 0/3] media: rcar-{csi2,vin}: Move to full Virtual Channel routing per CSI-2 IP Date: Sat, 27 Nov 2021 17:41:32 +0100 Message-Id: <20211127164135.2617686-1-niklas.soderlund+renesas@ragnatech.se> X-Mailer: git-send-email 2.34.0 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org Hello, This series attempts to increase the flexibility of the R-Car VIN capture pipeline by allowing for free form Virtual Channel routing within the same CSI-2 IP block. When Gen3 support was first added to this R-Car VIN and CSI-2 driver the routing was centred around the CHSEL register which multiplex the different parallel buses that sits between the CSI-2 receivers source side and the VIN dma engines. This was a bad design as the multiplexing do allow for only a few combinations and do not play nice with many video streams in the system. For example it's only possible for CSI-2 Virtual Channels 0 and 1 of any given CSI-2 receiver to be used together with the scaler. Later datasheets have expanded the documentation and it is now possible to improve on this design by allowing any Virtual Channel to be routed to any R-Car VIN instance, provided that there exists a parallel bus between them. This increases the flexibility as all Virtual Channels can now be used together with the scaler for example. The redesign is not however perfect. While the new design allows for many more routes, two constrains limit a small portion of routes that was possible in the old design but are no more. - It is no longer possible to route the same CSI-2 and VC to more then one VIN at a time. This was theoretically possible before if the specific SoC allowed for the same CSI-2 and VC to be routed to two different VIN capture groups. - It is no longer possible to simultaneously mix links from two CSI-2 IP blocks to the same VIN capture group. For example if VIN2 is capturing from CSI40 then VIN{0,1,3} must also capture from CSI40. While VIN{4,5,6,7} is still free to capture from any other CSI-2 IP in the system. Once all VIN{0,1,2,3} links to CSI40 are disabled that VIN capture group is free again to capture from any other CSI-2 IP it is connected to. At the core of the redesign is greater cooperator of the R-Car VIN and CSI-2 drivers in configuring the routing. The VIN driver is after this change only responsible to configure the full VIN capture groups parallel buses to be to a particular CSI-2 IP. While the configuration of which CSI-2 Virtual Channel is outputted on which of the R-Car CSI-2 IP output ports is handled by the CSI-2 driver. Before this change the CSI-2 Virtual Channel to output port was static in the CSI-2 driver and the different links only manipulated the VIN capture groups CHSEL register. With this change both the CHSEl register and the CSI-2 routing VCDT registers are modified for greater flexibility. Patch 1/3 and 2/3 are cleanup patches moving code around preparing for the real work in 3/3. The work is based on the latest media-tree. Kind Regards, Niklas Söderlund Niklas Söderlund (3): media: rcar-vin: Refactor link notify media: rcar-vin: Breakout media link creation media: rcar-{csi2,vin}: Move to full Virtual Channel routing per CSI-2 IP drivers/media/platform/rcar-vin/rcar-core.c | 387 ++++++-------------- drivers/media/platform/rcar-vin/rcar-csi2.c | 58 ++- drivers/media/platform/rcar-vin/rcar-dma.c | 2 +- drivers/media/platform/rcar-vin/rcar-vin.h | 18 +- 4 files changed, 173 insertions(+), 292 deletions(-)