Re: [PATCH] [media] s5p-jpeg: Only build suspend/resume for PM

From: Geert Uytterhoeven
Date: Mon Oct 13 2014 - 15:53:32 EST


Hi Mauro,

On Mon, Oct 13, 2014 at 6:24 PM, Mauro Carvalho Chehab
<m.chehab@xxxxxxxxxxx> wrote:
> Em Mon, 13 Oct 2014 18:16:10 +0200
> Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> escreveu:
>> On Thu, Oct 2, 2014 at 9:21 AM, Thierry Reding <thierry.reding@xxxxxxxxx> wrote:
>> > From: Thierry Reding <treding@xxxxxxxxxx>
>> >
>> > If power management is disabled these function become unused, so there
>> > is no reason to build them. This fixes a couple of build warnings when
>> > PM(_SLEEP,_RUNTIME) is not enabled.
>>
>> Thanks!
>>
>> Despite the availability of your patch, this build warning has
>> migrated to mainline.
>
> That's because I didn't have any time yet to backport the fixes for
> 3.18 and send those to -next. Also, while warnings are annoying,
> a warning like that is not really an urgent matter, as gcc should
> remove the dead code anyway.

Understood.

> I should be handling fixes next week, after my return from LinuxCon EU,
> gstreamer conf, audio mini-summit and media summit. This will be a too
> busy week.

I hope you'll enjoy the busy conference week. We're all hamsters in the same
threadmill, CU ;-)

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/