All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rob Herring <robh@kernel.org>
To: Andrew Turner <andrew@fubar.geek.nz>
Cc: Tom Rini <trini@konsulko.com>, Kumar Gala <kumar.gala@linaro.org>,
	"ksummit-discuss@lists.linuxfoundation.org"
	<ksummit-discuss@lists.linuxfoundation.org>,
	Rob Herring <rob.herring@linaro.org>,
	"devicetree-spec@vger.kernel.org"
	<devicetree-spec@vger.kernel.org>,
	Pantelis Antoniou <pantelis.antoniou@konsulko.com>,
	Andy Gross <andy.gross@linaro.org>,
	Lucas Stach <l.stach@pengutronix.de>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	David Gibson <david@gibson.dropbear.id.au>
Subject: Re: [Ksummit-discuss] Devicetree Workshop at Kernel Summit Prague (26 Oct 2017)
Date: Wed, 18 Oct 2017 18:53:40 -0500	[thread overview]
Message-ID: <CAL_JsqJTE5p0zjm-66D2143dEx+xQ0uKCV-20kMt-kq=SOa-8Q@mail.gmail.com> (raw)
In-Reply-To: <FD387FCC-3F95-48DF-B412-3384EF96FCFC@fubar.geek.nz>

On Wed, Oct 18, 2017 at 6:28 PM, Andrew Turner <andrew@fubar.geek.nz> wrote:
>
>> On 18 Oct 2017, at 18:59, Tom Rini <trini@konsulko.com> wrote:
>>
>> On Wed, Oct 18, 2017 at 12:09:58PM +0100, Mark Brown wrote:
>>> On Wed, Oct 18, 2017 at 06:35:24PM +0800, Chen-Yu Tsai wrote:
>>>
>>>> I'd like to add something on the topic of non-Linux projects. In this
>>>> case it's diverging DT bindings from U-boot:
>>>
>>>> https://patchwork.ozlabs.org/patch/823158/
>>>
>>>> U-boot already has a set of devicetree binding additions:
>>>> https://github.com/u-boot/u-boot/tree/master/doc/device-tree-bindings
>>>
>>>> The patch in question wants to ab(use) the regulator-name property for
>>>> driver instance binding. In my opinion this is not going to fly, as
>>>> boards are free to define the names. This either sees no use other than
>>>> as a dirty workaround for dts files that aren't following the PMIC
>>>> regulator bindings (regulator node names should follow well defined,
>>>> identifying names), or results in divergence of the DT files.
>>>
>>> One meta issue I'm seeing here is that the u-boot people appear to have
>>> their own divergent copy of some of the binding documents.
>>
>> Putting on my U-Boot hat now, it's mostly unintentional and something in
>> general (yes, the initial topic here is not such an example) we try and
>> avoid, or use u-boot, as the prefix on as it's something that had been
>> previously rejected or deemed inappropriate to be in the upstream
>> version of the binding.
>>
>> But perhaps it's time to try and force the issue again, given what Rob
>> and others have said in other parts of the thread.
>
> From the FreeBSD perspective I’d like it if there was a common repo for all devicetree consumers to share. We are trying to not have FreeBSD specific properties as this has caused issues in the past where we had (and still have) FreeBSD specific dts files. We are trying to remove these as drivers are updated to handle the common bindings.

Are you aware of this repo[1]? I don't have a sense for how widely
used it is. If not, it is intended to provide a common repository of
binding docs and dts files. If so, what are your issues with using it?
It's generated from the kernel tree with git-filter-branch and through
the kernel tree is the only way to add things currently. But there's
no requirement that you add a Linux driver to submit a binding or dts
change. We could consider taking patches against the tree directly,
and the maintainers (me) can fixup the paths and apply to the kernel
tree.

If there's bindings in the kernel tree you think are crap and Linux
specific, I'd like to know that too. We should start flagging those.

> I have also spoken with some NetBSD and OpenBSD developers. They are both using devicetree to handle device enumeration. Having all 5 projects using a common set of dts files and binding would simplify keeping them in sync.

There's more than 5 likely: linux, 3x BSD, u-boot, barebox, zephyr,
ARM trusted firmware?, UEFI?, ?

Rob

