Messages in this thread Patch in this message | | | Date | Thu, 24 Apr 2008 18:34:11 -0300 | From | Mauro Carvalho Chehab <> | Subject | Re: [v4l-dvb-maintainer] no help text for DVB_TUNER_ITD1000 |
| |
On Thu, 24 Apr 2008 14:08:21 -0700 Brandon Philips <bphilips@suse.de> wrote:
> On 17:28 Thu 24 Apr 2008, Mauro Carvalho Chehab wrote: > > On Thu, 24 Apr 2008 23:22:57 +0300 Adrian Bunk <bunk@kernel.org> wrote: > > > The DVB_TUNER_ITD1000 option introduced by > > > commit 1881ee89e0fe03ac5bba9045acb3bea1818f9466 > > > (V4L/DVB (7571): mt312: Cleanup buffer variables of read/write functions) > > > lacks a help text. > > > > > > Please add a help text. > > > > Thanks for pointing. > > > > I just noticed and prepared a patch. I should soon add it to devel branch > > at -git for people to review. > > How is adding this Kconfig related to the patch that Matthias posted? > Once again, some very strange folding is happening as things are moving > into git. > > http://article.gmane.org/gmane.linux.drivers.dvb/41017/match=mt312+cleanup+buffer+variables+read+write+functions > > vs. > > http://git.kernel.org/?p=linux/kernel/git/mchehab/v4l-dvb.git;a=commitdiff;h=1881ee89e0fe03ac5bba9045acb3bea1818f9466;hp=dbad108bdcb30629c850f5606949510da010a686
Argh!
Matthias, sorry for the mess.
> Could I take a look at the scripts that are used to generate the git > tree? This is the second time in the last few weeks that something like > this has happened.
This error weren't generated by the script. The enclosed patch is the resulting patch from the scripts. It is the correct one.
I'll investigate what's happened here, and do some git integrity checks. Maybe stgit did something bad.
Cheers, Mauro Changeset: 7571 From: Matthias Schwarzott <zzam@gentoo.org> Commiter: Mauro Carvalho Chehab <mchehab@infradead.org> Date: Sat Apr 12 15:04:46 2008 +0000 Subject: mt312: Cleanup buffer variables of read/write functions
Change type of buffer variables from void* to u8* to save some casts.
Signed-off-by: Matthias Schwarzott <zzam@gentoo.org> Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org> ---
diff -upNr oldtree/drivers/media/dvb/frontends/mt312.c linux/drivers/media/dvb/frontends/mt312.c --- oldtree/drivers/media/dvb/frontends/mt312.c 2008-04-14 22:20:08.000000000 -0300 +++ linux/drivers/media/dvb/frontends/mt312.c 2008-04-14 22:20:05.000000000 -0300 @@ -58,7 +58,7 @@ static int debug; #define MT312_PLL_CLK 10000000UL /* 10 MHz */ static int mt312_read(struct mt312_state *state, const enum mt312_reg_addr reg, - void *buf, const size_t count) + u8 *buf, const size_t count) { int ret; struct i2c_msg msg[2]; @@ -84,7 +84,7 @@ static int mt312_read(struct mt312_state int i; dprintk("R(%d):", reg & 0x7f); for (i = 0; i < count; i++) - printk(" %02x", ((const u8 *) buf)[i]); + printk(" %02x", buf[i]); printk("\n"); } @@ -92,7 +92,7 @@ static int mt312_read(struct mt312_state } static int mt312_write(struct mt312_state *state, const enum mt312_reg_addr reg, - const void *src, const size_t count) + const u8 *src, const size_t count) { int ret; u8 buf[count + 1]; @@ -102,7 +102,7 @@ static int mt312_write(struct mt312_stat int i; dprintk("W(%d):", reg & 0x7f); for (i = 0; i < count; i++) - printk(" %02x", ((const u8 *) src)[i]); + printk(" %02x", src[i]); printk("\n"); } @@ -463,7 +463,7 @@ static int mt312_read_snr(struct dvb_fro int ret; u8 buf[2]; - ret = mt312_read(state, M_SNR_H, &buf, sizeof(buf)); + ret = mt312_read(state, M_SNR_H, buf, sizeof(buf)); if (ret < 0) return ret; @@ -478,7 +478,7 @@ static int mt312_read_ucblocks(struct dv int ret; u8 buf[2]; - ret = mt312_read(state, RS_UBC_H, &buf, sizeof(buf)); + ret = mt312_read(state, RS_UBC_H, buf, sizeof(buf)); if (ret < 0) return ret; | |