From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
To: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: "Rafael J. Wysocki" <rafael@kernel.org>,
Heikki Krogerus <heikki.krogerus@linux.intel.com>,
Mika Westerberg <mika.westerberg@linux.intel.com>,
Linus Walleij <linus.walleij@linaro.org>,
Ard Biesheuvel <ard.biesheuvel@linaro.org>,
linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
platform-driver-x86@vger.kernel.org
Subject: Re: [PATCH v5 10/14] software node: rename is_array to is_inline
Date: Mon, 14 Oct 2019 10:37:20 +0300 [thread overview]
Message-ID: <20191014073720.GH32742@smile.fi.intel.com> (raw)
In-Reply-To: <20191011230721.206646-11-dmitry.torokhov@gmail.com>
On Fri, Oct 11, 2019 at 04:07:17PM -0700, Dmitry Torokhov wrote:
> We do not need a special flag to know if we are dealing with an array,
> as we can get that data from ratio between element length and the data
> size, however we do need a flag to know whether the data is stored
> directly inside property_entry or separately.
> - if (prop->is_array)
> + if (!prop->is_inline)
> - if (p->is_array) {
> + if (!p->is_inline) {
> - if (src->is_array) {
> + if (!src->is_inline) {
May we have positive conditionals instead?
> + * @is_inline: True when the property value is stored directly in
I think word 'directly' is superfluous here.
Or, perhaps, 'stored directly' -> 'embedded'
> + * &struct property_entry instance.
> + * @pointer: Pointer to the property when it is stored separately from
> + * the &struct property_entry instance.
'separately from' -> 'outside' ?
> + * @value: Value of the property when it is stored inline.
'stored inline' -> 'embedded in the &struct...' ?
--
With Best Regards,
Andy Shevchenko
next prev parent reply other threads:[~2019-10-14 7:37 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-10-11 23:07 [PATCH v5 00/14] software node: add support for reference properties Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 01/14] software node: remove DEV_PROP_MAX Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 02/14] software node: introduce PROPERTY_ENTRY_ARRAY_XXX_LEN() Dmitry Torokhov
2019-10-15 12:02 ` Andy Shevchenko
2019-10-11 23:07 ` [PATCH v5 03/14] efi/apple-properties: use PROPERTY_ENTRY_U8_ARRAY_LEN Dmitry Torokhov
2019-10-15 12:01 ` Andy Shevchenko
2019-10-15 18:02 ` Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 04/14] software node: mark internal macros with double underscores Dmitry Torokhov
2019-10-15 12:03 ` Andy Shevchenko
2019-10-15 18:09 ` Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 05/14] software node: clean up property_copy_string_array() Dmitry Torokhov
2019-10-15 12:07 ` Andy Shevchenko
2019-10-15 18:12 ` Dmitry Torokhov
2019-10-16 7:53 ` Andy Shevchenko
2019-10-16 17:00 ` Dmitry Torokhov
2019-10-17 7:02 ` Andy Shevchenko
2019-10-11 23:07 ` [PATCH v5 06/14] software node: get rid of property_set_pointer() Dmitry Torokhov
2019-10-15 12:09 ` Andy Shevchenko
2019-10-15 18:14 ` Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 07/14] software node: remove property_entry_read_uNN_array functions Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 08/14] software node: unify PROPERTY_ENTRY_XXX macros Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 09/14] software node: simplify property_entry_read_string_array() Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 10/14] software node: rename is_array to is_inline Dmitry Torokhov
2019-10-14 7:37 ` Andy Shevchenko [this message]
2019-10-15 18:22 ` Dmitry Torokhov
2019-10-16 7:59 ` Andy Shevchenko
2019-10-16 16:54 ` Dmitry Torokhov
2019-10-17 7:16 ` Andy Shevchenko
2019-10-17 16:00 ` Dmitry Torokhov
2019-10-17 16:02 ` Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 11/14] software node: move small properties inline when copying Dmitry Torokhov
2019-10-15 12:20 ` Andy Shevchenko
2019-10-15 18:25 ` Dmitry Torokhov
2019-10-16 7:48 ` Andy Shevchenko
2019-10-16 16:01 ` Dmitry Torokhov
2019-10-16 16:18 ` Andy Shevchenko
2019-10-16 16:23 ` Andy Shevchenko
2019-10-16 16:44 ` Dmitry Torokhov
2019-10-17 16:04 ` Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 12/14] software node: implement reference properties Dmitry Torokhov
2019-10-11 23:07 ` [PATCH v5 13/14] platform/x86: intel_cht_int33fe: use inline " Dmitry Torokhov
2019-10-16 8:01 ` Andy Shevchenko
2019-10-11 23:07 ` [PATCH v5 14/14] software node: remove separate handling of references Dmitry Torokhov
2019-10-14 7:38 ` [PATCH v5 00/14] software node: add support for reference properties Andy Shevchenko
2019-10-14 23:57 ` Dmitry Torokhov
2019-10-16 8:02 ` Andy Shevchenko
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=20191014073720.GH32742@smile.fi.intel.com \
--to=andriy.shevchenko@linux.intel.com \
--cc=ard.biesheuvel@linaro.org \
--cc=dmitry.torokhov@gmail.com \
--cc=heikki.krogerus@linux.intel.com \
--cc=linus.walleij@linaro.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mika.westerberg@linux.intel.com \
--cc=platform-driver-x86@vger.kernel.org \
--cc=rafael@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.