From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (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 026FA3C7689 for ; Fri, 31 Jul 2026 08:24:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785486271; cv=none; b=tc4h5qU1ymKY0bUlj0DJE0x0AFpOtgM048sSoYE/wlfBfDzhswL8M2T8RGpPen488OBjxPt28Z8jLlDkhx8L5XLcJiBp9zcirS/lhL374DnnKEu5N2V3uqta0afytWckH7NklzZ1gfwVB9yknJBB2+bEK5q4AXxMjRwKmrr9rac= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1785486271; c=relaxed/simple; bh=tZ6wcj1R/7T6ikjhyiGVXOIeTMRdFJS8++6PntsJuug=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=nuyuAV8r4u/lCWguYuJsCSy4thGzhqZt6Xm62C03cvMfnqL49h0FxS+OODGKYPdlFdUqeSokyc69vAAF2FpJtIebZO/YA/V/VaQVa1/BQ6VgvDr7IAamadvB26MhPo1UEJzMcvanTBwRtUOegUyjmblZuukQBvxSy3UBCyCS5eM= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=UWORjrLn; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=S5z82HG4; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="UWORjrLn"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="S5z82HG4" Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 66V7vRb92885269 for ; Fri, 31 Jul 2026 08:24:23 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= tfhVrR1XwzbRp56jy68NgRK5yU4D4Jc9JJLcVAiXFzE=; b=UWORjrLnvLDQEMRQ 2SfhQgqbKwcF+gfv+l/hWvPIZE7VRQEBEOyFMK81ce0QbOBABMJ7L0JHkylEGPJY 2rr682LuZg02RdkWuRS2+s9tHHBpR+WdD0j/3353PqzlsD5nOCw0EE+ZF+hTw0B/ +5yJQ85GlPRQJSmq0GKYGn9VXnU+0PceLLKQwMZYRuBxiryBP4k8jRjKbixaWM5Y 9QmyBv1pn3wvEU7Q0qixvlb0mkG8iuV+ODijGj7w4jXMJ+v3P22h1rKd+giSyYuR gExKpNayUDopwynCNxOq8DCUd8Y6JC/uQuLWzfTDXlqYHS8YEv0lOaHxi+76W2Vx qN1leg== Received: from mail-qv1-f71.google.com (mail-qv1-f71.google.com [209.85.219.71]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4frq9w86g2-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Fri, 31 Jul 2026 08:24:22 +0000 (GMT) Received: by mail-qv1-f71.google.com with SMTP id 6a1803df08f44-8e9489f62bfso8489206d6.2 for ; Fri, 31 Jul 2026 01:24:22 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1785486262; x=1786091062; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from:references :cc:to:subject:user-agent:mime-version:date:message-id:from:to:cc :subject:date:message-id:reply-to:content-type; bh=tfhVrR1XwzbRp56jy68NgRK5yU4D4Jc9JJLcVAiXFzE=; b=S5z82HG4fpDMcCFkJpizaeIWDj6bu1SA7kljLszKHPtN+fQ8AnXTkcIBWHUbhWrw8Z T5KyGppwgQJflDuXR6oesDsAw1aXo+/FVbQ8UBidv4SAuwu7pS56oExCT5iUL8MUkbQm kun+Mj8MqMtb8NgT+ozOGIxYxrJ30Q6lDruNAEnpM/5atBPcy2frdt9TdWO7C+8kb0cZ EchMALislLDMq7ZZoZlqGoXh4iWg7DxgaMjkRUI43gA6gQd1TXGS6fY4uVHR+t7SIZXZ 6kPH2777N8HD9+67zvuOD0W9mBakZSFfvShX4rWMXwqrsTNJwSz5Kh8Rwo+r0aTMKxOE uuBQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1785486262; x=1786091062; h=content-transfer-encoding:content-type:in-reply-to:from:references :cc:to:subject:user-agent:mime-version:date:message-id:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=tfhVrR1XwzbRp56jy68NgRK5yU4D4Jc9JJLcVAiXFzE=; b=GqTRNRsrDy8wf3atI0LG/uWBU3M5DkuJDxotQwuVw+50fhmLZGw+IhJGeyUv5iD0a+ 45RJK4kuwUQ7BM9ci0dtWE8mLRYPJxVCuMCF2XymyfDjOASUm08e9BsLfdD/GMYOFkCJ XCVgbB8WTqdkpF/Xsz3hvajOj6cwPsq1ADCOTT3YkjQbcmen+KF9tRkahtx3aFp5DUOj 7X3tCPl2Tc6y0NjNmd8USz6Zpb+qDlC5IdIWDR6U+bu6gqLLxC5Olfe4e91linNuBJ0A FNb2wt+TEHk70+5a6u1KRLCAk6VftHDx6oUQxICP/tbhGEp16A5PG6aXnLy4xSR5FkE1 sMbA== X-Forwarded-Encrypted: i=1; AHgh+RoXXFhVWkqP4Q3D6C7sKvPgkRqi8vX5oJCTa5nUy8PSZNdS4hvR4r377dMVDPm2xfVInU1Qz73AM7FJF4o=@vger.kernel.org X-Gm-Message-State: AOJu0Yx3Adds+TrkdY2NzJOMGCnq/rrVYWrmrunxTIEUjOH7rmEIFJWv ODmdk42Gv9DQTnwXUxzdNDUNZ61Tq7CmkkYCyVmRoQhIdv2Pyms6jA2P2rMi0e4kA9VjRxuOR/c +UY8GgCRc0AqD1A5J+Aa+ThvNGED10ZENIxNdQunJULt8vGGPWVs3UPyuuRcGoW4fYRM= X-Gm-Gg: AR+sD12CJQvwjC7pOATmUdwKXT/w5n/GqY88nzdfzyEygsciPfnbARkRitjd3lO24pr Wq/6lRtQKDmIRGCFjshWnQdiYtXrlJpYpo4UEK4LoxmZmN9rBs/7DBbYd107sme8qRfg/m6CDyb Tb7BANX5Wy+qU38nu6LaeiT4A4Xx7DF8skwmq8Sy+BrZNwAng9HC1fmlJAMqvV18vEkd2ff3Va1 YJEvAU5y/qP5mZGP2CGffRif6Q3KbzgI1u3o5LlyxPzXrA2EqSZInrxewUnzciIL/zLYgK32Qap xYNdEVgws4jpuIS21QWMrj9OJ/RAZUARjcUOhAyPu+imu/fdI5ojKe5d+kW+9w8Q4WakOSnpCDr EMbRyKngbBnZ3Buytuqd+NWC6rxY08MG68yO+gULnepyWh2Uwoa5xyWfz8W+MLbqC X-Received: by 2002:a05:6214:5d04:b0:8f3:9d85:405b with SMTP id 6a1803df08f44-908421b4b9emr17416916d6.16.1785486261891; Fri, 31 Jul 2026 01:24:21 -0700 (PDT) X-Received: by 2002:a05:6214:5d04:b0:8f3:9d85:405b with SMTP id 6a1803df08f44-908421b4b9emr17416506d6.16.1785486261404; Fri, 31 Jul 2026 01:24:21 -0700 (PDT) Received: from [10.111.175.212] (Global_NAT1_IAD_FW.qualcomm.com. [129.46.232.65]) by smtp.gmail.com with ESMTPSA id 6a1803df08f44-908435eab41sm5401746d6.34.2026.07.31.01.24.17 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 31 Jul 2026 01:24:20 -0700 (PDT) Message-ID: Date: Fri, 31 Jul 2026 16:24:15 +0800 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 2/2] media: i2c: Add Samsung S5KJN5 image sensor driver To: Vladimir Zapolskiy , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Sakari Ailus Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20260724-sk5jn5-v2-0-871d3b9a2e47@oss.qualcomm.com> <20260724-sk5jn5-v2-2-871d3b9a2e47@oss.qualcomm.com> <2b5753f2-d5f4-4957-b865-f06b842af3de@linaro.org> From: Wenmeng Liu In-Reply-To: <2b5753f2-d5f4-4957-b865-f06b842af3de@linaro.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Authority-Analysis: v=2.4 cv=DPq/JSNb c=1 sm=1 tr=0 ts=6a6c5bb6 cx=c_pps a=UgVkIMxJMSkC9lv97toC5g==:117 a=C3Dk8TwHQYyIj7nOf9RCJw==:17 a=IkcTkHD0fZMA:10 a=RAioF0-LDSMA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=DJpcGTmdVt4CTyJn9g5Z:22 a=EUspDBNiAAAA:8 a=VwQbUJbxAAAA:8 a=z7GYzWamlFIOCxjLmCIA:9 a=3ZKOabzyN94A:10 a=QEXdDO2ut3YA:10 a=1HOtulTD9v-eNWfpl4qZ:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzMxMDA1OSBTYWx0ZWRfX04ZD5oUUcn5g Zxw3Ra9QciAxJLpvinJfFI3yo1rdoh8YdoIAPNEzS1FUWKmTpOclixH5q3/Qii0zouVmgX4mI9c TpgfdLBtrXaX555JYYG8+L/4eeoPhpweP8++39DHQpxXb0nP8Spo1vqH6WPP5cw5/C50ayGG5EH TXXOoLDd4ROxe45UttWynimDYHc3YYW61u3Qo8g1kCXSttaGaVicbLZdzaj0Vme2+N03mZ01uEs 4Axrn0Sr/zRVZ3YINYE77QmY3EOZDIssJyAbAQNl80Qq98Be+vKVvXDH27wWPs8Qyjq5raZY5ax nxdDd7A4N33RxPzy7qEBUJCjlTMlimZH5wz4xFPpi/dTRPrKtB548jcKoKRMfgIlcVy8EQ3tvO0 FF5d5QLNOxb4kVe+3PfmgPT3hbFsgD7t+9TYmtcZf0H3JV/3W24Gnoan0ZboyxxvPiPlgo5tlS9 EN40XPnTMVubRhGNK+g== X-Proofpoint-ORIG-GUID: XSn8yk3rZ7r-kJuauv1ZdpsDqYC7SZ41 X-Proofpoint-Spam-Info: AW1haW4tMjYwNzMxMDA1OSBTYWx0ZWRfXx6UtluTSuZ4+ dDgUZvTgT3lYFL8OvX4AnVlMmv6LATnOI2N5e6WeNyaix1ZZ1XFqzCvcm5epI4PumiLPAsmuppP bNGEjHO1Wq/AJZORFQ8psFmICecbGrI= X-Proofpoint-GUID: XSn8yk3rZ7r-kJuauv1ZdpsDqYC7SZ41 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-07-31_03,2026-07-30_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 phishscore=0 spamscore=0 clxscore=1015 malwarescore=0 impostorscore=0 priorityscore=1501 lowpriorityscore=0 adultscore=0 bulkscore=0 suspectscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2607310059 Hi Vladimir, On 7/25/2026 3:22 PM, Vladimir Zapolskiy wrote: > Hi Wenmeng. > > On 7/24/26 12:50, Wenmeng Liu wrote: >> Add a V4L2 sub-device driver for the Samsung S5KJN5, a 50 MP GBRG 10-bit > > I believe "GBRG" detalization here will be incorrect, if you consider image > flips or odd crops. > >> RAW MIPI CSI-2 image sensor. The driver supports the 4096x3072 mode and >> exposes the standard controls (exposure, gain, vblank/hblank, test >> pattern). >> >> Signed-off-by: Wenmeng Liu >> --- >>   MAINTAINERS                |    1 + >>   drivers/media/i2c/Kconfig  |   11 + >>   drivers/media/i2c/Makefile |    1 + >>   drivers/media/i2c/s5kjn5.c | 2835 ++++++++++++++++++++++++++++++++++ >> ++++++++++ >>   4 files changed, 2848 insertions(+) >> >> diff --git a/MAINTAINERS b/MAINTAINERS >> index >> 244c7ed5872aa0972a7b055d7e0b19d530820a08..6f6edae5de13efc0cda6c91ab8c9a40bfe2cbff9 100644 >> --- a/MAINTAINERS >> +++ b/MAINTAINERS >> @@ -24195,6 +24195,7 @@ M:    Wenmeng Liu >>   L:    linux-media@vger.kernel.org >>   S:    Supported >>   F:    Documentation/devicetree/bindings/media/i2c/samsung,s5kjn5.yaml >> +F:    drivers/media/i2c/s5kjn5.c >>   SAMSUNG S5P Security SubSystem (SSS) DRIVER >>   M:    Krzysztof Kozlowski >> diff --git a/drivers/media/i2c/Kconfig b/drivers/media/i2c/Kconfig >> index >> 7ffccba897ae5637bd0cb913e1e04f9c7d0c89f5..06b5dbc32cab328df8d10da04d91a708bff6fdda 100644 >> --- a/drivers/media/i2c/Kconfig >> +++ b/drivers/media/i2c/Kconfig >> @@ -824,6 +824,17 @@ config VIDEO_S5KJN1 >>         To compile this driver as a module, choose M here: the >>         module will be called s5kjn1. >> +config VIDEO_S5KJN5 >> +    tristate "Samsung S5KJN5 sensor support" >> +    depends on GPIOLIB >> +    select V4L2_CCI_I2C >> +    help >> +      This is a V4L2 sensor driver for Samsung S5KJN5 50MP GBRG > > Same comment as above, I believe "GBRG" should be either removed or > generalized to "Bayer" or "raw". > >> +      10-bit RAW MIPI CSI-2 camera sensor. >> + >> +      To compile this driver as a module, choose M here: the >> +      module will be called s5kjn5. >> + >>   config VIDEO_T4KA3 >>       tristate "Toshiba T4KA3 sensor support" >>       depends on ACPI || COMPILE_TEST >> diff --git a/drivers/media/i2c/Makefile b/drivers/media/i2c/Makefile >> index >> d04bd5724552e88fc2ad39f9bbaf9b4ce1763939..45019ce20a6c34c8139b8247b6904c3abb1a8fe7 100644 >> --- a/drivers/media/i2c/Makefile >> +++ b/drivers/media/i2c/Makefile >> @@ -133,6 +133,7 @@ obj-$(CONFIG_VIDEO_S5K3M5) += s5k3m5.o >>   obj-$(CONFIG_VIDEO_S5K5BAF) += s5k5baf.o >>   obj-$(CONFIG_VIDEO_S5K6A3) += s5k6a3.o >>   obj-$(CONFIG_VIDEO_S5KJN1) += s5kjn1.o >> +obj-$(CONFIG_VIDEO_S5KJN5) += s5kjn5.o >>   obj-$(CONFIG_VIDEO_SAA6588) += saa6588.o >>   obj-$(CONFIG_VIDEO_SAA6752HS) += saa6752hs.o >>   obj-$(CONFIG_VIDEO_SAA7110) += saa7110.o >> diff --git a/drivers/media/i2c/s5kjn5.c b/drivers/media/i2c/s5kjn5.c >> new file mode 100644 >> index >> 0000000000000000000000000000000000000000..865db808aabcf7d910acbdab8126b9fa308ef875 >> --- /dev/null >> +++ b/drivers/media/i2c/s5kjn5.c >> @@ -0,0 +1,2835 @@ >> +// SPDX-License-Identifier: GPL-2.0 >> +/* >> + * Samsung S5KJN5 50 MP GBRG 10-bit RAW MIPI CSI-2 sensor driver > > Same comment as above. > >> + * >> + * Copyright (c) Qualcomm Technologies, Inc. and/or its subsidiaries >> + */ >> + >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include >> +#include > > Let me ask, which interface or declaration from media/v4l2-ctrls.h > is used in the driver? I have a feeling that it's unused. > It is required. The driver call many interfaces form it, like v4l2_ctrl_new_int_menu/__v4l2_ctrl_handler_setup... And thanks a lot for the review. ACK for the rest, and I'll address them in the next revision. Thanks, Wenmeng