Re: [PATCH v2 02/10] media: adv748x: include everything adv748x.h needs into the file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

To: Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>
Subject: Re: [PATCH v2 02/10] media: adv748x: include everything adv748x.h needs into the file
From: Alex Riesen <alexander.riesen@xxxxxxxxxxx>
Date: Fri, 20 Mar 2020 09:23:31 +0100
Cc: Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>, "Hans Verkuil" <hverkuil-cisco@xxxxxxxxx>, Rob Herring <robh+dt@xxxxxxxxxx>, "Mark Rutland" <mark.rutland@xxxxxxx>, Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>, <devel@xxxxxxxxxxxxxxxxxxxx>, <linux-media@xxxxxxxxxxxxxxx>, <linux-kernel@xxxxxxxxxxxxxxx>, <devicetree@xxxxxxxxxxxxxxx>, <linux-renesas-soc@xxxxxxxxxxxxxxx>
In-reply-to: <20200319174802.GH14585@pendragon.ideasonboard.com>
Mail-followup-to: Alex Riesen <alexander.riesen@xxxxxxxxxxx>, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>, Kieran Bingham <kieran.bingham@xxxxxxxxxxxxxxxx>, Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>, Hans Verkuil <hverkuil-cisco@xxxxxxxxx>, Rob Herring <robh+dt@xxxxxxxxxx>, Mark Rutland <mark.rutland@xxxxxxx>, Kuninori Morimoto <kuninori.morimoto.gx@xxxxxxxxxxx>, devel@xxxxxxxxxxxxxxxxxxxx, linux-media@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, devicetree@xxxxxxxxxxxxxxx, linux-renesas-soc@xxxxxxxxxxxxxxx
References: <cover.1584639664.git.alexander.riesen@cetitec.com> <fe109d58eaa34d68cad0f34bb048f827b336e024.1584639664.git.alexander.riesen@cetitec.com> <20200319174802.GH14585@pendragon.ideasonboard.com>

Hi Laurent,

Laurent Pinchart, Thu, Mar 19, 2020 18:48:02 +0100:
> On Thu, Mar 19, 2020 at 06:41:48PM +0100, Alex Riesen wrote:
> > To follow the established practice of not depending on others to
> > pull everything in.
> 
> Good idea. While at it, could you include "adv748x.h" as the very first
> header in at least one of the C files ? That will help ensuring the
> header stays self-contained in the future.

Done.

References:
- [PATCH v2 00/10] media: adv748x: add support for HDMI audio
  - From: Alex Riesen
- [PATCH v2 02/10] media: adv748x: include everything adv748x.h needs into the file
  - From: Alex Riesen
- Re: [PATCH v2 02/10] media: adv748x: include everything adv748x.h needs into the file
  - From: Laurent Pinchart

Prev by Date: [PATCH v3] media: rcar-csi2: Correct the selection of hsfreqrange
Next by Date: Re: [PATCH v2 05/10] media: adv748x: add support for HDMI audio
Previous by thread: Re: [PATCH v2 02/10] media: adv748x: include everything adv748x.h needs into the file
Next by thread: [PATCH v2 03/10] media: adv748x: reduce amount of code for bitwise modifications of device registers
Index(es):
- Date
- Thread

[Index of Archives] [Linux Samsung SOC] [Linux Wireless] [Linux Kernel] [ATH6KL] [Linux Bluetooth] [Linux Netdev] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Samba] [Device Mapper]