Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756126AbcKKLT2 (ORCPT ); Fri, 11 Nov 2016 06:19:28 -0500 Received: from foss.arm.com ([217.140.101.70]:40630 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755487AbcKKLT0 (ORCPT ); Fri, 11 Nov 2016 06:19:26 -0500 Date: Fri, 11 Nov 2016 11:18:31 +0000 From: Mark Rutland To: Jan Glauber Cc: Will Deacon , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v4 1/5] arm64: perf: Basic uncore counter support for Cavium ThunderX SOC Message-ID: <20161111111830.GA11945@leverpostej> References: <73173d6ad2430eead5e9da40564a90a60961b6d9.1477741719.git.jglauber@cavium.com> <20161110165405.GH4418@leverpostej> <20161111103921.GE16907@hardcore> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161111103921.GE16907@hardcore> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1119 Lines: 37 On Fri, Nov 11, 2016 at 11:39:21AM +0100, Jan Glauber wrote: > Hi Mark, > > thanks for reviewing. One question below, > On Thu, Nov 10, 2016 at 04:54:06PM +0000, Mark Rutland wrote: > > On Sat, Oct 29, 2016 at 01:55:29PM +0200, Jan Glauber wrote: > > > +#include > > > +#include > > > +#include > > > > I believe the following includes are necessary for APIs and/or data > > explicitly referenced by the driver code: [...] > Should I also add includes that are already in the included by uncore_cavium.h? Please do. > I usually avoid includes that come through the "local" header file. Generally, when you explcitly use some macro/function/data in a file, that file should have the relevant include. If something's only used in the header (e.g. hidden in a macro or inline function), then we only need that include in the header. For example: uncore_cavium.h uses container_of(), and should include . Also, uncore_cavium.c also uses container_of() directly for something unrelated, and should also include . Thanks, Mark.