From: Timur Tabi <timur-N01EOCouUvQ@public.gmane.org>
To: Sebastian Frias <sf84-QFKgK+z4sOrR7s880joybQ@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>
Cc: devicetree <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Mason <slash.tmp-GANU6spQydw@public.gmane.org>,
Linux ARM
<linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org>,
LKML <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: ARM, SoC: About the use DT-defined properties by 3rd-party drivers
Date: Mon, 12 Sep 2016 11:21:29 -0500 [thread overview]
Message-ID: <57D6D609.1090702@tabi.org> (raw)
In-Reply-To: <57D6D2A9.3010006-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
Sebastian Frias wrote:
>> You're not changing the code, but you are creating a binding.
>> Bindings
>>> are intended to be stable (i.e. a working DTB from today should
>>> continue to work in future), and thus there are ramifications.
> Ok, but who is responsible for such guarantee?
The developers who create the driver and modify it. Only upstream
contributions count.
> How is it enforced and verified?
The upstream reviewers verify it, and if they see a problem, they reject
the patch.
> Exactly, that's why to I'm having trouble to understand why there is
> so much insistence on "getting the DT 100% right", since a HW change
> could imply that what made 100% sense yesterday, does not today.
I'm not sure what you're getting at. Everything is best-effort. The
binding for a given device is supposed to accurately, but minimally,
describe the hardware so that the driver can program the device
properly. Specific properties are added to the binding to handle
specific cases. If a binding includes a description of properties that
are not used by the driver, the developer is typically asked to remove
those.
So I don't understand the "100% right" comment.
> Could you be more precise on those two issues? Namely: "the effort"
> and the "lack of benefit for the community"?
1) The effort = the effort by upstream developers to review the binding.
2) lack of benefit = proprietary and out-of-tree drivers do not benefit
the community.
> I can understand the effort it takes to review a binding and some
> driver, but if there's no driver, why would it matter if the DT
> binding is 100% right?
Why bother creating a binding if there's no driver to use it?
> Hence, why would it take more effort?
> Furthermore, if there's no driver, there's no backward compatibility
> to guarantee. Shouldn't it require less effort?
If a developer creates a binding without a driver, then it's very hard
to know if that binding is correct. That's like creating a recipe
without attempting to make the food. How do you know it actually tastes
good? No one would ever ask a chef to create a "technically correct"
recipe without cooking it first and tasting it.
> Also, what sort of "benefits" does the community expects or
> requires? Because the idea behind the proposal is to put the HW
> description in DT, so basically the HW would be documented. Maybe
> there wouldn't be a driver right away, but the HW description would
> allow for drivers to exist.
I'm not sure how many times we need to repeat this, but the idea that
the DT binding documentation can be used as official documentation for
the hardware is absurd. What's wrong with the ACTUAL hardware
documentation provided by the manufacturer in PDF format? The DT
binding has no value without an actual driver.
--
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
next prev parent reply other threads:[~2016-09-12 16:21 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-08-24 14:29 ARM,SoC: About the use DT-defined properties by 3rd-party drivers Sebastian Frias
2016-08-28 20:36 ` ARM, SoC: " Timur Tabi
2016-09-12 12:29 ` ARM,SoC: " Sebastian Frias
[not found] ` <57D69FB1.2020801-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
2016-09-12 12:38 ` Mark Rutland
2016-09-12 13:15 ` Sebastian Frias
2016-09-12 13:23 ` Timur Tabi
2016-09-12 14:01 ` ARM, SoC: " Mark Rutland
2016-09-12 14:26 ` Warner Losh
2016-09-12 16:29 ` Sebastian Frias
2016-09-12 16:45 ` Warner Losh
2016-09-12 16:49 ` Timur Tabi
2016-09-12 17:07 ` Mark Rutland
2016-09-12 17:06 ` Mark Rutland
2016-09-12 16:07 ` Sebastian Frias
[not found] ` <57D6D2A9.3010006-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
2016-09-12 16:21 ` Timur Tabi [this message]
2016-09-12 16:26 ` Sebastian Frias
2016-09-12 16:56 ` Mark Rutland
2016-09-13 10:04 ` Sebastian Frias
[not found] ` <57D7CF17.2050905-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
2016-09-13 11:37 ` Timur Tabi
[not found] ` <57D7E4EC.6050509-N01EOCouUvQ@public.gmane.org>
2016-09-13 13:23 ` Mark Rutland
2016-09-13 13:12 ` Mark Rutland
2016-09-13 14:22 ` Sebastian Frias
[not found] ` <57D80B91.4020306-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
2016-09-13 14:51 ` Mark Rutland
2016-09-14 8:32 ` Sebastian Frias
2016-09-13 14:55 ` Sebastian Frias
[not found] ` <57D8137F.8040508-QFKgK+z4sOrR7s880joybQ@public.gmane.org>
2016-09-13 15:47 ` Mark Rutland
2016-09-14 8:24 ` Sebastian Frias
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=57D6D609.1090702@tabi.org \
--to=timur-n01eocouuvq@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=mark.rutland-5wv7dgnIgG8@public.gmane.org \
--cc=sf84-QFKgK+z4sOrR7s880joybQ@public.gmane.org \
--cc=slash.tmp-GANU6spQydw@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).