[1] https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Andrew Turner <andrew-/GQUgJ3qWXsIvnNNuaVxrw@public.gmane.org>
Cc: Tom Rini <trini-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>,
	Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>,
	Grant Likely
	<grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>,
	Kumar Gala <kumar.gala-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"ksummit-discuss-cunTk1MwBs98uUxBSJOaYoYkZiVZrdSR2LY78lusg7I@public.gmane.org"
	<ksummit-discuss-cunTk1MwBs98uUxBSJOaYoYkZiVZrdSR2LY78lusg7I@public.gmane.org>,
	Rob Herring <rob.herring-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"devicetree-spec-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-spec-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Pantelis Antoniou
	<pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Andy Gross <andy.gross-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	David Gibson
	<david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>,
	Lucas Stach <l.stach-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
Subject: Re: [Ksummit-discuss] Devicetree Workshop at Kernel Summit Prague (26 Oct 2017)
Date: Wed, 18 Oct 2017 18:53:40 -0500	[thread overview]
Message-ID: <CAL_JsqJTE5p0zjm-66D2143dEx+xQ0uKCV-20kMt-kq=SOa-8Q@mail.gmail.com> (raw)
In-Reply-To: <FD387FCC-3F95-48DF-B412-3384EF96FCFC-/GQUgJ3qWXsIvnNNuaVxrw@public.gmane.org>

On Wed, Oct 18, 2017 at 6:28 PM, Andrew Turner <andrew-/GQUgJ3qWXsIvnNNuaVxrw@public.gmane.org> wrote:
>
>> On 18 Oct 2017, at 18:59, Tom Rini <trini-OWPKS81ov/FWk0Htik3J/w@public.gmane.org> wrote:
>>
>> On Wed, Oct 18, 2017 at 12:09:58PM +0100, Mark Brown wrote:
>>> On Wed, Oct 18, 2017 at 06:35:24PM +0800, Chen-Yu Tsai wrote:
>>>
>>>> I'd like to add something on the topic of non-Linux projects. In this
>>>> case it's diverging DT bindings from U-boot:
>>>
>>>> https://patchwork.ozlabs.org/patch/823158/
>>>
>>>> U-boot already has a set of devicetree binding additions:
>>>> https://github.com/u-boot/u-boot/tree/master/doc/device-tree-bindings
>>>
>>>> The patch in question wants to ab(use) the regulator-name property for
>>>> driver instance binding. In my opinion this is not going to fly, as
>>>> boards are free to define the names. This either sees no use other than
>>>> as a dirty workaround for dts files that aren't following the PMIC
>>>> regulator bindings (regulator node names should follow well defined,
>>>> identifying names), or results in divergence of the DT files.
>>>
>>> One meta issue I'm seeing here is that the u-boot people appear to have
>>> their own divergent copy of some of the binding documents.
>>
>> Putting on my U-Boot hat now, it's mostly unintentional and something in
>> general (yes, the initial topic here is not such an example) we try and
>> avoid, or use u-boot, as the prefix on as it's something that had been
>> previously rejected or deemed inappropriate to be in the upstream
>> version of the binding.
>>
>> But perhaps it's time to try and force the issue again, given what Rob
>> and others have said in other parts of the thread.
>
> From the FreeBSD perspective I’d like it if there was a common repo for all devicetree consumers to share. We are trying to not have FreeBSD specific properties as this has caused issues in the past where we had (and still have) FreeBSD specific dts files. We are trying to remove these as drivers are updated to handle the common bindings.

Are you aware of this repo[1]? I don't have a sense for how widely
used it is. If not, it is intended to provide a common repository of
binding docs and dts files. If so, what are your issues with using it?
It's generated from the kernel tree with git-filter-branch and through
the kernel tree is the only way to add things currently. But there's
no requirement that you add a Linux driver to submit a binding or dts
change. We could consider taking patches against the tree directly,
and the maintainers (me) can fixup the paths and apply to the kernel
tree.

If there's bindings in the kernel tree you think are crap and Linux
specific, I'd like to know that too. We should start flagging those.

> I have also spoken with some NetBSD and OpenBSD developers. They are both using devicetree to handle device enumeration. Having all 5 projects using a common set of dts files and binding would simplify keeping them in sync.

