Received: by 2002:a05:6a10:1a4d:0:0:0:0 with SMTP id nk13csp169732pxb; Mon, 7 Feb 2022 08:39:35 -0800 (PST) X-Google-Smtp-Source: ABdhPJxfWtTI3tX3wbS+I7Ep6LywHHhOPww4LepptofTgGSGVkhjU+SG8BUp0FPxTG3ENBenu9mA X-Received: by 2002:a17:906:cede:: with SMTP id si30mr427534ejb.669.1644251975601; Mon, 07 Feb 2022 08:39:35 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1644251975; cv=none; d=google.com; s=arc-20160816; b=rgj90rfL1Zj1143RMSZ8sJzlh91j68xKfKHcJ34RkoXKIzqeCdOLYrBJLFwsoqgj3u B+sGjaVF5QMbREetYhE+JemU2lXLnhYCuoQZLE5cDq57RFhtLNPhXYKUZSwlzRTDjUON Lyd2IhkWSeaB3ZLdUbibKVgdU/VbvzzxTt3GzI/CedqVnEZVxrC5w55GKq9eQ7wqlV9N dAEVXw/QR+0ExF7Io+5mmPTeXUsLMAE0o9bIO97XNgtwadScIS7F/NovVyOnlvhhRRd9 MQYdK6YJQ36An0VUvE1odkQt2Q0MYSOcBM3cF+ZYl4tsvpLlIh7XWrZUeUYWR7JZOeYM U53A== 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=iqZrLHxxMKYXcwDk+JH/xBRKoG0uPSeGw2pYz3/v6kY=; b=rL4443zcy3aqXfkoL35fLuSZmylafKjs7H8v8XEjNhFGsWvtBFD3g1AE6kO2xswECM XB/wNsitKv9Lq4vW77EBUxP1mQ0iqQcqmQs7apsrJ0Ify7068Xp2MuGU+VG0+rUpyXkf EoBKSvMhsFL4kgwRXfcMehnzUOxw25MSMnBNGxEc1hDLHLwK1Ca2JvpNVefhoiVO5sqD tE2JoO6jXSZ5gFkm283GUrrml4D5taHCWyRrQRj6TUsJjruFMcv4q0gd7Qn1Dmav6VHR rRyL5GKb+89JseF82xeWbnsaxUR+BquHJSfUtdT/i8+TBjrulU7LC7Vd9kzr39bL3D0t OGEQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@canonical.com header.s=20210705 header.b=qnPp9d4x; 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 dr22si3125078ejc.277.2022.02.07.08.39.09; Mon, 07 Feb 2022 08:39:35 -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=qnPp9d4x; 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 S1357231AbiBDIw3 (ORCPT + 99 others); Fri, 4 Feb 2022 03:52:29 -0500 Received: from smtp-relay-internal-0.canonical.com ([185.125.188.122]:36942 "EHLO smtp-relay-internal-0.canonical.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1357225AbiBDIw0 (ORCPT ); Fri, 4 Feb 2022 03:52:26 -0500 Received: from mail-lj1-f197.google.com (mail-lj1-f197.google.com [209.85.208.197]) (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 8FD21405B4 for ; Fri, 4 Feb 2022 08:52:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1643964744; bh=iqZrLHxxMKYXcwDk+JH/xBRKoG0uPSeGw2pYz3/v6kY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=qnPp9d4xFaxTg3mjbDdwampcBXqqfo8DgkBf3yT/Edg/I4LN6PG/4BlRLSTKOk+yL iMlPX/FYl0tcjL790Q2ZD8tAn8JF4m4GJo2fMB4GVQg4DDH0dFtu0Le3/tE3FBel/5 AKBrh1u8NG9PLNOvZOWPZHfsb8YJtRl7G4w3fDgf+gFCAntwq26Ef43atxjovTutxJ wEc7NH1a5pVVQ59uIG1n7f840Ywhyr6HeOuQ68NyCo+ziLdt8raXl9UVhQmAC+jrSF TXTsfNhWA28SR38YNHj26zj03cekT0C9/iwbiBx4c+qcvQXtWF85t/v+dt5swWFz5G CY8qwSYtILxkw== Received: by mail-lj1-f197.google.com with SMTP id v9-20020a2e2f09000000b0023021c0c47aso917755ljv.11 for ; Fri, 04 Feb 2022 00:52:24 -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=iqZrLHxxMKYXcwDk+JH/xBRKoG0uPSeGw2pYz3/v6kY=; b=rMlNipyVrYfTrKDOVk/gXHV9A3Z5a/6u5Bn3tcoHGU8y+dKikE3iETr3yy9k8X0Md6 4+GfdHCAZtmK9Of9m4S41fdtO8sp0AQljLBhnr8SHYwLb8SauFrr+MZzAbpKUnSPR2PQ bAn9YKJeUoJlBOzwZNTKRgqnp7Goni4uCTpcC0+e50nIuEDBl2dY2ih2C5iMzSk/chxr pMIELrkfJKXfZPJk63B8EoJ1zA9U5YfHKSfheUmZDw3caFQ7QLq0xwvCk4yC3YtfoCqV b4Jco2UO2PteS5fsXzhImUla/70MsmYkYs8SiI3ZQSKqRgJbP+krYNsf2K9DCAOB/SG6 NdYw== X-Gm-Message-State: AOAM530muKT3YBH99EchUE8baNSG1SgTP73bQW1nbwu3XuWWQYfX2Ej9 js9P/1hfMHtSXiFzokDZDDLfcRLBR2ocjCrBsaxVVcHjzf8LzOQx7lfbd6bW5WxN6ghEHm5kI95 KBbMoW4PGhv6da4k6q6ihM57LgsFeJ+6eQtTWyVa+rg== X-Received: by 2002:a05:6402:3492:: with SMTP id v18mr1976668edc.345.1643964732877; Fri, 04 Feb 2022 00:52:12 -0800 (PST) X-Received: by 2002:a05:6402:3492:: with SMTP id v18mr1976597edc.345.1643964732578; Fri, 04 Feb 2022 00:52:12 -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 z8sm418238ejc.197.2022.02.04.00.52.10 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 04 Feb 2022 00:52:11 -0800 (PST) Message-ID: Date: Fri, 4 Feb 2022 09:52:10 +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: "Verdun, Jean-Marie" , Rob Herring , "Hawkins, Nick" Cc: 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> <1FEAAC19-1DD7-40D7-98FD-30C2659E7C0F@hpe.com> From: Krzysztof Kozlowski In-Reply-To: <1FEAAC19-1DD7-40D7-98FD-30C2659E7C0F@hpe.com> 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 18:07, Verdun, Jean-Marie wrote: > > > 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. > > Hi Krysztof, > > Accept our apologies if you think you lose your time, it is clearly not our > intent. > > This is the first time that we (I mean the team) introduce a new arch into > the linux kernel and I must admit that we had hard time to understand > from which angle we needed to start. > > I will probably write a Documentation afterward, as it is easy to find doc > on how to introduce a patch or a driver, but not when you want to > introduce a new chip. > > We are trying to do our best, and clearly want to follow all of your inputs. > Mistakes happen and they are clearly not intentional and not driven in > a way to make you lose your time. > > Helping others, and teaching something new is definitely a way to > optimize your time and this is what you are currently doing with us. > > We appreciate it and hope you will too. I understand, I also maybe over-reacted on this. Just please go through the comments you got for first submission and either apply them or respond why you disagree. The next submissions (patchset split into several commits) should be a v3, preferably with cover letter (git format-patch --cover-letter -v3 ...) where you can document also changes you did to the patchset. It looks for example like this: https://lore.kernel.org/linux-samsung-soc/31da451b-a36c-74fb-5667-d4193284c6cd@canonical.com/T/#mf98d2ac27a8481dc69dd110f9861c8318cade252 or like this (where changelogs are in each patch, although ordering is not correct because dt-bindings should be the first in the series): https://lore.kernel.org/all/20220103233948.198119-1-Mr.Bossman075@gmail.com/ Best regards, Krzysztof