From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932266AbcGFQsX (ORCPT ); Wed, 6 Jul 2016 12:48:23 -0400 Received: from hqemgate15.nvidia.com ([216.228.121.64]:7862 "EHLO hqemgate15.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754490AbcGFQsS (ORCPT ); Wed, 6 Jul 2016 12:48:18 -0400 X-PGP-Universal: processed; by hqnvupgp08.nvidia.com on Wed, 06 Jul 2016 09:46:39 -0700 Subject: Re: [PATCH V2 05/10] firmware: tegra: add BPMP support To: Alexandre Courbot , Joseph Lo References: <20160705090431.5852-1-josephl@nvidia.com> <20160705090431.5852-6-josephl@nvidia.com> CC: Stephen Warren , Thierry Reding , "linux-tegra@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , Rob Herring , Mark Rutland , Peter De Schrijver , "devicetree@vger.kernel.org" , Jassi Brar , Linux Kernel Mailing List , Catalin Marinas , Will Deacon From: Matt Longnecker Message-ID: <577D363A.4010003@nvidia.com> Date: Wed, 6 Jul 2016 09:47:54 -0700 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: X-Originating-IP: [172.18.213.133] X-ClientProxiedBy: HQMAIL102.nvidia.com (172.18.146.10) To HQMAIL102.nvidia.com (172.18.146.10) Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Alex, On 07/06/2016 04:39 AM, Alexandre Courbot wrote: >> diff --git a/include/soc/tegra/bpmp_abi.h b/include/soc/tegra/bpmp_abi.h >> >new file mode 100644 >> >index 000000000000..0aaef5960e29 >> >--- /dev/null >> >+++ b/include/soc/tegra/bpmp_abi.h >> >@@ -0,0 +1,1601 @@ >> >+/* >> >+ * Copyright (c) 2014-2016, NVIDIA CORPORATION. All rights reserved. >> >+ * >> >+ * This program is free software; you can redistribute it and/or modify it >> >+ * under the terms and conditions of the GNU General Public License, >> >+ * version 2, as published by the Free Software Foundation. >> >+ * >> >+ * This program is distributed in the hope it will be useful, but WITHOUT >> >+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or >> >+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License for >> >+ * more details. >> >+ * >> >+ * You should have received a copy of the GNU General Public License >> >+ * along with this program. If not, see. >> >+ */ >> >+ >> >+#ifndef_ABI_BPMP_ABI_H_ >> >+#define_ABI_BPMP_ABI_H_ >> >+ >> > ... > > There is a lot of stuff in this file, most of which we are not using > now - this is ok, but unless this is a file synced from an outside > resource maybe we should trim the structures we don't need and add > them as we make use of them? It helps dividing the work in bite-size > chunks. > > Regarding the documentation format of this file, is this valid kernel > documentation since the adoption of Sphynx? Or is it whatever the > origin is using? bpmp_abi.h is meant to be delivered as is from an NVIDIA internal repo to a variety of OS'es. Each of them has a different documentation standard and coding standard. I'd like to avoid trimming parts from the file (or even worse modifying parts of the file) so that future deliveries are trivial. Thanks, Matt Longnecker