On 6/18/24 01:29, Tom Rini wrote:
On Fri, Jun 14, 2024 at 04:13:54PM +0200, Stefan Roese wrote:
On 6/12/24 18:50, Tom Rini wrote:
On Wed, Jun 12, 2024 at 05:13:37PM +0100, Jiaxun Yang wrote:
在2024年6月12日六月 下午5:00，Tom Rini写道：
[...]
configs/octeon_nic23_defconfig | 1 -
2 files changed, 1 insertion(+), 2 deletions(-)
This seems similar to:
https://patchwork.ozlabs.org/project/uboot/patch/[email protected]/
at least for CI. And for outside CI, I'm OK with just having the value
be changed in the defconfig as needed. We do support using config
fragments, so keeping such changes locally isn't too hard.
So the default value is a little bit too hard even for some of the actual
hardware.
Right, there's some platforms where it's too small and we should just
bump it up. I think for now the default is what we want it to be for
most platforms.
The current default value is definitely too small, especially when CI
is involved (I did not have this in mind when implementing), so:
Acked-by: Stefan Roese <[email protected]>
Can we please get either this, or
https://patchwork.ozlabs.org/project/uboot/patch/[email protected]/
merged for master? The number of false negatives in CI due to this is
big issue for getting more contributors to use CI. Thanks.
Yes.
Applied to u-boot-watchdog/master
Thanks,
Stefan
