public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Pavel Herrmann <morpheus.ibis@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v4 15/15] dm: Expand and improve the device lifecycle docs
Date: Fri, 06 Jun 2014 21:52:53 +0200	[thread overview]
Message-ID: <1552477.9gIaszZOQa@bloomfield> (raw)
In-Reply-To: <1402082012-4152-16-git-send-email-sjg@chromium.org>

Hi,

I found some typos in this

On Friday 06 of June 2014 13:13:32 Simon Glass wrote:
> The lifecycle of a device is an important part of driver model. Add to the
> existing documentation and clarify it.

...snip...

> +This means that instead of having lots of U_BOOT_DEVICE() declarations in
> +the board file, we put these in the device tree. This approach allows a lot
> +more generality, since the same board file can support many types of
> boards +(e,g. with the same SoC) just by using different device trees. An
> added +benefit is that the Linux device tree can be used, thus further
> simplifying +the task of board-bring up either for U-Boot or Linux devs
> (whoever gets to +the baord first!).

s/baord/board/

...snip...

> +Note that compared to Linux, U-Boot's driver model has a separate step of
> +probe/remove which is independent of bind/unbind. This is partly because in
> +U-Boot it may be expensive to prove devices and we don't want to do it
> until +they are needed, or perhaps until after relocation.

s/prove/probe/

...snip...

> +   d. All parent devices are probed. It is not possible to activate a
> device +   unless its parents (all the way up to the root device) are
> activated. +   This means (for example) that an I2C driver will require
> that its bus +   be activated.
s/parents/predecessors/
in a tree, a node can have up to one parent. parent of a parent (recursively) 
is a predecessor.

...snip...

> +   e. The device is marked inactive. Note that it is still bound, so the
> +   device structure itself is not freed at this point. Should the device be
> +   activated again, then the cycle starts again at step 4 above.

if there were no drastic changes since I last checked, this would go to 
activation/probe(), which is described in step 2


regards
Pavel Herrmann

  reply	other threads:[~2014-06-06 19:52 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-06 19:13 [U-Boot] [PATCH v4 0/15] Collected driver model bug-fixes and docs Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 01/15] Add an I/O tracing feature Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 02/15] arm: Support iotrace feature Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 03/15] sandbox: " Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 04/15] Makefile: Support include files for .dts files Simon Glass
2014-06-06 20:29   ` Jon Loeliger
2014-06-06 20:44     ` Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 05/15] dm: Rename struct device_id to udevice_id Simon Glass
2014-06-06 20:26   ` Jon Loeliger
2014-06-06 20:30     ` Jon Loeliger
2014-06-06 20:36     ` Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 06/15] dm: Update README to encourage conversion to driver model Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 07/15] dm: Use case-insensitive comparison for GPIO banks Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 08/15] dm: Add missing header files in lists and root Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 09/15] dm: Cast away the const-ness of the global_data pointer Simon Glass
2014-06-08  5:00   ` Marek Vasut
2014-06-12  3:26     ` Simon Glass
2014-06-12  3:29       ` Simon Glass
2014-06-17 23:42       ` Marek Vasut
2014-06-18  5:36         ` Simon Glass
2014-06-20  4:07           ` Simon Glass
2014-06-20  4:12             ` Marek Vasut
2014-06-20 20:34               ` Simon Glass
2014-06-20 21:45                 ` Marek Vasut
2014-06-06 19:13 ` [U-Boot] [PATCH v4 10/15] dm: Allow driver model tests only for sandbox Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 11/15] dm: Fix printf() strings in the 'dm' command Simon Glass
2014-06-08 22:46   ` Andre Renaud
2014-06-12  3:34     ` Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 12/15] tegra: dts: Bring in GPIO bindings from linux Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 13/15] tegra: Enable driver model Simon Glass
2014-06-06 19:13 ` [U-Boot] [PATCH v4 14/15] dm: Tidy up four minor code nits Simon Glass
2014-06-08  5:01   ` Marek Vasut
2014-06-06 19:13 ` [U-Boot] [PATCH v4 15/15] dm: Expand and improve the device lifecycle docs Simon Glass
2014-06-06 19:52   ` Pavel Herrmann [this message]
2014-06-06 20:42     ` 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=1552477.9gIaszZOQa@bloomfield \
    --to=morpheus.ibis@gmail.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