u-boot.git/drivers/core/lists.c, branch v2016.05

dm: core: Display the error number when driver binding fails

2016-01-24T04:07:18+00:00

This is often -96 (-EPFNOSUPPORT) which indicates that the uclass is not
compiled in. Display the error number to make this easier to spot.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng

of: clean up OF_CONTROL ifdef conditionals

2015-08-18T17:46:05+00:00

We have flipped CONFIG_SPL_DISABLE_OF_CONTROL.  We have cleansing
devices, $(SPL_) and CONFIG_IS_ENABLED(), so we are ready to clear
away the ugly logic in include/fdtdec.h:

 #ifdef CONFIG_OF_CONTROL
 # if defined(CONFIG_SPL_BUILD) && !defined(SPL_OF_CONTROL)
 #  define OF_CONTROL 0
 # else
 #  define OF_CONTROL 1
 # endif
 #else
 # define OF_CONTROL 0
 #endif

Now CONFIG_IS_ENABLED(OF_CONTROL) is the substitute.  It refers to
CONFIG_OF_CONTROL for U-boot proper and CONFIG_SPL_OF_CONTROL for
SPL.

Also, we no longer have to cancel CONFIG_OF_CONTROL in
include/config_uncmd_spl.h and scripts/Makefile.spl.

Signed-off-by: Masahiro Yamada 
Reviewed-by: Tom Rini 
Reviewed-by: Simon Glass 
Acked-by: Linus Walleij

dm: core: Use debug() instead of printf() for failures

2015-07-21T23:39:22+00:00

To avoid bloating SPL code, use debug() where possible in the driver model
core code. The error code is already returned, and can be investigated as
needed.

Signed-off-by: Simon Glass

dm: core: Add a function to bind a driver for a device tree node

2015-04-30T03:02:32+00:00

Some device tree nodes do not have compatible strings but do require
drivers. This is pretty rare, and somewhat unfortunate. Add a function
to permit creation of a driver for any device tree node.

Signed-off-by: Simon Glass

dm: core: Rename driver data function to dev_get_driver_data()

2015-04-18T17:11:19+00:00

The existing get_get_of_data() function provides access to both the driver's
compatible string and its driver data. However only the latter is actually
useful. Update the interface to reflect this and fix up existing users.

Signed-off-by: Simon Glass 
Reviewed-by: Marek Vasut

dm: core: remove unnecessary return condition in uclass lookup

2014-11-22T09:16:49+00:00

These conditions never happen.
 - There is no real uclass with UCLASS_INVALID id.
 - uclass never becomes NULL because ll_entry_start() always returns
   a valid pointer.

Signed-off-by: Masahiro Yamada 
Acked-by: Simon Glass

dm: core: remove unnecessary return condition in driver lookup

2014-11-22T09:16:49+00:00

The variable "drv" never becomes NULL because ll_entry_start()
always returns a valid pointer even if there are no entries.

The case "n_ents == 0" is covered by the following "for" loop.

Signed-off-by: Masahiro Yamada 
Acked-by: Simon Glass

dm: core: remove meaningless if conditional

2014-11-22T09:16:48+00:00

If the variable "ret" is equal to "-ENOENT", it is trapped at [1] and
never reaches [2].  At [3], the condition "ret != -ENOENT" is always
true.

  if (ret == -ENOENT) {                       <------------------ [1]
          continue;
  } else if (ret == -ENODEV) {
          dm_dbg("Device '%s' has no compatible string\n", name);
          break;
  } else if (ret) {                           <------------------ [2]
          dm_warn("Device tree error at offset %d\n", offset);
          if (!result || ret != -ENOENT)      <------------------ [3]
                  result = ret;
          break;
  }

Signed-off-by: Masahiro Yamada 
Acked-by: Simon Glass

dm: Add a function to bind a device by driver name

2014-11-22T09:16:47+00:00

In some cases we need to manually bind a device to a particular driver.
Add a function to do this.

Signed-off-by: Simon Glass 
Reviewed-by: Jagannadha Sutradharudu Teki 
Acked-by: Heiko Schocher

dm: core: Allow access to the device's driver_id data

2014-11-22T09:16:45+00:00

When the device is created from a device tree node, it matches a compatible
string. Allow access to that string and the associated data.

Signed-off-by: Simon Glass 
Reviewed-by: Tom Rini 
Acked-by: Heiko Schocher