From: "Justin P. Mattock" <justinmattock@gmail.com>
To: Randy Dunlap <rdunlap@xenotime.net>
Cc: trivial@kernel.org, linux-kernel@vger.kernel.org,
pebolle@tiscali.nl, mad_soft@inbox.ru
Subject: Re: [RFC V5]Trivial: Fix typos in various parts of the kernel, and fix some comments.
Date: Mon, 28 Nov 2011 20:32:27 -0800 [thread overview]
Message-ID: <4ED4605B.8040306@gmail.com> (raw)
In-Reply-To: <4ED3D95E.2040702@xenotime.net>
On 11/28/2011 10:56 AM, Randy Dunlap wrote:
> On 11/25/2011 09:18 AM, Justin P. Mattock wrote:
>> From: "Justin P. Mattock"<justinmattock@gmail.com>
>>
>> Note:resend after some changes rev5.
>> The below patch fixes some typos in various parts of the kernel, as well as fixes some comments.
>> Please let me know if I missed anything, and I will try to get it changed and resent.
>
> Mostly looks good to me, with a few exceptions noted below.
>
>> Signed-off-by: Justin P. Mattock<justinmattock@gmail.com>
>>
>> ---
>> arch/arm/plat-omap/include/plat/serial.h | 2 +-
>> arch/powerpc/include/asm/io.h | 2 +-
>> arch/powerpc/include/asm/keylargo.h | 2 +-
>> arch/powerpc/mm/numa.c | 2 +-
>> arch/sparc/kernel/smp_64.c | 2 +-
>> drivers/acpi/acpica/hwxface.c | 2 +-
>> drivers/block/xen-blkback/xenbus.c | 2 +-
>> drivers/char/ipmi/ipmi_bt_sm.c | 2 +-
>> drivers/edac/ppc4xx_edac.c | 2 +-
>> drivers/media/video/zoran/zoran_driver.c | 2 +-
>> drivers/message/fusion/lsi/mpi_ioc.h | 2 +-
>> drivers/net/irda/nsc-ircc.c | 2 +-
>> drivers/net/irda/via-ircc.c | 4 ++--
>> drivers/net/irda/w83977af_ir.c | 2 +-
>> drivers/net/wimax/i2400m/i2400m.h | 2 +-
>> drivers/net/wireless/rtlwifi/rtl8192de/hw.c | 4 ++--
>> drivers/parport/parport_mfc3.c | 2 +-
>> drivers/scsi/aic7xxx/aicasm/aicasm.c | 2 +-
>> drivers/scsi/ips.c | 2 +-
>> drivers/scsi/qla4xxx/ql4_fw.h | 2 +-
>> drivers/scsi/vmw_pvscsi.c | 2 +-
>> .../comedi/drivers/addi-data/hwdrv_apci3200.c | 2 +-
>> drivers/staging/cxt1e1/libsbew.h | 2 +-
>> drivers/staging/ft1000/ft1000-pcmcia/ft1000_hw.c | 4 ++--
>> drivers/staging/ft1000/ft1000-usb/ft1000_hw.c | 2 +-
>> drivers/staging/iio/industrialio-trigger.c | 2 +-
>> drivers/staging/sep/sep_driver.c | 2 +-
>> drivers/staging/tidspbridge/Kconfig | 2 +-
>> drivers/staging/tidspbridge/rmgr/dbdcd.c | 2 +-
>> drivers/usb/host/hwa-hc.c | 2 +-
>> drivers/usb/host/imx21-hcd.c | 2 +-
>> drivers/usb/otg/fsl_otg.c | 2 +-
>> drivers/uwb/i1480/dfu/usb.c | 2 +-
>> fs/btrfs/inode.c | 2 +-
>> fs/ext3/inode.c | 2 +-
>> fs/ext4/inode.c | 2 +-
>> fs/nfsd/nfs4state.c | 2 +-
>> fs/ocfs2/file.c | 2 +-
>> fs/xfs/xfs_file.c | 6 +++---
>> fs/xfs/xfs_log_cil.c | 2 +-
>> include/drm/drmP.h | 2 +-
>> include/linux/wanrouter.h | 2 +-
>> include/net/mac80211.h | 2 +-
>> net/ipv4/ip_fragment.c | 2 +-
>> net/mac80211/work.c | 2 +-
>> net/sctp/endpointola.c | 2 +-
>> 46 files changed, 51 insertions(+), 51 deletions(-)
>
>
>> diff --git a/arch/powerpc/include/asm/io.h b/arch/powerpc/include/asm/io.h
>> index 45698d5..2fd3645 100644
>> --- a/arch/powerpc/include/asm/io.h
>> +++ b/arch/powerpc/include/asm/io.h
>> @@ -394,7 +394,7 @@ __do_out_asm(_rec_outl, "stwbrx")
>> #endif /* CONFIG_PPC32 */
>>
>> /* The "__do_*" operations below provide the actual "base" implementation
>> - * for each of the defined acccessor. Some of them use the out_* functions
>> + * for each of the defined accessor. Some of them use the out_* functions
>
> accessors.
>
>> * directly, some of them still use EEH, though we might change that in the
>> * future. Those macros below provide the necessary argument swapping and
>> * handling of the IO base for PIO.
>> diff --git a/drivers/media/video/zoran/zoran_driver.c b/drivers/media/video/zoran/zoran_driver.c
>> index d4d05d2..c624403 100644
>> --- a/drivers/media/video/zoran/zoran_driver.c
>> +++ b/drivers/media/video/zoran/zoran_driver.c
>> @@ -1550,7 +1550,7 @@ static int zoran_enum_fmt(struct zoran *zr, struct v4l2_fmtdesc *fmt, int flag)
>> if (zoran_formats[i].flags& flag&& num++ == fmt->index) {
>> strncpy(fmt->description, zoran_formats[i].name,
>> sizeof(fmt->description) - 1);
>> - /* fmt struct pre-zeroed, so adding '\0' not neeed */
>> + /* fmt struct pre-zeroed, so adding '\0' not need */
>
> not needed */
>
>> fmt->pixelformat = zoran_formats[i].fourcc;
>> if (zoran_formats[i].flags& ZORAN_FORMAT_COMPRESSED)
>> fmt->flags |= V4L2_FMT_FLAG_COMPRESSED;
>> diff --git a/drivers/usb/host/imx21-hcd.c b/drivers/usb/host/imx21-hcd.c
>> index 2ee18cf..8936bfa 100644
>> --- a/drivers/usb/host/imx21-hcd.c
>> +++ b/drivers/usb/host/imx21-hcd.c
>> @@ -473,7 +473,7 @@ static void free_epdmem(struct imx21 *imx21, struct usb_host_endpoint *ep)
>> /* End handling */
>> /* =========================================== */
>>
>> -/* Endpoint now idle - release it's ETD(s) or asssign to queued request */
>> +/* Endpoint now idle - release it's ETD(s) or assign to queued request */
>
> its
>
>> static void ep_idle(struct imx21 *imx21, struct ep_priv *ep_priv)
>> {
>> int i;
>
>
> thanks.
>
Cool Thanks for having a look! I went and did the changes, and resent
if I missed any let me know.
Thanks!
Justin P. Mattock
prev parent reply other threads:[~2011-11-29 4:32 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-25 17:18 [RFC V5]Trivial: Fix typos in various parts of the kernel, and fix some comments Justin P. Mattock
2011-11-28 18:56 ` Randy Dunlap
2011-11-29 4:32 ` Justin P. Mattock [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4ED4605B.8040306@gmail.com \
--to=justinmattock@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mad_soft@inbox.ru \
--cc=pebolle@tiscali.nl \
--cc=rdunlap@xenotime.net \
--cc=trivial@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox