On 9/24/07, Jiri Slaby <jirislaby@xxxxxxxxx> wrote:
> On 09/24/2007 07:24 PM, Luis R. Rodriguez wrote:
> > * #if 0's need to go, lets just remove this code instead.
>
> Which code do you mean exactly,
These hunks:
+#if 0
/*
* Get power mode (sleep state)
* TODO:Remove ?
@@ -1385,7 +1148,7 @@ ath5k_hw_get_power_mode(struct ath_hw *hal)
AR5K_TRACE;
return hal->ah_power_mode;
}
-
+#endif
/***********************\
DMA Related Functions
@@ -1816,6 +1579,7 @@ enum ath5k_int ath5k_hw_set_intr(struct ath_hw
*hal, enum ath5k_int new_mask)
return old_mask;
}
+#if 0
/*
* Enable HW radar detection
*/
@@ -1858,7 +1622,7 @@ ath5k_hw_radar_alert(struct ath_hw *hal, bool enable)
/*Re-enable interrupts*/
ath5k_hw_reg_write(hal, AR5K_IER_ENABLE, AR5K_IER);
}
-
> there are also parts that might be needed in the future?
Then we can add then in the future. We can also remove turbo for this
same reason, I don't see us introducing it for a while. I should have
just removed turbo with my 'internalizing turbo' as its useless code
right now. But can patch work can be sent in later though. Mind you
madwifi-old-openhal will still have all this code we're not using
right now. So we'll always be able to pull from there.
Luis
-
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
