Message ID | 20240124164646.1506814-1-hugo@hugovil.com |
---|---|
Headers |
Return-Path: <linux-kernel+bounces-37368-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7300:2553:b0:103:945f:af90 with SMTP id p19csp1119574dyi; Wed, 24 Jan 2024 08:57:40 -0800 (PST) X-Google-Smtp-Source: AGHT+IH4rbTBqOFdV81Pn/5WYvjhYgmvXP8iJpYvE8+TMk+FOKwAZH2OGdjKw+4mpp6Ty35IRsfR X-Received: by 2002:a05:6902:1ac9:b0:dbe:3ddf:57e8 with SMTP id db9-20020a0569021ac900b00dbe3ddf57e8mr1101115ybb.55.1706115460681; Wed, 24 Jan 2024 08:57:40 -0800 (PST) ARC-Seal: i=2; a=rsa-sha256; t=1706115460; cv=pass; d=google.com; s=arc-20160816; b=UZanP5RN3GIPnzPIK4Abb9/LbUTPlXJEj7Wk3WU+VyafiRV6/fIfWz5U6LxzgBjwDU DZD/Lc29+un15gXBVzWclSjHwDnUNVXGam887rsLBwZGMyKgxSJJ1PcuFloU+bSoGu/f 1ej3z8YTDr4e5IpqEU0VlMHWAogPDfBzdI12DYuGYyl2ZLBgBvdK7wlNAjNLC7MsZUpz JmLygncakKZjyajV0eqFyCkx1p6pfNsiP3XQKcmA7/so0pGC6jZNu2ZwfvsyhzQvNyfb MmacRp+2HzJjEbtwE/wkkM9hq8CBCNhr9SiJeqMk14shjCknkyL0jygvytTAtjYR2q/M QCGw== ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=subject:content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:cc:to:from :dkim-signature; bh=mnUWpMCy5DToKNWP1PJXR/S74CtvHvrqkEFrG4JVcmw=; fh=hwulkhypVQWzRkEwR6SVi+5t+9KS51RC5S7dj7FQiPo=; b=vPZB5Aw/zhfiyrf2kLtnxs9U5zfoozZhNVEODxWcZoj6TWXnVq+vm5EJLaC659U3xK oFHEfgQRmI5o0bqyjQdW3zqNFEZxXYJcpFqMyGBYmw0nVhApvICa/OpLjNQO1ZDeDk99 R1JKHxsRFX8GJmccx8aY7Op1ChaVx4gS2820QG9zxFa4rRpli74ySCFvFgAdu+jBAS5a uIVNifa6IbvaQS45yBXyWAirdsAccTohIMl5hIC8NwEdPU1GuFgskM7mBPgCMQ23FFEe zlzRNz0lkwFOaUYC5z9HK50Kvdq2UqaiKlVaJHET1MUknOCDgH1zGMwUb6FBXtPO8Y+W PNrA== ARC-Authentication-Results: i=2; mx.google.com; dkim=pass header.i=@hugovil.com header.s=x header.b=gQceyD+1; arc=pass (i=1 spf=pass spfdomain=hugovil.com dkim=pass dkdomain=hugovil.com); spf=pass (google.com: domain of linux-kernel+bounces-37368-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-37368-ouuuleilei=gmail.com@vger.kernel.org" Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id t18-20020a0ca692000000b00680c838d0fdsi10494501qva.169.2024.01.24.08.57.40 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 24 Jan 2024 08:57:40 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-37368-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@hugovil.com header.s=x header.b=gQceyD+1; arc=pass (i=1 spf=pass spfdomain=hugovil.com dkim=pass dkdomain=hugovil.com); spf=pass (google.com: domain of linux-kernel+bounces-37368-ouuuleilei=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-37368-ouuuleilei=gmail.com@vger.kernel.org" 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 ny.mirrors.kernel.org (Postfix) with ESMTPS id 774261C22BAA for <ouuuleilei@gmail.com>; Wed, 24 Jan 2024 16:57:40 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 94CB9823A2; Wed, 24 Jan 2024 16:56:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=hugovil.com header.i=@hugovil.com header.b="gQceyD+1" Received: from mail.hugovil.com (mail.hugovil.com [162.243.120.170]) (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 C3DE37E584; Wed, 24 Jan 2024 16:56:36 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=162.243.120.170 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706115398; cv=none; b=pF+C15it9twTyWu2XgWy2f5WOIse1F7j2KRBAXth46WjvfBfuJs/2MPi1VlOz1tLhD05xIdVFHhCVGzU2LU24ZJMHbz2C+5FNzGLRluJiEBOKqBJKVFYQYK2z8fcNMjbPXZzNBzqOuZJuXpKqxxFqAZbgRnvLOh5jOMz9IC4QVQ= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1706115398; c=relaxed/simple; bh=J/sQhWMEvt/vn+lfCEXPe62LcDsGsyF7kH6x+xd5LL0=; h=From:To:Cc:Date:Message-Id:MIME-Version:Subject; b=na5K5xKxWDbMQAvl9+8+ay5DRP9QjC3kewHG0pUDaFzmnYEiHRV++EZ/4O1ygeKB8MXsu+RZ8hjYpAU4EbLKkdM+uUdxsMTWQWK4tHGYtlS6t2h5q1mj1UNx3lnLp7o7FENYzTIDDW2OMl/PqFD/yfOUOGIymbhnlpamOLfpB4w= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=hugovil.com; spf=pass smtp.mailfrom=hugovil.com; dkim=pass (1024-bit key) header.d=hugovil.com header.i=@hugovil.com header.b=gQceyD+1; arc=none smtp.client-ip=162.243.120.170 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=hugovil.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=hugovil.com DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=hugovil.com ; s=x; h=Subject:Content-Transfer-Encoding:MIME-Version:Message-Id:Date:Cc:To :From:subject:date:message-id:reply-to; bh=mnUWpMCy5DToKNWP1PJXR/S74CtvHvrqkEFrG4JVcmw=; b=gQceyD+1WuSt+4y40mjd6PxMF1 FehwBP+JKLpk5+tVsOi7HMq60F3ateXgIJLo4dQZismkca2AN6krYcA6HWfPqTmoZqt/POizCd7Ks QYHCyImjGHZ3MX4GaiFJOgRyB6AbaIyjL90geHBph/FQtpKGpuFKo9KGD9Lolf7P5fm4=; Received: from modemcable168.174-80-70.mc.videotron.ca ([70.80.174.168]:45420 helo=pettiford.lan) by mail.hugovil.com with esmtpa (Exim 4.92) (envelope-from <hugo@hugovil.com>) id 1rSgY6-0007ym-NO; Wed, 24 Jan 2024 11:56:27 -0500 From: Hugo Villeneuve <hugo@hugovil.com> To: robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, linux-imx@nxp.com, leoyang.li@nxp.com, hvilleneuve@dimonoff.com Cc: krzysztof.kozlowski@linaro.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, hugo@hugovil.com, andy.shevchenko@gmail.com Date: Wed, 24 Jan 2024 11:46:44 -0500 Message-Id: <20240124164646.1506814-1-hugo@hugovil.com> X-Mailer: git-send-email 2.39.2 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-SA-Exim-Connect-IP: 70.80.174.168 X-SA-Exim-Mail-From: hugo@hugovil.com X-Spam-Level: X-Spam-Report: * -1.0 ALL_TRUSTED Passed through trusted hosts only via SMTP * -0.0 T_SCC_BODY_TEXT_LINE No description available. Subject: [PATCH v2 0/2] board: imx8mn-rve-gateway: fix compatible description X-SA-Exim-Version: 4.2.1 (built Wed, 08 May 2019 21:11:16 +0000) X-SA-Exim-Scanned: Yes (on mail.hugovil.com) X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1788989445670823474 X-GMAIL-MSGID: 1788991725424135280 |
Series |
board: imx8mn-rve-gateway: fix compatible description
|
|
Message
Hugo Villeneuve
Jan. 24, 2024, 4:46 p.m. UTC
From: Hugo Villeneuve <hvilleneuve@dimonoff.com>
Hello,
this patch series fixes compatible description for the RVE gateway board.
Thank you.
Link: [v1] https://lore.kernel.org/all/20240124154422.3600920-1-hugo@hugovil.com/raw
Changes for V2:
-Add Fixes tags
Hugo Villeneuve (2):
dt-bindings: arm: fsl: remove redundant company name
arm64: dts: imx8mn-rve-gateway: remove redundant company name
Documentation/devicetree/bindings/arm/fsl.yaml | 2 +-
arch/arm64/boot/dts/freescale/imx8mn-rve-gateway.dts | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
base-commit: 615d300648869c774bd1fe54b4627bb0c20faed4
Comments
On 24/01/2024 17:46, Hugo Villeneuve wrote: > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > Company name in compatible description appears twice, which is not really > helpful, so remove it from product name. It's just a string. There is no bug, at least you did not describe where the bug is. This has probably little impact on user-space, but there could be such which actually reads compatibles and depends on them. Or bootloader or whatever. Therefore to me such cosmetic change of correct compatible is not worth ABI impact. Best regards, Krzysztof
On Wed, 24 Jan 2024 18:00:00 +0100 Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > On 24/01/2024 17:46, Hugo Villeneuve wrote: > > From: Hugo Villeneuve <hvilleneuve@dimonoff.com> > > > > Company name in compatible description appears twice, which is not really > > helpful, so remove it from product name. > > It's just a string. There is no bug, at least you did not describe where > the bug is. Hi Krzysztof, in fact the fix is based on comments you made in a review for another board that I submitted: https://lore.kernel.org/all/6035c4cc-afe1-456d-a4d0-16992da09a2e@linaro.org/ ------------------ Company name appears twice, which is not really helpful. What's the true name? Gateway EVK? Then keep only this. ----------------- > This has probably little impact on user-space, but there could be such > which actually reads compatibles and depends on them. Or bootloader or > whatever. Therefore to me such cosmetic change of correct compatible is > not worth ABI impact. Yes, normally you would be right, but in this specific case, the board is a prototype developped by my company and we are still at the prototype stage, so there is zero ABI impact. Maybe I can add this to the cover letter if you want me to submit a V3. Hugo Villeneuve
On 24/01/2024 18:08, Hugo Villeneuve wrote: > On Wed, 24 Jan 2024 18:00:00 +0100 > Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> wrote: > >> On 24/01/2024 17:46, Hugo Villeneuve wrote: >>> From: Hugo Villeneuve <hvilleneuve@dimonoff.com> >>> >>> Company name in compatible description appears twice, which is not really >>> helpful, so remove it from product name. >> >> It's just a string. There is no bug, at least you did not describe where >> the bug is. > > Hi Krzysztof, > in fact the fix is based on comments you made in a > review for another board that I submitted: > > https://lore.kernel.org/all/6035c4cc-afe1-456d-a4d0-16992da09a2e@linaro.org/ > > ------------------ > Company name appears twice, which is not really helpful. What's the true > name? Gateway EVK? Then keep only this. You are mixing arguments. You want to change released compatible based on style comment. Style comments are reasonable comments for compatibles while they are patches. Not released commits. > ----------------- > >> This has probably little impact on user-space, but there could be such >> which actually reads compatibles and depends on them. Or bootloader or >> whatever. Therefore to me such cosmetic change of correct compatible is >> not worth ABI impact. > > Yes, normally you would be right, but in this specific case, the board > is a prototype developped by my company and we are still at the > prototype stage, so there is zero ABI impact. > > Maybe I can add this to the cover letter if you want me to submit a V3. Any explanation and possible ABI impact should be in commit msg. Anyway, one patchset per day, so people have chance to respond... Best regards, Krzysztof