Re: [PATCH v2] rfkill: Regulator consumer driver for rfkill

[Antonio Ospite]

On Tue, 12 Apr 2011 13:41:28 +0200
Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote:

> > +	dev_info(&pdev->dev, "initialized\n");
> 
> Is that message really useful?
>

I can print the pdata->name here as well as an excuse to keep it,
reality is that I just like visual feedback in dmesg when a driver has
been loaded.

> Other than that, looks good to me.
> 
> johannes
>

In another message you said that the comment in rfkill-regulator.h
still seemed a little wrong to you, would you elaborate more about what
you were referring to?

Thanks,
   Antonio

-- 
Antonio Ospite
http://ao2.it

PGP public key ID: 0x4553B001

A: Because it messes up the order in which people normally read text.
   See http://en.wikipedia.org/wiki/Posting_style
Q: Why is top-posting such a bad thing?

Attachment: pgp00000.pgp
Description: PGP signature