From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f53.google.com (mail-ed1-f53.google.com [209.85.208.53]) (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 B732C1E260A; Wed, 27 Aug 2025 04:47:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.53 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756270055; cv=none; b=tMo67o41kKRG4ICM9Xx/LdWJPCsEzJlUkAsNsbwduHjG24H6uy8ajGuteS7XnKRwHb6hSXbp6KrfTEYTBKIfXVNt5WMBgo35Qlq+TZif5k5zG6B7EHhEM6wiomsSIMNmVv4veMc0n8ifbksfxIL/uAA8OPjZvve3+l9PtdB2bfs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1756270055; c=relaxed/simple; bh=FH79sR3wwQwKOwJz4iMC16W4fPbO9VVTpyHth1+ZjzM=; h=Date:From:To:CC:Subject:In-Reply-To:References:Message-ID: MIME-Version:Content-Type; b=eUJVBN4phxCxKVtxej6wsNLEVtTyR965TmefXXIQhQM2hLvSZcLDQB9PzUbDVZ9XZgil+0araU2EuoyA0auPJHbmaSPlFU2AXjMn8v0xKKODWv1hED4bh0cjJIERSo1v1O4zM5rZBJbLRGXhdAFAyHufuyNIQYkhO2q1sojfiOQ= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=f/pZ4G7i; arc=none smtp.client-ip=209.85.208.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="f/pZ4G7i" Received: by mail-ed1-f53.google.com with SMTP id 4fb4d7f45d1cf-61c325a4d18so6873287a12.0; Tue, 26 Aug 2025 21:47:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1756270052; x=1756874852; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:references :in-reply-to:user-agent:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to; bh=8TCMaff2/OOBuiGXhxSyW4vUb1EhEINJOzc43HDpiqA=; b=f/pZ4G7ihgGrs8QRe50LTu7+MNAo3IZiZyCH0Ay5BnG9fLql38HrUv4R0p8/54NYku f+dAKyQ8DJ8DcUIeOE4QiwwSuSe3DGHMte6OhlAYNxPvY1SheYGo6SWWmbynMmzdcrmV 51HRtriv1lhEOrqIHgikgnmjw+WwiBZBlcMV08dPW6dq0kab3IQWDTPRvE7qcHdsbiZa HCXb95NNLxB9xWpM7i7RtvWoSXtkI3rO9/Rj95+kF4Ja8vnluZA5D7v4VWZYr2oIIU6F WxkdWKsPEYLhoXh5XjJdtS0BYCaZIQCjVfGtNbQy6wnEN9YXLbTODs49nAX6VBVP0/Ms 03ww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1756270052; x=1756874852; h=content-transfer-encoding:mime-version:message-id:references :in-reply-to:user-agent:subject:cc:to:from:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=8TCMaff2/OOBuiGXhxSyW4vUb1EhEINJOzc43HDpiqA=; b=CAs2ynzaBHIaJIitls0K3a/VF8QzSr5YEbijOV8CF3esX5eMSc0tfEeypl8M5Q9R3O +Ii4nyyfhVXXS/cd1Tz86f12+KCHXJ/IUljPdjre+34a5F3a0RKz92e0grHo/IpdKJi3 mJjvVSVQvga4eZq7jesH/LIOqZ8YnZ2fwTSfHPvWJVF5YLyACxBoXT5ijq/gLHEef9FW NBej4dhWvACp4Ef2Y7pKFubZqLL0xQjycdNjUuh5KFkS3tVNOv6X8k9UbF+OEDFBf4cv MFsj8miFJ4u/stv7blGTCKE3mwagkKg3OH3IT1VMScBMrQvdLA4MX1Mf1iiIyBDjmewp Rl/Q== X-Forwarded-Encrypted: i=1; AJvYcCU4X2+hOoXu8VH/+IAtxYe4UldWnFB7R/2G+YFlR9Rgy60qxP+N5+5dC87UKmay4KqH5u/qirJNDe8g@vger.kernel.org, AJvYcCU7yipZjCcyYls+F+RlQRw9x6mo07S5WPPZ6MHGkkFx3pD9n8TEYWGQT9STAeB22AZuLCK/Jmr8QPa5qU9/@vger.kernel.org, AJvYcCVOQyD6I6DUHDqPMvkuqe+3Xm9xwNWmOZydQjH5aC3FXNw5Xw7VcNNXeNtxwYp/jdSHTVDn9ggjGDik@vger.kernel.org, AJvYcCX4TBigD90ptHRcIbyIKcMPlF7f9vNSO9nRiePjlAr/Y9HCv9Ux4yx3wHCnF+hVI/cY9r5ImtTpWT21x3o=@vger.kernel.org X-Gm-Message-State: AOJu0Yz6jW8cvsZYWha1hMKaoCHaS/tDo9yqbBS23m1AIlxbbORw8cd3 4LZjblkXUnxvJBF2zyOvYG21pNZi8brDKm1/c+dvb9lyZ3O0aN+CLPAg X-Gm-Gg: ASbGncvDoMK95UpPk4vmy7G6+9QFFSNCk6INeB5m3BgOJ/qcezg84oeU06GYmb3m7FQ N5raJ2cDYPi12Nbw+fAli+CX+kDzdgmy59ia+InehUR7vJFi33hN56AQzX35JLFo0Z6/hWR40ij KhC8cljJRbbu4aaHyt2G7wzrZiNrQ37PnsuU+jXvgkMWQXbT24JHtEByIO39v4GGaMc8uMQg1Vk HE6nrGNrT/duGU0h1b69kfduosI8WqcQ9l/bES6utBSiaM4xYpXw339e79uhJkcaOlAr8/A35O3 hAHF1BGuuZG0aD9qjFWg7suzERUEGJk4/O0vq6pI5C07bxflXZZAxZDo39UeCYcmNW14w1AU/Bf IsCAhexAJPmfPb1NBdUozgiv5 X-Google-Smtp-Source: AGHT+IG9JdOvxAI0JpZC7UN53cppIId0lE2cbYaF9wAYpBd5bYrwqo7A7y/TJvWUTacmrbOk5BBATg== X-Received: by 2002:a05:6402:44d2:b0:618:fe3:f4c with SMTP id 4fb4d7f45d1cf-61c1b6f1fbemr12637501a12.29.1756270051831; Tue, 26 Aug 2025 21:47:31 -0700 (PDT) Received: from [127.0.0.1] ([5.248.55.4]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-61c90ffd677sm2710258a12.46.2025.08.26.21.47.31 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 26 Aug 2025 21:47:31 -0700 (PDT) Date: Wed, 27 Aug 2025 07:47:29 +0300 From: Svyatoslav To: Mikko Perttunen , Thierry Reding , Thierry Reding , Jonathan Hunter , Sowjanya Komatineni , Luca Ceresoli , David Airlie , Simona Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Peter De Schrijver , Prashant Gaikwad , Michael Turquette , Stephen Boyd , Mauro Carvalho Chehab , Greg Kroah-Hartman , Dmitry Osipenko , Charan Pedumuru CC: linux-media@vger.kernel.org, linux-tegra@vger.kernel.org, dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, linux-staging@lists.linux.dev Subject: =?US-ASCII?Q?Re=3A_=5BPATCH_v1_05/19=5D_staging=3A_media=3A_tegra-?= =?US-ASCII?Q?video=3A_expand_VI_and_VIP_support_to_Tegra30?= User-Agent: K-9 Mail for Android In-Reply-To: <2271797.NgBsaNRSFp@senjougahara> References: <20250819121631.84280-1-clamor95@gmail.com> <20250819121631.84280-6-clamor95@gmail.com> <2271797.NgBsaNRSFp@senjougahara> Message-ID: <4BD9010B-3F5B-4EE3-B57C-A20DFAEC5276@gmail.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable 27 =D1=81=D0=B5=D1=80=D0=BF=D0=BD=D1=8F 2025=E2=80=AF=D1=80=2E 07:29:40 GM= T+03:00, Mikko Perttunen =D0=BF=D0=B8=D1=88=D0=B5= : >On Tuesday, August 19, 2025 9:16=E2=80=AFPM Svyatoslav Ryhel wrote: >> Exisitng VI and VIP implementation for Tegra20 is fully compatible with >> Tegra30=2E >>=20 >> Signed-off-by: Svyatoslav Ryhel >> --- >> drivers/staging/media/tegra-video/Makefile | 1 + >> drivers/staging/media/tegra-video/vi=2Ec | 3 +++ >> drivers/staging/media/tegra-video/vi=2Eh | 2 +- >> drivers/staging/media/tegra-video/video=2Ec | 4 ++++ >> drivers/staging/media/tegra-video/vip=2Ec | 5 ++++- >> 5 files changed, 13 insertions(+), 2 deletions(-) >>=20 >> diff --git a/drivers/staging/media/tegra-video/Makefile >> b/drivers/staging/media/tegra-video/Makefile index >> 6c7552e05109=2E=2E96380b5dbd8b 100644 >> --- a/drivers/staging/media/tegra-video/Makefile >> +++ b/drivers/staging/media/tegra-video/Makefile >> @@ -6,5 +6,6 @@ tegra-video-objs :=3D \ >> csi=2Eo >>=20 >> tegra-video-$(CONFIG_ARCH_TEGRA_2x_SOC) +=3D tegra20=2Eo >> +tegra-video-$(CONFIG_ARCH_TEGRA_3x_SOC) +=3D tegra20=2Eo >> tegra-video-$(CONFIG_ARCH_TEGRA_210_SOC) +=3D tegra210=2Eo >> obj-$(CONFIG_VIDEO_TEGRA) +=3D tegra-video=2Eo >> diff --git a/drivers/staging/media/tegra-video/vi=2Ec >> b/drivers/staging/media/tegra-video/vi=2Ec index c9276ff76157=2E=2E71be= 205cacb5 >> 100644 >> --- a/drivers/staging/media/tegra-video/vi=2Ec >> +++ b/drivers/staging/media/tegra-video/vi=2Ec >> @@ -1959,6 +1959,9 @@ static const struct of_device_id >> tegra_vi_of_id_table[] =3D { #if defined(CONFIG_ARCH_TEGRA_2x_SOC) >> { =2Ecompatible =3D "nvidia,tegra20-vi", =2Edata =3D &tegra20_vi_soc= }, >> #endif >> +#if defined(CONFIG_ARCH_TEGRA_3x_SOC) >> + { =2Ecompatible =3D "nvidia,tegra30-vi", =2Edata =3D &tegra20_vi_soc= }, >> +#endif >> #if defined(CONFIG_ARCH_TEGRA_210_SOC) >> { =2Ecompatible =3D "nvidia,tegra210-vi", =2Edata =3D &tegra210_vi_so= c }, >> #endif >> diff --git a/drivers/staging/media/tegra-video/vi=2Eh >> b/drivers/staging/media/tegra-video/vi=2Eh index 1e6a5caa7082=2E=2Ecac0= c0d0e225 >> 100644 >> --- a/drivers/staging/media/tegra-video/vi=2Eh >> +++ b/drivers/staging/media/tegra-video/vi=2Eh >> @@ -296,7 +296,7 @@ struct tegra_video_format { >> u32 fourcc; >> }; >>=20 >> -#if defined(CONFIG_ARCH_TEGRA_2x_SOC) >> +#if defined(CONFIG_ARCH_TEGRA_2x_SOC) || defined(CONFIG_ARCH_TEGRA_3x_= SOC) >> extern const struct tegra_vi_soc tegra20_vi_soc; >> #endif >> #if defined(CONFIG_ARCH_TEGRA_210_SOC) >> diff --git a/drivers/staging/media/tegra-video/video=2Ec >> b/drivers/staging/media/tegra-video/video=2Ec index >> 074ad0dc56ca=2E=2Ea25885f93cd7 100644 >> --- a/drivers/staging/media/tegra-video/video=2Ec >> +++ b/drivers/staging/media/tegra-video/video=2Ec >> @@ -127,6 +127,10 @@ static const struct of_device_id host1x_video_subd= evs[] >> =3D { { =2Ecompatible =3D "nvidia,tegra20-vip", }, >> { =2Ecompatible =3D "nvidia,tegra20-vi", }, >> #endif >> +#if defined(CONFIG_ARCH_TEGRA_3x_SOC) >> + { =2Ecompatible =3D "nvidia,tegra30-vip", }, >> + { =2Ecompatible =3D "nvidia,tegra30-vi", }, >> +#endif >> #if defined(CONFIG_ARCH_TEGRA_210_SOC) >> { =2Ecompatible =3D "nvidia,tegra210-csi", }, >> { =2Ecompatible =3D "nvidia,tegra210-vi", }, >> diff --git a/drivers/staging/media/tegra-video/vip=2Ec >> b/drivers/staging/media/tegra-video/vip=2Ec index 5ec717f3afd5=2E=2E00e= 08a9971d5 >> 100644 >> --- a/drivers/staging/media/tegra-video/vip=2Ec >> +++ b/drivers/staging/media/tegra-video/vip=2Ec >> @@ -263,13 +263,16 @@ static void tegra_vip_remove(struct platform_devi= ce >> *pdev) pm_runtime_disable(&pdev->dev); >> } >>=20 >> -#if defined(CONFIG_ARCH_TEGRA_2x_SOC) >> +#if defined(CONFIG_ARCH_TEGRA_2x_SOC) || defined(CONFIG_ARCH_TEGRA_3x_= SOC) >> extern const struct tegra_vip_soc tegra20_vip_soc; >> #endif >>=20 >> static const struct of_device_id tegra_vip_of_id_table[] =3D { >> #if defined(CONFIG_ARCH_TEGRA_2x_SOC) >> { =2Ecompatible =3D "nvidia,tegra20-vip", =2Edata =3D &tegra20_vip_so= c }, >> +#endif >> +#if defined(CONFIG_ARCH_TEGRA_3x_SOC) >> + { =2Ecompatible =3D "nvidia,tegra30-vip", =2Edata =3D &tegra20_vip_so= c }, >> #endif >> { } >> }; > >If tegra30-vip is compatible with tegra20-vip, we don't need to add the= =20 >compatible string into the driver=2E Just mark it as 'compatible =3D=20 >"nvidia,tegra30-vip", "nvidia,tegra20-vip";' in the device tree (and as R= ob=20 >alluded, have this compat string pair as an option in the device tree sch= ema)=2E > While I am fine with using fallback but it may be a good idea to have a se= parate compatible so in case tegra30 would need a specific set of ops (tegr= a20 and tegra30 VIs are not exact match) no additional changes into schema = would be required=2E >Cheers, >Mikko > > >