Received: by 2002:a05:6a10:8c0a:0:0:0:0 with SMTP id go10csp3970512pxb; Mon, 1 Feb 2021 09:09:00 -0800 (PST) X-Google-Smtp-Source: ABdhPJweDcz7TE2xfTCo+rJcHq3U12ZtQSZjJVIorrHnuq5FkWamg27r3RBA0ID42f4PPkkkaANK X-Received: by 2002:a50:fd12:: with SMTP id i18mr19937224eds.220.1612199339843; Mon, 01 Feb 2021 09:08:59 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1612199339; cv=none; d=google.com; s=arc-20160816; b=IzmTZVhsuj8J3ALM2G55Srnts/ZbJbod23Tejj8uIP+dItMlbIAaP/qCRQsMgt5UOL IYwSvGtYrNWKg2V5/XLNvUCTGNdg3/jzeeAyTM3Ai/gJWySxHvJyZlbSFmlJuv6Kgzj0 lMtBr9+0s+JXK9KxTbjgEcimUIm8vAEVXbMXAiSXQNB6JOlXuFJQtG3WL12sQKFr6BQC +LnIX1fv1w/g4+Ul5M0Fkbi8DHnYpNVZ8hJmCLW17d/fF7+DYcO8gIFAHRwwvaC1bn18 iIDYWqMr25GixwJdXKzJy7j9lZmhYcxmoqU7Eskq4RLJiRoB/T5EjA63eHyfT2YCBUmV NinQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=rV0K3F4zmeFdrVc02lSWJwOMPX58ML+v7RlA79PMicU=; b=MZ8QtvIPSo+3S1NOxBuP20SQr5LWWXnBmJ8sZk68GzW6N8w3srr2iEDWm6Lmp+n4NZ MqJ/rd98Zi3p+fnK19YsUTpRb/4XogmlErP9rQJlMLqBLqH4dBDbrMb4U86PBf+EGete CaKm/2lL/I/He/t/VyNk8WvlH0wc/FOWomDiQDsI5U2b7Ap4sdbx+xeVlQN0aRDH5NU2 rfo5ex2tiypD9/jBv8wptwlDbCB+CUcakXHTiMMAq7qrChhGwn+PIDdE4npSDoAGA2rV spjf2ghc4Fc+MZs0ggteI97vzwjyxdVP6de2Z/8S/g/v+qVFyyc3syUaJvR2rIQL0tUi jl4Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=jJ0VbsO9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id a17si11282618edn.146.2021.02.01.09.08.26; Mon, 01 Feb 2021 09:08:59 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=jJ0VbsO9; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230298AbhBARGZ (ORCPT + 99 others); Mon, 1 Feb 2021 12:06:25 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34144 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229549AbhBARGU (ORCPT ); Mon, 1 Feb 2021 12:06:20 -0500 Received: from mail-pg1-x534.google.com (mail-pg1-x534.google.com [IPv6:2607:f8b0:4864:20::534]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CADCEC06174A for ; Mon, 1 Feb 2021 09:05:40 -0800 (PST) Received: by mail-pg1-x534.google.com with SMTP id g15so12476871pgu.9 for ; Mon, 01 Feb 2021 09:05:40 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=rV0K3F4zmeFdrVc02lSWJwOMPX58ML+v7RlA79PMicU=; b=jJ0VbsO9t5DSWgyjy9Zb5XMirkM538/h0z7vYekgbJ8VlWdqeZod2gMdQ3QMxRPHl6 ygozOMFrOcOukUqgY50TWQWWL+JHiDtB3fgFN6bnBU0eRFYxCMNM6CxbhOVgXYTEm0YB GZ0dLFspeAGIXaZ1qhcVxW8VoAvjHOXLfO9jKMS6/P9O/hdoqSqnlhSDDleO3FXavl3K q4sFyAI678eZ5SzBFKb6N3hkzRjyIVszFsB6nmnzk7y6QOSGh6sU89b3k9G7XDVxR5uI rkgKoN7TWGUU8ze+8YxFQi7Agv2KIurgquVNzY+RFDsO20cMqBtIEofyFUWfAHuhx4OA MHjw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=rV0K3F4zmeFdrVc02lSWJwOMPX58ML+v7RlA79PMicU=; b=DV1kjqbUyOB4Evw2k+q5BkHQzLLWu9BUINCwCeCMdr3VNuloe7TB+GkXb9FBqBzIcO aGLlw4rjeD17C5Fpg8sm6QrwtmWghbGf8cn48/kvMnnMB/NBYWP/JETpyYMg7Jkug3z7 zbirp4EZkUpCMedcMdWJfKAVO3VVMW/Nmv9XZFIkN6bSCDuZBttKKG0aMBGmxyd+aSG1 sM8z9YmySKtnDIUpANZRqYhGwPyA/bI0O5DWiH3IsnqsMYvK+17CacUh/wQbgyEnF9XX qfRIEZ7O7f5DO4ZgCe0rp+FzueY7c4sZ16GjgqmqnhysWAuSOOEUseQalLK1rlt15l+U hOXQ== X-Gm-Message-State: AOAM533gLChPqTdInPXAtsPQ8Y9qMHBWyi+LE+/+2CQRepSb1B58xlpj 5x9EMNWFkGuh26VxR7J4d6p/5w== X-Received: by 2002:a63:1f21:: with SMTP id f33mr18240572pgf.31.1612199140061; Mon, 01 Feb 2021 09:05:40 -0800 (PST) Received: from xps15 (S0106889e681aac74.cg.shawcable.net. [68.147.0.187]) by smtp.gmail.com with ESMTPSA id i9sm18878851pfo.146.2021.02.01.09.05.38 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Feb 2021 09:05:39 -0800 (PST) Date: Mon, 1 Feb 2021 10:05:37 -0700 From: Mathieu Poirier To: Qi Liu Cc: suzuki.poulose@arm.com, mike.leach@linaro.org, coresight@lists.linaro.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linuxarm@openeuler.org Subject: Re: [PATCH] coresight: Remove duplicate header files of coresight drivers Message-ID: <20210201170537.GA1399621@xps15> References: <1612184564-26613-1-git-send-email-liuqi115@huawei.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1612184564-26613-1-git-send-email-liuqi115@huawei.com> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Good day, On Mon, Feb 01, 2021 at 09:02:44PM +0800, Qi Liu wrote: > Remove duplicate included header files, as coresight-priv.h is included in > these coresight drivers. > > Signed-off-by: Qi Liu > --- > drivers/hwtracing/coresight/coresight-catu.c | 2 -- > drivers/hwtracing/coresight/coresight-core.c | 3 --- > drivers/hwtracing/coresight/coresight-cpu-debug.c | 3 --- > drivers/hwtracing/coresight/coresight-cti-core.c | 3 --- > drivers/hwtracing/coresight/coresight-cti-platform.c | 1 - > drivers/hwtracing/coresight/coresight-cti.h | 1 - > drivers/hwtracing/coresight/coresight-etb10.c | 4 ---- > drivers/hwtracing/coresight/coresight-etm-perf.c | 1 - > drivers/hwtracing/coresight/coresight-etm3x-sysfs.c | 1 - > drivers/hwtracing/coresight/coresight-etm4x-core.c | 5 ----- > drivers/hwtracing/coresight/coresight-etm4x-sysfs.c | 1 - > drivers/hwtracing/coresight/coresight-funnel.c | 3 --- > drivers/hwtracing/coresight/coresight-platform.c | 2 -- > drivers/hwtracing/coresight/coresight-replicator.c | 3 --- > drivers/hwtracing/coresight/coresight-stm.c | 3 --- > drivers/hwtracing/coresight/coresight-tmc-core.c | 3 --- > drivers/hwtracing/coresight/coresight-tmc-etf.c | 1 - > drivers/hwtracing/coresight/coresight-tmc-etr.c | 1 - > drivers/hwtracing/coresight/coresight-tpiu.c | 4 ---- > 19 files changed, 45 deletions(-) The idea is to make sure every file stands on its own without dependencies on other header files. Otherwise it is difficult and expensive to move files around or make changes to any header file. As such I will not move forward with this set. Thanks, Mathieu > > diff --git a/drivers/hwtracing/coresight/coresight-catu.c b/drivers/hwtracing/coresight/coresight-catu.c > index a61313f..fcc0367 100644 > --- a/drivers/hwtracing/coresight/coresight-catu.c > +++ b/drivers/hwtracing/coresight/coresight-catu.c > @@ -7,10 +7,8 @@ > * Author: Suzuki K Poulose > */ > > -#include > #include > #include > -#include > #include > #include > > diff --git a/drivers/hwtracing/coresight/coresight-core.c b/drivers/hwtracing/coresight/coresight-core.c > index 4ba801d..5fb8b12 100644 > --- a/drivers/hwtracing/coresight/coresight-core.c > +++ b/drivers/hwtracing/coresight/coresight-core.c > @@ -7,17 +7,14 @@ > #include > #include > #include > -#include > #include > #include > #include > #include > #include > #include > -#include > #include > #include > -#include > > #include "coresight-etm-perf.h" > #include "coresight-priv.h" > diff --git a/drivers/hwtracing/coresight/coresight-cpu-debug.c b/drivers/hwtracing/coresight/coresight-cpu-debug.c > index e1d2324..f8df512 100644 > --- a/drivers/hwtracing/coresight/coresight-cpu-debug.c > +++ b/drivers/hwtracing/coresight/coresight-cpu-debug.c > @@ -4,15 +4,12 @@ > * > * Author: Leo Yan > */ > -#include > -#include > #include > #include > #include > #include > #include > #include > -#include > #include > #include > #include > diff --git a/drivers/hwtracing/coresight/coresight-cti-core.c b/drivers/hwtracing/coresight/coresight-cti-core.c > index 61dbc1a..ed7ab82 100644 > --- a/drivers/hwtracing/coresight/coresight-cti-core.c > +++ b/drivers/hwtracing/coresight/coresight-cti-core.c > @@ -4,18 +4,15 @@ > * Author: Mike Leach > */ > > -#include > #include > #include > #include > #include > #include > #include > -#include > #include > #include > #include > -#include > #include > #include > > diff --git a/drivers/hwtracing/coresight/coresight-cti-platform.c b/drivers/hwtracing/coresight/coresight-cti-platform.c > index ccef04f..60b0580 100644 > --- a/drivers/hwtracing/coresight/coresight-cti-platform.c > +++ b/drivers/hwtracing/coresight/coresight-cti-platform.c > @@ -2,7 +2,6 @@ > /* > * Copyright (c) 2019, The Linaro Limited. All rights reserved. > */ > -#include > #include > #include > #include > diff --git a/drivers/hwtracing/coresight/coresight-cti.h b/drivers/hwtracing/coresight/coresight-cti.h > index acf7b54..0a3b313 100644 > --- a/drivers/hwtracing/coresight/coresight-cti.h > +++ b/drivers/hwtracing/coresight/coresight-cti.h > @@ -7,7 +7,6 @@ > #ifndef _CORESIGHT_CORESIGHT_CTI_H > #define _CORESIGHT_CORESIGHT_CTI_H > > -#include > #include > #include > #include > diff --git a/drivers/hwtracing/coresight/coresight-etb10.c b/drivers/hwtracing/coresight/coresight-etb10.c > index 0cf6f0b..8d5bc04 100644 > --- a/drivers/hwtracing/coresight/coresight-etb10.c > +++ b/drivers/hwtracing/coresight/coresight-etb10.c > @@ -10,17 +10,13 @@ > #include > #include > #include > -#include > #include > #include > #include > #include > #include > #include > -#include > #include > -#include > -#include > #include > #include > #include > diff --git a/drivers/hwtracing/coresight/coresight-etm-perf.c b/drivers/hwtracing/coresight/coresight-etm-perf.c > index bdc34ca..01aba63 100644 > --- a/drivers/hwtracing/coresight/coresight-etm-perf.c > +++ b/drivers/hwtracing/coresight/coresight-etm-perf.c > @@ -4,7 +4,6 @@ > * Author: Mathieu Poirier > */ > > -#include > #include > #include > #include > diff --git a/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c b/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > index e8c7649..5cf6660 100644 > --- a/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > +++ b/drivers/hwtracing/coresight/coresight-etm3x-sysfs.c > @@ -5,7 +5,6 @@ > */ > > #include > -#include > #include > #include "coresight-etm.h" > #include "coresight-priv.h" > diff --git a/drivers/hwtracing/coresight/coresight-etm4x-core.c b/drivers/hwtracing/coresight/coresight-etm4x-core.c > index b20b6ff..0438ef7 100644 > --- a/drivers/hwtracing/coresight/coresight-etm4x-core.c > +++ b/drivers/hwtracing/coresight/coresight-etm4x-core.c > @@ -3,13 +3,11 @@ > * Copyright (c) 2014, The Linux Foundation. All rights reserved. > */ > > -#include > #include > #include > #include > #include > #include > -#include > #include > #include > #include > @@ -20,14 +18,11 @@ > #include > #include > #include > -#include > #include > #include > -#include > #include > #include > #include > -#include > #include > > #include > diff --git a/drivers/hwtracing/coresight/coresight-etm4x-sysfs.c b/drivers/hwtracing/coresight/coresight-etm4x-sysfs.c > index 989ce7b..347ccb7 100644 > --- a/drivers/hwtracing/coresight/coresight-etm4x-sysfs.c > +++ b/drivers/hwtracing/coresight/coresight-etm4x-sysfs.c > @@ -5,7 +5,6 @@ > */ > > #include > -#include > #include > #include "coresight-etm4x.h" > #include "coresight-priv.h" > diff --git a/drivers/hwtracing/coresight/coresight-funnel.c b/drivers/hwtracing/coresight/coresight-funnel.c > index 071c723..c79ded9 100644 > --- a/drivers/hwtracing/coresight/coresight-funnel.c > +++ b/drivers/hwtracing/coresight/coresight-funnel.c > @@ -15,9 +15,6 @@ > #include > #include > #include > -#include > -#include > -#include > #include > > #include "coresight-priv.h" > diff --git a/drivers/hwtracing/coresight/coresight-platform.c b/drivers/hwtracing/coresight/coresight-platform.c > index 3629b78..ca22409 100644 > --- a/drivers/hwtracing/coresight/coresight-platform.c > +++ b/drivers/hwtracing/coresight/coresight-platform.c > @@ -13,8 +13,6 @@ > #include > #include > #include > -#include > -#include > #include > #include > > diff --git a/drivers/hwtracing/coresight/coresight-replicator.c b/drivers/hwtracing/coresight/coresight-replicator.c > index 7e2a2b7..36788de 100644 > --- a/drivers/hwtracing/coresight/coresight-replicator.c > +++ b/drivers/hwtracing/coresight/coresight-replicator.c > @@ -6,14 +6,11 @@ > */ > > #include > -#include > #include > #include > #include > -#include > #include > #include > -#include > #include > #include > #include > diff --git a/drivers/hwtracing/coresight/coresight-stm.c b/drivers/hwtracing/coresight/coresight-stm.c > index 9979177..75f0bd8 100644 > --- a/drivers/hwtracing/coresight/coresight-stm.c > +++ b/drivers/hwtracing/coresight/coresight-stm.c > @@ -17,17 +17,14 @@ > */ > #include > #include > -#include > #include > #include > -#include > #include > #include > #include > #include > #include > #include > -#include > #include > > #include "coresight-priv.h" > diff --git a/drivers/hwtracing/coresight/coresight-tmc-core.c b/drivers/hwtracing/coresight/coresight-tmc-core.c > index 8169dff..4fff281 100644 > --- a/drivers/hwtracing/coresight/coresight-tmc-core.c > +++ b/drivers/hwtracing/coresight/coresight-tmc-core.c > @@ -9,7 +9,6 @@ > #include > #include > #include > -#include > #include > #include > #include > @@ -21,8 +20,6 @@ > #include > #include > #include > -#include > -#include > > #include "coresight-priv.h" > #include "coresight-tmc.h" > diff --git a/drivers/hwtracing/coresight/coresight-tmc-etf.c b/drivers/hwtracing/coresight/coresight-tmc-etf.c > index 989d965..b263f75 100644 > --- a/drivers/hwtracing/coresight/coresight-tmc-etf.c > +++ b/drivers/hwtracing/coresight/coresight-tmc-etf.c > @@ -6,7 +6,6 @@ > > #include > #include > -#include > #include > #include > #include "coresight-priv.h" > diff --git a/drivers/hwtracing/coresight/coresight-tmc-etr.c b/drivers/hwtracing/coresight/coresight-tmc-etr.c > index bf5230e..1ab0b28 100644 > --- a/drivers/hwtracing/coresight/coresight-tmc-etr.c > +++ b/drivers/hwtracing/coresight/coresight-tmc-etr.c > @@ -5,7 +5,6 @@ > */ > > #include > -#include > #include > #include > #include > diff --git a/drivers/hwtracing/coresight/coresight-tpiu.c b/drivers/hwtracing/coresight/coresight-tpiu.c > index d5dfee9..c5675f9 100644 > --- a/drivers/hwtracing/coresight/coresight-tpiu.c > +++ b/drivers/hwtracing/coresight/coresight-tpiu.c > @@ -9,12 +9,8 @@ > #include > #include > #include > -#include > #include > #include > -#include > -#include > -#include > #include > > #include "coresight-priv.h" > -- > 2.8.1 >