From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 14/16] tegra: dts: Add serial port details
Date: Mon, 4 Aug 2014 14:11:48 -0400 [thread overview]
Message-ID: <20140804181148.GT19374@bill-the-cat> (raw)
In-Reply-To: <53DFC74C.2010807@wwwdotorg.org>
On Mon, Aug 04, 2014 at 11:47:56AM -0600, Stephen Warren wrote:
> On 08/04/2014 04:43 AM, Simon Glass wrote:
> >On 1 August 2014 15:50, Stephen Warren <swarren@wwwdotorg.org> wrote:
> ...
> >>DT schemas/bindings MUST be identical between U-Boot, Linux, FreeBSD,
> >>Barebox, ... (all of which use DT). As such, all the DT bindings MUST be
> >>discussed on the devicetree mailing list.
> >>
> >>Since you're the author of the patch, it's your responsibility to have that
> >>discussion.
> >
> >Are you referring to the linux,stdout-path discussion, or something
> >more DT-generic?I suppose we could have a 'u-boot,console' for our
> >part. But in any case you are talking about code and a convention that
> >is already in mainline U-Boot.
>
> I'm saying that any and all additions or changes to DT
> schemas/bindings must be discussed on the devicetree mailing list,
> not made/reviewed in isolation on only the U-Boot mailing list.
>
> > While I accept that we might change to
> >something DT-generic if Linux points the way to something better, I
> >don't want to stop using it just because Linux hasn't decided yet. The
> >early console stuff and early debug UART stuff in Linux is not yet a
> >shining example of perfection.
>
> I strongly believe that if U-Boot continues to use DT, the current
> DT usage in U-Boot needs to be actively moved in line with the
> bindings that the Linux kernel, Barebox, FreeBSD, ... use. I'd
> prefer this to happen even before U-Boot starts making additional
> use of DT, so the conversion doesn't get forgotten. However, I
> suppose it's a bit draconian to prevent further usage until the
> existing usage is cleaned up, except where new usage introduces
> additional dependencies on any current usage that's inconsistent
> with the standard bindings.
I don't disagree (and at ELC I was trying to see who, if anyone, had
been reaching out to the FreeBSD folks since their DT files were very
far from what Linux uses afaict for am33xx), but I'm also not seeing
anything from non-Linux kernel folks on the devicetrees ML, or at least
my quick search failed.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20140804/8a41703d/attachment.pgp>
next prev parent reply other threads:[~2014-08-04 18:11 UTC|newest]
Thread overview: 42+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-30 9:49 [U-Boot] [PATCH v3 0/16] Introduce driver model serial uclass Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 01/16] serial: Set up the 'priv' pointer when creating a serial device Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 02/16] dm: Adjust lists_bind_fdt() to return the bound device Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 03/16] dm: Add a uclass for serial devices Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 04/16] Set up stdio earlier when using driver model Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 05/16] sandbox: Convert serial driver to use " Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 06/16] sandbox: serial: Support a coloured console Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 07/16] sandbox: dts: Add a serial console node Simon Glass
2014-07-31 20:20 ` Stephen Warren
2014-07-31 22:13 ` Simon Glass
2014-07-31 23:09 ` Stephen Warren
2014-08-01 15:46 ` Jon Loeliger
2014-08-01 16:53 ` Tom Rini
2014-08-01 21:37 ` Simon Glass
2014-08-01 21:40 ` Simon Glass
2014-08-01 21:52 ` Stephen Warren
2014-07-30 9:49 ` [U-Boot] [PATCH v3 08/16] dm: exynos: Mark exynos5 console as pre-reloc Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 09/16] dm: exynos: Move serial to driver model Simon Glass
2014-07-30 15:38 ` Tom Rini
2014-07-30 15:43 ` Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 10/16] dm: Make driver model available before board_init() Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 11/16] dm: serial: Move baud rate calculation to ns16550.c Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 12/16] dm: serial: Collect common baud rate code in ns16550 Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 13/16] dm: serial: Add driver model support for ns16550 Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 14/16] tegra: dts: Add serial port details Simon Glass
2014-07-31 20:16 ` Stephen Warren
2014-07-31 22:10 ` Simon Glass
2014-07-31 23:06 ` Stephen Warren
2014-08-01 21:32 ` Simon Glass
2014-08-01 21:50 ` Stephen Warren
2014-08-04 10:43 ` Simon Glass
2014-08-04 17:47 ` Stephen Warren
2014-08-04 18:11 ` Tom Rini [this message]
2014-08-04 18:47 ` Jeroen Hofstee
2014-08-04 20:14 ` Simon Glass
2014-08-05 16:05 ` Stephen Warren
2014-07-30 9:49 ` [U-Boot] [PATCH v3 15/16] RFC: dm: tegra: Enable driver model for serial Simon Glass
2014-07-31 20:18 ` Stephen Warren
2014-07-31 22:11 ` Simon Glass
2014-07-30 9:49 ` [U-Boot] [PATCH v3 16/16] dm: tegra: Use V_NS16550_CLK only in SPL builds Simon Glass
2014-07-31 20:22 ` Stephen Warren
2014-07-31 22:14 ` Simon Glass
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=20140804181148.GT19374@bill-the-cat \
--to=trini@ti.com \
--cc=u-boot@lists.denx.de \
/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