19 Oct, 2007
1 commit
-
Found these while looking at printk uses.
Add missing newlines to dev_ uses
Add missing KERN_ prefixes to multiline dev_s
Fixed a wierd->weird spelling typo
Added a newline to a printkSigned-off-by: Joe Perches
Cc: "Luck, Tony"
Cc: Jens Axboe
Cc: Mark M. Hoffman
Cc: Roland Dreier
Cc: Tilman Schmidt
Cc: David Woodhouse
Cc: Jeff Garzik
Cc: Stephen Hemminger
Cc: Greg KH
Cc: Jeremy Fitzhardinge
Cc: Geert Uytterhoeven
Cc: Alessandro Zummo
Cc: David Brownell
Cc: James Smart
Cc: Andrew Vasquez
Cc: "Antonino A. Daplas"
Cc: Evgeniy Polyakov
Cc: Russell King
Cc: Jaroslav Kysela
Cc: Takashi Iwai
Signed-off-by: Andrew Morton
Signed-off-by: Linus Torvalds
17 Oct, 2007
1 commit
-
Slab constructors currently have a flags parameter that is never used. And
the order of the arguments is opposite to other slab functions. The object
pointer is placed before the kmem_cache pointer.Convert
ctor(void *object, struct kmem_cache *s, unsigned long flags)
to
ctor(struct kmem_cache *s, void *object)
throughout the kernel
[akpm@linux-foundation.org: coupla fixes]
Signed-off-by: Christoph Lameter
Signed-off-by: Andrew Morton
Signed-off-by: Linus Torvalds
16 Oct, 2007
1 commit
-
Commit 90833fdab89da02fc0276224167f0a42e5176f41 ("[ARM] 4554/1: replace
consistent_sync() with flush_ioremap_region()") introduced a new
"flush_ioremap_region()" function to be used by the MTD mainstone-flash
and lubbock-flash drivers to fix a regression from around 2.6.18.Those drivers were independently merged into a single driver by Todd
Poynor in commit e644f7d6289456657996df4192de76c5d0a9f9c7 ("[MTD] MAPS:
Merge Lubbock and Mainstone drivers into common PXA2xx driver")Later, those two commits were merged into the main MTD tree by commit
b160292cc216a50fd0cd386b0bda2cd48352c73b ("Merge Linux 2.6.23") by David
Woodhouse, but in that merge, the fix to use flush_iomap_region() got
lost (as it was to files that now no longer existed).This reinstates the fix in the new driver.
Noticed-by: Russell King
Tested-and-acked-by: Nicolas Pitre
Cc: David Woodhouse
Cc: Jared Hulbert
Cc: Todd Poynor
Signed-off-by: Linus Torvalds
15 Oct, 2007
1 commit
-
... should be unsigned int
Signed-off-by: Al Viro
Signed-off-by: Linus Torvalds
14 Oct, 2007
16 commits
-
This allows the mtdconcat driver to work with NAND flash devices that
support sub-page writes.Signed-off-by: Chris Paulson-Ellis
Signed-off-by: David Woodhouse -
When the erase callback performs some other action on the flash, it's
highly likely to deadlock unless we actually release the chip lock
before calling it.This patch mirrors that same change already done for NAND.
Signed-off-by: Adrian Hunter
Signed-off-by: David Woodhouse -
The OneNAND driver was confusing JFFS2 by returning positive error
codes.Signed-off-by: Adrian Hunter
Acked-by: Kyungmin Park
Signed-off-by: David Woodhouse -
Ensure OneNAND's block locking operations are synchronized
like all other operations.Signed-off-by: Adrian Hunter
Acked-by: Kyungmin Park
Signed-off-by: David Woodhouse -
Fix the following warning:
drivers/mtd/ubi/eba.c: In function 'ubi_eba_init_scan':
drivers/mtd/ubi/eba.c:1116: warning: 'err' may be used uninitialized in this functionPointed-to-by: Andrew Morton
Signed-off-by: Artem Bityutskiy -
Signed-off-by: Artem Bityutskiy
-
When the UBI device is nearly full, i.e. all LEBs are mapped, we have
only one spare LEB left - the one we reserved for WL purposes. Well,
I do not count the LEBs which were reserved for bad PEB handling -
suppose NOR flash for simplicity. If an "atomic LEB change operation"
is run, and the WL unit is moving a LEB, we have no spare LEBs to
finish the operation and fail, which is not good. Moreover, if there
are 2 or more simultanious "atomic LEB change" requests, only one of
them has chances to succeed, the other will fail with -ENOSPC. Not
good either.This patch does 2 things:
1. Reserves one PEB for the "atomic LEB change" operation.
2. Serealize the operations so that only on of them may run
at a time (by means of a mutex).Pointed-to-by: Brijesh Singh
Signed-off-by: Artem Bityutskiy -
More handy since word hexdump prints in host endian.
Signed-off-by: Artem Bityutskiy
-
Similar reason as in case of the previous patch: it causes
deadlocks if a filesystem with writeback support works on top
of UBI. So pre-allocate needed buffers when attaching MTD device.
We also need mutexes to protect the buffers, but they do not
cause much contantion because they are used in recovery, torture,
and WL copy routines, which are called seldom.Signed-off-by: Artem Bityutskiy
-
Use GFP_NOFS flag when allocating memory on I/O path, because otherwise
we may deadlock the filesystem which works on top of us. We observed
the deadlocks with UBIFS. Example:VFS->FS lock a lock->UBI->kmalloc()->VFS writeback->FS locks the same
lock again.Signed-off-by: Artem Bityutskiy
-
Signed-off-by: Artem Bityutskiy
-
I can't find anything guaranteeing that 'ubi_num' cannot be
Signed-off-by: Artem Bityutskiy -
I hit those situations and found out lack of print messages. Add more prints
when erase problems occur.Signed-off-by: Artem Bityutskiy
-
Fix "symbol shadows an earlier one" warnings. Although they are harmless
but it does not hurt to fix them and make sparse happy.Signed-off-by: Artem Bityutskiy
-
Coverity (1769) found the following problem: if the erase counter
overflow check triggers, ec_hdr is leaked.Moving the allocation after the overflow check should take care of it.
Signed-off-by: Florin Malita
Signed-off-by: Artem Bityutskiy
13 Oct, 2007
6 commits
-
This is the driver for latest Blackfin on-chip nand flash controller
- use nand_chip and mtd_info common nand driver interface
- provide both PIO and dma operation
- compiled with ezkit bf548 configuration
- use hardware 1-bit ECC
- tested with YAFFS2 and can mount YAFFS2 filesystem as rootfsChangeLog from try#1
- use hweight32() instead of count_bits()
- replace bf54x with bf5xx and BF54X with BF5XX
- compare against plat->page_size in 2 cases when enable hardware ECCChangeLog from try#2
- passed nand_test suites
- use cpu_relax() instead of busy wait loop
- some coding style issue pointed out by AndrewSigned-off-by: Bryan Wu
Signed-off-by: Andrew Morton
Signed-off-by: David Woodhouse -
When we press ctrl-alt-del,kernel_restart_prepare will invoke
cfi_intelext_reboot which will set flash to read array mode, but later
when device_shutdown is invoked which may put current work queue to
sleep and other process may be scheduled to running and programming
flash in not FL_READY mode again. So we can't boot up if this flash is
used for bootloader.Signed-off-by: Andrew Morton
Signed-off-by: David Woodhouse -
drivers/mtd/nand/alauda.c: In function 'alauda_bounce_read':
drivers/mtd/nand/alauda.c:412: warning: comparison of distinct pointer types lacks a castSigned-off-by: Andrew Morton
Signed-off-by: David Woodhouse -
Signed-off-by: Kyungmin Park
Signed-off-by: David Woodhouse
07 Oct, 2007
4 commits
-
When the erase callback performs some other action on the flash, it's
highly likely to deadlock unless we actually release the chip lock
before calling it.Signed-off-by: David Woodhouse
-
Originally from Marcelo; modified to put the original timing registers
back instead of 0xFFFFFFFF.Signed-off-by: David Woodhouse
-
It was only the very early prototypes which made the mistake of using
the same device ident for all three functions on the device -- don't
bother trying to express that in the PCI match table, since the tools
don't cope. We can check in the probe routine instead, just in case.
Also remember to terminate the table.Signed-off-by: David Woodhouse
25 Sep, 2007
1 commit
-
Fixup the includes which have been moved around
when changing the s3c24xx arch support.Signed-off-by: Ben Dooks
Signed-off-by: David Woodhouse
24 Sep, 2007
4 commits
-
This patch make the OneNAND driver much less racy. It fixes
our "onenand_wait: read timeout!" heisenbugs. The reason of
these bugs was that the driver did not lock the chip when
accessing OTP, and it screwed up OneNAND state when the OTP
was read while JFFS2 was doing FS checking.This patch also fixes other races I spotted:
1. BBT was not protected
2. Access to ecc_stats was not protectedNow the chip is locked when BBT is accessed.
To fix all of these I basically split all interface functions
on 'function()' and 'function_nolock()' parts.I tested this patch on N800 hardware - it fixes our problems.
But I tested a little different version because our OneNAND
codebase is slightly out-of-date. But it should be OK.This patch also includes the prin fixes I posted before.
Signed-off-by: Artem Bityutskiy
Signed-off-by: David Woodhouse -
The Vermilion Range Expansion Bus supports four chip selects, each of which
has 64MiB of address space. The 2nd BAR of the Expansion Bus PCI Device
is a 256MiB memory region containing the address spaces for all four of
the chip selects, with start addresses hardcoded on 64MiB boundaries.This map driver only supports NOR flash on chip select 0. The buswidth
(either 8 bits or 16 bits) is determined by reading the Expansion Bus Timing
and Control Register for Chip Select 0 (EXP_TIMING_CS0).Signed-off-by: Andy Lowe
Signed-off-by: David Woodhouse -
The CFI probe routine is capable of detecting flash banks consisting of
identical chips mapped to physically discontiguous addresses. (One
common way this can occur is if a flash bank is populated with chips of
less capacity than the hardware was designed to support.) The CFI
point() routine currently ignores any such gaps. This patch fixes
the CFI point() routine so that it truncates any request that would
span a gap.Signed-off-by: Andy Lowe
Signed-off-by: Nicolas Pitre
Signed-off-by: David Woodhouse -
Replace Lubbock and Mainstone board drivers with common PXA2xx driver,
convert to platform driver (corresponding platform device changes merged
to kernel.org for 2.6.15), add power management callbacks.Signed-off-by: Todd Poynor
Signed-off-by: Nicolas Pitre
Signed-off-by: David Woodhouse
20 Sep, 2007
4 commits
-
This patch includes a whole batch of smallish cleanups for
drivers/mtd/physmap_of.c.- A bunch of uneeded #includes are removed
- We switch to the modern linux/of.h etc. in place of
asm/prom.h
- Use some helper macros to avoid some ugly inline #ifdefs
- A few lines of unreachable code are removed
- A number of indentation / line-wrapping fixes
- More consistent use of kernel idioms such as if (!p) instead
of if (p == NULL)
- Clarify some printk()s and other informative strings.
- parse_obsolete_partitions() now returns 0 if no partition
information is found, instead of returning -ENOENT which the caller
had to handle specially.
- (the big one) Despite the name, this driver really has
nothing to do with drivers/mtd/physmap.c. The fact that the flash
chips must be physically direct mapped is a constrant, but doesn't
really say anything about the actual purpose of this driver, which is
to instantiate MTD devices based on information from the device tree.
Therefore the physmap name is replaced everywhere within the file with
"of_flash". The file itself and the Kconfig option is not renamed for
now (so that the diff is actually a diff). That can come later.Signed-off-by: David Gibson
Signed-off-by: Josh Boyer -
The latest physmap_of driver has a small error where it will fail the probe
with:physmap-flash: probe of fff00000.small-flas failed with error -2
if there are no partition subnodes in the device tree and the old style binding
is not used. Since partition definitions are optional, the probe should still
succeed.Signed-off-by: Josh Boyer
Acked-by: David Gibson -
Fix a couple drivers that do not correctly terminate their pci_device_id
lists. This results in garbage being spewed into modules.pcimap when the
module happens to not have 28 NULL bytes following the table, and/or the
last PCI ID is actually truncated from the table when calculating the
modules.alias PCI aliases, cause those unfortunate device IDs to not
auto-load.Signed-off-by: Kees Cook
Acked-by: Corey Minyard
Cc: David Woodhouse
Acked-by: Jeff Garzik
Cc: Greg KH
Signed-off-by: Andrew Morton
Signed-off-by: Linus Torvalds
13 Sep, 2007
1 commit
-
This replaces the binding for flash chips in booting-without-of.txt
with an clarified and improved version. It also makes
drivers/mtd/maps/physmap_of.c recognize this new binding. Finally it
revises the Ebony device tree source to use the new binding as an
example.Signed-off-by: David Gibson
Acked-by: Segher Boessenkool
Signed-off-by: Paul Mackerras