devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).