The patch titled
orinoco replace hermes_write_words() with hermes_write_bytes()
has been removed from the -mm tree. Its filename is
orinoco-replace-hermes_write_words-with-hermes_write_bytes.patch
This patch was probably dropped from -mm because
it has now been merged into a subsystem tree or
into Linus's tree, or because it was folded into
its parent patch in the -mm tree.
From: Pavel Roskin <proski@xxxxxxx>
The new function can write an odd number of bytes, thus making padding
unnecessary.
Signed-off-by: Pavel Roskin <proski@xxxxxxx>
Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
---
drivers/net/wireless/hermes.c | 17 ++++++++---------
drivers/net/wireless/hermes.h | 7 +++++--
drivers/net/wireless/spectrum_cs.c | 7 +++----
3 files changed, 16 insertions(+), 15 deletions(-)
diff -puN drivers/net/wireless/hermes.c~orinoco-replace-hermes_write_words-with-hermes_write_bytes drivers/net/wireless/hermes.c
--- 25/drivers/net/wireless/hermes.c~orinoco-replace-hermes_write_words-with-hermes_write_bytes Fri Apr 7 15:37:28 2006
+++ 25-akpm/drivers/net/wireless/hermes.c Fri Apr 7 15:37:28 2006
@@ -400,8 +400,7 @@ int hermes_bap_pread(hermes_t *hw, int b
}
/* Write a block of data to the chip's buffer, via the
- * BAP. Synchronization/serialization is the caller's problem. len
- * must be even.
+ * BAP. Synchronization/serialization is the caller's problem.
*
* Returns: < 0 on internal failure (errno), 0 on success, > 0 on error from firmware
*/
@@ -411,7 +410,7 @@ int hermes_bap_pwrite(hermes_t *hw, int
int dreg = bap ? HERMES_DATA1 : HERMES_DATA0;
int err = 0;
- if ( (len < 0) || (len % 2) )
+ if (len < 0)
return -EINVAL;
err = hermes_bap_seek(hw, bap, id, offset);
@@ -419,7 +418,7 @@ int hermes_bap_pwrite(hermes_t *hw, int
goto out;
/* Actually do the transfer */
- hermes_write_words(hw, dreg, buf, len/2);
+ hermes_write_bytes(hw, dreg, buf, len);
out:
return err;
@@ -427,7 +426,7 @@ int hermes_bap_pwrite(hermes_t *hw, int
/* Write a block of data to the chip's buffer with padding if
* neccessary, via the BAP. Synchronization/serialization is the
- * caller's problem. len must be even.
+ * caller's problem.
*
* Returns: < 0 on internal failure (errno), 0 on success, > 0 on error from firmware
*/
@@ -437,7 +436,7 @@ int hermes_bap_pwrite_pad(hermes_t *hw,
int dreg = bap ? HERMES_DATA1 : HERMES_DATA0;
int err = 0;
- if (len < 0 || len % 2 || data_len > len)
+ if (len < 0 || data_len > len)
return -EINVAL;
err = hermes_bap_seek(hw, bap, id, offset);
@@ -445,13 +444,13 @@ int hermes_bap_pwrite_pad(hermes_t *hw,
goto out;
/* Transfer all the complete words of data */
- hermes_write_words(hw, dreg, buf, data_len/2);
+ hermes_write_bytes(hw, dreg, buf, data_len);
/* If there is an odd byte left over pad and transfer it */
if (data_len & 1) {
u8 end[2];
end[1] = 0;
end[0] = ((unsigned char *)buf)[data_len - 1];
- hermes_write_words(hw, dreg, end, 1);
+ hermes_write_bytes(hw, dreg, end, 2);
data_len ++;
}
/* Now send zeros for the padding */
@@ -534,7 +533,7 @@ int hermes_write_ltv(hermes_t *hw, int b
count = length - 1;
- hermes_write_words(hw, dreg, value, count);
+ hermes_write_bytes(hw, dreg, value, count << 1);
err = hermes_docmd_wait(hw, HERMES_CMD_ACCESS | HERMES_CMD_WRITE,
rid, NULL);
diff -puN drivers/net/wireless/hermes.h~orinoco-replace-hermes_write_words-with-hermes_write_bytes drivers/net/wireless/hermes.h
--- 25/drivers/net/wireless/hermes.h~orinoco-replace-hermes_write_words-with-hermes_write_bytes Fri Apr 7 15:37:28 2006
+++ 25-akpm/drivers/net/wireless/hermes.h Fri Apr 7 15:37:28 2006
@@ -408,10 +408,13 @@ static inline void hermes_read_words(str
ioread16_rep(hw->iobase + off, buf, count);
}
-static inline void hermes_write_words(struct hermes *hw, int off, const void *buf, unsigned count)
+static inline void hermes_write_bytes(struct hermes *hw, int off,
+ const char *buf, unsigned count)
{
off = off << hw->reg_spacing;
- iowrite16_rep(hw->iobase + off, buf, count);
+ iowrite16_rep(hw->iobase + off, buf, count >> 1);
+ if (unlikely(count & 1))
+ iowrite8(buf[count - 1], hw->iobase + off);
}
static inline void hermes_clear_words(struct hermes *hw, int off, unsigned count)
diff -puN drivers/net/wireless/spectrum_cs.c~orinoco-replace-hermes_write_words-with-hermes_write_bytes drivers/net/wireless/spectrum_cs.c
--- 25/drivers/net/wireless/spectrum_cs.c~orinoco-replace-hermes_write_words-with-hermes_write_bytes Fri Apr 7 15:37:28 2006
+++ 25-akpm/drivers/net/wireless/spectrum_cs.c Fri Apr 7 15:37:28 2006
@@ -343,8 +343,7 @@ spectrum_plug_pdi(hermes_t *hw, struct p
/* do the actual plugging */
spectrum_aux_setaddr(hw, pdr_addr(pdr));
- hermes_write_words(hw, HERMES_AUXDATA, pdi->data,
- pdi_len(pdi) / 2);
+ hermes_write_bytes(hw, HERMES_AUXDATA, pdi->data, pdi_len(pdi));
return 0;
}
@@ -424,8 +423,8 @@ spectrum_load_blocks(hermes_t *hw, const
while (dblock_addr(blk) != BLOCK_END) {
spectrum_aux_setaddr(hw, blkaddr);
- hermes_write_words(hw, HERMES_AUXDATA, blk->data,
- blklen / 2);
+ hermes_write_bytes(hw, HERMES_AUXDATA, blk->data,
+ blklen);
blk = (struct dblock *) &blk->data[blklen];
blkaddr = dblock_addr(blk);
_
Patches currently in -mm which might be from proski@xxxxxxx are
origin.patch
git-wireless.patch
-
To unsubscribe from this list: send the line "unsubscribe mm-commits" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
