Received: by 2002:a05:6a10:1a4d:0:0:0:0 with SMTP id nk13csp2590957pxb; Thu, 3 Feb 2022 09:42:40 -0800 (PST) X-Google-Smtp-Source: ABdhPJxLHkBeQ06VXx/REQuIRJldDTt/6obSJ4D4aDCTMNhS0ZriPGTZiv6sJfGJL+3b4aaQyDtq X-Received: by 2002:a17:907:2da7:: with SMTP id gt39mr30928108ejc.563.1643910159993; Thu, 03 Feb 2022 09:42:39 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643910159; cv=none; d=google.com; s=arc-20160816; b=wy26imYIp9nkS5gBEZ2dtuHlk+83wQ4+aUad1rwHlIwkA3rWElUus+hhV/9toVEcy8 nygYJZXV0LCNHqrCSH7YJL03B1V1IuaR3iZ2gW5B4FiyEFbwIyzPDZ4wzRsNHbelqy7Q hejqvW2cym627MxQR4HJcZLCXJ8N9pIhjlhEynAEsSlBGtI/jZwBASgQEgTRQZ/iqzwl t8IiMTKjjvBeFd8Oor2CQlhHHpwEVQzjjCX31u4MV4vtL17jdP/VqHpSyb6/WfHj+qUq Drq0yxFtVcOhMf75Ix2cOScurHoGfpmPFPE+pTqeSsK1Fj5j4h6ArTS33hdzeMc+g/PX KJuQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=JKwlxtatWZQCVG4GF4BszucWwjVQTt2PxoebrxL5ciY=; b=SPmq+Ts6tNDZDtTYTng3TkmawGrKV69EKmp5Pv5Ugf5AFJI7Jl+A7RuCx3YratNeUg vLIVkgRfJeKPTn7l6FCjlp9t+k2A19mlvK6EJB/Ut5lNN9j2nGqhj5/cp90opynKkUI5 PeQ/5GYQYehURj6vTgxL3gAAxzed9h5SbRxF4xa/rGefwX4nv+MRD3Cg4qNa6F5BjwM9 NxdbZtr+YsiXz/cnJsQexd0hTDw0cPCqOxL4yTE5ckXenXIoTZYo51yrpromVxaTAP4e in8FEa29ljYeAZ1Z8UAfvyihAcmDDABMEj8nWJyQ3SFpa0lLI7fxypvpTeV+eD4EP2PG PCQA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@canonical.com header.s=20210705 header.b=i6yA7t9C; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=canonical.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id he18si13844865ejc.503.2022.02.03.09.42.12; Thu, 03 Feb 2022 09:42:39 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@canonical.com header.s=20210705 header.b=i6yA7t9C; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=canonical.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1351566AbiBCO4H (ORCPT + 99 others); Thu, 3 Feb 2022 09:56:07 -0500 Received: from smtp-relay-internal-0.canonical.com ([185.125.188.122]:33912 "EHLO smtp-relay-internal-0.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1351573AbiBCO4B (ORCPT ); Thu, 3 Feb 2022 09:56:01 -0500 Received: from mail-ed1-f69.google.com (mail-ed1-f69.google.com [209.85.208.69]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-internal-0.canonical.com (Postfix) with ESMTPS id 0F31A4031B for ; Thu, 3 Feb 2022 14:56:00 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1643900160; bh=JKwlxtatWZQCVG4GF4BszucWwjVQTt2PxoebrxL5ciY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=i6yA7t9Cv2ukmUXvHhtNRdUE9hNLWOnYQ2g7APjSZAMHCMKpeANi/B1ZtgoguxDA5 Nm8VGjt1PXrQCyAjvavzejgvqNwBlvEgSHGjKPyLksrXFul6IDWxivqgsLUUGcCUjS fYm9h6Pyw0RXim93IOsKpnABcf1QYGK+V8qRcDxd11bioFzHas2El667YHdN3q+EGT 2qj40Z0/CY+7lpUpF8OMylyrNyz35bRpUel+rnHAY/Czwj0Tnr1Wlb+rKxCPcM4sIj 7RfT/OhKWfuiBotVjERE1wg8fEM53q7Hpi+txS+3Vpb2pmHIpb1VidkUD8X5eEaxz4 C9+xoRt97/5cg== Received: by mail-ed1-f69.google.com with SMTP id l14-20020aa7cace000000b003f7f8e1cbbdso1552838edt.20 for ; Thu, 03 Feb 2022 06:56:00 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:message-id:date:mime-version:user-agent:subject :content-language:to:cc:references:from:in-reply-to :content-transfer-encoding; bh=JKwlxtatWZQCVG4GF4BszucWwjVQTt2PxoebrxL5ciY=; b=73sG/nsZnoOGhLxWVo1j2fVbFpGpu6UecMzgXJfQ2vX9zoGr4kKCGZWcbN2jeBP3w7 UOF5iFj753GOfoDdT3At55OV1wnnit/tnuI2ZPaPcLd8VYGjEJuu7xWL41QT2Bs3o/mU 3CqFiuZZotjbNQQQ0BJ4RKclLvzi/SbjuuV+URrU4Zz4UkjfHCcLaDWfB/s4qSp7tVbS W7lKeaij4TwUMIMNRJYj2FyN91FsPxW86DN6fFaIf746eHddcXPTKJnyhp+Kb2AJOt0U lPYt4sXTo++cOpP8AkWdbFVpftXKWz23SEM96Zl7ibaNvOSey0+PFrBTJRcLPPBH3pnO LDCg== X-Gm-Message-State: AOAM532GI5ZqVb2gpV+AyGqA13AFJAb7XI7W3iXnPFPHxhEEMXsYwX2d vdzDOkc/93HMZXZIH3h2OhLdtIcZ2tMjimTCZ6jRdxmhIGPCrjywJc+EK9fEbHmf7EfAWKlb3gL xDqBVkSgRW4hOVtuwrGXXY5g9lF895EZsQ3rbe69y+A== X-Received: by 2002:a17:907:c14:: with SMTP id ga20mr1992518ejc.243.1643900159544; Thu, 03 Feb 2022 06:55:59 -0800 (PST) X-Received: by 2002:a17:907:c14:: with SMTP id ga20mr1992450ejc.243.1643900159293; Thu, 03 Feb 2022 06:55:59 -0800 (PST) Received: from [192.168.0.81] (xdsl-188-155-168-84.adslplus.ch. [188.155.168.84]) by smtp.gmail.com with ESMTPSA id z19sm7573934eja.18.2022.02.03.06.55.57 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 03 Feb 2022 06:55:58 -0800 (PST) Message-ID: Date: Thu, 3 Feb 2022 15:55:56 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 Subject: Re: [PATCH] HPE BMC GXP SUPPORT Content-Language: en-US To: Rob Herring , nick.hawkins@hpe.com Cc: verdun@hpe.com, David Airlie , Daniel Vetter , Linus Walleij , Bartosz Golaszewski , Corey Minyard , Miquel Raynal , Richard Weinberger , Vignesh Raghavendra , "David S. Miller" , Jakub Kicinski , Thierry Reding , =?UTF-8?Q?Uwe_Kleine-K=c3=b6nig?= , Lee Jones , Greg Kroah-Hartman , Mark Brown , "Rafael J. Wysocki" , Daniel Lezcano , Amit Kucheria , Zhang Rui , Thomas Gleixner , Wim Van Sebroeck , Guenter Roeck , Russell King , Arnd Bergmann , Olof Johansson , SoC Team , Shawn Guo , Stanislav Jakubek , Sam Ravnborg , Hao Fang , "Russell King (Oracle)" , Mark Rutland , Marc Zyngier , Ard Biesheuvel , Anshuman Khandual , Wang Kefeng , Lukas Bulwahn , Masahiro Yamada , dri-devel , devicetree@vger.kernel.org, "linux-kernel@vger.kernel.org" , "open list:GPIO SUBSYSTEM" , Linux I2C , openipmi-developer@lists.sourceforge.net, MTD Maling List , netdev , Linux PWM List , "open list:SERIAL DRIVERS" , linux-spi , "open list:THERMAL" , Linux USB List , LINUX-WATCHDOG , linux-arm-kernel References: <20220202165315.18282-1-nick.hawkins@hpe.com> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 03/02/2022 15:29, Rob Herring wrote: > On Wed, Feb 2, 2022 at 10:55 AM wrote: >> >> From: Nick Hawkins >> (...) >> + >> + vuart_a: vuart_a@80fd0200 { > > serial@... Maybe it does not look like, but this is actually a v2. Nick was asked to change the naming for the nodes already in v1. Unfortunately it did not happen, so we have vuart, spifi, vic and more. It is a waste of reviewers' time to ask them to perform the same review twice or to ignore their comments. > >> + compatible = "hpe,gxp-vuart"; >> + reg = <0x80fd0200 0x100>; >> + interrupts = <2>; >> + interrupt-parent = <&vic1>; >> + clock-frequency = <1846153>; >> + reg-shift = <0>; >> + status = "okay"; >> + serial-line = <3>; >> + vuart_cfg = <&vuart_a_cfg>; >> + }; (...) >> diff --git a/Documentation/devicetree/bindings/vendor-prefixes.yaml b/Documentation/devicetree/bindings/vendor-prefixes.yaml >> index 294093d45a23..913f722a6b8d 100644 >> --- a/Documentation/devicetree/bindings/vendor-prefixes.yaml >> +++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml >> @@ -514,7 +514,9 @@ patternProperties: >> "^hoperun,.*": >> description: Jiangsu HopeRun Software Co., Ltd. >> "^hp,.*": >> - description: Hewlett Packard >> + description: Hewlett Packard Inc. > > Why are you changing this one? I guess this is squashing of my patch: https://lore.kernel.org/all/20220127075229.10299-1-krzysztof.kozlowski@canonical.com/ which is fine to me, but vendor changve should be a separate commit with its own explanation. Now it looks indeed weird. > >> + "^hpe,.*": > > You used HPE elsewhere... Lowercase is preferred. Best regards, Krzysztof