There's more than 5 likely: linux, 3x BSD, u-boot, barebox, zephyr,
ARM trusted firmware?, UEFI?, ?

Rob

[1] https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/

WARNING: multiple messages have this Message-ID (diff)
From: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Andrew Turner <andrew-/GQUgJ3qWXsIvnNNuaVxrw@public.gmane.org>
Cc: Tom Rini <trini-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>,
	Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
	Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>,
	Grant Likely
	<grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org>,
	Kumar Gala <kumar.gala-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"ksummit-discuss-cunTk1MwBs98uUxBSJOaYoYkZiVZrdSR2LY78lusg7I@public.gmane.org"
	<ksummit-discuss-cunTk1MwBs98uUxBSJOaYoYkZiVZrdSR2LY78lusg7I@public.gmane.org>,
	Rob Herring <rob.herring-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	"devicetree-spec-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-spec-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Pantelis Antoniou
	<pantelis.antoniou-OWPKS81ov/FWk0Htik3J/w@public.gmane.org>,
	"devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Andy Gross <andy.gross-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	David Gibson
	<david-xT8FGy+AXnRB3Ne2BGzF6laj5H9X9Tb+@public.gmane.org>,
	Lucas Stach <l.stach-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>
Subject: Re: [Ksummit-discuss] Devicetree Workshop at Kernel Summit Prague (26 Oct 2017)
Date: Wed, 18 Oct 2017 18:53:40 -0500	[thread overview]
Message-ID: <CAL_JsqJTE5p0zjm-66D2143dEx+xQ0uKCV-20kMt-kq=SOa-8Q@mail.gmail.com> (raw)
In-Reply-To: <FD387FCC-3F95-48DF-B412-3384EF96FCFC-/GQUgJ3qWXsIvnNNuaVxrw@public.gmane.org>

On Wed, Oct 18, 2017 at 6:28 PM, Andrew Turner <andrew-/GQUgJ3qWXsIvnNNuaVxrw@public.gmane.org> wrote:
>
>> On 18 Oct 2017, at 18:59, Tom Rini <trini-OWPKS81ov/FWk0Htik3J/w@public.gmane.org> wrote:
>>
>> On Wed, Oct 18, 2017 at 12:09:58PM +0100, Mark Brown wrote:
>>> On Wed, Oct 18, 2017 at 06:35:24PM +0800, Chen-Yu Tsai wrote:
>>>
>>>> I'd like to add something on the topic of non-Linux projects. In this
>>>> case it's diverging DT bindings from U-boot:
>>>
>>>> https://patchwork.ozlabs.org/patch/823158/
>>>
>>>> U-boot already has a set of devicetree binding additions:
>>>> https://github.com/u-boot/u-boot/tree/master/doc/device-tree-bindings
>>>
>>>> The patch in question wants to ab(use) the regulator-name property for
>>>> driver instance binding. In my opinion this is not going to fly, as
>>>> boards are free to define the names. This either sees no use other than
>>>> as a dirty workaround for dts files that aren't following the PMIC
>>>> regulator bindings (regulator node names should follow well defined,
>>>> identifying names), or results in divergence of the DT files.
>>>
>>> One meta issue I'm seeing here is that the u-boot people appear to have
>>> their own divergent copy of some of the binding documents.
>>
>> Putting on my U-Boot hat now, it's mostly unintentional and something in
>> general (yes, the initial topic here is not such an example) we try and
>> avoid, or use u-boot, as the prefix on as it's something that had been
>> previously rejected or deemed inappropriate to be in the upstream
>> version of the binding.
>>
>> But perhaps it's time to try and force the issue again, given what Rob
>> and others have said in other parts of the thread.
>
> From the FreeBSD perspective I’d like it if there was a common repo for all devicetree consumers to share. We are trying to not have FreeBSD specific properties as this has caused issues in the past where we had (and still have) FreeBSD specific dts files. We are trying to remove these as drivers are updated to handle the common bindings.

Are you aware of this repo[1]? I don't have a sense for how widely
used it is. If not, it is intended to provide a common repository of
binding docs and dts files. If so, what are your issues with using it?
It's generated from the kernel tree with git-filter-branch and through
the kernel tree is the only way to add things currently. But there's
no requirement that you add a Linux driver to submit a binding or dts
change. We could consider taking patches against the tree directly,
and the maintainers (me) can fixup the paths and apply to the kernel
tree.

