From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej1-f48.google.com (mail-ej1-f48.google.com [209.85.218.48]) (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 29954382F2C for ; Fri, 13 Mar 2026 08:32:20 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.218.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773390742; cv=none; b=JqL7BRpVz6IECdYg0fEW2Tpj2jP4an6cVbYV/7x7I+8m/Yxiuaqvq4Gtx+aacj4czG6MlNjyNoEYUmXmXChz8UK9VQQIx4T1oXjgbmdJZJGGu1XziyomgWiCbtLSdfl01GA/7FYVLAlDb5D78StAwayTw4CcEspxxZLIwVOhf7A= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1773390742; c=relaxed/simple; bh=gF7U4AhLl2B54UlbLoigUUUH6Jsm/F5mlTaV5RfBX1c=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=J+e89y19Yx4dlpl5mLNgij6QV0L5mLB8ngfciR7cbxKj+WPD7Lm0/C7HM+fVL6v1dQSaGjSp+rAttfDEV5/+LFp9OBwGjjZR4qJAesg6L/iiHEyHOGuyrO3Aj8SdGbGK9On0uixwasnpb0LJw+dG+2FSG3v7t8WvP5v9qNs6ncA= 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=Ve4+KjZM; arc=none smtp.client-ip=209.85.218.48 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="Ve4+KjZM" Received: by mail-ej1-f48.google.com with SMTP id a640c23a62f3a-b942b36de08so336094366b.1 for ; Fri, 13 Mar 2026 01:32:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1773390739; x=1773995539; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=zbcDxBVVYzOw7c6OTfNiMQmEPeMGSouBlRwpnguHKAQ=; b=Ve4+KjZMZXY4pNNaIPy6oKForvFFVjldMaURWwmxmzsmi+eV3MBTRRAd1SMZIX345h CuM9UPCoPnm84rEkVOfCzztFXPmTAlGoJBaGYyyhDd+hIbw2oLs+ZICv/LMKPCSCoPau HvE2s0thCUwFl1Lp4ruYJ7XJgjfwEHOYBowlXC5BHuqsyW5NJv2OWwL5ShpyhPmnxoyI IaTH1CNYSzxrimyyURz3spByUlwxn+KkTXJBZTy4lWDfNEIzHdB7/laV0bVJSJPK2lrJ 2ERci+y02pPLrqM5DXhh8IXd4bDpOuswI7THsBPfv/Ywl+B54JNSA58bAKWTK286/sJQ Jvqg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1773390739; x=1773995539; h=content-transfer-encoding:in-reply-to:from:content-language :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; bh=zbcDxBVVYzOw7c6OTfNiMQmEPeMGSouBlRwpnguHKAQ=; b=iLYOZJ58uC1DJJ8NSaam2ZXU+lgi/9bEyGQrDi22AiO2jJodqjaEPbIS8l6k9OoR7h EFDT9Cm/n9Lhyc3zd41CNKckAaHA5IiKL6uw+DaM/1qpt4JLrKO4ROAkuNOiyA1m9rGZ 0XYONKGyEzNL5g5XVUcegLSJoFNRabkWACqH2W/ei4uLrxWnKaKAQqU/qsBMnZAYLgsL bX1uZrQJJ/rcq76Cx4phXg7W7pdiP3fbnrGTr2QXVmVwsOwzRwZnTTLTMTfCrb16Vvfp PI43m0QryCBhh4WojL0kOImnOLJOGbCk+ea9MV5K0e/OnxnpaGyYaXmSVJ/lDsoNI5LA i/+Q== X-Forwarded-Encrypted: i=1; AJvYcCX/+gqLoLp5Biu54isAl2yBpF0fxLZc6R6Hm4tdip4xN+7CIoxd3tiw/MMHCKnGoPxiJ+hwk0KI9ewL73A=@vger.kernel.org X-Gm-Message-State: AOJu0Yw80VmP6Eef2qcYCJ7GDBvn1gapQijmB4Deu3KQdpcfehw/gqj7 S1btkJfSj6AoP2F5v6DIMnqvpORNXKNfdKd8h2Gt8EDIBgB7keQiofcr X-Gm-Gg: ATEYQzzdvlHR03MAtjE/HwWoEXpiofpi820GCDumFhAFKfghSKH1nXQYw6WJL8pLDkC bfabOo0WzTUbMkbm6X3kGBgU8XcyinSQgjILc3Flvfn2gwFVveN6EBPIIQry/KI34xn7IEjlZB+ xO09ZWhyHxDxqjWdLVdtUhfYAUceip/UpqCZ145uiqGEaW3ukVmBH7Umuq9nB1W6q96oC3MXCNU iQxFxMLN2NKVmQIENAacVYDNZJQcMXy6h/KcxNMnfEfIs2PY51mnYpAXe+j7dqu3dbcOLC1GQds /1xjJBIRtccRXLV/FLi+mRVu9fX2PBIfkBBjgRMKy6UbUDNKBGDpQx/9mPMiYZ39u6opjlEUQS9 hAFu0hH9RfsfuKZ6kfrbpya8gyScfh+2Z0ybwPImwZUn6DYCM+acC7UVEYlQVRHb1+kWlsWHUOm 5RKgWlZUqDbv0wxISANI/iSimV+vUv7yhLWWNziiay1eFBYVul3fyyg9SU/ER/Vs8U9w== X-Received: by 2002:a17:907:9285:b0:b97:73ae:e2e with SMTP id a640c23a62f3a-b9773ae119bmr31732166b.18.1773390738986; Fri, 13 Mar 2026 01:32:18 -0700 (PDT) Received: from ?IPV6:2a02:8109:8617:d700:5998:31f2:ed8:c4f? ([2a02:8109:8617:d700:5998:31f2:ed8:c4f]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b976cba6ec4sm29152166b.5.2026.03.13.01.32.17 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 13 Mar 2026 01:32:18 -0700 (PDT) Message-ID: <2bcc0442-cdd7-4093-831c-0bd4652edf56@gmail.com> Date: Fri, 13 Mar 2026 09:32:18 +0100 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 v9 00/21] media: i2c: add Maxim GMSL2/3 serializer and deserializer drivers To: Sakari Ailus , dumitru.ceclan@analog.com Cc: Tomi Valkeinen , Mauro Carvalho Chehab , Laurent Pinchart , Julien Massot , Rob Herring , =?UTF-8?Q?Niklas_S=C3=B6derlund?= , Greg Kroah-Hartman , Cosmin Tanislav , mitrutzceclan@gmail.com, linux-media@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-staging@lists.linux.dev, linux-gpio@vger.kernel.org, =?UTF-8?Q?Niklas_S=C3=B6derlund?= , Martin Hecht , Tomi Valkeinen , Cosmin Tanislav , Cory Keitz References: <20260311-gmsl2-3_serdes-v9-0-41499f09004f@analog.com> Content-Language: en-US From: Martin Hecht In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hi Sakari, hi Dumitru, I'm testing v8 already and will migrate next week to that version. Below some further comments. On 3/11/26 08:58, Sakari Ailus wrote: > Hi Dumitru, > > On Wed, Mar 11, 2026 at 09:17:12AM +0200, Dumitru Ceclan via B4 Relay wrote: >> The following deserializers are supported: >> * MAX96712 (already exists in staging) >> * MAX96714 (already exists) >> * MAX96714F (already exists) >> * MAX96714R (GMSL2) >> * MAX96716 (GMSL2) >> * MAX96724 (already exists as part of existing MAX96712 driver) >> * MAX96724F (GMSL2) >> * MAX96724R (GMSL2) >> * MAX9296A (GMSL2) >> * MAX96792A (GMSL3) >> >> The following serializers are supported: >> * MAX96717 (already exists) >> * MAX9295A (GMSL2) >> * MAX96793 (GMSL3) > > It'd be nice to have more reviews as well as comments from the users of the > existing drivers especially on how this works for them. > > Are there differences in functionality or UAPI compared to the drivers > already in upstream? Compared with the current mainline drivers from Julian (what offere a good basic functionality) there are a lot or more features enabled. One ov them is support for routing the channels as well as i2c-atr instead of i2c-gate only. Both helps to setup up more complex and reconfigurable routing of the video channels in complex systems with many sensors/cameras/streams. Additionally a lot more and never devices are supported by that patchset including the new GMSL3 devices what are also backward compatible to GMSL2. Both link types are supported now. Generation of test patterns in pixel mode has been added. Also log-status returns a lot of very useful information about the link mode (tunnel mode versus pixel mode) and potential errors / packet losses over cable by presenting the counter registers. That helps to monitor the quality and reliability of the GMSL links as well as CSI links. > > ... > >> The following v4l2-compliance test still fails: >> fail: v4l2-test-subdevs.cpp(371): fmt.code == 0 || fmt.code == ~0U >> fail: v4l2-test-subdevs.cpp(418): checkMBusFrameFmt(node, fmt.format) >> test Active VIDIOC_SUBDEV_G/S_FMT: FAIL > > Could you post the full report here, please? > BR Martin