From: "Jesper Juhl" <jesper.juhl@gmail.com>
To: "Adrian Bunk" <bunk@stusta.de>
Cc: "Alan Cox" <alan@lxorguk.ukuu.org.uk>,
jcm@jonmasters.org, "Jan Engelhardt" <jengelh@computergmbh.de>,
"Andy Whitcroft" <apw@shadowen.org>,
"Andrew Morton" <akpm@osdl.org>,
"Randy Dunlap" <rdunlap@xenotime.net>,
"Joel Schopp" <jschopp@austin.ibm.com>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] update checkpatch.pl to version 0.03
Date: Fri, 8 Jun 2007 16:34:01 +0200 [thread overview]
Message-ID: <9a8748490706080734h6ce5959fq6e7cb802d9a555cc@mail.gmail.com> (raw)
In-Reply-To: <20070608123942.GZ5500@stusta.de>
On 08/06/07, Adrian Bunk <bunk@stusta.de> wrote:
[snip]
>
> It's not only about MODULE_AUTHOR, if you consider it rude to limit
> people's names to ASCII, then don't forget that we have printk's like
> Linux agpgart interface v0.102 (c) Dave Jones
>
> What happens if the maintainer changes and it's now
> Linux agpgart interface v0.103 (c) Dave Ønes
>
> Does the console handle it correctly during boot?
> Can all tools that process the syslog cope with it?
>
> Perhaps the answer is in both cases "yes", but it's a completely
> untested area.
>
> We really must have all bugs shaken out and all users using fixed tools
> _before_ we can start outputting UTF-8 - limiting people's names to
> ASCII in not ideal, but IMHO causing breakages for users is a much
> bigger problem.
>
I haven't looked at it in depth yet, but it would seem we already have
a few files that need to be looked at with this in mind. Looks like
it's not exactely a new problem (although all the following could be
in comments of course)...
$ find ./ -name "*.[ch]" | xargs file | grep -i utf
./arch/arm/mach-pxa/leds-trizeps4.c: UTF-8
Unicode C program text
./arch/arm/mach-pxa/trizeps4.c: UTF-8
Unicode C program text
./arch/powerpc/platforms/cell/spufs/file.c: UTF-8
Unicode C program text
./drivers/acpi/asus_acpi.c: UTF-8
Unicode C program text
./drivers/char/drm/r128_drv.h: UTF-8
Unicode C program text
./drivers/char/drm/radeon_irq.c: UTF-8
Unicode C program text
./drivers/char/drm/drm_drawable.c: UTF-8
Unicode C program text
./drivers/char/drm/drm_pci.c: UTF-8
Unicode C program text
./drivers/char/drm/drm_core.h: UTF-8
Unicode C program text
./drivers/char/hw_random/omap-rng.c: UTF-8
Unicode C program text
./drivers/char/esp.c: UTF-8
Unicode C program text
./drivers/char/watchdog/iTCO_vendor_support.c: UTF-8
Unicode C program text
./drivers/i2c/busses/i2c-iop3xx.c: UTF-8
Unicode C program text
./drivers/infiniband/core/multicast.c: UTF-8
Unicode C program text
./drivers/infiniband/core/sa.h: UTF-8
Unicode C program text
./drivers/infiniband/core/sa_query.c: UTF-8
Unicode C program text
./drivers/mtd/chips/cfi_cmdset_0001.c: UTF-8
Unicode C program text
./drivers/mtd/chips/cfi_probe.c: UTF-8
Unicode C program text
./drivers/mtd/devices/block2mtd.c: UTF-8
Unicode C program text
./drivers/mtd/devices/phram.c: UTF-8
Unicode English text
./drivers/mtd/maps/cfi_flagadm.c: UTF-8
Unicode C program text
./drivers/mtd/maps/dbox2-flash.c: UTF-8
Unicode C program text
./drivers/mtd/maps/mtx-1_flash.c: UTF-8
Unicode C program text
./drivers/mtd/nand/ts7250.c: UTF-8
Unicode C program text
./drivers/mtd/nand/cafe_nand.c: UTF-8
Unicode C program text
./drivers/mtd/nand/cmx270_nand.c: UTF-8
Unicode C program text
./drivers/mtd/nand/cs553x_nand.c: UTF-8
Unicode C program text
./drivers/mtd/nand/edb7312.c: UTF-8
Unicode C program text
./drivers/mtd/nand/h1910.c: UTF-8
Unicode C program text
./drivers/mtd/mtdsuper.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/build.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/cdev.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/debug.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/debug.h: UTF-8
Unicode C program text
./drivers/mtd/ubi/gluebi.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/io.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/kapi.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/misc.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/scan.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/scan.h: UTF-8
Unicode C program text
./drivers/mtd/ubi/ubi.h: UTF-8
Unicode C program text
./drivers/mtd/ubi/upd.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/vmt.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/vtbl.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/wl.c: UTF-8
Unicode C program text
./drivers/mtd/ubi/eba.c: UTF-8
Unicode C program text
./drivers/net/irda/toim3232-sir.c: UTF-8
Unicode English text
./drivers/net/irda/kingsun-sir.c: UTF-8
Unicode Pascal program text
./drivers/net/atl1/atl1_hw.h: UTF-8
Unicode C program text
./drivers/scsi/atari_NCR5380.c:
UTF-8 Unicode C program text
./drivers/scsi/jazz_esp.c:
UTF-8 Unicode C program text
./drivers/usb/misc/iowarrior.c:
UTF-8 Unicode C program text
./drivers/usb/misc/auerswald.c:
UTF-8 Unicode C program text
./drivers/video/atafb_iplan2p2.c:
UTF-8 Unicode C program text
./drivers/video/atafb_iplan2p4.c:
UTF-8 Unicode C program text
./drivers/video/atafb_iplan2p8.c:
UTF-8 Unicode C program text
./fs/afs/afs_vl.h:
UTF-8 Unicode C program text
./fs/jffs2/acl.c:
UTF-8 Unicode C program text
./fs/jffs2/acl.h:
UTF-8 Unicode C program text
./fs/jffs2/background.c:
UTF-8 Unicode C program text
./fs/jffs2/build.c:
UTF-8 Unicode C program text
./fs/jffs2/compr.c:
UTF-8 Unicode C program text
./fs/jffs2/compr.h:
UTF-8 Unicode C program text
./fs/jffs2/compr_rtime.c:
UTF-8 Unicode C program text
./fs/jffs2/compr_rubin.c:
UTF-8 Unicode C program text
./fs/jffs2/compr_zlib.c:
UTF-8 Unicode C program text
./fs/jffs2/debug.c:
UTF-8 Unicode C program text
./fs/jffs2/debug.h:
UTF-8 Unicode C program text
./fs/jffs2/dir.c:
UTF-8 Unicode C program text
./fs/jffs2/erase.c:
UTF-8 Unicode C program text
./fs/jffs2/file.c:
UTF-8 Unicode C program text
./fs/jffs2/fs.c:
UTF-8 Unicode C program text
./fs/jffs2/gc.c:
UTF-8 Unicode C program text
./fs/jffs2/ioctl.c:
UTF-8 Unicode C program text
./fs/jffs2/jffs2_fs_i.h:
UTF-8 Unicode C program text
./fs/jffs2/malloc.c:
UTF-8 Unicode C program text
./fs/jffs2/nodelist.c:
UTF-8 Unicode C program text
./fs/jffs2/nodelist.h:
UTF-8 Unicode C program text
./fs/jffs2/os-linux.h:
UTF-8 Unicode C program text
./fs/jffs2/read.c:
UTF-8 Unicode C program text
./fs/jffs2/security.c:
UTF-8 Unicode C program text
./fs/jffs2/summary.c:
UTF-8 Unicode C program text
./fs/jffs2/summary.h:
UTF-8 Unicode C program text
./fs/jffs2/symlink.c:
UTF-8 Unicode C program text
./fs/jffs2/wbuf.c:
UTF-8 Unicode C program text
./fs/jffs2/write.c:
UTF-8 Unicode C program text
./fs/jffs2/xattr.h:
UTF-8 Unicode C program text
./fs/jffs2/xattr_trusted.c:
UTF-8 Unicode C program text
./fs/jffs2/xattr_user.c:
UTF-8 Unicode C program text
./fs/jffs2/readinode.c:
UTF-8 Unicode C program text
./fs/jffs2/super.c:
UTF-8 Unicode C program text
./fs/jffs2/jffs2_fs_sb.h:
UTF-8 Unicode C program text
./fs/jffs2/nodemgmt.c:
UTF-8 Unicode C program text
./fs/jffs2/scan.c:
UTF-8 Unicode C program text
./fs/jffs2/writev.c:
UTF-8 Unicode C program text
./fs/jffs2/xattr.c:
UTF-8 Unicode C program text
./fs/nls/nls_utf8.c:
ASCII C program text
./include/asm-arm/arch-aaec2000/aaec2000.h:
UTF-8 Unicode C program text
./include/asm-arm/arch-integrator/platform.h:
UTF-8 Unicode C program text
./include/asm-arm/arch-omap/board.h:
UTF-8 Unicode C program text
./include/asm-arm/arch-omap/dma.h:
UTF-8 Unicode C program text
./include/asm-arm/arch-omap/gpio.h:
UTF-8 Unicode C program text
./include/asm-arm/arch-pxa/trizeps4.h:
UTF-8 Unicode C program text
./include/asm-m68k/atariints.h: UTF-8
Unicode C program text
./include/asm-m68k/atarihw.h: UTF-8
Unicode C program text
./include/asm-ppc/hydra.h: UTF-8
Unicode C program text
./include/linux/i2c-algo-bit.h: UTF-8
Unicode C program text
./include/linux/i2c-algo-pcf.h: UTF-8
Unicode C program text
./include/linux/i2c.h: UTF-8
Unicode C program text
./include/linux/irda.h: UTF-8
Unicode Pascal program text
./include/linux/meye.h: UTF-8
Unicode C program text
./include/linux/mtd/super.h: UTF-8
Unicode C program text
./include/linux/mtd/mtd.h: UTF-8
Unicode C program text
./include/linux/mtd/ubi.h: UTF-8
Unicode C program text
./include/linux/sonypi.h: UTF-8
Unicode C program text
./include/mtd/ubi-header.h: UTF-8
Unicode C program text
./include/mtd/ubi-user.h: UTF-8
Unicode C program text
./include/net/irda/irda.h: UTF-8
Unicode Pascal program text
./include/net/irda/iriap.h: UTF-8
Unicode Pascal program text
./include/net/irda/iriap_event.h: UTF-8
Unicode Pascal program text
./include/net/irda/irias_object.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlan_client.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlan_common.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlan_eth.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlan_event.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlan_filter.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlan_provider.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlap.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlmp.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlmp_event.h: UTF-8
Unicode Pascal program text
./include/net/irda/irlmp_frame.h: UTF-8
Unicode Pascal program text
./include/net/irda/irmod.h: UTF-8
Unicode Pascal program text
./include/net/irda/irqueue.h: UTF-8
Unicode English text
./include/net/irda/irttp.h: UTF-8
Unicode Pascal program text
./include/net/irda/parameters.h: UTF-8
Unicode Pascal program text
./include/net/irda/timer.h: UTF-8
Unicode Pascal program text
./include/net/irda/wrapper.h: UTF-8
Unicode Pascal program text
./include/net/irda/af_irda.h: UTF-8
Unicode Pascal program text
./kernel/sys.c: UTF-8 Unicode
C program text
./sound/drivers/mts64.c: UTF-8 Unicode
C program text
./sound/oss/es1371.c: UTF-8 Unicode
C program text
./sound/oss/pas2_pcm.c: UTF-8 Unicode
C program text
./sound/oss/trident.c: UTF-8 Unicode
C program text
./sound/pci/ice1712/prodigy192.c: UTF-8 Unicode
C program text
./sound/pci/mixart/mixart.c: UTF-8 Unicode
C program text
--
Jesper Juhl <jesper.juhl@gmail.com>
Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please http://www.expita.com/nomime.html
next prev parent reply other threads:[~2007-06-08 14:34 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-06-04 9:46 Andy Whitcroft
2007-06-04 9:55 ` [PATCH] update feature-removal-schedule.txt to include deprecated functions Andy Whitcroft
2007-06-04 15:49 ` [PATCH] update checkpatch.pl to version 0.03 jschopp
2007-06-04 16:51 ` Andy Whitcroft
2007-06-04 17:22 ` jschopp
2007-06-05 18:45 ` Andy Whitcroft
2007-06-05 9:56 ` Andy Whitcroft
2007-06-04 16:25 ` Jan Engelhardt
2007-06-04 18:41 ` Andrew Morton
2007-06-04 19:08 ` Andy Whitcroft
2007-06-04 19:08 ` Rene Herman
2007-06-04 20:04 ` Randy Dunlap
2007-06-05 18:39 ` Andy Whitcroft
2007-06-08 9:31 ` Andy Whitcroft
2007-06-08 10:08 ` Rene Herman
2007-06-05 8:14 ` Heiko Carstens
2007-06-06 9:05 ` Jesper Juhl
2007-06-07 14:28 ` Jan Engelhardt
2007-06-07 14:39 ` Jesper Juhl
2007-06-07 19:34 ` Adrian Bunk
2007-06-07 22:22 ` Alan Cox
2007-06-07 23:21 ` Adrian Bunk
2007-06-07 23:41 ` Alan Cox
2007-06-08 0:04 ` Adrian Bunk
2007-06-08 4:37 ` Jon Masters
2007-06-08 8:58 ` Jan-Benedict Glaw
2007-06-08 10:52 ` Alan Cox
2007-06-08 12:39 ` Adrian Bunk
2007-06-08 14:34 ` Jesper Juhl [this message]
2007-06-08 14:42 ` Adrian Bunk
2007-06-08 15:16 ` Jan Engelhardt
2007-06-08 15:37 ` Jon Masters
2007-06-08 15:42 ` Alan Cox
2007-06-08 16:39 ` Adrian Bunk
2007-06-08 18:43 ` Jan Engelhardt
2007-06-08 16:03 ` Roland Dreier
2007-06-07 23:49 ` Adrian Bunk
2007-06-07 19:32 ` Adrian Bunk
2007-06-07 22:18 ` Alan Cox
2007-06-06 11:49 ` Jesper Juhl
2007-06-07 11:46 ` Andy Whitcroft
2007-06-07 11:52 ` Jesper Juhl
2007-06-07 15:16 ` checkpatch.pl: should be executable Andy Whitcroft
2007-06-07 15:33 ` jschopp
2007-06-07 14:22 ` [PATCH] update checkpatch.pl to version 0.03 Jan Engelhardt
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=9a8748490706080734h6ce5959fq6e7cb802d9a555cc@mail.gmail.com \
--to=jesper.juhl@gmail.com \
--cc=akpm@osdl.org \
--cc=alan@lxorguk.ukuu.org.uk \
--cc=apw@shadowen.org \
--cc=bunk@stusta.de \
--cc=jcm@jonmasters.org \
--cc=jengelh@computergmbh.de \
--cc=jschopp@austin.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=rdunlap@xenotime.net \
/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
all inboxes | Powered by JetHome®