From: Jonathan Cameron <jic23@kernel.org>
To: Cristina Georgiana Opriceana <cristina.opriceana@gmail.com>
Cc: "outreachy-kernel@googlegroups.com"
<outreachy-kernel@googlegroups.com>,
"linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
Lars-Peter Clausen <lars@metafoo.de>,
Hartmut Knaack <knaack.h@gmx.de>,
Peter Meerwald <pmeerw@pmeerw.net>
Subject: Re: [PATCH v2 1/2] Staging: iio: Place driver in sleep mode on error
Date: Sat, 28 Mar 2015 13:45:21 +0000 [thread overview]
Message-ID: <5516B071.4000301@kernel.org> (raw)
In-Reply-To: <CAFA9rWNH8nwSQv75eQMOdxNGgF6bEGOjDCLw5pvdoALUjkh2xA@mail.gmail.com>
On 28/03/15 13:10, Cristina Georgiana Opriceana wrote:
> On Sat, Mar 28, 2015 at 2:13 PM, Jonathan Cameron <jic23@kernel.org> wrote:
>> On 22/03/15 00:26, Cristina Opriceana wrote:
>>> Put device in sleep mode if an error is encountered after
>>> initialization in order to avoid wasting power.
>> The patch title should definitely include the name of the driver in question!
>> Otherwise you'll not generally get reviews from the relevant people.
>> Staging:iio:hmc5843 Place driver in sleep mode on error during probe.
>>
>> Also try to cc relevant people and the relevant list.
>> See MAINTAINERS and you'll notice we have both a list and a
>> set of designated reviewers who would like to be cc'd on IIO patches.
>
> Okay, thanks for your feedback, I'll keep that in mind for the future work
> because Greg has already merged this on his staging-next branch.
Fair enough!
J
>
> Thanks,
> Cristina
> --
> To unsubscribe from this list: send the line "unsubscribe linux-iio" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
next prev parent reply other threads:[~2015-03-28 13:45 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-22 0:20 [PATCH v2 0/2] Add small improvements to iio/magnetometer hmc5843 driver Cristina Opriceana
2015-03-22 0:26 ` [PATCH v2 1/2] Staging: iio: Place driver in sleep mode on error Cristina Opriceana
[not found] ` <CAEnQRZDkMngLNm61qLdEA8_4reJ+VzQKhxaJxXu4r5drUPaP6w@mail.gmail.com>
2015-03-22 10:16 ` [Outreachy kernel] " Daniel Baluta
2015-03-28 12:13 ` Jonathan Cameron
2015-03-28 13:10 ` Cristina Georgiana Opriceana
2015-03-28 13:45 ` Jonathan Cameron [this message]
2015-03-22 0:28 ` [PATCH v2 2/2] Staging: iio: Fix file header to match standards Cristina Opriceana
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=5516B071.4000301@kernel.org \
--to=jic23@kernel.org \
--cc=cristina.opriceana@gmail.com \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=outreachy-kernel@googlegroups.com \
--cc=pmeerw@pmeerw.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.