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/20240524210817.1953298-1-ra...
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 sr@denx.de
Can we please get either this, or https://patchwork.ozlabs.org/project/uboot/patch/20240524210817.1953298-1-ra... 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