[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20101129220046.GB8335@opensource.wolfsonmicro.com>
Date: Mon, 29 Nov 2010 22:00:46 +0000
From: Mark Brown <broonie@...nsource.wolfsonmicro.com>
To: Joe Perches <joe@...ches.com>
Cc: Liam Girdwood <lrg@...mlogic.co.uk>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] regulator: Add and use rdev_<level> macros
On Mon, Nov 29, 2010 at 11:54:50AM -0800, Joe Perches wrote:
> +#define rdev_debug(rdev, fmt, ...) \
> + pr_debug("%s: " fmt, rdev_get_name(rdev), ##__VA_ARGS__)
> +
This looks good but I'd prefer if this were rdev_dbg() rather than
rdev_debug() to match dev_dbg() - it feels like that would be a better
match given that we're supplying an rdev and it's less typing to boot.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists