All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Éric Piel" <eric.piel-VkQ1JFuSMpfAbQlEx87xDw@public.gmane.org>
To: Sebastian Reichel <sre-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>,
	Sebastian Reichel <sre-GFxCN5SEZAc@public.gmane.org>,
	Daniel Mack <zonque-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
Cc: Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
	Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	Ian Campbell
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	arnd-r2nGTMty4D4@public.gmane.org,
	Greg Kroah-Hartman
	<gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>
Subject: Re: [PATCH 0/4] lis3lv02d: update DT binding for use with Nokia N900
Date: Fri, 13 Dec 2013 23:31:39 +0100	[thread overview]
Message-ID: <52AB8ACB.6000101@tremplin-utc.net> (raw)
In-Reply-To: <1386968094-24554-1-git-send-email-sre-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>

On 13-12-13 21:54, Sebastian Reichel wrote:
> Hi,
>
> The lis302 has already a DT binding described in [0],
> which descibes misc. hardware properties. The current
> binding does not support all values needed to convert
> the Nokia N900's platform data to DT.
>
> This patchset introduces support for describing inverted
> axis, configuration of second wakeup unit and wakeup
> threshold support.
Hi,

All the three patches related to the lis3lv02d driver look fine to me.
Here is my signed-off for patches 2, 3, and 4:
Signed-off-by: Éric Piel <eric.piel-VkQ1JFuSMpfAbQlEx87xDw@public.gmane.org>

I've got 2 side questions:
  * How do you write a negative value in a DTS file? Is this just the 
very standard way (i.e., -), or do you need to use a specific technique 
(such as 2-completemnt on 32 bits)? If it's not the standard way, it 
should probably be documented somewhere.
  * Who wants to carry these patches? Can they all go via the devicetree 
branch? Or maybe Arnd or Greg would prefer to pick them via the misc 
driver branch? Both ways are fine for me...

Cheers,
Éric
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2013-12-13 22:31 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-13 20:54 [PATCH 0/4] lis3lv02d: update DT binding for use with Nokia N900 Sebastian Reichel
     [not found] ` <1386968094-24554-1-git-send-email-sre-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org>
2013-12-13 20:54   ` [PATCH 1/4] of: introduce of_property_read_s32 Sebastian Reichel
2013-12-13 20:54   ` [PATCH 2/4] lis3lv02d: DT: use s32 to support negative values Sebastian Reichel
2013-12-13 20:54   ` [PATCH 3/4] lis3lv02d: DT: add wakeup unit 2 and wakeup threshold Sebastian Reichel
2013-12-13 20:54   ` [PATCH 4/4] Documentation: DT: lis302: update wakeup binding Sebastian Reichel
2013-12-13 22:31   ` Éric Piel [this message]
     [not found]     ` <52AB8ACB.6000101-VkQ1JFuSMpfAbQlEx87xDw@public.gmane.org>
2013-12-13 22:59       ` [PATCH 0/4] lis3lv02d: update DT binding for use with Nokia N900 Sebastian Reichel
     [not found]         ` <20131213225920.GA20534-SfvFxonMDyemK9LvCR3Hrw@public.gmane.org>
2014-01-08 11:17           ` Sebastian Reichel

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=52AB8ACB.6000101@tremplin-utc.net \
    --to=eric.piel-vkq1jfusmpfabqlex87xdw@public.gmane.org \
    --cc=arnd-r2nGTMty4D4@public.gmane.org \
    --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org \
    --cc=ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org \
    --cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
    --cc=pawel.moll-5wv7dgnIgG8@public.gmane.org \
    --cc=rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org \
    --cc=sre-8fiUuRrzOP0dnm+yROfE0A@public.gmane.org \
    --cc=sre-GFxCN5SEZAc@public.gmane.org \
    --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
    --cc=zonque-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    /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.