From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-qk1-f182.google.com (mail-qk1-f182.google.com [209.85.222.182]) (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 096F863B9 for ; Thu, 9 Jan 2025 14:07:00 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.222.182 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736431622; cv=none; b=FxO399Um1JvONxuWp2EWBA1aETH4T8jBTjMzQflwTkE8hOeYqXRwMYI0HdSMe6fzENDfE10hVSOJ/R4beX/gF0dEL6Mv7LkGkI3ut4abIppXnq6MQWLsOtgsZTeI8MWlCqlSddRJz2mudvX9UMJOVT5TYFHgnjFLkZwo6iaOHxQ= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1736431622; c=relaxed/simple; bh=Ym1yDocfM13GAXNnJN+26yjZUBZZ3aFj68yIdbE9lfY=; h=Message-ID:Subject:From:To:Cc:Date:In-Reply-To:References: Content-Type:MIME-Version; b=WfA3IHVpZX62CyJtwN9Y4h+DsXTqg313FRlZZYd5z4cSvaS1QKBf/YStYM4Y0ywCBJjWW7hlGbWxYgbyAp5hbudMG+djYiJttvt4G4zFDGG3sjryXpqSl/OtNCforNfDbcEoG/66CqtZx06KRlgDDu6UrSBapfDmdHwQ2HGNDs4= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ndufresne.ca; spf=none smtp.mailfrom=ndufresne.ca; dkim=pass (2048-bit key) header.d=ndufresne-ca.20230601.gappssmtp.com header.i=@ndufresne-ca.20230601.gappssmtp.com header.b=pRSqPvf0; arc=none smtp.client-ip=209.85.222.182 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=ndufresne.ca Authentication-Results: smtp.subspace.kernel.org; spf=none smtp.mailfrom=ndufresne.ca Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=ndufresne-ca.20230601.gappssmtp.com header.i=@ndufresne-ca.20230601.gappssmtp.com header.b="pRSqPvf0" Received: by mail-qk1-f182.google.com with SMTP id af79cd13be357-7b6eb531e13so44315485a.0 for ; Thu, 09 Jan 2025 06:07:00 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ndufresne-ca.20230601.gappssmtp.com; s=20230601; t=1736431620; x=1737036420; darn=vger.kernel.org; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:from:to:cc:subject :date:message-id:reply-to; bh=bywW9vMeiiYOYy2QNzu7ZKqXWuFCGn5oBIdbY0P2hyg=; b=pRSqPvf0JjzAAh42MC3kLlu9ZH8Yx9NH5Iqvr5cdeKRBKwK+6W/r3T0K7WTKUUTRbD vNgCDwhcMQNSGN/SiIb+OhMsRvmAXr1p1o/+45OEdnmVvNuj44BNgkP+y/motY7lMUay x71WWsExf9DMToEjzy6UApIVeBSQySwc32AUEjF5VJ+PVeLhDLaUcK7GW1UetNN0I71B omEtUTugBw1G1PayiX28W95IDPo5HraRLByFyyH5Km5ZruZq2PiywaznhaksvYl1QJ8+ yIQbQCE5e+6KOys+daNY+HlbgjYUfln9zCw8ZIoS6NcB7RLll0TyNQSNfjYPHxMHneqx SXwA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1736431620; x=1737036420; h=mime-version:user-agent:content-transfer-encoding:references :in-reply-to:date:cc:to:from:subject:message-id:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=bywW9vMeiiYOYy2QNzu7ZKqXWuFCGn5oBIdbY0P2hyg=; b=Ifc2rvU2hc96dFGacrqA7GfBI3Xvrs+51xm1+0UNUxO2hZv11Peoj9D4jyGmRr6oTI OzjoRqD5qjXO/PbQ/D5MOzQE/GImPhYjjNutNmk2Qf5GEM1uW4vhFnNIrJqr3TeKuqOQ 8mDUMSq2y7jbxXYxzqFeCxwES+89rg62USHEhwslmiaHytZKwF9RLscKmKLFQ5XcCyM5 D2I0g7gHn1YkY5BExatWlxKuiyI4BP1buetbw9Gn2yMx1vZ69bRgK8itfvd6ZY7UhVql 4VeogBYWaXSdqsRNnvJyKTxxTMjfUftr09PY9FiD7VI7iDcLfqocU3kVCZ9b7DYKWJIq QTLA== X-Forwarded-Encrypted: i=1; AJvYcCWksIHXklJuDWA40hXjU2JXWI91uIbkH+D2dVaKX/z+34X02t82O2pAhBBbXR7fWJbcjygGV7lkrnKQ@vger.kernel.org X-Gm-Message-State: AOJu0Ywi/bLidqwXdVTl9O7qmQ0u1ZdwU7Hzqk6MsE9FgTT1NHJyyWsm bE0EU2ytD4LzZ5JH8wCNMXeWdaUr6WepwyriDqzxLayubuHgGwGVf4Sw28ZD5SQ= X-Gm-Gg: ASbGncv2v12Ype9f5V+Ie10tmr9ztndR5iVKFjmhHGdCzBI98lYke6dUEa86eQPO+9G DAVQWD9d/v1iUVYHm4Mh/xMXPyIGguRDb6kqDxjPsq1N+Dm+2xi0XVpi9q20A/wJp65jMLDz0ta Hpzl2TumCqx9PhDZhEVLpXGZka8FEYcx+L8KN9k/UY8Vou0VgfLcCzlRXhOSwVX4mP6ZzLtLccI unA+/YKb83jqqNq9n5dd/RDZj7Ga0tu56NE8H13HtfzI27nPEfmnqB5dQ== X-Google-Smtp-Source: AGHT+IEscSxY4Do3TXZ6G7JOMUAekHEsOUeBZGyyHfF7ljBFczTdIIJ78AZHYR6zSk6tQnyqAKAV5A== X-Received: by 2002:a05:620a:414a:b0:7b6:dddb:b88 with SMTP id af79cd13be357-7bcd976277dmr964450385a.38.1736431620006; Thu, 09 Jan 2025 06:07:00 -0800 (PST) Received: from nicolas-tpx395.localdomain ([2606:6d00:15:862e::7a9]) by smtp.gmail.com with ESMTPSA id af79cd13be357-7bce327bb28sm69478785a.64.2025.01.09.06.06.58 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 09 Jan 2025 06:06:59 -0800 (PST) Message-ID: Subject: Re: [V1,00/12] Enable jpeg enc & dec multi-hardwares for MT8196 From: Nicolas Dufresne To: "kyrie.wu" , Hans Verkuil , Mauro Carvalho Chehab , Rob Herring , Matthias Brugger , Tzung-Bi Shih Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , AngeloGioacchino Del Regno , Bin Liu , kyrie wu , linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org Date: Thu, 09 Jan 2025 09:06:58 -0500 In-Reply-To: <20250109133513.20151-1-kyrie.wu@mediatek.com> References: <20250109133513.20151-1-kyrie.wu@mediatek.com> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable User-Agent: Evolution 3.54.2 (3.54.2-1.fc41) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Hi, Le jeudi 09 janvier 2025 =C3=A0 21:35 +0800, kyrie.wu a =C3=A9crit=C2=A0: > This series adds support for mt8196 multi-hardwares jpeg enc & dec, > by first adding mt8196 jpegdec and jpegenc compatible to install > kernel driver. Add smmu setting to support smmu and iommu at the > same time. > Secondly refactor buffer and clock setting to support multi-hw jpeg > working. > Lastly, fix some bugs, including resolution change handleing, stop > streaming sw flow and others. >=20 > This series has been tested with MT8196 tast test. > Encoding and decoding worked for this chip. >=20 > Patches 1-3 Adds jpeg encoder and decoder compatible. > Patches 4 add jpeg smmu sid setting. > Patches 5 fix jpeg hw count setting to support different chips. > Patches 6 refactor jpeg buffer payload setting to handle buffer > size bug while resolution changed. > Patches 7 reconstruct jpeg dst buffer layout. > Patches 8 fix multi-core stop streaming flow > Patches 9 refactor multi-core clk suspend/resume setting > Patches 10 fix decoding buffer number setting timing issue > Patches 11 refactor decoding resolution change operation > Patches 12 fix remove buffer operation >=20 > --- > This series patches dependent on: > [1] > https://patchwork.kernel.org/project/linux-mediatek/patch/20240808092555.= 12999-1-jianhua.lin@mediatek.com/ >=20 > kyrie.wu (11): would you mind fixing the --author in your configuration, write your name instead of your user name. UT8 is allowed, many people will use a ASCI spel= ling (some approximation) and then add their real name in parenthesis. regards, Nicolas > dt-bindings: mediatek: Add mediatek, mt8196-jpgdec compatible > dt-bindings: mediatek: Add mediatek, mt8196-jpgenc compatible > media: mediatek: jpeg: add jpeg compatible > media: mediatek: jpeg: add jpeg smmu sid setting > media: mediatek: jpeg: fix jpeg hw count setting > media: mediatek: jpeg: refactor jpeg buffer payload setting > media: mediatek: jpeg: refactor jpeg dst buffer layout > media: mediatek: jpeg: fix stop streaming flow for multi-core > media: mediatek: jpeg: refactor multi-core clk suspend and resume > setting > media: mediatek: jpeg: fix decoding buffer number setting timing issue > media: mediatek: jpeg: refactor decoding resolution change operation > media: mediatek: jpeg: fix remove buffer operation for multi-core >=20 > ....yaml =3D> mediatek,multi-core-jpegdec.yaml} | 10 +- > ....yaml =3D> mediatek,multi-core-jpegenc.yaml} | 10 +- > .../platform/mediatek/jpeg/mtk_jpeg_core.c | 126 ++++++++++++------ > .../platform/mediatek/jpeg/mtk_jpeg_core.h | 17 ++- > .../platform/mediatek/jpeg/mtk_jpeg_dec_hw.c | 116 +++++++++++++++- > .../platform/mediatek/jpeg/mtk_jpeg_dec_hw.h | 4 + > .../platform/mediatek/jpeg/mtk_jpeg_enc_hw.c | 113 +++++++++++++++- > .../platform/mediatek/jpeg/mtk_jpeg_enc_hw.h | 4 + > 8 files changed, 343 insertions(+), 57 deletions(-) > rename Documentation/devicetree/bindings/media/{mediatek,mt8195-jpegdec.= yaml =3D> mediatek,multi-core-jpegdec.yaml} (95%) > rename Documentation/devicetree/bindings/media/{mediatek,mt8195-jpegenc.= yaml =3D> mediatek,multi-core-jpegenc.yaml} (94%) >=20