mirror of
				https://github.com/smaeul/u-boot.git
				synced 2025-10-30 11:38:15 +00:00 
			
		
		
		
	Historically, the reset_cpu() function had an `addr` parameter which was
meant to pass in an address of the reset vector location, where the CPU
should reset to.  This feature is no longer used anywhere in U-Boot as
all reset_cpu() implementations now ignore the passed value.  Generic
code has been added which always calls reset_cpu() with `0` which means
this feature can no longer be used easily anyway.
Over time, many implementations seem to have "misunderstood" the
existence of this parameter as a way to customize/parameterize the reset
(e.g.  COLD vs WARM resets).  As this is not properly supported, the
code will almost always not do what it is intended to (because all
call-sites just call reset_cpu() with 0).
To avoid confusion and to clean up the codebase from unused left-overs
of the past, remove the `addr` parameter entirely.  Code which intends
to support different kinds of resets should be rewritten as a sysreset
driver instead.
This transformation was done with the following coccinelle patch:
    @@
    expression argvalue;
    @@
    - reset_cpu(argvalue)
    + reset_cpu()
    @@
    identifier argname;
    type argtype;
    @@
    - reset_cpu(argtype argname)
    + reset_cpu(void)
    { ... }
Signed-off-by: Harald Seiler <hws@denx.de>
Reviewed-by: Simon Glass <sjg@chromium.org>
		
	
			
		
			
				
	
	
		
			46 lines
		
	
	
		
			661 B
		
	
	
	
		
			C
		
	
	
	
	
	
			
		
		
	
	
			46 lines
		
	
	
		
			661 B
		
	
	
	
		
			C
		
	
	
	
	
	
| // SPDX-License-Identifier: GPL-2.0+
 | |
| /*
 | |
|  * (C) Copyright 2007
 | |
|  * Nobuhiro Iwamatsu <iwamatsu@nigauri.org>
 | |
|  */
 | |
| 
 | |
| #include <common.h>
 | |
| #include <command.h>
 | |
| #include <irq_func.h>
 | |
| #include <cpu_func.h>
 | |
| #include <net.h>
 | |
| #include <netdev.h>
 | |
| #include <asm/processor.h>
 | |
| 
 | |
| int checkcpu(void)
 | |
| {
 | |
| 	puts("CPU: SH4\n");
 | |
| 	return 0;
 | |
| }
 | |
| 
 | |
| int cpu_init (void)
 | |
| {
 | |
| 	return 0;
 | |
| }
 | |
| 
 | |
| int cleanup_before_linux (void)
 | |
| {
 | |
| 	disable_interrupts();
 | |
| 	return 0;
 | |
| }
 | |
| 
 | |
| int do_reset(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
 | |
| {
 | |
| 	disable_interrupts();
 | |
| 	reset_cpu();
 | |
| 	return 0;
 | |
| }
 | |
| 
 | |
| int cpu_eth_init(struct bd_info *bis)
 | |
| {
 | |
| #ifdef CONFIG_SH_ETHER
 | |
| 	sh_eth_initialize(bis);
 | |
| #endif
 | |
| 	return 0;
 | |
| }
 |