Received: by 2002:ab2:3319:0:b0:1ef:7a0f:c32d with SMTP id i25csp571861lqc; Fri, 8 Mar 2024 06:08:35 -0800 (PST) X-Forwarded-Encrypted: i=3; AJvYcCVDA2nt7eJMvjM7ADkrdry5djRNIq8FQzuTHGY90vqS16zAmNlM8fo2M2iHS5pN5ZYNz1qtqa1dgQ7/G9a2ZtRjZGC0p96BtMbsKBIrsQ== X-Google-Smtp-Source: AGHT+IH3Ob1hySeimY0JgRwdkhKqw3LvJVaA5+de1jUrqYKNuAd+2ca0L0zPRWF5qBJHxi1TTsH1 X-Received: by 2002:a05:6402:5c4:b0:565:3aa7:565f with SMTP id n4-20020a05640205c400b005653aa7565fmr1968393edx.8.1709906915775; Fri, 08 Mar 2024 06:08:35 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1709906915; cv=pass; d=google.com; s=arc-20160816; b=k+cw2kFAoFtV/LPiCxj5Kpcvl3Wh9Tq2+AFs5Im3+dIHe8SyGXmCmtFPcD2YI/T5iu P9HygxEeMx3mxHhbOozrNF5/kozT64WIqjL8L9LSsmx+0NbG5lSoai3dxFCourP0UpgS /vH5a6467XCiwygzOETtoSHcbSWDhSrHrCt6LXAbHce9tImf/tOWAKE2ucVSFxYEshrj XIbZUKgviAuLwxz4CXeDBa0BykxUP6V2G+am15W5/tzhxqC8JBWWSzKrdXMwqm7w3sT7 gL8gAoG1lO3/AHv4fSBMa61x5tEqZEiEthR03N1LpMK7Sxtvy0vTuonU0SwEgc+fAHOq dnQw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:date:message-id:dkim-signature; bh=NN7d8FE0ogsAfIcWLSfEOWtWn8yJt4v11ZeoqlxB6R4=; fh=hhRniLwKtH5RiNv7JKms6YZ3g6ZZ/KrTZ5GldPNW7Kg=; b=sYxyZeWL2hHE6gZCxolnYDARyDYTBemPaw3vl3wlyZfmf7jp6ZWfB4eiqusoKwxTP8 xCiP8X47YFSmSGBreNH0kKG0g2d4QRVhBzG0s9+iFExWxZNgG+YKYQUyELD4AwsLif9K k9Mg+rU3psLTEioCBi23C5onF2dESYF6cJdruPCa6zWiEx2apgyS07C/cWTRr05alPuc oqW6RiOxvL2o9691hYn2EAPSJDxAEPeaPsTa+PoMfuByQWwMS5MguJ7J3q1yCbIo7cgi btcLt3lBbkpkh9Cp/d0Lu0lTJRSLDWUWLBn+sfblgfA0yQaQ5mg2dAiA/XyTVDXl+dCT X8Mg==; dara=google.com ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=yNyS0Ruj; arc=pass (i=1 spf=pass spfdomain=collabora.com dkim=pass dkdomain=collabora.com dmarc=pass fromdomain=collabora.com); spf=pass (google.com: domain of linux-kernel+bounces-97141-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-97141-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com Return-Path: Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id t1-20020a056402524100b00565895b787dsi8128722edd.130.2024.03.08.06.08.35 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 08 Mar 2024 06:08:35 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-97141-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass header.i=@collabora.com header.s=mail header.b=yNyS0Ruj; arc=pass (i=1 spf=pass spfdomain=collabora.com dkim=pass dkdomain=collabora.com dmarc=pass fromdomain=collabora.com); spf=pass (google.com: domain of linux-kernel+bounces-97141-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-97141-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE) header.from=collabora.com 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 807EC1F269BE for ; Fri, 8 Mar 2024 14:08:35 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 54B1F40861; Fri, 8 Mar 2024 14:08:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=collabora.com header.i=@collabora.com header.b="yNyS0Ruj" 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 F3ACE63B8; Fri, 8 Mar 2024 14:08:21 +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=1709906903; cv=none; b=HtawIOVUnY0BVDPfT0sLUvWEze4fk7Cl5e0YddFRUB7602b7XHjilSaT8erwIV3i7d/VWZpyhqEs22cOgzR7+t+HcqfY2pnPd2hoK5XSgpkxjIyOxkBDVvKpLhH2HalPqvVOxvsrcz50F7kYkD7XauGStRGaJ8V/FLITseK0VT0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1709906903; c=relaxed/simple; bh=wefvkTTCoZvNRPlXyzWk06NTVJGu8R7LMpdXHxDxiy4=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=c/6YBoJIQxE5y0iyzOqoweN2uPm8/BuT0mj3KnJwcMfXFm1eV7By8hRdtrFaz19e21vd92hQvm71wogg0aZudRq4CrleRfarSmJf6xOyfkKA8+zJ9AoSKSLKVPzjWda3yoAWNfhU8MHEq+vQPoIdIvytKyY/Ue1wqxIfVFMy0yA= 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=yNyS0Ruj; 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=1709906894; bh=wefvkTTCoZvNRPlXyzWk06NTVJGu8R7LMpdXHxDxiy4=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=yNyS0RujkB0DNybLfVf7NH+HhpN7qHTqtzXps3SMSxsZLl3SEQt/MLmDCgMMdRPSu TOINSTCC1zvfZJ3tGoYriRGN+mJpeIm5+ehnQxObZ+4NTj2xr1sb74ftZIJUJvZoRF pf4I4l2w/Qt1sb7eLkl2SgiY+g8Zxr6p/OkCu8doCjzxd1xvgG0flLd8cwDxkmhiCa pQY+Zy7l0ELeg9ij6/+xzoOUEXAIVInbeAWPvo9ZBUMxt0jaPjTLGmlc041eqQehoO a70KroxXvRhwHc9TcUmWE0fqElbNsIGxrdtMSwKzIdn2Kl9jXXF6JUaxdGS6PU0kgu U8LzlTBhJpbQA== Received: from [100.74.67.65] (cola.collaboradmins.com [195.201.22.229]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: jmassot) by madrid.collaboradmins.com (Postfix) with ESMTPSA id 5DF8737820F1; Fri, 8 Mar 2024 14:08:13 +0000 (UTC) Message-ID: <8f6972a1-e174-4c0e-808e-afece9b529bf@collabora.com> Date: Fri, 8 Mar 2024 15:08:12 +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 v4 2/4] dt-bindings: media: add Maxim MAX96714 GMSL2 Deserializer Content-Language: en-US To: Conor Dooley Cc: linux-media@vger.kernel.org, kernel@collabora.com, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, mchehab@kernel.org, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, sakari.ailus@iki.fi References: <20240305152608.287527-1-julien.massot@collabora.com> <20240305152608.287527-3-julien.massot@collabora.com> <20240307-retract-aloof-9ff1fde79a82@spud> From: Julien Massot In-Reply-To: <20240307-retract-aloof-9ff1fde79a82@spud> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Hi Conor, Thanks for reviewing my patchset. On 3/7/24 20:21, Conor Dooley wrote: > On Tue, Mar 05, 2024 at 04:26:06PM +0100, Julien Massot wrote: >> Add DT bindings for Maxim MAX96714 GMSL2 Deserializer. >> >> Signed-off-by: Julien Massot >> --- >> Change since v3: >> - Renamed file to maxim,max96714.yaml dropped the 'f' suffix > > Why? The filename should match the compatible, which /does/ have an f. All the work has been done on MAX96714F variant of this Maxim GMSL2 deserializer. The driver and the binding remain suitable for all variants of this chipset, since they share the same register mapping, similar features etc.. MAX96714 exists in different variant: MAX96714 / MAX96714F / MAX96714K that will be easy to add support for this binding and driver later. The MAX96714 name looks the most suitable. Please have a look at this discussion on the V3 version https://lore.kernel.org/lkml/ZdXYpc2csVnhtZH9@valkosipuli.retiisi.eu > >> - Removed mention to C-PHY since it's not supported by MAX96714 deserializers >> - Removed bus-type requirement on CSI endpoint since the device only support D-PHY >> - Removed the clock-lanes property in the dt example >> >> Change since v2: >> - remove reg description >> - rename enable gpio to powerdown >> - use generic node name: i2c, serializer, deserializer >> --- >> --- >> .../bindings/media/i2c/maxim,max96714.yaml | 169 ++++++++++++++++++ >> 1 file changed, 169 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/media/i2c/maxim,max96714.yaml > >> +properties: >> + compatible: >> + const: maxim,max96714f > >> + i2c-gate: >> + $ref: /schemas/i2c/i2c-controller.yaml > > There is an i2c-gate binding, you should reference it here instead. Ok, I will post a new version with a reference to the i2c-gate binding. > >> + unevaluatedProperties: false >> + description: | > > This | is not needed, there's no formatting to preserve. Ok I will drop the '|' > >> + The MAX96714 will pass through and forward the I2C requests from the >> + incoming I2C bus over the GMSL2 link. Therefore it supports an i2c-gate >> + subnode to configure a serializer. Regards, -- Julien Massot Senior Software Engineer Collabora Ltd. Platinum Building, St John's Innovation Park, Cambridge CB4 0DS, UK Registered in England & Wales, no. 5513718