Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 89955520 authored by Shawn Guo's avatar Shawn Guo
Browse files

ARM: imx: get rid of variable timer_base 



We now have pointer to imx_timer structure available where timer base
address is needed, so we can just kill global timer_base by using
imxtm->base instead.

Signed-off-by: default avatarShawn Guo <shawn.guo@linaro.org>
parent 24f74ad1

arch/arm/mach-imx/time.c

+26 −29
Original line number Original line Diff line number Diff line
@@ -104,45 +104,43 @@ struct imx_gpt_data { 
			      struct clock_event_device *ced);

			      struct clock_event_device *ced);

};

};





static void __iomem *timer_base;



static inline struct imx_timer *to_imx_timer(struct clock_event_device *ced)

static inline struct imx_timer *to_imx_timer(struct clock_event_device *ced)

{

{

	return container_of(ced, struct imx_timer, ced);

	return container_of(ced, struct imx_timer, ced);

}

}





static inline void gpt_irq_disable(void)

static inline void gpt_irq_disable(struct imx_timer *imxtm)

{

{

	unsigned int tmp;

	unsigned int tmp;





	if (timer_is_v2())

	if (timer_is_v2())

		writel_relaxed(0, timer_base + V2_IR);

		writel_relaxed(0, imxtm->base + V2_IR);

	else {

	else {

		tmp = readl_relaxed(timer_base + MXC_TCTL);

		tmp = readl_relaxed(imxtm->base + MXC_TCTL);

		writel_relaxed(tmp & ~MX1_2_TCTL_IRQEN, timer_base + MXC_TCTL);

		writel_relaxed(tmp & ~MX1_2_TCTL_IRQEN, imxtm->base + MXC_TCTL);

	}

	}

}

}





static inline void gpt_irq_enable(void)

static inline void gpt_irq_enable(struct imx_timer *imxtm)

{

{

	if (timer_is_v2())

	if (timer_is_v2())

		writel_relaxed(1<<0, timer_base + V2_IR);

		writel_relaxed(1<<0, imxtm->base + V2_IR);

	else {

	else {

		writel_relaxed(readl_relaxed(timer_base + MXC_TCTL) | MX1_2_TCTL_IRQEN,

		writel_relaxed(readl_relaxed(imxtm->base + MXC_TCTL) | MX1_2_TCTL_IRQEN,

			timer_base + MXC_TCTL);

			imxtm->base + MXC_TCTL);

	}

	}

}

}





static void gpt_irq_acknowledge(void)

static void gpt_irq_acknowledge(struct imx_timer *imxtm)

{

{

	if (timer_is_v1()) {

	if (timer_is_v1()) {

		if (cpu_is_mx1())

		if (cpu_is_mx1())

			writel_relaxed(0, timer_base + MX1_2_TSTAT);

			writel_relaxed(0, imxtm->base + MX1_2_TSTAT);

		else

		else

			writel_relaxed(MX2_TSTAT_CAPT | MX2_TSTAT_COMP,

			writel_relaxed(MX2_TSTAT_CAPT | MX2_TSTAT_COMP,

				timer_base + MX1_2_TSTAT);

				imxtm->base + MX1_2_TSTAT);

	} else if (timer_is_v2())

	} else if (timer_is_v2())

		writel_relaxed(V2_TSTAT_OF1, timer_base + V2_TSTAT);

		writel_relaxed(V2_TSTAT_OF1, imxtm->base + V2_TSTAT);

}

}





static void __iomem *sched_clock_reg;

static void __iomem *sched_clock_reg;
@@ -178,29 +176,31 @@ static int __init mxc_clocksource_init(struct imx_timer *imxtm) 
/* clock event */

/* clock event */





static int mx1_2_set_next_event(unsigned long evt,

static int mx1_2_set_next_event(unsigned long evt,

			      struct clock_event_device *unused)

			      struct clock_event_device *ced)

