u-boot.git/test/dm/devres.c, branch v2024.01

dm: treewide: Use uclass_first_device_err when accessing one device

2022-10-18T03:17:12+00:00

There is a number of users that use uclass_first_device to access the
first and (assumed) only device in uclass.

Some check the return value of uclass_first_device and also that a
device was returned which is exactly what uclass_first_device_err does.

Some are not checking that a device was returned and can potentially
crash if no device exists in the uclass. Finally there is one that
returns NULL on error either way.

Convert all of these to use uclass_first_device_err instead, the return
value will be removed from uclass_first_device in a later patch.

Signed-off-by: Michal Suchanek 
Reviewed-by: Simon Glass

test: dm: devres: Remove use-after-free

2022-04-29T15:11:36+00:00

Use-after-free shouldn't be used, even in tests. It's bad practice and
makes the test brittle.

Signed-off-by: Andrew Scull 
Cc: Simon Glass 
Reviewed-by: Simon Glass

dm: treewide: Rename ofdata_to_platdata() to of_to_plat()

2020-12-13T23:51:09+00:00

This name is far too long. Rename it to remove the 'data' bits. This makes
it consistent with the platdata->plat rename.

Signed-off-by: Simon Glass

dm: treewide: Rename 'platdata' variables to just 'plat'

2020-12-13T23:51:08+00:00

We use 'priv' for private data but often use 'platdata' for platform data.
We can't really use 'pdata' since that is ambiguous (it could mean private
or platform data).

Rename some of the latter variables to end with 'plat' for consistency.

Signed-off-by: Simon Glass

dm: Rename DM test flags to make them more generic

2020-08-08T02:31:32+00:00

The test flags used by driver model are currently not available to other
tests. Rather than creating two sets of flags, make these flags generic
by changing the DM_ prefix to UT_ and moving them to the test.h header.

This will allow adding other test flags without confusion.

Signed-off-by: Simon Glass

test/dm: check if devices exist

2020-07-25T20:46:57+00:00

Running 'ut dm' on the sandbox without -D or -d results in segmentation
faults due to NULL pointer dereferences.

Check that device pointers are non-NULL before using them.

Use ut_assertnonnull() for pointers instead of ut_assert().

Signed-off-by: Heinrich Schuchardt 
Tested-by: Philippe Reynes

Revert "Merge tag 'dm-pull-20jul20' of git://git.denx.de/u-boot-dm"

2020-07-24T12:42:06+00:00

This reverts commit 5d3a21df6694ebd66d5c34c9d62a26edc7456fc7, reversing
changes made to 56d37f1c564107e27d873181d838571b7d7860e7.

Unfortunately this is causing CI failures:
https://travis-ci.org/github/trini/u-boot/jobs/711313649

Signed-off-by: Tom Rini

test/dm: check if devices exist

2020-07-20T17:37:47+00:00

Running 'ut dm' on the sandbox without -D or -d results in segmentation
faults due to NULL pointer dereferences.

Check that device pointers are non-NULL before using them.

Use ut_assertnonnull() for pointers instead of ut_assert().

Signed-off-by: Heinrich Schuchardt 
Tested-by: Philippe Reynes

common: Drop log.h from common header

2020-05-19T01:19:18+00:00

Move this header out of the common header.

Signed-off-by: Simon Glass

dm: core: Require users of devres to include the header

2020-02-06T02:33:46+00:00

At present devres.h is included in all files that include dm.h but few
make use of it. Also this pulls in linux/compat which adds several more
headers. Drop the automatic inclusion and require files to include devres
themselves. This provides a good indication of which files use devres.

Signed-off-by: Simon Glass 
Reviewed-by: Anatolij Gustschin