From: "Heiko Stübner" <heiko@sntech.de>
To: grant.likely@linaro.org
Cc: robh+dt@kernel.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>
Subject: [PATCH 0/3] of: functions to count number of elements and convert regulators
Date: Wed, 12 Feb 2014 00:59:25 +0100 [thread overview]
Message-ID: <2573561.oCYNnnL0gm@phil> (raw)
In a different thread [0] Mark Rutland suggested that drivers should not
repeatedly open-code the counting of array elements in a property as well
as handling the format and endianes of the DTB, as these should be limited
to the of_ helper functions.
Therefore the first patch introduces a set of helper functions for counting
the number of u8,...,u64 elements in a property.
The second and third patch convert the two regulator drivers that use this
pattern to instead use both of_property_count_u32_elemens as well as
of_property_read_u32_index.
gpio-regulator change tested on a s3c2416-based device, ti-abb-regulator
compile-tested only.
[0] https://lkml.org/lkml/2014/1/16/172
Heiko Stuebner (3):
of: add functions to count number of elements in a property
regulator: gpio-regulator: do not open-code counting and access of dt array elements
regulator: ti-abb-regulator: do not open-code counting and access of dt array elements
drivers/of/base.c | 32 ++++++++++++++
drivers/regulator/gpio-regulator.c | 15 +++----
drivers/regulator/ti-abb-regulator.c | 43 +++++++++----------
include/linux/of.h | 76 ++++++++++++++++++++++++++++++++++
4 files changed, 134 insertions(+), 32 deletions(-)
--
1.7.10.4
next reply other threads:[~2014-02-11 23:59 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-11 23:59 Heiko Stübner [this message]
2014-02-12 0:00 ` [PATCH 1/3] of: add functions to count number of elements in a property Heiko Stübner
2014-02-12 0:00 ` Heiko Stübner
2014-02-12 15:48 ` Mark Brown
2014-02-12 0:01 ` [PATCH 2/3] regulator: gpio-regulator: do not open-code counting and access of dt array elements Heiko Stübner
2014-02-12 0:01 ` Heiko Stübner
2014-02-12 15:51 ` Mark Brown
2014-02-12 0:01 ` [PATCH 3/3] regulator: ti-abb-regulator: " Heiko Stübner
2014-02-12 0:01 ` Heiko Stübner
2014-02-12 15:50 ` Mark Brown
2014-02-12 15:50 ` Mark Brown
2014-02-12 9:25 ` [PATCH 0/3] of: functions to count number of elements and convert regulators Mark Rutland
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=2573561.oCYNnnL0gm@phil \
--to=heiko@sntech.de \
--cc=broonie@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=grant.likely@linaro.org \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=robh+dt@kernel.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.