If there's bindings in the kernel tree you think are crap and Linux
specific, I'd like to know that too. We should start flagging those.

> I have also spoken with some NetBSD and OpenBSD developers. They are both using devicetree to handle device enumeration. Having all 5 projects using a common set of dts files and binding would simplify keeping them in sync.

There's more than 5 likely: linux, 3x BSD, u-boot, barebox, zephyr,
ARM trusted firmware?, UEFI?, ?

Rob

[1] https://git.kernel.org/pub/scm/linux/kernel/git/devicetree/devicetree-rebasing.git/

  reply	other threads:[~2017-10-18 23:54 UTC|newest]

Thread overview: 126+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-10-09 20:39 [Ksummit-discuss] Devicetree Workshop at Kernel Summit Prague (26 Oct 2017) Grant Likely
2017-10-09 20:39 ` Grant Likely
2017-10-14 12:34 ` [Ksummit-discuss] " Thomas Petazzoni
2017-10-14 12:34   ` Thomas Petazzoni
2017-10-17 13:30   ` Grant Likely
2017-10-17 13:30     ` Grant Likely
2017-10-16  5:36 ` Michal Simek
2017-10-16  5:36   ` Michal Simek
2017-10-16 14:11   ` Rob Herring
2017-10-16 14:11     ` Rob Herring
2017-10-18 14:04     ` Michal Simek
2017-10-18 14:04       ` Michal Simek
2017-10-18 14:28       ` Andre Przywara
2017-10-18 14:28         ` Andre Przywara
2017-10-18 15:32         ` Rob Herring
2017-10-18 15:32           ` Rob Herring
2017-10-18 16:05           ` Andre Przywara
2017-10-18 16:05             ` Andre Przywara
2017-10-18 16:20             ` Pantelis Antoniou
2017-10-18 16:20               ` Pantelis Antoniou
2017-10-16 16:40   ` Ben Dooks
2017-10-16 16:40     ` Ben Dooks
2017-10-16 18:44     ` Heiko Stübner
2017-10-16 18:44       ` Heiko Stübner
2017-10-16 19:45     ` Rob Herring
2017-10-16 19:45       ` Rob Herring
2017-10-17 13:38       ` Grant Likely
2017-10-17 13:38         ` Grant Likely
2017-10-17 23:45         ` Frank Rowand
2017-10-17 23:45           ` Frank Rowand
2017-10-17 13:32   ` Grant Likely
2017-10-17 13:32     ` Grant Likely
2017-10-18 10:08     ` Thomas Petazzoni
2017-10-18 10:08       ` Thomas Petazzoni
2017-10-16 16:42 ` Ben Dooks
2017-10-16 16:42   ` Ben Dooks
2017-10-17 13:34   ` Grant Likely
2017-10-17 13:34     ` Grant Likely
2017-10-17  9:48 ` Boris Brezillon
2017-10-17  9:48   ` Boris Brezillon
2017-10-17 13:21   ` Tom Rini
2017-10-17 13:21     ` Tom Rini
2017-10-17 13:48   ` Grant Likely
2017-10-17 13:48     ` Grant Likely
2017-10-17 16:21     ` Ian Lepore
2017-10-17 16:21       ` Ian Lepore
2017-10-17 17:02       ` Kumar Gala
2017-10-17 17:02         ` Kumar Gala
2017-10-17 17:24         ` Geert Uytterhoeven
2017-10-17 17:24           ` Geert Uytterhoeven
2017-10-17 17:24           ` Geert Uytterhoeven
2017-10-17 19:03           ` Bird, Timothy
2017-10-17 19:03             ` Bird, Timothy
2017-10-18 12:14             ` Grant Likely
2017-10-18 12:14               ` Grant Likely
2017-10-18 12:14               ` Grant Likely
2017-10-18 12:59               ` Pantelis Antoniou
2017-10-18 12:59                 ` Pantelis Antoniou
2017-10-18 13:18                 ` Alexandre Belloni
2017-10-18 13:18                   ` Alexandre Belloni
2017-10-18 13:21                   ` Geert Uytterhoeven
2017-10-18 13:21                     ` Geert Uytterhoeven
2017-10-18 17:41                     ` Bird, Timothy
2017-10-18 17:41                       ` Bird, Timothy
2017-10-18 18:00                       ` Rob Herring
2017-10-18 18:00                         ` Rob Herring
2017-10-18 21:10                       ` Alexandre Belloni
2017-10-18 21:10                         ` Alexandre Belloni
2017-10-18 16:18                   ` David Woodhouse
2017-10-18 16:18                     ` David Woodhouse
2017-10-18 14:13                 ` Rob Herring
2017-10-18 14:13                   ` Rob Herring
2017-10-18 17:45                   ` Bird, Timothy
2017-10-18 17:45                     ` Bird, Timothy
2017-10-18 14:07           ` Kumar Gala
2017-10-18 14:07             ` Kumar Gala
2017-10-18 14:07             ` Kumar Gala
2017-10-17 17:25       ` Rob Herring
2017-10-17 17:25         ` Rob Herring
2017-10-18 10:11       ` Thomas Petazzoni
2017-10-18 10:11         ` Thomas Petazzoni
2017-10-18 10:35   ` Chen-Yu Tsai
2017-10-18 10:35     ` Chen-Yu Tsai
2017-10-18 11:09     ` Mark Brown
2017-10-18 11:09       ` Mark Brown
2017-10-18 17:59       ` Tom Rini
2017-10-18 17:59         ` Tom Rini
2017-10-18 23:28         ` Andrew Turner
2017-10-18 23:28           ` Andrew Turner
2017-10-18 23:28           ` Andrew Turner
2017-10-18 23:53           ` Rob Herring [this message]
2017-10-18 23:53             ` Rob Herring
2017-10-18 23:53             ` Rob Herring
2017-10-19 14:00             ` Alexandre Torgue
2017-10-19 14:00               ` Alexandre Torgue
2017-10-19 14:00               ` Alexandre Torgue
2017-10-19 14:59               ` Rob Herring
2017-10-19 14:59                 ` Rob Herring
2017-10-19 14:59                 ` Rob Herring
2017-10-19 18:46                 ` Frank Rowand
2017-10-19 18:46                   ` Frank Rowand
2017-10-19 18:46                   ` Frank Rowand
2017-10-20  9:55                   ` Alexandre Torgue
2017-10-20  9:55                     ` Alexandre Torgue
2017-10-20  9:55                     ` Alexandre Torgue
2017-10-20 10:01                     ` David Gibson
2017-10-20 10:01                       ` David Gibson
2017-10-20 13:37                     ` Rob Herring
2017-10-20 13:37                       ` Rob Herring
2017-10-22  8:25                       ` David Gibson
2017-10-22  8:25                         ` David Gibson
2017-10-20 13:47                 ` Alexandre Torgue
2017-10-20 13:47                   ` Alexandre Torgue
2017-10-20 13:47                   ` Alexandre Torgue
2017-10-19  0:04         ` Mark Brown
2017-10-19  0:04           ` Mark Brown
2017-10-19 11:10 ` Grant Likely
2017-10-19 11:10   ` Grant Likely
2017-10-24  7:37   ` [Ksummit-discuss] " Boris Brezillon
2017-10-24  7:37     ` Boris Brezillon
2017-10-25 14:40     ` Maxime Ripard
2017-10-25 14:40       ` Maxime Ripard
2017-10-26  5:47   ` Frank Rowand
2017-10-26  5:47     ` Frank Rowand
2017-10-26  7:17   ` [Ksummit-discuss] " Grant Likely
2017-10-26  7:17     ` Grant Likely

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='CAL_JsqJTE5p0zjm-66D2143dEx+xQ0uKCV-20kMt-kq=SOa-8Q@mail.gmail.com' \
    --to=robh@kernel.org \
    --cc=andrew@fubar.geek.nz \
    --cc=andy.gross@linaro.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=devicetree-spec@vger.kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=ksummit-discuss@lists.linuxfoundation.org \
    --cc=kumar.gala@linaro.org \
    --cc=l.stach@pengutronix.de \
    --cc=pantelis.antoniou@konsulko.com \
    --cc=rob.herring@linaro.org \
    --cc=trini@konsulko.com \
    /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.