From f201f0e2aa67c9424c6293a15a3761c8c93f8c3d Mon Sep 17 00:00:00 2001
From: Shaun Ruffell <sruffell@digium.com>
Date: Mon, 20 Sep 2010 20:33:07 +0000
Subject: dahdi: Use 'chan' convenience pointer in dahdi_transmit.

I believe this makes it a little more clear what is happening.  Also could
provide a potential speedup if the span doesn't need to be constantly
dereferenced (although an optimizer might make this moot).  Regardless, this
change doesn't hurt.

Review: https://reviewboard.asterisk.org/r/905/

Signed-off-by: Shaun Ruffell <sruffell@digium.com>

git-svn-id: http://svn.asterisk.org/svn/dahdi/linux/trunk@9362 a0bf4364-ded3-4de4-8d8a-66a801d63aff
---
 drivers/dahdi/dahdi-base.c | 42 +++++++++++++++++++++---------------------
 1 file changed, 21 insertions(+), 21 deletions(-)

diff --git a/drivers/dahdi/dahdi-base.c b/drivers/dahdi/dahdi-base.c
index 34d63e8..99ed34d 100644
--- a/drivers/dahdi/dahdi-base.c
+++ b/drivers/dahdi/dahdi-base.c
@@ -8427,22 +8427,22 @@ int dahdi_transmit(struct dahdi_span *span)
 	unsigned long flags;
 
 	for (x=0;x<span->channels;x++) {
-		spin_lock_irqsave(&span->chans[x]->lock, flags);
-		if (span->chans[x]->flags & DAHDI_FLAG_NOSTDTXRX) {
-			spin_unlock_irqrestore(&span->chans[x]->lock, flags);
+		struct dahdi_chan *const chan = span->chans[x];
+		spin_lock_irqsave(&chan->lock, flags);
+		if (chan->flags & DAHDI_FLAG_NOSTDTXRX) {
+			spin_unlock_irqrestore(&chan->lock, flags);
 			continue;
 		}
-		if (span->chans[x] == span->chans[x]->master) {
-			if (span->chans[x]->otimer) {
-				span->chans[x]->otimer -= DAHDI_CHUNKSIZE;
-				if (span->chans[x]->otimer <= 0) {
-					__rbs_otimer_expire(span->chans[x]);
-				}
+		if (chan == chan->master) {
+			if (chan->otimer) {
+				chan->otimer -= DAHDI_CHUNKSIZE;
+				if (chan->otimer <= 0)
+					__rbs_otimer_expire(chan);
 			}
-			if (span->chans[x]->flags & DAHDI_FLAG_AUDIO) {
-				__dahdi_real_transmit(span->chans[x]);
+			if (chan->flags & DAHDI_FLAG_AUDIO) {
+				__dahdi_real_transmit(chan);
 			} else {
-				if (span->chans[x]->nextslave) {
+				if (chan->nextslave) {
 					u_char data[DAHDI_CHUNKSIZE];
 					int pos=DAHDI_CHUNKSIZE;
 					/* Process master/slaves one way */
@@ -8452,7 +8452,7 @@ int dahdi_transmit(struct dahdi_span *span)
 						do {
 							if (pos==DAHDI_CHUNKSIZE) {
 								/* Get next chunk */
-								__dahdi_transmit_chunk(span->chans[x], data);
+								__dahdi_transmit_chunk(chan, data);
 								pos = 0;
 							}
 							span->chans[z]->writechunk[y] = data[pos++];
@@ -8461,21 +8461,21 @@ int dahdi_transmit(struct dahdi_span *span)
 					}
 				} else {
 					/* Process independents elsewise */
-					__dahdi_real_transmit(span->chans[x]);
+					__dahdi_real_transmit(chan);
 				}
 			}
-			if (span->chans[x]->sig == DAHDI_SIG_DACS_RBS) {
-				if (chans[span->chans[x]->confna]) {
+			if (chan->sig == DAHDI_SIG_DACS_RBS) {
+				struct dahdi_chan *const conf =
+							chans[chan->confna];
+				if (conf && (chan->txsig != conf->rxsig)) {
 				    	/* Just set bits for our destination */
-					if (span->chans[x]->txsig != chans[span->chans[x]->confna]->rxsig) {
-						span->chans[x]->txsig = chans[span->chans[x]->confna]->rxsig;
-						span->ops->rbsbits(span->chans[x], chans[span->chans[x]->confna]->rxsig);
-					}
+					chan->txsig = conf->rxsig;
+					span->ops->rbsbits(chan, conf->rxsig);
 				}
 			}
 
 		}
-		spin_unlock_irqrestore(&span->chans[x]->lock, flags);
+		spin_unlock_irqrestore(&chan->lock, flags);
 	}
 	if (span->mainttimer) {
 		span->mainttimer -= DAHDI_CHUNKSIZE;
-- 
cgit v1.2.3