From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-lf1-f47.google.com (mail-lf1-f47.google.com [209.85.167.47]) (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 30293245027 for ; Tue, 10 Jun 2025 23:00:51 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.167.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749596454; cv=none; b=kwIHXuLexmbevVrLnGb6B9thblQerlR/w0CEVpVzAQ8KA3sILObV0Xdj5owFeu1QgXzMbDuAzydZwwJY5N2Lpt5fCNmd37egfPLf28mv06iWtWkbomqexpID0z65QkdL2gi3UuMcC7AI9UbWtMOtyTA+hQ0jWG84waqLvXl0jgU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1749596454; c=relaxed/simple; bh=YGpY68rA/JhjEp+LxAFTrZMk9ii1rggbyBCVjH5Hi/0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=UBjcUIxgxLmnqCzjCjkcwLYDVi3zHQxugVEWxQwqQTiH607Eo9YLOXpAmPccMz071GmQANFMpWRU75GqV88JnauOEKnJF7HNyqFrftcDLsC2sOhHreSMwz8ncLDN+ewkg1w8bqDQSkyCPAeBCLOAVCC76GzAoRFROHoQnBDpGmI= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=E1Ulayps; arc=none smtp.client-ip=209.85.167.47 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="E1Ulayps" Received: by mail-lf1-f47.google.com with SMTP id 2adb3069b0e04-551fdd1c4b3so831431e87.0 for ; Tue, 10 Jun 2025 16:00:51 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1749596450; x=1750201250; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=pv6eC8BWJe/QC9w4Z6dJHOpH2uERhEZbO922xRkpRTM=; b=E1Ulaypso42LWSiGsujPuX+rpA6xoQfdDwYYL0bI+3Y23Ptx5xiOCiV0vD8IaFhVnx mGeefsNjmdCZvIxjiDDfwk2EmhuyW7G8BTq4n1Qa0kRj2AVvQxoWy5NqNdigbRTkm132 9P13jlQ1D6fnVc1pSOmFKcR9nPHdKc3DBDFlfOu9V665f1bF2VBo6AwHhNvIrbfE62VG 8WMVcioP4Z+9qAkYFrZ+5m3gwC37gu0TjEFYqP2hqzYBArpfQtgHuvayjLyHlfUkGq18 PvA5pv7CmWp5vwG+PjUh50vf6tmsr75NfbZkMuP0d9topJo1To1/8X9zkXqkNFCYuBgG kLNA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1749596450; x=1750201250; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=pv6eC8BWJe/QC9w4Z6dJHOpH2uERhEZbO922xRkpRTM=; b=kf+FSFVazTVKyZJemeRfeEheMvvMsUxyBiKkaci2JXGs8otAQ8+892Rk74p/MPK9tX 6XFpAH99Abf+AF7ocfQy7u/P30jbU31a1XMrBNha8RoFQcYVMxzkaZq8h2P6qH1eGs+f ufvPkZTDwQtTgDSoUDt4g/qz89tbEhqx5YxnIi/FCTCTLUuj2mrAU9UYV14u90ETk/5E 9EUzdqRC2AZiko2xMbmFG2bYnxC13cAHdkq7EbPFgZ+jTZPl2Z7IUwwYmHLHNmuaRIcO m0Pg6PHqJ08tKH3Ng0TDekSvpUgV1aqgPKh6xUdHgMyDt4WO8rVzinj6ap1BptQCmWHb VRcQ== X-Forwarded-Encrypted: i=1; AJvYcCUMotkcMPqgJENlbN3/Ae64HJlsbCMuhKHrdQvsDhBj7vmNQemfzakKkLGfilzPKtn0WM4bIpWSCZ9BgQ==@vger.kernel.org X-Gm-Message-State: AOJu0YxByD0XGCu3hvkxn3ukxL9+8JRXE1QCu7G4d2n9B8yUxEZ5U4Kt Ms6s8li4ppkbuP+WNYslg0FT+V8IDUUBF3Zux34MJB7osPC2EUuaByUZq8aKg2i2kZA= X-Gm-Gg: ASbGncvJg2qfVui1fdb+Wut0uXvRcqtNetlgv4GPgFMUjpQopzIDnK1IRJI0qku+TAF pSJeIIwBOAK4vze3o7eVXVynIgP03GUhN0mUbcr1M851Lrdb9mn8O+PqEybimmDFtZjCkX3i+hc GE8QFaGXgowBJJ3XilmbB1uDqgjG9y73zC77t9DUtnSwpxsVRpUm7hJhyLkFz8TYMOLSlhWBZWD MS6zMlyAHKjsxWchfchCryjziHc4lAmmxSa5hqOy8H+5fAJ4JFixJLVBslJUqRFQ/rYm5ouaEes x1QP45A0YmvdfYU8n+nU4cWTmjhxD/63cHW797MrWHWVfSvDJ+OY3y9JGQKa1eXCfHv//SSmD/B q0ullaLu01XaKE1RKvMIbe9N10PmZVMlBTtgzmiKeK16q9SM1O78= X-Google-Smtp-Source: AGHT+IHxOB/GIvprDibRGH8b/xJoWimp0/bpa4Tr3aOnKL4YrIP9uIWQke3mgMvMH8CXv6c9NpUI5A== X-Received: by 2002:a05:6512:b09:b0:550:ecdf:a7f9 with SMTP id 2adb3069b0e04-5539c246fb3mr134143e87.10.1749596450228; Tue, 10 Jun 2025 16:00:50 -0700 (PDT) Received: from [192.168.1.4] (88-112-131-206.elisa-laajakaista.fi. [88.112.131.206]) by smtp.gmail.com with ESMTPSA id 2adb3069b0e04-553926a7d2dsm578480e87.157.2025.06.10.16.00.49 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 10 Jun 2025 16:00:49 -0700 (PDT) Message-ID: <0943821e-603a-4ee6-9bcb-e5fe690358c5@linaro.org> Date: Wed, 11 Jun 2025 02:00:48 +0300 Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/2] dt-bindings: media: qcom,x1e80100-camss: Sort interconnect alphabetically Content-Language: ru-RU To: Dmitry Baryshkov Cc: Bryan O'Donoghue , Robert Foss , Todor Tomov , Mauro Carvalho Chehab , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Hans Verkuil , linux-arm-msm@vger.kernel.org, linux-media@vger.kernel.org, devicetree@vger.kernel.org References: <20250610083318.2773727-1-vladimir.zapolskiy@linaro.org> <3e8f8220-1fad-437e-9fa4-5eb628891110@linaro.org> <97e51ab0-737b-496e-81df-b73c9f598bb0@linaro.org> <35muvo7h7ynfvzjt6jomasr54xaomfgt5etjc3uuczhfxww2ds@u5xsayanthx7> From: Vladimir Zapolskiy In-Reply-To: <35muvo7h7ynfvzjt6jomasr54xaomfgt5etjc3uuczhfxww2ds@u5xsayanthx7> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 6/11/25 01:21, Dmitry Baryshkov wrote: > On Tue, Jun 10, 2025 at 06:10:33PM +0300, Vladimir Zapolskiy wrote: >> On 6/10/25 18:02, Bryan O'Donoghue wrote: >>> On 10/06/2025 13:45, Vladimir Zapolskiy wrote: >>>>> >>>>> How is this a Fixes: ? >>>> >>>> I call it the fix to the dt-bindings documentation, then what is this >>>> change, if it's not a fix?.. >>>> >>>> Anyway, if there is a strong disagreement about if it's a fix or not, >>>> the Fixes tag can be dropped from the change, since it's so secondary. >>> >>> Since we don't have a committed upstream user I don't think this is an >>> ABI break. >> >> Well, Dmitry says it's an ABI break... It would be beneficial to come to >> a common understanding here. >> >>> But I also don't think it warrants a Fixes: tag either, there's no bug. >> >> There is no bug, but there are Documentation/ changes with Fixes tags, >> it's okay. > > Fixes means that there was a bug / issue that needs to be fixed. For > example, if there was a user for the bindings and the user had these That's "for example" only, I don't think it's an all-descriptive definition. From Documentation/process/submitting-patches.rst: A Fixes: tag indicates that the patch fixes an issue in a previous commit. In my opinion this is quite applicable here, the "fixed issue" in the device tree binding documentation file is well set, and anyone can get it from the provided commit message. Anyway, per the ask I'll remove the Fixes: tag and resubmit these changes, thank you for the patch review and discussion. > values in a different order, then changing schema to follow established > order would have been a fix. > >> >> I will resend the changes with whatever updates requested by both of you, >> if they do not contradict to each other. >> -- Best wishes, Vladimir