@@ -1147,6 +1147,10 @@ static DEVICE_ATTR(prop, 0644, prop##_show, prop##_store);
THRESHOLD_PROP_BUILDER(max_tx_thres);
THRESHOLD_PROP_BUILDER(max_rx_thres);
+static const char *dma_op_modes[] = {
+ "element", "threshold", "frame",
+};
+
static ssize_t dma_op_mode_show(struct device *dev,
struct device_attribute *attr, char *buf)
{
@@ -1157,15 +1161,7 @@ static ssize_t dma_op_mode_show(struct device *dev,
dma_op_mode = mcbsp->dma_op_mode;
spin_unlock_irq(&mcbsp->lock);
- return sprintf(buf, "current mode: %d\n"
- "possible mode values are:\n"
- "%d - %s\n"
- "%d - %s\n"
- "%d - %s\n",
- dma_op_mode,
- MCBSP_DMA_MODE_ELEMENT, "element mode",
- MCBSP_DMA_MODE_THRESHOLD, "threshold mode",
- MCBSP_DMA_MODE_FRAME, "frame mode");
+ return sprintf(buf, "%s\n", dma_op_modes[dma_op_mode]);
}
static ssize_t dma_op_mode_store(struct device *dev,
@@ -1173,12 +1169,19 @@ static ssize_t dma_op_mode_store(struct device *dev,
const char *buf, size_t size)
{
struct omap_mcbsp *mcbsp = dev_get_drvdata(dev);
- unsigned long val;
- int status;
-
- status = strict_strtoul(buf, 0, &val);
- if (status)
- return status;
+ const char * const *s;
+ char *p;
+ int len, mode = 0;
+
+ p = memchr(buf, '\n', size);
+ len = p ? p - buf : size;
+
+ for (s = &dma_op_modes[mode];
+ mode < ARRAY_SIZE(dma_op_modes); s++, mode++)
+ if (*s && len == strlen(*s) && !strncmp(buf, *s, len))
+ break;
+ if (mode == ARRAY_SIZE(dma_op_modes))
+ return -EINVAL;
spin_lock_irq(&mcbsp->lock);
@@ -1187,12 +1190,7 @@ static ssize_t dma_op_mode_store(struct device *dev,
goto unlock;
}
- if (val > MCBSP_DMA_MODE_FRAME || val < MCBSP_DMA_MODE_ELEMENT) {
- size = -EINVAL;
- goto unlock;
- }
-
- mcbsp->dma_op_mode = val;
+ mcbsp->dma_op_mode = mode;
unlock:
spin_unlock_irq(&mcbsp->lock);