| Message ID | 20240127084258.68302-1-eugen.hristev@collabora.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers |
Received: from am.mirrors.kernel.org ([147.75.80.249]) by linuxtv.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from <linux-media+bounces-4257-patchwork=linuxtv.org@vger.kernel.org>) id 1rTeHc-0007pz-2F for patchwork@linuxtv.org; Sat, 27 Jan 2024 08:43:28 +0000 Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 8EB3D1F235CD for <patchwork@linuxtv.org>; Sat, 27 Jan 2024 08:43:21 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 81A5A1DFE8; Sat, 27 Jan 2024 08:43:15 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="p0X6hNJ0" X-Original-To: linux-media@vger.kernel.org Received: from madrid.collaboradmins.com (madrid.collaboradmins.com [46.235.227.194]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 8B7281DDDA; Sat, 27 Jan 2024 08:43:13 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=46.235.227.194 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706344995; cv=none; b=RCOGjPM60aYw45ww0QJxGcH7i/X48vIohwZNWE+e4APEN64/nsEbr2OqfGe2NKLN4XZ3MfBlAqoaITOtz2kD9WeKAfoKA50bnCGWA7Xl76gjxojRXfJWNZzMv0CgbORoKG4naGJU/VchqF3izcOU0iwHgDFZDe1jMJBQNaZdGro= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706344995; c=relaxed/simple; bh=8dRsLzcjxuiEqHzBiXW/6CBdzTfn9KdLsULhFxOyGP8=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=ZGwyCfhAWayt2rFSK0ZeSaJ4C5uU2Ml9QFQZo8LXo5jL99l9rjTBbBdTvUQTdTxXkD26xTTWNAHf4Fpo+TcY6sqekCg/hSVbUT2wjhTCyiWytfTRwfLC6hQIdv07IXclZJKnQPCGluFZUrHORmZHYYrhVIVNubPnI2af5S2cJNk= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=collabora.com; spf=pass smtp.mailfrom=collabora.com; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b=p0X6hNJ0; arc=none smtp.client-ip=46.235.227.194 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=collabora.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=collabora.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1706344991; bh=8dRsLzcjxuiEqHzBiXW/6CBdzTfn9KdLsULhFxOyGP8=; h=From:To:Cc:Subject:Date:From; b=p0X6hNJ0/FNbHA3OoNK9VWsNQk1wTfyIk7rVN5WxanigwvN3uMMNOy+NbhOFePM+Q f3oG0AhM7w4NI8jCaIJjZKVgeapnd4i2FDK0z+TUH1WpLYmV6bsAnm1h8HgiabB9qB YmUB7LAN/0ZfJWFcX9y4XAeBTue3leCepR5Z2/9frosmd+l1cqEu1r2zOQEPFhdt1r 4/Yq2e6rdDMZgG6xxC9XzAEDeHg+4fa/68DtBZkeCPk126eyb6qeIpbhCbQlI1hb/u rMi4WKI61iFLIM0poe6SqENgedUxqXHar+CMB3MuAGXkx0mIcTS+F/6TTVdDkSCG38 29WBSqkdpARVw== Received: from eugen-station.. (cola.collaboradmins.com [195.201.22.229]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: ehristev) by madrid.collaboradmins.com (Postfix) with ESMTPSA id 21F1D37809D0; Sat, 27 Jan 2024 08:43:07 +0000 (UTC) From: Eugen Hristev <eugen.hristev@collabora.com> To: linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-mediatek@lists.infradead.org, bin.liu@mediatek.com Cc: linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, angelogioacchino.delregno@collabora.com, kernel@collabora.com, eugen.hristev@collabora.com, matthias.bgg@gmail.com Subject: [PATCH v3 1/2] dt-bindings: media: mediatek-jpeg-encoder: change max iommus count Date: Sat, 27 Jan 2024 10:42:57 +0200 Message-Id: <20240127084258.68302-1-eugen.hristev@collabora.com> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: <linux-media.vger.kernel.org> List-Subscribe: <mailto:linux-media+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-media+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-LSpam-Score: -3.0 (---) X-LSpam-Report: No, score=-3.0 required=5.0 tests=ARC_SIGNED=0.001,ARC_VALID=-0.1,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,DMARC_PASS=-0.001,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,RCVD_IN_DNSWL_MED=-2.3,SPF_HELO_NONE=0.001,SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no |
| Series |
[v3,1/2] dt-bindings: media: mediatek-jpeg-encoder: change max iommus count
|
|
Commit Message
Eugen Hristev
Jan. 27, 2024, 8:42 a.m. UTC
MT8186 has 4 iommus in the list, to cope with this situation, adjust the maxItems to 4 (instead of previous 2). Add also minItems as 2 to keep compatibility with current devices. Signed-off-by: Eugen Hristev <eugen.hristev@collabora.com> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> --- Changes in v3: minItems changed to 2. Changes in v2: fixed typo in subject .../devicetree/bindings/media/mediatek-jpeg-encoder.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
Comments
On Sat, 27 Jan 2024 10:42:57 +0200, Eugen Hristev wrote: > MT8186 has 4 iommus in the list, to cope with this situation, adjust > the maxItems to 4 (instead of previous 2). > Add also minItems as 2 to keep compatibility with current devices. > > Applied to v6.4-next/dts64, thanks! [1/2] dt-bindings: media: mediatek-jpeg-encoder: change max iommus count commit: b824b32dd5e98221cbe2e8bcccc6fb4134e35fc1 [2/2] arm64: dts: mediatek: mt8186: Add jpgenc node commit: 4c5b46fbf52d52b0f392f0fc3913560bad438e49 Best regards,
Il 29/01/24 11:41, AngeloGioacchino Del Regno ha scritto: > > On Sat, 27 Jan 2024 10:42:57 +0200, Eugen Hristev wrote: >> MT8186 has 4 iommus in the list, to cope with this situation, adjust >> the maxItems to 4 (instead of previous 2). >> Add also minItems as 2 to keep compatibility with current devices. >> >> > > Applied to v6.4-next/dts64, thanks! > Sorry, typo: v6.8-next/dts64 > [1/2] dt-bindings: media: mediatek-jpeg-encoder: change max iommus count > commit: b824b32dd5e98221cbe2e8bcccc6fb4134e35fc1 > [2/2] arm64: dts: mediatek: mt8186: Add jpgenc node > commit: 4c5b46fbf52d52b0f392f0fc3913560bad438e49 > > Best regards,
On 29/01/2024 11:41, AngeloGioacchino Del Regno wrote: > > On Sat, 27 Jan 2024 10:42:57 +0200, Eugen Hristev wrote: >> MT8186 has 4 iommus in the list, to cope with this situation, adjust >> the maxItems to 4 (instead of previous 2). >> Add also minItems as 2 to keep compatibility with current devices. >> >> > > Applied to v6.4-next/dts64, thanks! > > [1/2] dt-bindings: media: mediatek-jpeg-encoder: change max iommus count > commit: b824b32dd5e98221cbe2e8bcccc6fb4134e35fc1 Hm, not sure why you took this one, as DT-Bindings normally go through the subsystem maintainer trees. Otherwise merge conflicts can occur. Just saying. Matthias > [2/2] arm64: dts: mediatek: mt8186: Add jpgenc node > commit: 4c5b46fbf52d52b0f392f0fc3913560bad438e49 > > Best regards,
(Drop a bunch of people from CC) On Mon, Jan 29, 2024 at 6:48 PM AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> wrote: > > Il 29/01/24 11:41, AngeloGioacchino Del Regno ha scritto: > > > > On Sat, 27 Jan 2024 10:42:57 +0200, Eugen Hristev wrote: > >> MT8186 has 4 iommus in the list, to cope with this situation, adjust > >> the maxItems to 4 (instead of previous 2). > >> Add also minItems as 2 to keep compatibility with current devices. > >> > >> > > > > Applied to v6.4-next/dts64, thanks! > > > > Sorry, typo: v6.8-next/dts64 I was wondering, what script produces this email? AFAIK `b4 ty` gives a simple thank you letter without branch or commit hashes. ChenYu > > [1/2] dt-bindings: media: mediatek-jpeg-encoder: change max iommus count > > commit: b824b32dd5e98221cbe2e8bcccc6fb4134e35fc1 > > [2/2] arm64: dts: mediatek: mt8186: Add jpgenc node > > commit: 4c5b46fbf52d52b0f392f0fc3913560bad438e49 > > > > Best regards, > >
Il 29/01/24 16:46, Matthias Brugger ha scritto: > > > On 29/01/2024 11:41, AngeloGioacchino Del Regno wrote: >> >> On Sat, 27 Jan 2024 10:42:57 +0200, Eugen Hristev wrote: >>> MT8186 has 4 iommus in the list, to cope with this situation, adjust >>> the maxItems to 4 (instead of previous 2). >>> Add also minItems as 2 to keep compatibility with current devices. >>> >>> >> >> Applied to v6.4-next/dts64, thanks! >> >> [1/2] dt-bindings: media: mediatek-jpeg-encoder: change max iommus count >> commit: b824b32dd5e98221cbe2e8bcccc6fb4134e35fc1 > > Hm, not sure why you took this one, as DT-Bindings normally go through the > subsystem maintainer trees. Otherwise merge conflicts can occur. Just saying. > That was done so that we don't get devicetree validation issues on our branch; eventually, there were no driver changes to support that binding change, and it's there just to validate commit 2/2. Of course the media maintainers do know about this so there will be no merge conflict - but yes next time I'll let them pick these instead, it's probably easier for everyone, besides simply being the proper thing to do. Angelo > Matthias > > >> [2/2] arm64: dts: mediatek: mt8186: Add jpgenc node >> commit: 4c5b46fbf52d52b0f392f0fc3913560bad438e49 >> >> Best regards,
Il 30/01/24 04:24, Chen-Yu Tsai ha scritto: > (Drop a bunch of people from CC) > > On Mon, Jan 29, 2024 at 6:48 PM AngeloGioacchino Del Regno > <angelogioacchino.delregno@collabora.com> wrote: >> >> Il 29/01/24 11:41, AngeloGioacchino Del Regno ha scritto: >>> >>> On Sat, 27 Jan 2024 10:42:57 +0200, Eugen Hristev wrote: >>>> MT8186 has 4 iommus in the list, to cope with this situation, adjust >>>> the maxItems to 4 (instead of previous 2). >>>> Add also minItems as 2 to keep compatibility with current devices. >>>> >>>> >>> >>> Applied to v6.4-next/dts64, thanks! >>> >> >> Sorry, typo: v6.8-next/dts64 > > I was wondering, what script produces this email? > > AFAIK `b4 ty` gives a simple thank you letter without branch or commit > hashes. > That's exactly `b4 ty` :-) You can use a custom template, so that it reads something like `Applied to ${branch}, thanks!` Cheers, Angelo > ChenYu > >>> [1/2] dt-bindings: media: mediatek-jpeg-encoder: change max iommus count >>> commit: b824b32dd5e98221cbe2e8bcccc6fb4134e35fc1 >>> [2/2] arm64: dts: mediatek: mt8186: Add jpgenc node >>> commit: 4c5b46fbf52d52b0f392f0fc3913560bad438e49 >>> >>> Best regards, >> >>
diff --git a/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.yaml b/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.yaml index 37800e1908cc..83c020a673d6 100644 --- a/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.yaml +++ b/Documentation/devicetree/bindings/media/mediatek-jpeg-encoder.yaml @@ -38,7 +38,8 @@ properties: maxItems: 1 iommus: - maxItems: 2 + minItems: 2 + maxItems: 4 description: | Points to the respective IOMMU block with master port as argument, see Documentation/devicetree/bindings/iommu/mediatek,iommu.yaml for details.