Hi All,
Is it possible to include this series for next U-Boot release?
Regards,
Anup
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
Hi Holger,
On Tue, Nov 13, 2018 at 11:42 AM Holger Brunck wrote:
>
> Hi Mario,
>
> > Simplify the keymile config files once more by unrolling the
> > km/km83xx-common.h, and resolve the #ifdef logic as needed.
> >
> > Signed-off-by: Mario Six
> >
> > ---
> >
> > v1 -> v2:
> > No changes
> >
> >
Hi Tom,
On Mon, Nov 26, 2018 at 7:20 AM Tom Rini wrote:
>
> As the core of the subsystem has been converted along with some of the
> drivers, formalize a deadline for migration.
>
> Cc: Tuomas Tynkkynen
> Cc: Simon Glass
> Cc: Bin Meng
> Cc: Nikita Kiryanov
> Cc: York Sun
> Cc: Fabio Estevam
Hi Andy,
> Von: Bin Meng
> An: Andy Shevchenko ,
> Kopie: Bernhard Messerklinger automation.com>, U-Boot Mailing List
> Datum: 25.11.2018 15:50
> Betreff: Re: [U-Boot] [PATCH] x86: tsc: Add support for APL soc
>
> Hi Andy,
>
> On Sun, Nov 25, 2018 at 10:43 PM Andy Shevchenko
> wrote:
> >
>
Hi Tom,
Please pull the following patch from u-boot-riscv into your tree.
Thanks!
The following changes since commit 5830791d91d1200854ef78fcb32f808c8080f0f0:
Merge tag 'pull-tg18' of git://git.denx.de/u-boot-dm (2018-11-23 17:25:27
-0500)
are available in the Git repository at:
git://
[I've cut down the CC list a bit due to some gmail warnings]
On Mon, Nov 26, 2018 at 4:00 AM Simon Glass wrote:
>
> Hi Simon,
>
> On Sun, 25 Nov 2018 at 14:09, Simon Goldschmidt
> wrote:
> >
> > On Thu, Nov 22, 2018 at 9:50 PM Simon Glass wrote:
> > >
> > > Hi,
> > >
> > > On Thu, 22 Nov 2018 at
This function can be used from do_bootefi_exec() so that we use mostly the
same code for a normal EFI application and an EFI test.
Rename the function and use it in both places.
Signed-off-by: Simon Glass
---
Changes in v16: None
Changes in v15:
- Add a comment about a leaked device path
- Drop
The functions in bootefi are very long because they mix high-level code
and control with the low-level implementation. To help with this, create
functions which handle preparing for running the test and cleaning up
afterwards.
Also shorten the awfully long variable names here.
Signed-off-by: Simo
At present a few error conditions are not checked. Before refactoring
this code, add some basic checks. Note that this code still leaks memory
in the event of error. This will be tackled after the refactor.
Signed-off-by: Simon Glass
---
Changes in v16:
- Add new patch to check for failure to cr
There is still duplicated code in efi_loader for tests and normal
operation.
Add a new bootefi_run_prepare() function which holds common code used to
set up U-Boot to run EFI code. Make use of this from the existing
bootefi_test_prepare() function, as well as do_bootefi_exec().
Also shorten a few
This collects the patches previously sent to break up the very large
functions in efi_loader into smaller pieces. Now that the other sandbox
stuff is applied, perhaps it is time to apply these patches.
This also adds a few new patches to fix more recent breakages.
Unfortunately we still cannot ena
Add an entry for my maintainership of this tool.
Signed-off-by: Simon Glass
---
MAINTAINERS | 5 +
1 file changed, 5 insertions(+)
diff --git a/MAINTAINERS b/MAINTAINERS
index abdb6dcdb51..72e3390e2cb 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -358,6 +358,11 @@ S: Maintained
T: git
Now that many things are converted to Kconfig we can drop most of the
Signed-off-by: Simon Glass
---
common/main.c | 22 ++
1 file changed, 10 insertions(+), 12 deletions(-)
diff --git a/common/main.c b/common/main.c
index 9802bed229c..07b34bf2b05 100644
--- a/common/main.c
Hi Tom.
On Sun, 25 Nov 2018 at 18:12, Tom Rini wrote:
>
> On Sat, Nov 24, 2018 at 12:41:53PM -0700, Simon Glass wrote:
> > Hi Tom,
> >
> > On Fri, 23 Nov 2018 at 12:38, Tom Rini wrote:
> > >
> > > On Fri, Nov 23, 2018 at 05:04:46AM -0700, Simon Glass wrote:
> > > > Hi Tom,
> > > >
> > > > On Thu
On Sat, Nov 24, 2018 at 12:41:53PM -0700, Simon Glass wrote:
> Hi Tom,
>
> On Fri, 23 Nov 2018 at 12:38, Tom Rini wrote:
> >
> > On Fri, Nov 23, 2018 at 05:04:46AM -0700, Simon Glass wrote:
> > > Hi Tom,
> > >
> > > On Thu, 22 Nov 2018 at 16:31, Tom Rini wrote:
> > > >
> > > > On Thu, Nov 22, 20
On Thu, Nov 22, 2018 at 9:50 PM Simon Glass wrote:
>
> Hi,
>
> On Thu, 22 Nov 2018 at 10:02, Tom Rini wrote:
> >
> > On Thu, Nov 22, 2018 at 03:44:28PM +0100, Simon Goldschmidt wrote:
> > > Am Do., 22. Nov. 2018, 14:44 hat Tom Rini
> > > geschrieben:
> > >
> > > > On Thu, Nov 22, 2018 at 02:24:
On 11/25/18 7:32 PM, Philipp Tomsich wrote:
> The documentation comment for dm_rtc_set was referring to dm_rtc_put
> instead. Fix it.
>
> Signed-off-by: Philipp Tomsich
> ---
>
> include/rtc.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/rtc.h b/include/rtc.
On 14.11.2018 10:15, Jagan Teki wrote:
On Wed, Nov 14, 2018 at 2:39 PM Simon Goldschmidt
wrote:
On Wed, Nov 14, 2018 at 9:59 AM Jagan Teki wrote:
On Sat, Nov 10, 2018 at 1:14 AM Simon Goldschmidt
wrote:
On 06.11.2018 10:24, Neil Armstrong wrote:
There is no reason not to use the Linux "jed
The comment in uclass-id.h states that
"U-Boot uclasses start here - in alphabetical order"
but the subsequent list is not sorted alphabetically.
This reestablishes order.
Signed-off-by: Philipp Tomsich
---
include/dm/uclass-id.h | 8
1 file changed, 4 insertions(+), 4 deletions(-)
The documentation comment for dm_rtc_set was referring to dm_rtc_put
instead. Fix it.
Signed-off-by: Philipp Tomsich
---
include/rtc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/rtc.h b/include/rtc.h
index 0d964d5..2c3a574 100644
--- a/include/rtc.h
+++ b/includ
The CRC16-CCITT checksum function is useful for space-constrained
applications (such as obtaining a checksum across a 2KBit or 4KBit
EEPROM) in boot applications. It has not been accessible from boot
scripts until now (due to not having a dedicated command and not being
supported by the hash infrst
This merges the CRC16-CCITT headers into u-boot/crc.h to prepare for
rolling CRC16 into the hash infrastructure. Given that CRC8, CRC32
and CRC32-C already have their prototypes in a single header file, it
seems a good idea to also include CRC16-CCITT in the same.
Signed-off-by: Philipp Tomsich
As much of the USB system has been migrated to DM now, formalize a
deadline for migration.
Cc: Marek Vasut
Cc: Simon Glass
Signed-off-by: Tom Rini
---
Makefile | 7 +++
doc/driver-model/MIGRATION.txt | 10 ++
2 files changed, 17 insertions(+)
diff --git a/Ma
The biggest part of migration to using CONFIG_BLK is that we need to
have the various subsystems migrated first, so reword the plan here to
reference the new deadlines.
Cc: Simon Glass
Signed-off-by: Tom Rini
---
doc/driver-model/MIGRATION.txt | 12 +---
1 file changed, 5 insertions(+),
As the core of the subsystem has been converted along with some of the
drivers, formalize a deadline for migration.
Cc: Tuomas Tynkkynen
Cc: Simon Glass
Cc: Bin Meng
Cc: Nikita Kiryanov
Cc: York Sun
Cc: Fabio Estevam
Cc: Peng Fan
Cc: Stefano Babic
Cc: Stefan Roese
Signed-off-by: Tom Rini
Given that at this point the MMC subsystem itself has been migrated
along with a number of subsystem drivers, formalize a deadline for
migration.
Cc: Simon Glass
Cc: Jaehoon Chung
Signed-off-by: Tom Rini
---
Makefile | 7 +++
doc/driver-model/MIGRATION.txt | 9 +++
On 11/24/2018 11:11 PM, Lukasz Majewski wrote:
> Dear Marek,
>
> The following changes since commit
> 5830791d91d1200854ef78fcb32f808c8080f0f0:
>
> Merge tag 'pull-tg18' of git://git.denx.de/u-boot-dm (2018-11-23
> 17:25:27 -0500)
>
> are available in the git repository at:
>
> git://git.
On 11/25/2018 06:28 PM, Jagan Teki wrote:
(trimming the insane and utterly useless To: list)
> Dropped
> - sh_spi driver
> - CONFIG_SH_SPI,SH_SPI_BASE
>
> Dropped becuase
> - no active updates
> - no dm conversion
> - multiple pings for asking dm-conversion
I haven't received a single notificat
Look like all defconfigs are using DM_SPI, but it may grow
the size if enable it globally. but due to DM_SPI migration
policy remove non-dm code, and will discuss footprint issues if any?
Signed-off-by: Jagan Teki
---
Makefile | 6 +-
drivers/spi/Makefile | 2 +-
drivers/spi/
Dropped
- fsl_espi driver
Dropped becuase
- no active updates
- no dm conversion
- multiple pings for asking dm-conversion
- driver-model migration expiry
Signed-off-by: Jagan Teki
---
board/Arcturus/ucp1020/ucp1020.c | 6 +
drivers/spi/Kconfig | 7 -
drivers/spi/Makefile
Update the spi driver dm-conversion status.
Signed-off-by: Jagan Teki
---
doc/driver-model/MIGRATION.txt | 18 +-
1 file changed, 1 insertion(+), 17 deletions(-)
diff --git a/doc/driver-model/MIGRATION.txt b/doc/driver-model/MIGRATION.txt
index 5ebefd608b..b2d1488aeb 100644
---
Drop the non-dm code, those board which uses non-dm need
to define DM_SPI. u-boot build trigger the warning for
the same.
Signed-off-by: Jagan Teki
---
Makefile| 5 +-
drivers/spi/Makefile| 2 +-
drivers/spi/atmel_spi.c | 198
3 fi
Drop the non-dm code, those board which uses non-dm need
to define DM_SPI. u-boot build trigger the warning for
the same.
Signed-off-by: Jagan Teki
---
Makefile | 2 +-
drivers/spi/Makefile | 2 +-
drivers/spi/mxc_spi.c | 123 ++
3 files
Drop the non-dm code, those board which uses non-dm need
to define DM_SPI. u-boot build trigger the warning for
the same.
Signed-off-by: Jagan Teki
---
Makefile| 2 +-
drivers/spi/Makefile| 2 +-
drivers/spi/omap3_spi.c | 126
3 fi
> Well, f you have the hardware, then there's only one way to find out:
>
> 1) Apply patchset
> 2) Add following into boards defconfig:
> CONFIG_AXP_ALDO3_INRUSH_QUIRK=y
> 3) If it still fails to reboot, add following
> CONFIG_AXP_ALDO3_VOLT_SLOPE_08=y
Hmm... I tried it and it worked: it now reboo
Dropped
- lpc32xx_ssp driver
- CONFIG_LPC32XX_SSP, LPC32XX_SSP_TIMEOUT items
Dropped becuase
- no active updates
- no dm conversion
- multiple pings for asking dm-conversion
- no reponse for dm converted patch
- driver-model migration expiry
Signed-off-by: Jagan Teki
---
configs/devkit3250_defc
Dropped
- sh_spi driver
- CONFIG_SH_SPI,SH_SPI_BASE
Dropped becuase
- no active updates
- no dm conversion
- multiple pings for asking dm-conversion
- no reponse for dm converted patch
- driver-model migration expiry
Signed-off-by: Jagan Teki
---
configs/sh7752evb_defconfig | 1 -
configs/sh
Dropped
- mxs_spi driver
- CONFIG_MXS_SPI
Dropped becuase
- no active updates
- no dm conversion
- multiple pings for asking dm-conversion
- no reponse for dm converted patch
- driver-model migration expiry
Signed-off-by: Jagan Teki
---
configs/bg0900_defconfig| 1 -
configs/m
From: Mario Six
Instead of having a nested if block, just have two branches within the
overarching if block to eliminate one nesting level.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/drivers/spi/mpc8xx
Dropped
- cf_spi driver
- CF_DSPI code
- CONFIG_CF_SPI, CF_DSPI items
Dropped becuase
- no active usage
- no dm conversion
- multiple pings for asking dm-conversion
- driver-model migration expiry
Signed-off-by: Jagan Teki
---
arch/m68k/cpu/mcf5227x/cpu_init.c | 54 ---
arch/m68k/c
Dropped
- soft_spi_legacy driver
- CONFIG_SOFT_SPI
Dropped becuase
- no active updates
- no dm conversion
- multiple pings for asking dm-conversion
- driver-model migration expiry
Signed-off-by: Jagan Teki
---
drivers/spi/Kconfig | 12 +--
drivers/spi/Makefile | 1 -
drive
Support DM in the MPC8xxx SPI driver, and remove the legacy SPI
interface.
Signed-off-by: Mario Six
[jagan: build for board which doesn't enable DM_SPI]
Signed-off-by: Jagan Teki
---
Makefile | 3 +-
drivers/spi/Makefile | 2 +-
drivers/spi/mpc8xxx_spi.c | 144
From: Mario Six
Accesses to the register map are currently done by directly reading and
writing the structure.
Switch to the appropriate IO accessors instead.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 38 +++---
1 file changed, 19 insertions(+),
From: Mario Six
There are three variables that have camel-case names, which is not the
preferred naming style.
Give those variables more compliant names instead.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(
From: Mario Six
Replace the function name with a "%s" format string and the __func__
variable in debug statements (as proposed by checkpatch).
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/spi/mpc8xxx
From: Mario Six
Fix all "superfluous space after case" style errors.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/spi/mpc8xxx_spi.c b/drivers/spi/mpc8xxx_spi.c
index 0393765b6f..1424e7febe 100644
--- a/
From: Mario Six
Minize the time the adapter is disabled (via SPI_MODE_EN
clearing/setting) to just the character length setting, and only set up
the temporary data writing variable right before we need it, so there is
a more clear distinction between setting up the SPI adapter, and setting
up the
From: Mario Six
Decreasing the bit length and increasing the write data pointer should
be done when there are more than 32 bit of data, not 16 bit.
This did not produce incorrect behavior, because the only time where the
two checks produce different outcomes is the case of 16 < bitlen < 32,
and
From: Mario Six
Instead of having a table right before the code implementing the length
setting for documentation, have inline comments for the if branches
actually implementing the length setting described table's entries
(which is readable thanks to the set_char_len function).
Signed-off-by: M
From: Mario Six
Introduce the to_prescale_mod and set_char_len inline functions to make
the code more readable.
Note that the added "if (bitlen > 16)" check does not change the
semantics of the current code, and hence only preserves the current
error (this will be fixed in a later patch in the s
From: Mario Six
The transmission loop starts with setting some variables, which are only
used inside the loop. Reduce the scope to the loop to make the
declaration and initialization of these variables coincide.
In the case of char_size this also always initializes the variable
immediately with
From: Mario Six
The variable "char_size" holds the number of bits to be transferred in
the current loop iteration. A better name would be "xfer_bitlen", which
we rename this variable to.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 8
1 file changed, 4 insertions(+), 4 del
From: Mario Six
Replace pre-processor defines with proper enums, and use the BIT macro
where applicable.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 26 +++---
1 file changed, 19 insertions(+), 7 deletions(-)
diff --git a/drivers/spi/mpc8xxx_spi.c b/drivers/sp
From: Mario Six
Get rid of the is_read variable, and just keep the state of the "not
empty" and "not full" events in two boolean variables within the loop
body.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git
From: Mario Six
The comment before the transmission loop in conjunction with the
definition of SPI_TIMEOUT as 1000 implies that the loop is supposed to
have a timeout value of 1000 ms. But since there is no mdelay(1) or
similar in the loop body, the loop just runs 1000 times, without regard
for t
From: Mario Six
There are some comments on the same line as the code they document. Put
comments above the code lines they document, so the line length is not
unnecessarily increased.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 22 ++
1 file changed, 14 inserti
From: Mario Six
The function signatures in the driver are quite long as is. Use short
type names (uint etc.) to make them more readable.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 12 +---
1 file changed, 5 insertions(+), 7 deletions(-)
diff --git a/drivers/spi/mpc8xxx_s
From: Mario Six
We do nothing in the loop if the "not empty" event was not detected. To
simplify the logic, check if this is the case, and skip the execution of
the loop early to reduce the nesting level and flag checking.
Signed-off-by: Mario Six
---
drivers/spi/mpc8xxx_spi.c | 23 +++
ti_qspi now support dt along with platform data,
respective boards need to switch into dm for the same.
Cc: Tom Rini
Cc: Vitaly Andrianov
Cc: Peter Howard
Signed-off-by: Jagan Teki
---
Makefile | 2 +-
drivers/spi/Makefile | 2 +-
drivers/sp
kirkwood now support dt along with platform data,
respective boards need to switch into dm for the same.
Signed-off-by: Jagan Teki
---
Makefile| 2 +-
drivers/spi/Makefile| 2 +-
drivers/spi/kirkwood_spi.c | 218 ---
davinci_spi now support dt along with platform data,
respective boards need to switch into dm for the same.
Cc: Adam Ford
Cc: Vitaly Andrianov
Cc: Stefano Babic
Cc: Peter Howard
Cc: Tom Rini
Signed-off-by: Jagan Teki
---
Makefile | 9 +++
drivers/spi/Makefile | 2 +
This series fix, build issues and comments from previous
series [1].
We have not heard any positive feed back about dm-conversion on
spi drivers along with no response for the work done by Akash and me.
Plan is to send the same series remainder recursively based on
the dm conversion updates and f
The function part_init() will only be built when we have both
CONFIG_PARTITIONS and CONFIG_HAVE_BLOCK_DEVICE set. Protect the call to
this function with both of these tests now.
Cc: Simon Glass
Signed-off-by: Tom Rini
---
drivers/block/blk-uclass.c | 2 +-
1 file changed, 1 insertion(+), 1 del
Hi Andy,
On Sun, Nov 25, 2018 at 10:43 PM Andy Shevchenko
wrote:
>
> On Sun, Nov 25, 2018 at 4:31 PM Andy Shevchenko
> wrote:
> > On Sat, Nov 24, 2018 at 5:39 PM Bin Meng wrote:
>
> > > Are you saying that if doing cpu_mhz_from_cpuid() you can't get
> > > correct frequency? Can you investigate
On Sun, Nov 25, 2018 at 4:31 PM Andy Shevchenko
wrote:
> On Sat, Nov 24, 2018 at 5:39 PM Bin Meng wrote:
> > Are you saying that if doing cpu_mhz_from_cpuid() you can't get
> > correct frequency? Can you investigate why your core crystal clock is
> > always zero?
>
> Yes, this has to be investig
On Sat, Nov 24, 2018 at 5:39 PM Bin Meng wrote:
> On Fri, Nov 23, 2018 at 5:13 PM Bernhard Messerklinger
> wrote:
> > > APL means ApolloLake? Could you please spell it out?
> > I will fix this.
> > > > --- a/drivers/timer/tsc_timer.c
> > > > +++ b/drivers/timer/tsc_timer.c
> > > > @@ -64,6 +64,
We should use a logical or when combining logical values.
Signed-off-by: Heinrich Schuchardt
---
lib/efi_selftest/efi_selftest_loaded_image.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/lib/efi_selftest/efi_selftest_loaded_image.c
b/lib/efi_selftest/efi_selftest_loaded_i
Hi,
I just wanted to mention that the CONFIG whitelist file started out
two years ago at 8420 lines and is now only 4690 line, a reduction of
close to half, in about 550 commits. So the Kconfig conversion has
gone quite well.
Good news!
Regards,
Simon
Hi Alex,
On Sun, 25 Nov 2018 at 06:27, Alexander Graf wrote:
>
>
>
> On 25.11.18 13:53, Simon Glass wrote:
> > Hi Alex,
> >
> > On Sat, 24 Nov 2018 at 14:26, Alexander Graf wrote:
> >>
> >>
> >>
> >> On 22.11.18 21:46, Simon Glass wrote:
> >>> This function can be used from do_bootefi_exec() so
> At present this code casts addresses to pointers so cannot be used with
> sandbox. Update it to use mapmem instead.
>
> Signed-off-by: Simon Glass
Thanks, applied to efi-next
Alex
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/l
On 25.11.18 13:53, Simon Glass wrote:
> Hi Alex,
>
> On Sat, 24 Nov 2018 at 14:26, Alexander Graf wrote:
>>
>>
>>
>> On 22.11.18 21:46, Simon Glass wrote:
>>> This function can be used from do_bootefi_exec() so that we use mostly the
>>> same code for a normal EFI application and an EFI test.
>
Hi Alex,
On Sat, 24 Nov 2018 at 14:26, Alexander Graf wrote:
>
>
>
> On 22.11.18 21:46, Simon Glass wrote:
> > This function can be used from do_bootefi_exec() so that we use mostly the
> > same code for a normal EFI application and an EFI test.
> >
> > Rename the function and use it in both plac
Hi Jagan,
On Sat, Nov 24, 2018 at 05:41:03PM +0530, Jagan Teki wrote:
> On Thu, Nov 22, 2018 at 1:48 PM Quentin Schulz
> wrote:
> >
> > Hi Jagan,
> >
> > On Thu, Nov 22, 2018 at 01:31:25PM +0530, Jagan Teki wrote:
> > > On Thu, Nov 22, 2018 at 1:21 PM Quentin Schulz
> > > wrote:
> > > >
> > > >
73 matches
Mail list logo
