[PATCH trivial] lpfc: Grammar s/an negative/a negative/

2015-05-21 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven --- drivers/scsi/lpfc/lpfc_debugfs.c | 12 ++-- 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/drivers/scsi/lpfc/lpfc_debugfs.c b/drivers/scsi/lpfc/lpfc_debugfs.c index 513edcb0c2dae379..25aa9b98d53aa345 100644 --- a/drivers/scsi/lpfc

Re: Patch "sd: Disable support for 256 byte/sector disks" has been added to the 4.0-stable tree

2015-06-03 Thread Geert Uytterhoeven
12) { >> > > - /* only legitimate sector_size here is 256 */ >> > > - start_lba <<= 1; >> > > - end_lba <<= 1; >> > > - } else { >> > > - /* be careful ... don't want any overflows */ >

[PATCH 1/2] advansys: Do not byteswap sense_len on big endian

2015-06-24 Thread Geert Uytterhoeven
("advansys: use DMA-API for mapping sense buffer") Signed-off-by: Geert Uytterhoeven --- drivers/scsi/advansys.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/advansys.c b/drivers/scsi/advansys.c index 4305178e4e0147ba..1c1cd657c380c2e0 100644 --- a/dr

[PATCH 2/2] advansys: Make sure ret and share_irq are initialized

2015-06-24 Thread Geert Uytterhoeven
share_irq are indeed not initialized. Pre-initialize them to zero, and drop the now superfluous setting of share_irq to zero. Note that this is sort-of a false positive, as apparently ASC_NARROW_BOARD(boardp) can only be false for PCI boards, e.g. if CONFIG_PCI=y. Signed-off-by: Geert

Re: [PATCH 07/29] ncr5380: Cleanup TAG_NEXT and TAG_NONE macros

2014-10-03 Thread Geert Uytterhoeven
ces. > -*/ > +#define TAG_NONE 0xff > > /* > * These are "special" values for the irq and dma_channel fields of the > @@ -323,7 +320,7 @@ static irqreturn_t NCR5380_intr(int irq, Gr{oetje,eeting}s, Geert -- Ge

Re: [PATCH 10/29] ncr5380: Fix SCSI_IRQ_NONE bugs

2014-10-03 Thread Geert Uytterhoeven
t some drivers do #ifndef NO_IRQ #define NO_IRQ (-1) #endif and others do #ifndef NO_IRQ #define NO_IRQ 0 #endif Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technica

Re: [PATCH 12/29] ncr5380: Cleanup host info() methods

2014-10-03 Thread Geert Uytterhoeven
""); > + hostdata->info[sizeof(hostdata->info) - 1] = '\0'; snprintf() will make sure the string is zero-terminated, so doing this manually is not needed. This applies to all three occurrences. > } Gr{oetje,eeting}s, Geert -- Geert

Re: [PATCH 17/29] dmx3191d: Use IRQ_NONE

2014-10-03 Thread Geert Uytterhoeven
hich is related to irqreturn_t). Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journa

Re: [PATCH 19/29] mac_scsi: Add module option to Kconfig

2014-10-03 Thread Geert Uytterhoeven
IA1_BASE + 0x1; > + mac_scsi_drq = (unsigned char *) VIA1_BASE + 0x6000; > + mac_scsi_nodrq = (unsigned char *) VIA1_BASE + 0x12000; > } > > if (! setup_use_pdma) I don't think the above chunk belongs in this patch. Gr{oetje,eeting}s,

Re: [PATCH 21/29] mac_scsi: Convert to platform device

2014-10-03 Thread Geert Uytterhoeven
* not make use of its DMA or hardware handshaking logic. > +*/ > + mac_scsi_0_pdev.num_resources--; > + platform_device_register(&mac_scsi_0_pdev); What about removing the static mac_scsi_0_pdev and mac_scsi_1_pdev, and calling platform_device

Re: [PATCH 22/29] atari_scsi: Fix atari_scsi deadlocks on Falcon

2014-10-03 Thread Geert Uytterhoeven
local_irq_restore(flags); return res; } Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when

Re: [PATCH 23/29] atari_scsi: Convert to platform device

2014-10-03 Thread Geert Uytterhoeven
*/ > + if (MACH_IS_MEDUSA) > + atari_read_overruns = 4; > +#endif > + } else { > + /* Nothing to do for the interrupt: the ST-DMA is initialized > +* already. > +*/ > +#ifdef RE

Re: [PATCH 19/29] mac_scsi: Add module option to Kconfig

2014-10-03 Thread Geert Uytterhoeven
On Fri, Oct 3, 2014 at 12:49 PM, Finn Thain wrote: > On Fri, 3 Oct 2014, Geert Uytterhoeven wrote: >> On Thu, Oct 2, 2014 at 8:56 AM, Finn Thain >> wrote: >> > Allow mac_scsi to be built as a module. Replace the old validation of >> > __setup options with co

Re: [PATCH 23/29] atari_scsi: Convert to platform device

