[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20130805103307.GW9858@sirena.org.uk>
Date: Mon, 5 Aug 2013 11:33:07 +0100
From: Mark Brown <broonie@...nel.org>
To: Richard Fitzgerald <rf@...nsource.wolfsonmicro.com>
Cc: lgirdwood@...il.com, tiwai@...e.de, alsa-devel@...a-project.org,
linux-kernel@...r.kernel.org
Subject: Re: [alsa-devel] [PATCH v3] ASoC: core: init delayed_work for
codec-codec links
On Fri, Aug 02, 2013 at 11:51:56AM +0100, Richard Fitzgerald wrote:
> +static void codec2codec_close_delayed_work(struct work_struct *work)
> +{
> +}
> +
Having a commment here was a really good idea since this is the sort of
thing I'd expect someone to come along and try to clean up otherwise.
I'd also expect just passing NULL to INIT_DELAYED_WORK() (with a big
comment attached) to do the right thing, though perhaps it doesn't.
Download attachment "signature.asc" of type "application/pgp-signature" (837 bytes)
Powered by blists - more mailing lists