xref: /rk3399_ARM-atf/plat/arm/board/fvp/fvp_pm.c (revision 1298ae023484077dcab596eb8e963cef96fe9605)
13fc4124cSDan Handley /*
26355f234SVikram Kanigiri  * Copyright (c) 2013-2016, ARM Limited and Contributors. All rights reserved.
33fc4124cSDan Handley  *
43fc4124cSDan Handley  * Redistribution and use in source and binary forms, with or without
53fc4124cSDan Handley  * modification, are permitted provided that the following conditions are met:
63fc4124cSDan Handley  *
73fc4124cSDan Handley  * Redistributions of source code must retain the above copyright notice, this
83fc4124cSDan Handley  * list of conditions and the following disclaimer.
93fc4124cSDan Handley  *
103fc4124cSDan Handley  * Redistributions in binary form must reproduce the above copyright notice,
113fc4124cSDan Handley  * this list of conditions and the following disclaimer in the documentation
123fc4124cSDan Handley  * and/or other materials provided with the distribution.
133fc4124cSDan Handley  *
143fc4124cSDan Handley  * Neither the name of ARM nor the names of its contributors may be used
153fc4124cSDan Handley  * to endorse or promote products derived from this software without specific
163fc4124cSDan Handley  * prior written permission.
173fc4124cSDan Handley  *
183fc4124cSDan Handley  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
193fc4124cSDan Handley  * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
203fc4124cSDan Handley  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
213fc4124cSDan Handley  * ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE
223fc4124cSDan Handley  * LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR
233fc4124cSDan Handley  * CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF
243fc4124cSDan Handley  * SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
253fc4124cSDan Handley  * INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN
263fc4124cSDan Handley  * CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
273fc4124cSDan Handley  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
283fc4124cSDan Handley  * POSSIBILITY OF SUCH DAMAGE.
293fc4124cSDan Handley  */
303fc4124cSDan Handley 
313fc4124cSDan Handley #include <arch_helpers.h>
323fc4124cSDan Handley #include <arm_config.h>
333fc4124cSDan Handley #include <assert.h>
343fc4124cSDan Handley #include <debug.h>
353fc4124cSDan Handley #include <errno.h>
363fc4124cSDan Handley #include <mmio.h>
373fc4124cSDan Handley #include <platform.h>
383fc4124cSDan Handley #include <plat_arm.h>
393fc4124cSDan Handley #include <psci.h>
403fc4124cSDan Handley #include <v2m_def.h>
413fc4124cSDan Handley #include "drivers/pwrc/fvp_pwrc.h"
423fc4124cSDan Handley #include "fvp_def.h"
433fc4124cSDan Handley #include "fvp_private.h"
443fc4124cSDan Handley 
453fc4124cSDan Handley 
462204afdeSSoby Mathew #if ARM_RECOM_STATE_ID_ENC
472204afdeSSoby Mathew /*
482204afdeSSoby Mathew  *  The table storing the valid idle power states. Ensure that the
492204afdeSSoby Mathew  *  array entries are populated in ascending order of state-id to
502204afdeSSoby Mathew  *  enable us to use binary search during power state validation.
512204afdeSSoby Mathew  *  The table must be terminated by a NULL entry.
522204afdeSSoby Mathew  */
532204afdeSSoby Mathew const unsigned int arm_pm_idle_states[] = {
542204afdeSSoby Mathew 	/* State-id - 0x01 */
552204afdeSSoby Mathew 	arm_make_pwrstate_lvl1(ARM_LOCAL_STATE_RUN, ARM_LOCAL_STATE_RET,
562204afdeSSoby Mathew 			ARM_PWR_LVL0, PSTATE_TYPE_STANDBY),
572204afdeSSoby Mathew 	/* State-id - 0x02 */
582204afdeSSoby Mathew 	arm_make_pwrstate_lvl1(ARM_LOCAL_STATE_RUN, ARM_LOCAL_STATE_OFF,
592204afdeSSoby Mathew 			ARM_PWR_LVL0, PSTATE_TYPE_POWERDOWN),
602204afdeSSoby Mathew 	/* State-id - 0x22 */
612204afdeSSoby Mathew 	arm_make_pwrstate_lvl1(ARM_LOCAL_STATE_OFF, ARM_LOCAL_STATE_OFF,
622204afdeSSoby Mathew 			ARM_PWR_LVL1, PSTATE_TYPE_POWERDOWN),
632204afdeSSoby Mathew 	0,
642204afdeSSoby Mathew };
652204afdeSSoby Mathew #endif
662204afdeSSoby Mathew 
673fc4124cSDan Handley /*******************************************************************************
683fc4124cSDan Handley  * Function which implements the common FVP specific operations to power down a
693fc4124cSDan Handley  * cpu in response to a CPU_OFF or CPU_SUSPEND request.
703fc4124cSDan Handley  ******************************************************************************/
713fc4124cSDan Handley static void fvp_cpu_pwrdwn_common(void)
723fc4124cSDan Handley {
733fc4124cSDan Handley 	/* Prevent interrupts from spuriously waking up this cpu */
7427573c59SAchin Gupta 	plat_arm_gic_cpuif_disable();
753fc4124cSDan Handley 
763fc4124cSDan Handley 	/* Program the power controller to power off this cpu. */
773fc4124cSDan Handley 	fvp_pwrc_write_ppoffr(read_mpidr_el1());
783fc4124cSDan Handley }
793fc4124cSDan Handley 
803fc4124cSDan Handley /*******************************************************************************
813fc4124cSDan Handley  * Function which implements the common FVP specific operations to power down a
823fc4124cSDan Handley  * cluster in response to a CPU_OFF or CPU_SUSPEND request.
833fc4124cSDan Handley  ******************************************************************************/
843fc4124cSDan Handley static void fvp_cluster_pwrdwn_common(void)
853fc4124cSDan Handley {
863fc4124cSDan Handley 	uint64_t mpidr = read_mpidr_el1();
873fc4124cSDan Handley 
883fc4124cSDan Handley 	/* Disable coherency if this cluster is to be turned off */
896355f234SVikram Kanigiri 	fvp_interconnect_disable();
903fc4124cSDan Handley 
913fc4124cSDan Handley 	/* Program the power controller to turn the cluster off */
923fc4124cSDan Handley 	fvp_pwrc_write_pcoffr(mpidr);
933fc4124cSDan Handley }
943fc4124cSDan Handley 
95f14d1886SSoby Mathew static void fvp_power_domain_on_finish_common(const psci_power_state_t *target_state)
96f14d1886SSoby Mathew {
97f14d1886SSoby Mathew 	unsigned long mpidr;
98f14d1886SSoby Mathew 
99f14d1886SSoby Mathew 	assert(target_state->pwr_domain_state[ARM_PWR_LVL0] ==
100f14d1886SSoby Mathew 					ARM_LOCAL_STATE_OFF);
101f14d1886SSoby Mathew 
102f14d1886SSoby Mathew 	/* Get the mpidr for this cpu */
103f14d1886SSoby Mathew 	mpidr = read_mpidr_el1();
104f14d1886SSoby Mathew 
105f14d1886SSoby Mathew 	/* Perform the common cluster specific operations */
106f14d1886SSoby Mathew 	if (target_state->pwr_domain_state[ARM_PWR_LVL1] ==
107f14d1886SSoby Mathew 					ARM_LOCAL_STATE_OFF) {
108f14d1886SSoby Mathew 		/*
109f14d1886SSoby Mathew 		 * This CPU might have woken up whilst the cluster was
110f14d1886SSoby Mathew 		 * attempting to power down. In this case the FVP power
111f14d1886SSoby Mathew 		 * controller will have a pending cluster power off request
112f14d1886SSoby Mathew 		 * which needs to be cleared by writing to the PPONR register.
113f14d1886SSoby Mathew 		 * This prevents the power controller from interpreting a
114f14d1886SSoby Mathew 		 * subsequent entry of this cpu into a simple wfi as a power
115f14d1886SSoby Mathew 		 * down request.
116f14d1886SSoby Mathew 		 */
117f14d1886SSoby Mathew 		fvp_pwrc_write_pponr(mpidr);
118f14d1886SSoby Mathew 
119f14d1886SSoby Mathew 		/* Enable coherency if this cluster was off */
1206355f234SVikram Kanigiri 		fvp_interconnect_enable();
121f14d1886SSoby Mathew 	}
122f14d1886SSoby Mathew 
123f14d1886SSoby Mathew 	/*
124f14d1886SSoby Mathew 	 * Clear PWKUPR.WEN bit to ensure interrupts do not interfere
125f14d1886SSoby Mathew 	 * with a cpu power down unless the bit is set again
126f14d1886SSoby Mathew 	 */
127f14d1886SSoby Mathew 	fvp_pwrc_clr_wen(mpidr);
128f14d1886SSoby Mathew }
129f14d1886SSoby Mathew 
130f14d1886SSoby Mathew 
1313fc4124cSDan Handley /*******************************************************************************
13238dce70fSSoby Mathew  * FVP handler called when a CPU is about to enter standby.
1333fc4124cSDan Handley  ******************************************************************************/
13438dce70fSSoby Mathew void fvp_cpu_standby(plat_local_state_t cpu_state)
1353fc4124cSDan Handley {
13638dce70fSSoby Mathew 
13738dce70fSSoby Mathew 	assert(cpu_state == ARM_LOCAL_STATE_RET);
13838dce70fSSoby Mathew 
1393fc4124cSDan Handley 	/*
1403fc4124cSDan Handley 	 * Enter standby state
1413fc4124cSDan Handley 	 * dsb is good practice before using wfi to enter low power states
1423fc4124cSDan Handley 	 */
1433fc4124cSDan Handley 	dsb();
1443fc4124cSDan Handley 	wfi();
1453fc4124cSDan Handley }
1463fc4124cSDan Handley 
1473fc4124cSDan Handley /*******************************************************************************
14838dce70fSSoby Mathew  * FVP handler called when a power domain is about to be turned on. The
14938dce70fSSoby Mathew  * mpidr determines the CPU to be turned on.
1503fc4124cSDan Handley  ******************************************************************************/
15138dce70fSSoby Mathew int fvp_pwr_domain_on(u_register_t mpidr)
1523fc4124cSDan Handley {
1533fc4124cSDan Handley 	int rc = PSCI_E_SUCCESS;
1543fc4124cSDan Handley 	unsigned int psysr;
1553fc4124cSDan Handley 
1563fc4124cSDan Handley 	/*
1570f09c8f7SSandrine Bailleux 	 * Ensure that we do not cancel an inflight power off request for the
1580f09c8f7SSandrine Bailleux 	 * target cpu. That would leave it in a zombie wfi. Wait for it to power
1590f09c8f7SSandrine Bailleux 	 * off and then program the power controller to turn that CPU on.
1603fc4124cSDan Handley 	 */
1613fc4124cSDan Handley 	do {
1623fc4124cSDan Handley 		psysr = fvp_pwrc_read_psysr(mpidr);
1633fc4124cSDan Handley 	} while (psysr & PSYSR_AFF_L0);
1643fc4124cSDan Handley 
1653fc4124cSDan Handley 	fvp_pwrc_write_pponr(mpidr);
1663fc4124cSDan Handley 	return rc;
1673fc4124cSDan Handley }
1683fc4124cSDan Handley 
1693fc4124cSDan Handley /*******************************************************************************
17038dce70fSSoby Mathew  * FVP handler called when a power domain is about to be turned off. The
17138dce70fSSoby Mathew  * target_state encodes the power state that each level should transition to.
1723fc4124cSDan Handley  ******************************************************************************/
17338dce70fSSoby Mathew void fvp_pwr_domain_off(const psci_power_state_t *target_state)
1743fc4124cSDan Handley {
17538dce70fSSoby Mathew 	assert(target_state->pwr_domain_state[ARM_PWR_LVL0] ==
17638dce70fSSoby Mathew 					ARM_LOCAL_STATE_OFF);
1773fc4124cSDan Handley 
1783fc4124cSDan Handley 	/*
17938dce70fSSoby Mathew 	 * If execution reaches this stage then this power domain will be
18038dce70fSSoby Mathew 	 * suspended. Perform at least the cpu specific actions followed
18138dce70fSSoby Mathew 	 * by the cluster specific operations if applicable.
1823fc4124cSDan Handley 	 */
1833fc4124cSDan Handley 	fvp_cpu_pwrdwn_common();
1843fc4124cSDan Handley 
18538dce70fSSoby Mathew 	if (target_state->pwr_domain_state[ARM_PWR_LVL1] ==
18638dce70fSSoby Mathew 					ARM_LOCAL_STATE_OFF)
1873fc4124cSDan Handley 		fvp_cluster_pwrdwn_common();
1883fc4124cSDan Handley 
1893fc4124cSDan Handley }
1903fc4124cSDan Handley 
1913fc4124cSDan Handley /*******************************************************************************
19238dce70fSSoby Mathew  * FVP handler called when a power domain is about to be suspended. The
19338dce70fSSoby Mathew  * target_state encodes the power state that each level should transition to.
1943fc4124cSDan Handley  ******************************************************************************/
19538dce70fSSoby Mathew void fvp_pwr_domain_suspend(const psci_power_state_t *target_state)
1963fc4124cSDan Handley {
1973fc4124cSDan Handley 	unsigned long mpidr;
1983fc4124cSDan Handley 
19938dce70fSSoby Mathew 	/*
20038dce70fSSoby Mathew 	 * FVP has retention only at cpu level. Just return
20138dce70fSSoby Mathew 	 * as nothing is to be done for retention.
20238dce70fSSoby Mathew 	 */
20338dce70fSSoby Mathew 	if (target_state->pwr_domain_state[ARM_PWR_LVL0] ==
20438dce70fSSoby Mathew 					ARM_LOCAL_STATE_RET)
2053fc4124cSDan Handley 		return;
2063fc4124cSDan Handley 
20738dce70fSSoby Mathew 	assert(target_state->pwr_domain_state[ARM_PWR_LVL0] ==
20838dce70fSSoby Mathew 					ARM_LOCAL_STATE_OFF);
20938dce70fSSoby Mathew 
2103fc4124cSDan Handley 	/* Get the mpidr for this cpu */
2113fc4124cSDan Handley 	mpidr = read_mpidr_el1();
2123fc4124cSDan Handley 
2133fc4124cSDan Handley 	/* Program the power controller to enable wakeup interrupts. */
2143fc4124cSDan Handley 	fvp_pwrc_set_wen(mpidr);
2153fc4124cSDan Handley 
2163fc4124cSDan Handley 	/* Perform the common cpu specific operations */
2173fc4124cSDan Handley 	fvp_cpu_pwrdwn_common();
2183fc4124cSDan Handley 
2193fc4124cSDan Handley 	/* Perform the common cluster specific operations */
22038dce70fSSoby Mathew 	if (target_state->pwr_domain_state[ARM_PWR_LVL1] ==
22138dce70fSSoby Mathew 					ARM_LOCAL_STATE_OFF)
2223fc4124cSDan Handley 		fvp_cluster_pwrdwn_common();
2233fc4124cSDan Handley }
2243fc4124cSDan Handley 
2253fc4124cSDan Handley /*******************************************************************************
22638dce70fSSoby Mathew  * FVP handler called when a power domain has just been powered on after
22738dce70fSSoby Mathew  * being turned off earlier. The target_state encodes the low power state that
22838dce70fSSoby Mathew  * each level has woken up from.
2293fc4124cSDan Handley  ******************************************************************************/
23038dce70fSSoby Mathew void fvp_pwr_domain_on_finish(const psci_power_state_t *target_state)
2313fc4124cSDan Handley {
232f14d1886SSoby Mathew 	fvp_power_domain_on_finish_common(target_state);
2333fc4124cSDan Handley 
2343fc4124cSDan Handley 	/* Enable the gic cpu interface */
23527573c59SAchin Gupta 	plat_arm_gic_pcpu_init();
23627573c59SAchin Gupta 
23727573c59SAchin Gupta 	/* Program the gic per-cpu distributor or re-distributor interface */
23827573c59SAchin Gupta 	plat_arm_gic_cpuif_enable();
2393fc4124cSDan Handley }
2403fc4124cSDan Handley 
2413fc4124cSDan Handley /*******************************************************************************
24238dce70fSSoby Mathew  * FVP handler called when a power domain has just been powered on after
24338dce70fSSoby Mathew  * having been suspended earlier. The target_state encodes the low power state
24438dce70fSSoby Mathew  * that each level has woken up from.
2453fc4124cSDan Handley  * TODO: At the moment we reuse the on finisher and reinitialize the secure
2463fc4124cSDan Handley  * context. Need to implement a separate suspend finisher.
2473fc4124cSDan Handley  ******************************************************************************/
24838dce70fSSoby Mathew void fvp_pwr_domain_suspend_finish(const psci_power_state_t *target_state)
2493fc4124cSDan Handley {
25038dce70fSSoby Mathew 	/*
25138dce70fSSoby Mathew 	 * Nothing to be done on waking up from retention from CPU level.
25238dce70fSSoby Mathew 	 */
25338dce70fSSoby Mathew 	if (target_state->pwr_domain_state[ARM_PWR_LVL0] ==
25438dce70fSSoby Mathew 					ARM_LOCAL_STATE_RET)
25538dce70fSSoby Mathew 		return;
25638dce70fSSoby Mathew 
257f14d1886SSoby Mathew 	fvp_power_domain_on_finish_common(target_state);
258f14d1886SSoby Mathew 
259f14d1886SSoby Mathew 	/* Enable the gic cpu interface */
26027573c59SAchin Gupta 	plat_arm_gic_cpuif_enable();
2613fc4124cSDan Handley }
2623fc4124cSDan Handley 
2633fc4124cSDan Handley /*******************************************************************************
2643fc4124cSDan Handley  * FVP handlers to shutdown/reboot the system
2653fc4124cSDan Handley  ******************************************************************************/
2663fc4124cSDan Handley static void __dead2 fvp_system_off(void)
2673fc4124cSDan Handley {
2683fc4124cSDan Handley 	/* Write the System Configuration Control Register */
2693fc4124cSDan Handley 	mmio_write_32(V2M_SYSREGS_BASE + V2M_SYS_CFGCTRL,
2703fc4124cSDan Handley 		V2M_CFGCTRL_START |
2713fc4124cSDan Handley 		V2M_CFGCTRL_RW |
2723fc4124cSDan Handley 		V2M_CFGCTRL_FUNC(V2M_FUNC_SHUTDOWN));
2733fc4124cSDan Handley 	wfi();
2743fc4124cSDan Handley 	ERROR("FVP System Off: operation not handled.\n");
2753fc4124cSDan Handley 	panic();
2763fc4124cSDan Handley }
2773fc4124cSDan Handley 
2783fc4124cSDan Handley static void __dead2 fvp_system_reset(void)
2793fc4124cSDan Handley {
2803fc4124cSDan Handley 	/* Write the System Configuration Control Register */
2813fc4124cSDan Handley 	mmio_write_32(V2M_SYSREGS_BASE + V2M_SYS_CFGCTRL,
2823fc4124cSDan Handley 		V2M_CFGCTRL_START |
2833fc4124cSDan Handley 		V2M_CFGCTRL_RW |
2843fc4124cSDan Handley 		V2M_CFGCTRL_FUNC(V2M_FUNC_REBOOT));
2853fc4124cSDan Handley 	wfi();
2863fc4124cSDan Handley 	ERROR("FVP System Reset: operation not handled.\n");
2873fc4124cSDan Handley 	panic();
2883fc4124cSDan Handley }
2893fc4124cSDan Handley 
290*1298ae02SJeenu Viswambharan static int fvp_node_hw_state(u_register_t target_cpu,
291*1298ae02SJeenu Viswambharan 			     unsigned int power_level)
292*1298ae02SJeenu Viswambharan {
293*1298ae02SJeenu Viswambharan 	unsigned int psysr;
294*1298ae02SJeenu Viswambharan 	int ret;
295*1298ae02SJeenu Viswambharan 
296*1298ae02SJeenu Viswambharan 	/*
297*1298ae02SJeenu Viswambharan 	 * The format of 'power_level' is implementation-defined, but 0 must
298*1298ae02SJeenu Viswambharan 	 * mean a CPU. We also allow 1 to denote the cluster
299*1298ae02SJeenu Viswambharan 	 */
300*1298ae02SJeenu Viswambharan 	if (power_level != ARM_PWR_LVL0 && power_level != ARM_PWR_LVL1)
301*1298ae02SJeenu Viswambharan 		return PSCI_E_INVALID_PARAMS;
302*1298ae02SJeenu Viswambharan 
303*1298ae02SJeenu Viswambharan 	/*
304*1298ae02SJeenu Viswambharan 	 * Read the status of the given MPDIR from FVP power controller. The
305*1298ae02SJeenu Viswambharan 	 * power controller only gives us on/off status, so map that to expected
306*1298ae02SJeenu Viswambharan 	 * return values of the PSCI call
307*1298ae02SJeenu Viswambharan 	 */
308*1298ae02SJeenu Viswambharan 	psysr = fvp_pwrc_read_psysr(target_cpu);
309*1298ae02SJeenu Viswambharan 	if (psysr == PSYSR_INVALID)
310*1298ae02SJeenu Viswambharan 		return PSCI_E_INVALID_PARAMS;
311*1298ae02SJeenu Viswambharan 
312*1298ae02SJeenu Viswambharan 	switch (power_level) {
313*1298ae02SJeenu Viswambharan 	case ARM_PWR_LVL0:
314*1298ae02SJeenu Viswambharan 		ret = (psysr & PSYSR_AFF_L0) ? HW_ON : HW_OFF;
315*1298ae02SJeenu Viswambharan 		break;
316*1298ae02SJeenu Viswambharan 	case ARM_PWR_LVL1:
317*1298ae02SJeenu Viswambharan 		ret = (psysr & PSYSR_AFF_L1) ? HW_ON : HW_OFF;
318*1298ae02SJeenu Viswambharan 		break;
319*1298ae02SJeenu Viswambharan 	default:
320*1298ae02SJeenu Viswambharan 		assert(0);
321*1298ae02SJeenu Viswambharan 	}
322*1298ae02SJeenu Viswambharan 
323*1298ae02SJeenu Viswambharan 	return ret;
324*1298ae02SJeenu Viswambharan }
325*1298ae02SJeenu Viswambharan 
3263fc4124cSDan Handley /*******************************************************************************
327785fb92bSSoby Mathew  * Export the platform handlers via plat_arm_psci_pm_ops. The ARM Standard
328785fb92bSSoby Mathew  * platform layer will take care of registering the handlers with PSCI.
3293fc4124cSDan Handley  ******************************************************************************/
330785fb92bSSoby Mathew const plat_psci_ops_t plat_arm_psci_pm_ops = {
33138dce70fSSoby Mathew 	.cpu_standby = fvp_cpu_standby,
33238dce70fSSoby Mathew 	.pwr_domain_on = fvp_pwr_domain_on,
33338dce70fSSoby Mathew 	.pwr_domain_off = fvp_pwr_domain_off,
33438dce70fSSoby Mathew 	.pwr_domain_suspend = fvp_pwr_domain_suspend,
33538dce70fSSoby Mathew 	.pwr_domain_on_finish = fvp_pwr_domain_on_finish,
33638dce70fSSoby Mathew 	.pwr_domain_suspend_finish = fvp_pwr_domain_suspend_finish,
3373fc4124cSDan Handley 	.system_off = fvp_system_off,
3383fc4124cSDan Handley 	.system_reset = fvp_system_reset,
339f9e858b1SSoby Mathew 	.validate_power_state = arm_validate_power_state,
340*1298ae02SJeenu Viswambharan 	.validate_ns_entrypoint = arm_validate_ns_entrypoint,
341*1298ae02SJeenu Viswambharan 	.get_node_hw_state = fvp_node_hw_state
3423fc4124cSDan Handley };
343