[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <57896546.3010202@gmail.com>
Date: Fri, 15 Jul 2016 15:35:50 -0700
From: Steve Longerbeam <slongerbeam@...il.com>
To: Paul Gortmaker <paul.gortmaker@...driver.com>
Cc: p.zabel@...gutronix.de, dri-devel@...ts.freedesktop.org,
LKML <linux-kernel@...r.kernel.org>,
Steve Longerbeam <steve_longerbeam@...tor.com>
Subject: Re: [PATCH 01/16] gpu: ipu-v3: Add Video Deinterlacer unit
On 07/10/2016 05:02 PM, Paul Gortmaker wrote:
>
> +#include <linux/export.h>
> +#include <linux/module.h>
> You have a u32 field in a struct called "modules" but aside from that, I do not
> see anything in this code requiring module.h -- did I miss something?
>
> You might want export.h for EXPORT_SYMBOL though.
>
>
Hi Paul, yes module.h wasn't needed. Fixed in next version.
Steve
Powered by blists - more mailing lists