{

{

	struct imx_timer *imxtm = to_imx_timer(ced);

	unsigned long tcmp;

	unsigned long tcmp;





	tcmp = readl_relaxed(timer_base + MX1_2_TCN) + evt;

	tcmp = readl_relaxed(imxtm->base + MX1_2_TCN) + evt;





	writel_relaxed(tcmp, timer_base + MX1_2_TCMP);

	writel_relaxed(tcmp, imxtm->base + MX1_2_TCMP);





	return (int)(tcmp - readl_relaxed(timer_base + MX1_2_TCN)) < 0 ?

	return (int)(tcmp - readl_relaxed(imxtm->base + MX1_2_TCN)) < 0 ?

				-ETIME : 0;

				-ETIME : 0;

}

}





static int v2_set_next_event(unsigned long evt,

static int v2_set_next_event(unsigned long evt,

			      struct clock_event_device *unused)

			      struct clock_event_device *ced)

{

{

	struct imx_timer *imxtm = to_imx_timer(ced);

	unsigned long tcmp;

	unsigned long tcmp;





	tcmp = readl_relaxed(timer_base + V2_TCN) + evt;

	tcmp = readl_relaxed(imxtm->base + V2_TCN) + evt;





	writel_relaxed(tcmp, timer_base + V2_TCMP);

	writel_relaxed(tcmp, imxtm->base + V2_TCMP);





	return evt < 0x7fffffff &&

	return evt < 0x7fffffff &&

		(int)(tcmp - readl_relaxed(timer_base + V2_TCN)) < 0 ?

		(int)(tcmp - readl_relaxed(imxtm->base + V2_TCN)) < 0 ?

				-ETIME : 0;

				-ETIME : 0;

}

}
@@ -227,7 +227,7 @@ static void mxc_set_mode(enum clock_event_mode mode, 
	local_irq_save(flags);

	local_irq_save(flags);





	/* Disable interrupt in GPT module */

	/* Disable interrupt in GPT module */

	gpt_irq_disable();

	gpt_irq_disable(imxtm);





	if (mode != imxtm->cem) {

	if (mode != imxtm->cem) {

		u32 tcn = readl_relaxed(imxtm->base + imxtm->gpt->reg_tcn);

		u32 tcn = readl_relaxed(imxtm->base + imxtm->gpt->reg_tcn);
@@ -235,7 +235,7 @@ static void mxc_set_mode(enum clock_event_mode mode, 
		writel_relaxed(tcn - 3, imxtm->base + imxtm->gpt->reg_tcmp);

		writel_relaxed(tcn - 3, imxtm->base + imxtm->gpt->reg_tcmp);





		/* Clear pending interrupt */

		/* Clear pending interrupt */

		gpt_irq_acknowledge();

		gpt_irq_acknowledge(imxtm);

	}

	}





#ifdef DEBUG

#ifdef DEBUG
@@ -261,7 +261,7 @@ static void mxc_set_mode(enum clock_event_mode mode, 
	 * mode switching

	 * mode switching

	 */

	 */

		local_irq_save(flags);

		local_irq_save(flags);

		gpt_irq_enable();

		gpt_irq_enable(imxtm);

		local_irq_restore(flags);

		local_irq_restore(flags);

		break;

		break;

	case CLOCK_EVT_MODE_SHUTDOWN:

	case CLOCK_EVT_MODE_SHUTDOWN:
@@ -283,7 +283,7 @@ static irqreturn_t mxc_timer_interrupt(int irq, void *dev_id) 




	tstat = readl_relaxed(imxtm->base + imxtm->gpt->reg_tstat);

	tstat = readl_relaxed(imxtm->base + imxtm->gpt->reg_tstat);





	gpt_irq_acknowledge();

	gpt_irq_acknowledge(imxtm);





	ced->event_handler(ced);

	ced->event_handler(ced);
@@ -387,9 +387,6 @@ static const struct imx_gpt_data imx6dl_gpt_data = { 




static void __init _mxc_timer_init(struct imx_timer *imxtm)

static void __init _mxc_timer_init(struct imx_timer *imxtm)

{

{

	/* Temporary */

	timer_base = imxtm->base;



	switch (imxtm->type) {

	switch (imxtm->type) {

	case GPT_TYPE_IMX1:

	case GPT_TYPE_IMX1:

		imxtm->gpt = &imx1_gpt_data;

		imxtm->gpt = &imx1_gpt_data;