2014-10-06 Thread Geert Uytterhoeven
gt; > would like to hear Michael's views. >> > >> >> The IRQ is a good candidate to be passed via platform data. > > Geert didn't say so, but after thinking about his review comments I > imagine that he wants all the Atari IRQ numbers kept in one place

Re: [PATCH v4 22/23] mac_scsi: Fix pseudo DMA implementation

2016-05-19 Thread Geert Uytterhoeven
, HZ / 64)) { > + CP_IO_TO_MEM(s, d, n); This is now before the inclusion of NCR5380.c, causing In file included from drivers/scsi/mac_scsi.c:335: drivers/scsi/NCR5380.h:295: warning: ‘NCR5380_poll_politely’ declared inline after being called drivers/scsi/NCR5380.h:295: warning

Re: [PATCH v4 22/23] mac_scsi: Fix pseudo DMA implementation

2016-05-19 Thread Geert Uytterhoeven
Hi Finn, On Thu, May 19, 2016 at 2:02 PM, Finn Thain wrote: > On Thu, 19 May 2016, Geert Uytterhoeven wrote: >> In file included from drivers/scsi/mac_scsi.c:335: >> drivers/scsi/NCR5380.h:295: warning: `NCR5380_poll_politely' declared inline >> after being called >

[PATCH 42/54] MAINTAINERS: Add file patterns for scsi device tree bindings

2016-05-22 Thread Geert Uytterhoeven
Submitters of device tree binding documentation may forget to CC the subsystem maintainer if this is missing. Signed-off-by: Geert Uytterhoeven Cc: James E.J. Bottomley Cc: Martin K. Petersen Cc: linux-scsi@vger.kernel.org --- Please apply this patch directly if you want to be involved in

[PATCH RESEND] scsi/trivial: Spelling hsot -> host

2013-05-17 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven --- drivers/scsi/aic7xxx_old/aic7xxx.seq |2 +- drivers/scsi/scsi_error.c|2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/aic7xxx_old/aic7xxx.seq b/drivers/scsi/aic7xxx_old/aic7xxx.seq index 823ff28..dc3bb81

Re: btrfs zero divide

2013-08-14 Thread Geert Uytterhoeven
On Tue, Aug 13, 2013 at 6:32 PM, Geert Uytterhoeven wrote: > On Fri, 9 Aug 2013, Zach Brown wrote: >> On Fri, Aug 09, 2013 at 02:26:36PM +0200, Andreas Schwab wrote: >> > Josef Bacik writes: >> > >> > > So stripe_len shouldn't be 0, if it is you h

Re: btrfs zero divide

2013-08-14 Thread Geert Uytterhoeven
On Wed, Aug 14, 2013 at 10:40 AM, Geert Uytterhoeven wrote: > On Tue, Aug 13, 2013 at 6:32 PM, Geert Uytterhoeven > wrote: >> On Fri, 9 Aug 2013, Zach Brown wrote: >>> On Fri, Aug 09, 2013 at 02:26:36PM +0200, Andreas Schwab wrote: >>> > Josef Bacik writes: >

Re: Drivers: scsi: FLUSH timeout

2013-09-25 Thread Geert Uytterhoeven
he value > we currently have for I/O > timeout. So you want to replace it by "180 * HZ", which is ... another magic number? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal co

Re: scsi: Implement per-cpu logging buffer

2015-02-12 Thread Geert Uytterhoeven
se_hdr - 774+774 proc_keys_show - 770+770 Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technica

Re: scsi: Implement per-cpu logging buffer

2015-02-14 Thread Geert Uytterhoeven
Hi Hannes, On Sat, Feb 14, 2015 at 3:29 PM, Hannes Reinecke wrote: > @jejb, hch: should I do a new patch or update the existing one? As the existing one is already upstream, you should send a new patch. Thanks! Gr{oetje,eeting}s, Geert -- Geert Uytterhoe

[PATCH] target: initialize sense_reason_t ret in core_scsi3_emulate_pro_register()

2012-12-22 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven --- drivers/target/target_core_pr.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/target/target_core_pr.c b/drivers/target/target_core_pr.c index e35dbf8..c2e8026 100644 --- a/drivers/target/target_core_pr.c +++ b/driv

Re: [PATCH 2/2] [SCSI] a3000: use module_platform_driver_probe()

2013-03-05 Thread Geert Uytterhoeven
_unregister(&amiga_a3000_scsi_driver); >> -} >> -module_exit(amiga_a3000_scsi_exit); >> +module_platform_driver_probe(amiga_a3000_scsi_driver, >> amiga_a3000_scsi_probe); >> >> MODULE_DESCRIPTION("Amiga 3000 built-in SCSI"); >> MODULE_LICENSE(&q

[PATCH -next 2/2] sun3_scsi: switch to ->show_info()

2013-04-10 Thread Geert Uytterhoeven
Based on Al's changes to atari_scsi. Signed-off-by: Geert Uytterhoeven --- http://kisskb.ellerman.id.au/kisskb/buildresult/8543326/ drivers/scsi/sun3_NCR5380.c | 185 ++ drivers/scsi/sun3_scsi.c|2 +- drivers/scsi/sun3_scsi.h|2

[PATCH 1/2] sun3_scsi: Fill in missing scsi_host_template.proc_info method

2013-04-10 Thread Geert Uytterhoeven
Signed-off-by: Geert Uytterhoeven --- drivers/scsi/sun3_scsi.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/scsi/sun3_scsi.c b/drivers/scsi/sun3_scsi.c index 6e25889..b0dc034 100644 --- a/drivers/scsi/sun3_scsi.c +++ b/drivers/scsi/sun3_scsi.c @@ -626,6 +626,7

Re: [PATCH -next 2/2] sun3_scsi: switch to ->show_info()

2013-05-03 Thread Geert Uytterhoeven
Ping? Now the build failure is also in Linus' tree: http://kisskb.ellerman.id.au/kisskb/buildresult/8674437/ BTW, this patch depends on "[PATCH 1/2] sun3_scsi: Fill in missing scsi_host_template.proc_info method" On Wed, Apr 10, 2013 at 1:52 PM, Geert Uytterhoeven wrote:

Re: [RFC-v4 7/9] iscsi-target: Refactor TX queue logic + export response PDU creation

2013-05-03 Thread Geert Uytterhoeven
, 0, NULL, (u8 *)header_digest); and pr_debug("Built Reject PDU StatSN: 0x%08x, Reason: 0x%02x," " CID: %hu\n", ntohl(hdr->statsn), hdr->reason, conn->cid); Gr{oetje,eeting}s, Geert --

[PATCH] iscsi-target: Make buf param of iscsit_do_crypto_hash_buf() const void *

2013-05-03 Thread Geert Uytterhoeven
Make the "buf" input param of iscsit_do_crypto_hash_buf() "const void *". This allows to remove lots of casts in its callers. Signed-off-by: Geert Uytterhoeven --- drivers/target/iscsi/iscsi_target.c | 57 ++- 1 files changed, 23 inserti

Re: [PATCH] iscsi-target: Make buf param of iscsit_do_crypto_hash_buf() const void *

2013-05-03 Thread Geert Uytterhoeven
On Fri, May 3, 2013 at 11:15 PM, Geert Uytterhoeven wrote: > --- a/drivers/target/iscsi/iscsi_target.c > +++ b/drivers/target/iscsi/iscsi_target.c > @@ -3585,9 +3575,8 @@ static int iscsit_send_reject( > } > > if (conn->

Re: [PATCH] iscsi-target: Make buf param of iscsit_do_crypto_hash_buf() const void *

2013-05-05 Thread Geert Uytterhoeven
use u8 data_crc[ISCSI_DIGEST_SIZE] and > pad_bytes[ISCSI_PAD_LEN] now, and will post a patch soon. That's another good idea. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In per

[PATCH 02/12] [SCSI] SCSI_UFSHCD should depend on SCSI_DMA

2013-05-09 Thread Geert Uytterhoeven
fs/ufshcd.c:1498: undefined reference to `scsi_dma_unmap' drivers/built-in.o: In function `ufshcd_device_reset': drivers/scsi/ufs/ufshcd.c:1436: undefined reference to `scsi_dma_unmap' Signed-off-by: Geert Uytterhoeven Cc: Vinayak Holikatti Cc: James E.J. Bottomley Cc: linux-sc

Re: [PATCH RESEND 2/2] [SCSI] a3000: use module_platform_driver_probe()

2013-05-13 Thread Geert Uytterhoeven
On Thu, May 2, 2013 at 8:30 AM, Jingoo Han wrote: > This patch uses module_platform_driver_probe() macro which makes > the code smaller and simpler. > > Acked-by: Geert Uytterhoeven > Signed-off-by: Jingoo Han James, will you take this one? Or shall I queue it in the m68k tree?

Re: [PATCH 22/71] ncr5380: Eliminate selecting state

2015-11-29 Thread Geert Uytterhoeven
jiffies, end); And a similar loop for "Busy-wait for up to 20 ms". Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalis

Re: [PATCH 22/71] ncr5380: Eliminate selecting state

2015-11-29 Thread Geert Uytterhoeven
Hi Finn, On Sun, Nov 29, 2015 at 11:25 AM, Finn Thain wrote: > On Sun, 29 Nov 2015, Geert Uytterhoeven wrote: >> This still heavily depends on the processing time spent in >> NCR5380_read(). You should never use a value derived from >> loops_per_jiffy for a non-empty loop,

Re: [PATCH 73/71] ncr5380: Use runtime register mapping

2015-12-06 Thread Geert Uytterhoeven
hould appear on the next line where it is more visible. And adding a cpu_relax() to the empty loop body is another good visual cue. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations

Re: [PATCH v2 21/72] ncr5380: Sleep when polling, if possible

2015-12-06 Thread Geert Uytterhoeven
f you store the number of accesses per jiffy instead of per ms. Unlike the historical calibrating-delay-loop code, you don't wait for a jiffy change before starting the calibration. At first I thought that was OK, but on some platforms, HZ can be as low as 24, which means the result can vary by

Re: [PATCH v4 34/78] atari_NCR5380: Use arbitration timeout

2016-01-24 Thread Geert Uytterhoeven
shost_printk(KERN_ERR, instance, @@ -1297,10 +1293,6 @@ static struct scsi_cmnd *NCR5380_select(struct Scsi_Host *instance, spin_lock_irq(&hostdata->lock); - /* NCR5380_reselect() clears MODE_REG after a reselection interrupt */ - if (!(NCR5380_read(MODE_REG) &a

Re: [PATCH v4 34/78] atari_NCR5380: Use arbitration timeout

2016-01-25 Thread Geert Uytterhoeven
Hi Finn, On Mon, Jan 25, 2016 at 3:45 AM, Finn Thain wrote: > On Sun, 24 Jan 2016, Geert Uytterhoeven wrote: >> On Sun, Jan 3, 2016 at 6:05 AM, Finn Thain >> wrote: >> > Allow target selection to fail with a timeout instead of waiting in >> > infinite l

Re: [PATCH v4 34/78] atari_NCR5380: Use arbitration timeout

2016-01-26 Thread Geert Uytterhoeven
Hi Finn, On Tue, Jan 26, 2016 at 1:18 AM, Finn Thain wrote: > On Mon, 25 Jan 2016, Geert Uytterhoeven wrote: >> > In principle I think that Linux drivers should not carry workarounds >> > for emulators. >> >> Please consider ARAnyM is the current m68k workhorse,

Re: Build regressions/improvements in v4.3

2015-11-02 Thread Geert Uytterhoeven
On Mon, Nov 2, 2015 at 12:26 PM, Geert Uytterhoeven wrote: > JFYI, when comparing v4.3[1] to v4.3-rc7[3], the summaries are: > - build errors: +4/-7 + /home/kisskb/slave/src/drivers/scsi/advansys.c: error: implicit declaration of function 'free_dma' [-Werror=implicit-func

[PATCH] [SCSI] st: Remove obsolete scsi_tape.max_pfn

2015-11-15 Thread Geert Uytterhoeven
Its last user was removed 10 years ago, in commit 8b05b773b6030de5 ("[SCSI] convert st to use scsi_execute_async"). Signed-off-by: Geert Uytterhoeven --- drivers/scsi/st.h | 2 -- 1 file changed, 2 deletions(-) diff --git a/drivers/scsi/st.h b/drivers/scsi/st.h index b6486b

Re: [PATCH, resend] scsi: advansys: fix big-endian builds

2015-11-16 Thread Geert Uytterhoeven
warning on all big endian platforms. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists

Re: [PATCH 00/71] More fixes, cleanup and modernization for NCR5380 drivers

2015-11-20 Thread Geert Uytterhoeven
oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that.

Re: [2.6 patch] 53c7xx removal fallout

2007-10-25 Thread Geert Uytterhoeven
On Wed, 24 Oct 2007, Adrian Bunk wrote: > This patch does some additional cleanups after the 53c7xx removal. > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> Acked-by: Geert Uytterhoeven <[EMAIL PROTECTED]> > --- > > Documentation/dontdiff|2

Re: [2.6 patch] cleanup after APUS removal

2007-10-25 Thread Geert Uytterhoeven
On Wed, 24 Oct 2007, Adrian Bunk wrote: > After the APUS removal, some code can be removed. > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> Acked-by: Geert Uytterhoeven <[EMAIL PROTECTED]> > --- > > arch/m68k/amiga/chipram.c |2 > drivers/i

Re: [PATCH] SGIWD93: use cached memory access to make driver work on IP28

2007-11-27 Thread Geert Uytterhoeven
u32 _padding; /* align to quadword boundary */ > + u32 _padding[128/4 - 3];/* align to biggest cache line size */ ^ (128 - sizeof(struct hpc_dma_desc))/4? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- T

Re: [PATCH 09/28] blk_end_request: changing ps3disk (take 3)

2007-12-02 Thread Geert Uytterhoeven
todate, num_sectors)) { > - add_disk_randomness(req->rq_disk); > - blkdev_dequeue_request(req); > - end_that_request_last(req, uptodate); > - } > + __blk_end_request(req, uptodate, num_sectors << 9); ^ With kind regards, Geert Uytterhoeven S

Re: Patch submission question [not in the FAQ]

2007-12-09 Thread Geert Uytterhoeven
river or > are you looking for anybody who has a packed 'wait_queue_head_t' and > submitting a patch to fix it? Alpha is little endian. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [EMAIL P

Re: [PATCH 1/1] Remove of old NCR53C9x/esp family of drivers

2008-01-03 Thread Geert Uytterhoeven
rs and has hardware to test (some of) them? I don't think we can afford losing one third of our SCSI drivers... You can use the following as guidance: commit 5ff263667798946abc15314eae3f341345877d7a Author: Thomas Bogendoerfer <[EMAIL PROTECTED]> Date: Tue May 22 17:03:44 2007 -0700 [SCSI] jazz_esp: Converted to

Re: [PATCH 1/1] Remove of old NCR53C9x/esp family of drivers

2008-01-04 Thread Geert Uytterhoeven
On Thu, 3 Jan 2008, David Miller wrote: > From: Geert Uytterhoeven <[EMAIL PROTECTED]> > > On Thu, 3 Jan 2008, James Bottomley wrote: > > > On Thu, 2008-01-03 at 17:40 +0200, Boaz Harrosh wrote: > > > > As recommended by Christoph Hellwig. There is no use >

Re: [BUG] 2.6.24-git usb reset problems

2008-01-30 Thread Geert Uytterhoeven
;ses->sg_table, sizeof(scmd->sg_table)); > scmd->resid = ses->resid; > scmd->result = ses->result; > } > diff --git a/include/scsi/scsi_eh.h b/include/scsi/scsi_eh.h > index d21b891..d43dc83 100644 > --- a/include/scsi/scsi_eh.h > +++ b/incl

Re: [Cbe-oss-dev] [PATCH] ps3rom: sector size should be 512 bytes

2008-01-31 Thread Geert Uytterhoeven
find where it handles conversion from CD data frame sizes to 512-byte sectors. Am I missing something? With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium Phone:+32 (0)

Re: [Cbe-oss-dev] [PATCH] ps3rom: sector size should be 512 bytes

2008-01-31 Thread Geert Uytterhoeven
should be using > > scsi_execute_async(), what's the actual problem use case? > > > > The problem case is a SCSI Target Mode engine that receives a 2048 Byte > single sector ATAPI READ_10 request from the storage fabric, and uses > scsi_execute_async() (the only op

Re: [Cbe-oss-dev] [PATCH] ps3rom: sector size should be 512 bytes

2008-01-31 Thread Geert Uytterhoeven
On Thu, 31 Jan 2008, James Bottomley wrote: > On Thu, 2008-01-31 at 18:41 +0100, Geert Uytterhoeven wrote: > > James, do you have any comment about the first email in this thread, > > increasing > > scsi_host_template.max_sectors? > > Assuming your BOUNCE_SIZE is in

Re: [Cbe-oss-dev] [PATCH] ps3rom: sector size should be 512 bytes

2008-01-31 Thread Geert Uytterhoeven
s was allowed to be calcuated in non 512 byte sectors, > so please disregard my patch. > > Geert, .max_sectors for ps3rom.c using 512 byte sectors ends up being > 128, yes.? Yes. With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe

Re: [PATCH 1/1] Remove of old NCR53C9x/esp family of drivers

2008-01-31 Thread Geert Uytterhoeven
On Thu, 31 Jan 2008, James Bottomley wrote: > On Mon, 2008-01-07 at 07:07 +0100, Kars de Jong wrote: > > On do, 2008-01-03 at 20:05 +0100, Geert Uytterhoeven wrote: > > > On Thu, 3 Jan 2008, James Bottomley wrote: > > > > On Thu, 2008-01-03 at 17:40 +0200, Bo

Re: [PATCH] kill hotplug init/exit section annotations

2008-01-31 Thread Geert Uytterhoeven
ds CPU hotplug. Thank you for giving an exhaustive list of classes of machines Linux runs on! Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [EMAIL PROTECTED] In personal conversations with technic

[PATCH] ps3rom: sector size should be 512 bytes

2008-02-01 Thread Geert Uytterhoeven
s used, which limited /sys/block/sr0/queue/max_sectors_kb to 16 KiB (32 sectors). Signed-off-by: Aegis Lin <[EMAIL PROTECTED]> Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- drivers/scsi/ps3rom.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- a/drivers/s

Re: [GIT PATCH] final SCSI updates for 2.6.24 merge window

2008-02-08 Thread Geert Uytterhoeven
iller <[EMAIL PROTECTED]> > > Christoph Hellwig <[EMAIL PROTECTED]> > > Not immediately ... I anticipate a few "where'd my driver go?" type > questions from m68k for which this provides a useful reference to point > to ... Don't bother, we'

[PATCH] scsi: Do not call do_div() with a 64-bit divisor

2013-11-04 Thread Geert Uytterhoeven
should be unsigned int, too. Signed-off-by: Geert Uytterhoeven --- drivers/scsi/sd.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/scsi/sd.c b/drivers/scsi/sd.c index 5693f6d7eddb..d6645c70cceb 100644 --- a/drivers/scsi/sd.c +++ b/drivers/scsi/sd.c @@ -1607

[PATCH] [SCSI] Update kernel size increase for SCSI_CONSTANTS

2013-11-28 Thread Geert Uytterhoeven
Many error messages have been added since Linux-0.99.12 (August 14, 1993) Signed-off-by: Geert Uytterhoeven --- drivers/scsi/Kconfig |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/Kconfig b/drivers/scsi/Kconfig index fe25677a5511..8024add2375f 100644

Re: [PATCH 19/25] fix error return code

2013-12-29 Thread Geert Uytterhoeven
gt; > // > ( > if@p1 (\(ret < 0\|ret != 0\)) > { ... return ret; } > | > ret@p1 = 0 > ) > ... when != ret = e1 > when != &ret > *if(...) > { > ... when != ret = e2 > when forall > return ret; > } > > // > > Signed-off-by

Re: [PATCH] target_core_alua: silence GCC warning

2014-02-19 Thread Geert Uytterhoeven
uggest making core_alua_state_nonoptimized() return void, too. Currently it always returns zero. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. Bu

Re: [PATCH 00/16] sleep_on removal, second try

2014-02-27 Thread Geert Uytterhoeven
ntainers can take them if they prefer, which apparently already happened for 01/16. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org In personal conversations with technical people, I call myself a hacker. But when I'm

Re: [PATCH 1/3] m68k/atari - convert atari_scsi falcon_get_lock() to use wait_event()

2014-03-06 Thread Geert Uytterhoeven
f-by: Arnd Bergmann > Acked-by: Michael Schmitz > Cc: Geert Uytterhoeven > Cc: James E.J. Bottomley > Cc: linux-scsi@vger.kernel.org Thanks, applied with original authorship attribution, original subject matching SCSI oneline summary style, and proper SoB. Will queue for 3.15

Re: [patch 0/6] PS3 Storage Drivers for 2.6.23, take 4

2007-07-10 Thread Geert Uytterhoeven
On Wed, 4 Jul 2007, Geert Uytterhoeven wrote: > This is the fourth submission of the new PS3 storage drivers: > [1] ps3: Preallocate bootmem memory for the PS3 FLASH ROM storage driver > [2] ps3: Storage Driver Core > [3] ps3: Storage device registration routines. > [4] ps

Re: [patch 5/6] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-13 Thread Geert Uytterhoeven
On Fri, 13 Jul 2007, Jens Axboe wrote: > On Fri, Jul 13 2007, James Bottomley wrote: > > On Wed, 2007-07-04 at 15:22 +0200, Geert Uytterhoeven wrote: > > > + kaddr = kmap_atomic(sgpnt->page, KM_USER0); > > > +

Re: [patch 5/6] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-13 Thread Geert Uytterhoeven
On Fri, 13 Jul 2007, James Bottomley wrote: > On Fri, 2007-07-13 at 15:25 +0200, Geert Uytterhoeven wrote: > > kmap() just returns page_address() on ppc64, as there's no highmem. > > kunmap() is a no-op. > > > So technically I could just use page_address() direc

Re: PS3 Storage Driver O_DIRECT issue

2007-07-13 Thread Geert Uytterhoeven
Debian etch/lenny/sid) and kernel 2.6.22-g77320894. With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium Phone:+32 (0)2 700 8453 Fax: +32 (0)2 700 8622

Re: [patch 0/6] PS3 Storage Drivers for 2.6.23, take 4

2007-07-13 Thread Geert Uytterhoeven
On Tue, 10 Jul 2007, Geert Uytterhoeven wrote: > On Wed, 4 Jul 2007, Geert Uytterhoeven wrote: > > This is the fourth submission of the new PS3 storage drivers: > > [1] ps3: Preallocate bootmem memory for the PS3 FLASH ROM storage driver > > [2] ps3: Storage Driver Core &

Re: [patch 5/6] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-13 Thread Geert Uytterhoeven
sh_kernel_dcache_page()? `git grep' finds it in the following files only: Documentation/cachetlb.txt arch/parisc/kernel/cache.c arch/parisc/kernel/pacache.S include/asm-parisc/cacheflush.h include/linux/highmem.h With kind regards, Geert Uytterhoeven Software Architect Son

Re: [patch 5/6] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-13 Thread Geert Uytterhoeven
On Fri, 13 Jul 2007, James Bottomley wrote: > On Fri, 2007-07-13 at 17:10 +0200, Geert Uytterhoeven wrote: > > On Fri, 13 Jul 2007, Arnd Bergmann wrote: > > > On Friday 13 July 2007, James Bottomley wrote: > > > > > IC. > > > > > > > &

Re: [patch 5/6] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-16 Thread Geert Uytterhoeven
On Fri, 13 Jul 2007, Geert Uytterhoeven wrote: > Ah, that explains it. flush_dcache_page() is used in some drivers. > I'll update my patches. Thanks for the comments! Does this look OK? - Replaced KM_USER0 by KM_IRQ0 (all routines are either called from an interrupt ha

Re: [patch 5/6] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-16 Thread Geert Uytterhoeven
On Mon, 16 Jul 2007, Jens Axboe wrote: > On Mon, Jul 16 2007, Geert Uytterhoeven wrote: > > On Fri, 13 Jul 2007, Geert Uytterhoeven wrote: > > > Ah, that explains it. flush_dcache_page() is used in some drivers. > > > I'll update my patches. Thanks for the comm

Re: [patch 5/6] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-16 Thread Geert Uytterhoeven
On Mon, 16 Jul 2007, Geert Uytterhoeven wrote: > On Mon, 16 Jul 2007, Jens Axboe wrote: > > On Mon, Jul 16 2007, Geert Uytterhoeven wrote: > > > On Fri, 13 Jul 2007, Geert Uytterhoeven wrote: > > > > Ah, that explains it. flush_dcache_page() is used in some drivers.

Re: [patch 5/6] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-16 Thread Geert Uytterhoeven
On Mon, 16 Jul 2007, Jens Axboe wrote: > On Mon, Jul 16 2007, James Bottomley wrote: > > On Mon, 2007-07-16 at 14:16 +0200, Jens Axboe wrote: > > > On Mon, Jul 16 2007, Geert Uytterhoeven wrote: > > > > On Fri, 13 Jul 2007, Geert Uytterhoeven wrote:

[patch 0/3] PS3 Storage Drivers for 2.6.23, take 5

2007-07-16 Thread Geert Uytterhoeven
e the kmap updates, as requested by Jens Axboe and James E.J. Bottomley. I didn't hear anything from the misc device maintainer (for the FLASH ROM Storage Driver). Paulus, please apply. Thanks in advance! With kind regards, Geert Uytterhoeven Software Architect Sony Network and Soft

[patch 3/3] ps3: FLASH ROM Storage Driver

2007-07-16 Thread Geert Uytterhoeven
From: Geert Uytterhoeven <[EMAIL PROTECTED]> Add a FLASH ROM Storage Driver for the PS3: - Implemented as a misc character device driver - Uses a fixed 256 KiB buffer allocated from boot memory as the hypervisor requires the writing of aligned 256 KiB blocks CC: Geoff Levand &

[patch 1/3] ps3: Disk Storage Driver

2007-07-16 Thread Geert Uytterhoeven
From: Geert Uytterhoeven <[EMAIL PROTECTED]> Add a Disk Storage Driver for the PS3: - Implemented as a block device driver with a dynamic major - Disk names (and partitions) are of the format ps3d%c(%u) - Uses software scatter-gather with a 64 KiB bounce buffer as the hypervisor d

[patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-16 Thread Geert Uytterhoeven
From: Geert Uytterhoeven <[EMAIL PROTECTED]> Add a BD/DVD/CD-ROM Storage Driver for the PS3: - Implemented as a SCSI device driver - Uses software scatter-gather with a 64 KiB bounce buffer as the hypervisor doesn't support scatter-gather CC: Geoff Levand <[EMAIL PROTECTE

[PATCH] Don't define empty struct bsg_class_device if !CONFIG_BLK_DEV_BSG (was: Re: block/bsg.c)

2007-07-17 Thread Geert Uytterhoeven
Don't define an empty struct bsg_class_device if !CONFIG_BLK_DEV_BSG. It's embedded in struct request_queue, but there we have #if defined(CONFIG_BLK_DEV_BSG) struct bsg_class_device bsg_dev; #endif anyway. Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- At fi

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-18 Thread Geert Uytterhoeven
On Wed, 18 Jul 2007, Jan Engelhardt wrote: > On Jul 16 2007 18:15, Geert Uytterhoeven wrote: > >Add a Disk Storage Driver for the PS3: > > - Implemented as a block device driver with a dynamic major > > - Disk names (and partitions) are of the format ps3d%c(%u) > &g

Re: PS3 Storage Driver O_DIRECT issue

2007-07-18 Thread Geert Uytterhoeven
v->bounce_buf+offset, buf, size); With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zaventem · Belgium Phone:+32 (0)2 700 8453 Fax: +32 (0)2 700 8622 E-mail: [EMA

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Thu, 19 Jul 2007, Jens Axboe wrote: > On Wed, Jul 18 2007, Andrew Morton wrote: > > On Mon, 16 Jul 2007 18:15:40 +0200 > > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > > > From: Geert Uytterhoeven <[EMAIL PROTECTED]> > > > > &g

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Geert Uytterhoeven
Hi Andrew, On Wed, 18 Jul 2007, Andrew Morton wrote: > On Mon, 16 Jul 2007 18:15:40 +0200 > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > From: Geert Uytterhoeven <[EMAIL PROTECTED]> > > > > Add a Disk Storage Driver for the PS3: > > Y

Re: [patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-19 Thread Geert Uytterhoeven
least. If it can fail on any other > arch then we have a biiiiig problem. (Multiple instances of this) Thanks, fixed! With kind regards, Geert Uytterhoeven Software Architect Sony Network and Software Technology Center Europe The Corporate Village · Da Vincilaan 7-D1 · B-1935 Zave

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Thu, 19 Jul 2007, Cornelia Huck wrote: > On Thu, 19 Jul 2007 10:57:53 +0200 (CEST), > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > Were .probe()/.remove() made concurrent again? I thought that idea was > > dropped > > because it caused too many proble

Re: [patch 2/3] ps3: BD/DVD/CD-ROM Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Thu, 19 Jul 2007, Andrew Morton wrote: > On Thu, 19 Jul 2007 11:02:07 +0200 (CEST) Geert Uytterhoeven <[EMAIL > PROTECTED]> wrote: > > > On Wed, 18 Jul 2007, Andrew Morton wrote: > > > > +struct ps3rom_private { > > > > + struct ps3_storage_

Re: [patch 3/3] ps3: FLASH ROM Storage Driver

2007-07-19 Thread Geert Uytterhoeven
deeply-embedded return points in this function. Will change to `goto fail' and common error return. > > + if (*pos+count > size) { > > checkpatch missed stuff here. Checkpatch is silent... > > + dev_dbg(&dev->sbd.core, > > +

Re: [patch 1/3] ps3: Disk Storage Driver

2007-07-19 Thread Geert Uytterhoeven
On Thu, 19 Jul 2007, Cornelia Huck wrote: > On Thu, 19 Jul 2007 11:36:31 +0200 (CEST), > Geert Uytterhoeven <[EMAIL PROTECTED]> wrote: > > > We have a probe thread that checks for new storage devices and adds them to > > the > > bus with ps3_system_b

[patch 0/4] PS3 storage driver updates (was: Re: [patch 1/3] ps3: Disk Storage Driver)

2007-07-19 Thread Geert Uytterhoeven
Hi Andrew, On Thu, 19 Jul 2007, Andrew Morton wrote: > On Thu, 19 Jul 2007 10:57:53 +0200 (CEST) Geert Uytterhoeven <[EMAIL > PROTECTED]> wrote: > > On Wed, 18 Jul 2007, Andrew Morton wrote: > > > Your patchset significantly hits powerpc, scsi and block. So w

[patch 1/4] ps3disk: use correct bio vector size

2007-07-19 Thread Geert Uytterhoeven
ps3disk: use correct bio vector size This fixes the O_DIRECT corruptions, as reported by Olaf Hering (triggered by e.g. parted >= 1.8.0). Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- drivers/block/ps3disk.c |2 +- 1 files changed, 1 insertion(+), 1 deletion(-) ---

[patch 2/4] ps3disk: updates after final review

2007-07-19 Thread Geert Uytterhoeven
mask o Minor coding style fixes not reported by checkpatch Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- drivers/block/ps3disk.c | 44 +--- 1 files changed, 25 insertions(+), 19 deletions(-) --- a/drivers/block/ps3disk.c +++ b/drivers/blo

[patch 3/4] ps3rom: updates after final review

2007-07-19 Thread Geert Uytterhoeven
ps3rom: updates after final review: o Kill confusing ps3rom_priv() macro, open code it instead o kmap_atomic() cannot fail Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- drivers/scsi/ps3rom.c | 15 +-- 1 files changed, 5 insertions(+), 10 deletions(-) --- a/d

[patch 4/4] ps3flash: updates after final review

2007-07-19 Thread Geert Uytterhoeven
fixes not reported by checkpatch Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- drivers/char/ps3flash.c | 59 1 files changed, 35 insertions(+), 24 deletions(-) --- a/drivers/char/ps3flash.c +++ b/drivers/char/ps3flash.c @@ -34,7

[patch 3/3] scsi: wd33c93 needs

2007-07-20 Thread Geert Uytterhoeven
driver still compiles on MIPS (CONFIG_SGIWD93_SCSI=y) Signed-off-by: Geert Uytterhoeven <[EMAIL PROTECTED]> --- drivers/scsi/wd33c93.c |2 ++ 1 file changed, 2 insertions(+) --- a/drivers/scsi/wd33c93.c +++ b/drivers/scsi/wd33c93.c @@ -89,6 +89,8 @@ #include #include +#include

Re: [patch 3/3] scsi: wd33c93 needs

2007-07-20 Thread Geert Uytterhoeven
On Fri, 20 Jul 2007, James Bottomley wrote: > On Fri, 2007-07-20 at 18:40 +0200, Geert Uytterhoeven wrote: > > plain text document attachment (m68k-wd33c93-needs-asm-irq.diff) > > wd33c93 SCSI needs on m68k > > > > drivers/scsi/wd33c93.c: In function 'wd33c

Re: [PATCH] ps3rom: convert to use the data buffer accessors

2007-08-06 Thread Geert Uytterhoeven
is converts ps3rom driver to use the new accessors for the sg lists > and the parameters. > > Signed-off-by: FUJITA Tomonori <[EMAIL PROTECTED]> Acked-by: Geert Uytterhoeven <[EMAIL PROTECTED]> With kind regards, Geert Uytterhoeven Software Architect Sony Network and

  1   2   3   >