On Wed, 24 Jun 2020 at 22:11, Walter Lozano wrote:
>
> Add a test for dtoc taking into account the cd-gpios property.
>
> Signed-off-by: Walter Lozano
> ---
>
> tools/dtoc/dtoc_test_phandle_cd_gpios.dts | 42 ++
> tools/dtoc/test_dtoc.py | 67 +++
Drop the now-unused Python 2 code to keep code coverage at 100%.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/dtoc/fdt.py | 17 -
1 file changed, 4 insertions(+), 13 deletions(-)
Applied to u-boot-fdt
___
U-Boot mailing l
Some tests have crept in with Python 2 strings and constructs. Convert
then.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/binman/ftest.py | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
Applied to u-boot-fdt
___
U-Bo
On Thu, Oct 31, 2019 at 10:12:04AM -0600, Simon Glass wrote:
> Hi Tom,
>
> On Thu, 31 Oct 2019 at 09:50, Tom Rini wrote:
> >
> > On Thu, Oct 31, 2019 at 09:19:50AM -0600, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Thu, 31 Oct 2019 at 07:55, Tom Rini wrote:
> > > >
> > > > On Thu, Oct 31, 201
On Sun, Oct 27, 2019 at 09:47:39AM -0600, Simon Glass wrote:
> Unfortunately libfdt needs this value now, which is present in the
> stdint.h header. That file is just a placeholder in U-Boot and these sorts
> of constants appear in the linux/kernel.h header instead.
>
> To keep libfdt happy, add I
Convert buildman to Python 3 and make it use that, to meet the 2020
deadline.
Signed-off-by: Simon Glass
---
Changes in v2:
- Use integer division for multiprocessing.cpu_count()
- Use integer division in SetupBuild() and GetActionSummary()
- Use HTTPMessage.getheader() instead of HTTPMessage.ge
At present all the 'command' methods return bytes. Most of the time we
actually want strings, so change this. We still need to keep the internal
representation as bytes since otherwise unicode strings might break over
a read() boundary (e.g. 4KB), causing errors. But we can convert the end
result t
On Sun, Oct 27, 2019 at 09:47:40AM -0600, Simon Glass wrote:
> For better or worse libfdt recent grew a lot of code that checks the
> validity of the device tree in great detail. When using unsigned or
> unverified data this makes things safer, but it does add to code size.
>
> Add some controls t
This function should use a void * type, not char *. This causes an error:
TypeError: in method 'fdt_property_stub', argument 3 of type 'char const *'
Fix it.
Signed-off-by: Simon Glass
---
Changes in v2: None
scripts/dtc/pylibfdt/libfdt.i_shipped | 4 ++--
1 file changed, 2 insertions(+), 2
At present patman test fail in some environments which don't use utf-8
as the default file encoding. Add this explicitly.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add new patch to explicitly use unicode for file I/O in patman
tools/patman/func_test.py | 8
tools/patman/patchs
Update this test to use Python 3 to meet the 2020 deadline.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/dtoc/test_fdt.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Applied to u-boot-fdt
___
U-Boot mailing list
U-Boot@lists.de
Update this tool to use Python 3 to meet the 2020 deadline.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/patman/patman.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Applied to u-boot-fdt
___
U-Boot mailing list
U-Boot@lists.de
Convert this tool to Python 3 and make it use that, to meet the 2020
deadline.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/microcode-tool.py | 28 ++--
1 file changed, 14 insertions(+), 14 deletions(-)
Applied to u-boot-fdt
Convert this tool to Python 3 and make it use that, to meet the 2020
deadline.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/rkmux.py | 16
1 file changed, 8 insertions(+), 8 deletions(-)
Applied to u-boot-fdt
___
U-Boot
Update this test to use Python 3 to meet the 2020 deadline.
Also make it executable while we are here.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/dtoc/dtoc.py | 2 +-
tools/dtoc/test_dtoc.py | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)
mode change 100644 => 10075
Sync up the libfdt Python bindings with upstream, commit:
430419c (tests: fix some python warnings)
Signed-off-by: Simon Glass
---
Changes in v2: None
scripts/dtc/pylibfdt/libfdt.i_shipped | 45 ---
1 file changed, 33 insertions(+), 12 deletions(-)
Applied to u-boot-f
Bring over the fdt from this commit:
430419c (origin/master) tests: fix some python warnings
adding in the 'assumptions' series designed to reduce code size.
Signed-off-by: Simon Glass
---
lib/libfdt/fdt_ro.c | 420 ---
scripts/dtc/libfdt/Makefile.libf
This board appears to be very near its size limit and cannot accept the
new checking code in libfdt. Disable this code so this the board can
continue to build.
Signed-off-by: Simon Glass
---
configs/tbs2910_defconfig | 1 +
1 file changed, 1 insertion(+)
Applied to u-boot-fdt
_
Convert this tool to Python 3 and make it use that, to meet the 2020
deadline.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/moveconfig.py | 82 ++---
1 file changed, 41 insertions(+), 41 deletions(-)
Applied to u-boot-fdt
___
Build this swig module with Python 3.
Signed-off-by: Simon Glass
---
Changes in v2: None
scripts/dtc/pylibfdt/Makefile | 2 +-
scripts/dtc/pylibfdt/libfdt.i_shipped | 2 +-
scripts/dtc/pylibfdt/setup.py | 2 +-
tools/binman/entry.py | 16 ++--
too
When preparing to possible expand or contract an entry we reset the size
to the original value from the binman device-tree definition, which is
often None.
This causes binman to forget the original size of the entry. Remember this
so that it can be used when needed.
Signed-off-by: Simon Glass
--
Hi Robert,
On Wed, 13 Nov 2019 at 12:40, Robert Beckett wrote:
>
> fixes: 8ab452d5877638a97e5bdd521d119403b70b45f5
>
> When compiling list of cover letter cc addresses, using null as a
> separater, then encoding to utf-8 results in lots of "\x00" as
> separators. patman then doesnt understand tha
On Sun, 27 Oct 2019 at 10:55, Anatolij Gustschin wrote:
>
> s/Subprocress/Subprocess/
> s/easiler/easier/
> s/repositiory/repository/
> s/rangem/range/
> s/Retruns/Returns/
>
> Signed-off-by: Anatolij Gustschin
> ---
> tools/patman/cros_subprocess.py | 4 ++--
> tools/patman/gitutil.py |
On Sun, 27 Oct 2019 at 10:55, Anatolij Gustschin wrote:
>
> s/Subprocress/Subprocess/
> s/easiler/easier/
> s/repositiory/repository/
> s/rangem/range/
> s/Retruns/Returns/
>
> Signed-off-by: Anatolij Gustschin
> ---
> tools/patman/cros_subprocess.py | 4 ++--
> tools/patman/gitutil.py |
Am Mittwoch, 23. Oktober 2019, 16:46:41 CET schrieb Heiko Stuebner:
> From: Heiko Stuebner
>
> OP-TEE can get supplied with a devicetree and will then insert
> its firmware node and reserved-memory sections into it.
> As this devicetree often is not the one supplied to a later
> loaded kernel, a p
Am Mittwoch, 23. Oktober 2019, 16:46:39 CET schrieb Heiko Stuebner:
> From: Heiko Stuebner
>
> The phandlep pointer returning the phandle to the caller is optional
> and if it is not set when calling fdtdec_add_reserved_memory() it is
> highly likely that the caller is not interested in a phandle
On Sat, 26 Oct 2019 at 15:17, Heinrich Schuchardt wrote:
>
> Compiling arch/sandbox/cpu/os.c results in an error
>
> ../arch/sandbox/cpu/os.c: In function ‘os_find_text_base’:
> ../arch/sandbox/cpu/os.c:823:12: error: cast to pointer from
> integer of different size [-Werror=int-to-pointer-cast]
>
Am Mittwoch, 23. Oktober 2019, 16:46:41 CET schrieb Heiko Stuebner:
> From: Heiko Stuebner
>
> OP-TEE can get supplied with a devicetree and will then insert
> its firmware node and reserved-memory sections into it.
> As this devicetree often is not the one supplied to a later
> loaded kernel, a p
Hi Robert,
On Wed, 13 Nov 2019 at 12:40, Robert Beckett wrote:
>
> fixes: 8ab452d5877638a97e5bdd521d119403b70b45f5
>
> When compiling list of cover letter cc addresses, using null as a
> separater, then encoding to utf-8 results in lots of "\x00" as
> separators. patman then doesnt understand tha
On Sat, 26 Oct 2019 at 15:17, Heinrich Schuchardt wrote:
>
> Compiling arch/sandbox/cpu/os.c results in an error
>
> ../arch/sandbox/cpu/os.c: In function ‘os_find_text_base’:
> ../arch/sandbox/cpu/os.c:823:12: error: cast to pointer from
> integer of different size [-Werror=int-to-pointer-cast]
>
Hi Robert,
On Wed, 13 Nov 2019 at 12:40, Robert Beckett wrote:
>
> fixes: 8ab452d5877638a97e5bdd521d119403b70b45f5
>
> When compiling list of cover letter cc addresses, using null as a
> separater, then encoding to utf-8 results in lots of "\x00" as
> separators. patman then doesnt understand tha
On Sun, 27 Oct 2019 at 10:55, Anatolij Gustschin wrote:
>
> s/Subprocress/Subprocess/
> s/easiler/easier/
> s/repositiory/repository/
> s/rangem/range/
> s/Retruns/Returns/
>
> Signed-off-by: Anatolij Gustschin
> ---
> tools/patman/cros_subprocess.py | 4 ++--
> tools/patman/gitutil.py |
On Sat, 26 Oct 2019 at 15:17, Heinrich Schuchardt wrote:
>
> Compiling arch/sandbox/cpu/os.c results in an error
>
> ../arch/sandbox/cpu/os.c: In function ‘os_find_text_base’:
> ../arch/sandbox/cpu/os.c:823:12: error: cast to pointer from
> integer of different size [-Werror=int-to-pointer-cast]
>
Am Mittwoch, 23. Oktober 2019, 16:46:38 CET schrieb Heiko Stuebner:
> From: Heiko Stuebner
>
> The change adding fdtdec_add_reserved_memory() already protected the added
> phandle against the phandlep being NULL - making the phandlep var optional.
>
> But in the early code checking for an already
On Wed, 23 Oct 2019 at 07:44, Patrick Delaunay wrote:
>
> Remove the pinctrl_decode_pin_config() API, because this
> function is unused and not compatible with livetree
> (it uses fdtdec_get_bool instead of ofnode API).
>
> Signed-off-by: Patrick Delaunay
> ---
>
> drivers/pinctrl/pinctrl-uclass
Am Mittwoch, 23. Oktober 2019, 16:46:39 CET schrieb Heiko Stuebner:
> From: Heiko Stuebner
>
> The phandlep pointer returning the phandle to the caller is optional
> and if it is not set when calling fdtdec_add_reserved_memory() it is
> highly likely that the caller is not interested in a phandle
This script already works with Python 3. Make it use that by default so
that it can import the patman libraries.
Signed-off-by: Simon Glass
---
Changes in v2: None
tools/rmboard.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Applied to u-boot-dm, thanks!
On Fri, 27 Sep 2019 at 02:19, Lokesh Vutla wrote:
>
> From: Anatolij Gustschin
>
> The power domain associated with a device is enabled when probing,
> but currently the domain remains enabled when the device is removed.
> Some boards started to disable power domains for selected devices
> via cu
On Thu, 3 Oct 2019 at 21:41, AKASHI Takahiro wrote:
>
> Without this patch, compiling may potentially fail.
>
> Signed-off-by: AKASHI Takahiro
> ---
> arch/sandbox/include/asm/u-boot-sandbox.h | 2 ++
> 1 file changed, 2 insertions(+)
Reviewed-by: Simon Glass
Applied to u-boot-dm, thanks!
___
On Fri, 27 Sep 2019 at 02:19, Lokesh Vutla wrote:
>
> Add dev_power_domain_off() api to disable all the power-domains
> corresponding to a device
>
> Signed-off-by: Lokesh Vutla
> ---
> drivers/power/domain/power-domain-uclass.c | 34 +-
> include/power-domain.h
On Fri, 27 Sep 2019 at 02:19, Lokesh Vutla wrote:
>
> Enable DM_FLAG_DEFAULT_PD_CTRL_OFF for arm64 remote core so that
> pd can be enabled after loading the image.
>
> Signed-off-by: Lokesh Vutla
> ---
> drivers/remoteproc/ti_k3_arm64_rproc.c | 1 +
> 1 file changed, 1 insertion(+)
Reviewed-by:
On Mon, 30 Sep 2019 at 02:19, Patrick Delaunay wrote:
>
> At present the 'Index' column of 'dm tree' assumes there is
> two digits, this patch increase it to 3 digits.
>
> It also aligns output of 'dm uclass', assuming the same 3 digits index.
>
> The boards with CONFIG_PINCTRL_FULL activated have
On Thu, 26 Sep 2019 at 07:44, Jean-Jacques Hiblot wrote:
>
> When DEVRES is not set, devm_kmalloc_array() is spelled
> devm_kmaloc_array() (with one 'l' only).
> Fixing it so that the name is the same with and without DEVRES.
>
> Signed-off-by: Jean-Jacques Hiblot
> ---
> include/dm/device.h | 4
Hi Lokesh,
On Fri, 27 Sep 2019 at 02:19, Lokesh Vutla wrote:
>
> In some remoteproc cases, enabling the power domain of the core will
> start running the core. In such cases image should be loaded before
> enabling the power domain. But the current DM framework enables the
> power-domain by defau
On Tue, 22 Oct 2019 at 01:40, Kever Yang wrote:
>
> Use log() insted of debug() for uclass_find_device_by_seq function,
> since this print is very much and we can filter it out with log()
> interface.
>
> Signed-off-by: Kever Yang
> ---
>
> Changes in v2:
> - use log_debug() instead of log()
>
>
On Tue, 22 Oct 2019 at 02:05, Jean-Jacques Hiblot wrote:
>
> The FDT specification [0] gives a requirement of aligning properties on
> 32-bits. Make sure that the compiler is aware of this constraint when
> accessing 64-bits properties.
>
> [0]:
> https://github.com/devicetree-org/devicetree-spec
On Mon, 21 Oct 2019 at 21:10, Dmitry Torokhov wrote:
>
> There is a contributor in Linux kernel with a comma in their name, which
> confuses patman and results in invalid to- or cc- addresses on some
> patches. To avoid this, let's use \0 as a separator when generating cc
> file.
>
> Signed-off-by
On 22.10.19 01:26, Simon Glass wrote:
> This function is used in the bootstage report which may be trigged in TPL
> or TPL. Add a very basic implication of this function so that it builds.
> There is no attempt to get the formatting right, since this would add too
> much code size.
>
> Signed-off-b
The ctype array is brought into the image, adding 256 bytes, when it is
unlikely to be needed. The extra code for %p is only present when DEBUG
is defined, so let's drop ctype as well unless DEBUG is defined.
Signed-off-by: Simon Glass
---
Changes in v2: None
lib/tiny-printf.c | 7 +++
1 f
At present bootstage relocation assumes that it is possible to point back
to memory available before relocation, so it does not relocate the
strings. However this is not the case on some platforms, such as x86 which
uses the cache as RAM and loses access to this when the cache is enabled.
Move the
When stashing bootstage info, store the next ID so that it can be used
when the stash is restored. This avoids the ID starting at zero and
potentially overwriting existing entries.
Signed-off-by: Simon Glass
---
Changes in v2: None
common/bootstage.c | 11 +++
1 file changed, 7 inserti
At present there is a single shared address for bootstage data in both
TPL and SPL. If SPL unstashs TPL bootstage info and then stashes it again
to pass it to U-Boot, the new stash overwrites the strings of the old
stash.
Fix this by duplicating the strings into the malloc() region. This should
be
Hi Simon,
On Sat, Oct 12, 2019 at 6:22 AM Simon Glass wrote:
>
> This function writes to its address so the address should not be declared
> as const. Fix it.
>
> Signed-off-by: Simon Glass
> ---
>
> arch/sandbox/cpu/cpu.c| 3 +--
> arch/sandbox/include/asm/io.h | 11 +--
> 2 f
Make sure that the bloblist starts on an aligned boundary. This protects
against one of the early allocating causing the alignment to be lost.
Signed-off-by: Simon Glass
---
Changes in v2: None
common/board_f.c | 1 +
1 file changed, 1 insertion(+)
Applied to u-boot-dm, thanks!
__
The current code searches for empty records but these not existing with
bootstage now. This used to be needed when bootstage records were stored
in a spare array.
Drop the unnecessary code and fix a code-style nit at the same time.
Signed-off-by: Simon Glass
---
Changes in v2: None
common/boo
It is possible to enable bootstage in TPL. TPL can stash the info for SPL.
But at present this information is then lost because SPL does not read
from the stash.
Add support for SPL not being the first phase to enable bootstage.
Signed-off-by: Simon Glass
---
Changes in v2:
- Adjust SPL logic to
On Sat, Oct 12, 2019 at 6:28 AM Simon Glass wrote:
>
> From: Jean-Jacques Hiblot
>
> The test did reads after writes but didn't check the value.
> It probably was because the sandbox didn't implement the writeX/readX
> functions.
>
> Signed-off-by: Jean-Jacques Hiblot
> Updated to use sandbox_se
On Sat, Oct 12, 2019 at 6:22 AM Simon Glass wrote:
>
> This function needs a prototype so that tests can use it. Add one.
>
> Signed-off-by: Simon Glass
> ---
>
> arch/sandbox/include/asm/test.h | 11 +++
> 1 file changed, 11 insertions(+)
>
Reviewed-by: Bin Meng
Applied to u-boot-dm,
At present bootstage in TPL and SPL use the same ID so it is not possible
to see the timing of each. Separate out the IDs and use the correct one
depending on which phase we are at.
Example output:
Timer summary in microseconds (14 records):
MarkElapsed Stage
0 0 r
With a bit of code reordering we can support %p using the existing code
for ulong.
Move the %p code up and adjust the logic accordingly.
Signed-off-by: Simon Glass
---
Changes in v2:
- Add a new patch to support %p without DEBUG
lib/tiny-printf.c | 30 +-
1 file ch
On Sat, Oct 12, 2019 at 6:28 AM Simon Glass wrote:
>
> Quite a few tests use addresses or hex values for comparisons. Add hex
> output for test failures, e.g.:
>
>0x55ca22fa == reg: Expected 0x55ca22fa (1439310586),
> got 0x55ea22fb (1441407739)
>
> Signed-off-by: Simon Glass
> ---
>
On 12/10/2019 00:16, Simon Glass wrote:
> This function assumes that the 'val' parameter has no masked bits set.
> This is not defined by the function prototype though. Fix the function to
> mask the value and update the documentation.
>
> Signed-off-by: Simon Glass
> ---
>
> drivers/core/regmap
Fix the 'memeber' typo in doc/driver-model/design.rst.
Signed-off-by: Dario Binacchi
Reviewed-by: Simon Glass
---
Changes in v2:
- Add the commit message
doc/driver-model/design.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Applied to u-boot-dm, thanks!
When recording the console output for testing it is useful to be able to
read the output a line at a time to check that the output is correct. Also
we need to check that we get to the end of the output.
Add a console function to return the next line and another to see how must
data is left.
Signe
At present we reset the console buffer before each test but do not
actually set the recording flag. Without this, the output is not
recorded.
Update the code to set the flag before the test and clear it afterwards.
Signed-off-by: Simon Glass
---
test/dm/test-main.c | 4 ++--
1 file changed, 2
On Wed, 29 Jan 2020 at 10:45, Fabio Estevam wrote:
>
> Following the recommendation of adding '#define DEBUG' at the top
> of drivers/core/lists.c does not cause the debug messages to be
> shown. Change it to '#define LOG_DEBUG' instead, which actually
> makes it work as per doc/README.log.
>
> Wh
These functions do not modify the device so should use a const pointer to
it. Update the code accordingly.
Signed-off-by: Simon Glass
---
drivers/core/read.c | 97 +++--
include/dm/read.h | 205 +++-
2 files changed, 159 insertions(+),
These functions do not modify the device so should use a const pointer to
it. Update the code accordingly.
Signed-off-by: Simon Glass
---
drivers/core/fdtaddr.c | 26 +-
include/dm/fdtaddr.h | 26 +-
2 files changed, 26 insertions(+), 26 deletio
Tidy up a few places where const * should be used.
Signed-off-by: Simon Glass
---
drivers/pci/pci-uclass.c | 18 +-
include/fdtdec.h | 2 +-
include/pci.h| 16
lib/fdtdec.c | 2 +-
4 files changed, 19 insertions(+), 19 deletions
Update this header file to use const devices where possible, to permit
callers to also use const.
Signed-off-by: Simon Glass
---
drivers/core/device.c | 27 ++-
include/dm/device.h | 30 --
include/dm/read.h | 4 ++--
3 files changed, 3
On Tue, 28 Jan 2020 10:07:18 -0700
Simon Glass wrote:
Hi Simon,
> On Sat, 25 Jan 2020 at 16:18, André Przywara wrote:
> >
> > On 25/01/2020 16:20, Tom Rini wrote:
> >
> > Hi Tom,
> >
> > thanks for having a look!
> >
> >
> > > On Tue, Jan 21, 2020 at 10:23:17AM +, Andre Przywara wrote:
> >
Hi Thirupathaiah,
On Tue, 4 Feb 2020 at 10:08, Thirupathaiah Annapureddy
wrote:
>
> Hi All,
>
> May I know what are the next steps in making forward progress on this?
Can you please add a test for this? We need a sandbox driver of some
sort - see the existing sandbox TPM driver.
Regards,
Simon
On Fri, 17 Jan 2020 at 02:53, wrote:
>
> From: Matthias Brugger
>
> At kernel.org aarch64 toolchains are published in folder
> arm64. Fix the URL for that case, so that we can fetch
> toolchains on aarch64 machines.
>
> Signed-off-by: Matthias Brugger
>
> ---
>
> tools/buildman/toolchain.py | 2
On Sat, Jan 11, 2020 at 1:32 AM Sean Anderson wrote:
>
> If CONFIG_CMDLINE=n, common/cli.c calls board_run_command. This fails to
> link on most architectures. However, the sandbox architecture has an
> implementation which we can use.
>
> Signed-off-by: Sean Anderson
> ---
> Changes for v2:
> -
I am not sure how to add libsdl2-dev to the gitlab image, so disable
building sandbox with SDL for now.
Signed-off-by: Simon Glass
---
Changes in v2: None
.gitlab-ci.yml | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
Applied to u-boot-dm, thanks!
At present we only support two phases where devres can be used:
bind and probe. This is handled with a boolean. We want to add a new
phase (platdata), so change this to an enum.
Signed-off-by: Simon Glass
---
drivers/core/devres.c | 22 --
1 file changed, 16 insertions(+), 6
Add a driver which does devres allocations so that we can write tests for
devres.
Signed-off-by: Simon Glass
---
arch/sandbox/dts/test.dts | 4
include/dm/uclass-id.h| 1 +
include/test/test.h | 9
test/dm/test-fdt.c| 47 +++
The devres functionality has very few users in U-Boot, but it still should
have tests. Add a few basic tests of the main functions.
Signed-off-by: Simon Glass
---
drivers/core/devres.c | 13 +++
include/dm/devres.h | 20 +
test/dm/Makefile | 1 +
test/dm/devres.c | 178 +
Since the ofdata_to_platdata() method can allocate resources, add it as a
new devres phase.
Signed-off-by: Simon Glass
---
drivers/core/devres.c | 18 +-
include/test/test.h | 3 ++-
test/dm/devres.c | 14 +++---
test/dm/test-fdt.c| 11 +++
4 files ch
The malloc() implementations provides a way of finding out the approximate
amount of memory that is allocated. Add helper functions to make it easier
to access this and see changes over time. This is useful for tests that
want to check if memory has been allocated or freed.
Signed-off-by: Simon Gl
At present when CONFIG_DEBUG_DEVRES is enabled, U-Boot prints log messages
to the console with every devres allocation/free event. This causes most
tests to fail since the console output is not as expected.
In particular this prevents us from adding a device to sandbox which uses
devres in its bin
This flag is missing a comment. Add one.
Signed-off-by: Simon Glass
---
include/dm/device.h | 1 +
1 file changed, 1 insertion(+)
Applied to u-boot-dm, thanks!
Add a new internal function, device_ofdata_to_platdata() to handle
allocating private space associated with each device and reading the
platform data from the device tree.
Call this new function from device_probe().
Signed-off-by: Simon Glass
---
drivers/core/device.c| 29 +
We want to avoid allocating platform data twice. This could happen if
device_probe() is called after device_ofdata_to_platdata() for the same
device.
Add a flag to track whether device_ofdata_to_platdata() has been called on
a device. Check the flag to make sure it doesn't happen twice, and clear
This method is supposed to extract platform data from the device tree. It
should be done before the device itself is probed. Move it earlier in the
device_probe() function.
Signed-off-by: Simon Glass
---
drivers/core/device.c | 14 +++---
1 file changed, 7 insertions(+), 7 deletions(-)
Hi Cédric,
On Tue, 7 Jan 2020 at 21:20, Cédric Le Goater wrote:
>
> On 12/30/19 5:19 AM, Simon Glass wrote:
> > At present the clock driver reads its ofdata in the probe() method. This
> > is not correct although it is often harmless.
> >
> > However in this case it causes a problem, something li
Remove this duplicated code, since the 'fail' label does this immediately.
Signed-off-by: Simon Glass
---
drivers/core/device.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
Applied to u-boot-dm, thanks!
fdt_support.h is missing declarations for bd_t. Including asm/u-boot.h
pulls in the definition.
Signed-off-by: Sean Anderson
---
include/fdt_support.h | 1 +
1 file changed, 1 insertion(+)
Applied to u-boot-dm, thanks!
Update the device tree, sandbox i2c driver and tests to use the new
emulation parent to hold emulators.
Signed-off-by: Simon Glass
---
Changes in v2: None
arch/sandbox/dts/sandbox.dts | 25 +++
arch/sandbox/dts/sandbox_pmic.dtsi | 71 +++---
arch/sandbox/d
Some RTC chips have child drivers, e.g. to provide access to their
non-volatile RAM. Scan for these when binding.
Signed-off-by: Simon Glass
---
Changes in v2:
- Drop patches previously applied
- Add new patches to support the RTC change
drivers/rtc/rtc-uclass.c | 1 +
1 file changed, 1 inser
At present tests are quite slow to run, over a minute on my machine. This
presents a considerable barrier to bisecting for failures.
The slowest tests are the filesystem ones and the buildman --fetch-arch
test. Add a new 'qcheck' target that skips these tests. This reduces test
time down to about
Add two functions which can find a child device by uclass or by name.
The first is useful with Multi-Function-Devices (MFDs) to find one of a
particular type. The second is useful when only the name is known.
Signed-off-by: Simon Glass
---
Changes in v2: None
drivers/core/device.c | 34 +++
This function may be useful to code outside of the code driver-model
implementation. Export it and add a test.
Signed-off-by: Simon Glass
---
Changes in v2: None
drivers/core/uclass.c| 6 ++
include/dm/uclass-internal.h | 17 +
test/dm/test-fdt.c | 20 +++
On Sun, 18 Nov 2018 08:14:28 -0700
Simon Glass wrote:
> There is a newline missing from quite a few printf() strings in these
> pmic files. Fix them.
>
> Signed-off-by: Simon Glass
> ---
Reviewed-by: Lukasz Majewski
Best regards,
Lukasz Majewski
--
DENX Software Engineering GmbH, Man
This is currently at the top in the space for internal use. But this
uclass is used outside driver model and test code. Move it into the
correct alpha order.
Signed-off-by: Simon Glass
---
Changes in v2: None
include/dm/uclass-id.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Applied
At present this value is hard-coded in the function that generates a
square wave. Since sample rates vary between different hardware, it makes
more sense to have this as a parameter.
Update the function and its users.
Signed-off-by: Simon Glass
---
arch/sandbox/cpu/sdl.c| 2 +-
drivers/sou
Sandbox i2c works using emulation drivers which are currently children of
the i2c device:
rtc_0: rtc@43 {
reg = <0x43>;
compatible = "sandbox-rtc";
emul {
compatible = "sandbox,i2c-rtc";
};
};
This function currently outputs twice as much data as it should and
overwrites its buffer as a result. Fix it.
Signed-off-by: Simon Glass
---
drivers/sound/sound.c | 2 --
1 file changed, 2 deletions(-)
Applied to u-boot-dm/master, thanks!
___
U-Boot
Currently the else part of ofnode_get_addr_index function
does not fetch addresses based on the index but rather just
returns the base address. Fix that.
Signed-off-by: Keerthy
---
drivers/core/ofnode.c | 17 ++---
1 file changed, 10 insertions(+), 7 deletions(-)
Applied to u-boot-d
Hi Heiko,
> store fdt header member with name in U-Boot
> Environment variable with name .
>
> for example to get the total length of the fdt and store
> it in filesize, call:
>
> fdt header get filesize totalsize
>
> For membernames look into fdt header definition at
> scripts/dtc/libfdt/libfdt.
1 - 100 of 1121 matches
Mail list logo
