Re: [RFC PATCH 01/20] powerpc/cell: Remove support for IBM Cell Blades

2024-11-14 Thread Jeremy Kerr
ent symbols PPC_CELL_NATIVE, PPC_CELL_COMMON, CBE_RAS, > PPC_IBM_CELL_RESETBUTTON, PPC_IBM_CELL_POWERBUTTON, CBE_THERM, and > AXON_MSI. Then remove the associated C files and headers, and trim > unused header content (some is shared with PS3). Nice work, thanks for pulling this apart. For the series: Acked-by: Jeremy Kerr Cheers, Jeremy

Re: [PATCH 02/79] spufs: switch to new ctime accessors

2023-06-21 Thread Jeremy Kerr
Hi Jeff, > In later patches, we're going to change how the ctime.tv_nsec field is > utilized. Switch to using accessor functions instead of raw accesses > of inode->i_ctime. s/utilized/used/ :D All looks good on the spufs change: Acked-by: Jeremy Kerr (also, thanks

Re: [PATCH] Documentation: spufs: correct a duplicate word typo

2022-08-29 Thread Jeremy Kerr
Hi Randy, > Fix a typo of "or" which should be "of". Nice, people are still reading the spufs docs! :D Looks good to me, thanks for the patch. Reviewed-by: Jeremy Kerr Cheers, Jeremy

Re: [PATCH 6/6] i2c: Make remove callback return void

2022-06-29 Thread Jeremy Kerr
 struct mctp_i2c_dev *midev = NULL, *tmp = NULL; > @@ -1000,7 +1000,6 @@ static int mctp_i2c_remove(struct i2c_client *client) > mctp_i2c_free_client(mcli); > mutex_unlock(&driver_clients_lock); > /* Callers ignore return code */ > -       return 0;

Re: [PATCH v2] sched: Use BUG_ON

2021-07-01 Thread Jeremy Kerr
Hi Jason, > The BUG_ON macro simplifies the if condition followed by BUG, so that > we can use BUG_ON instead of if condition followed by BUG. [...] > -   if (spu_acquire(ctx)) > -   BUG();  /* a kernel thread never has signals pending */ > +   /* a kernel thread never has si

Re: [PATCH] powerpc/spufs: add CONFIG_COREDUMP dependency

2020-07-06 Thread Jeremy Kerr
; > Reported-by: kernel test robot > Signed-off-by: Arnd Bergmann Looks good to me, thanks. Acked-by: Jeremy Kerr Cheers, Jeremy

Re: [PATCH] powerpc/spufs: adjust list element pointer type

2020-05-08 Thread Jeremy Kerr
le. Reviewed-by: Jeremy Kerr Cheers, Jeremy

Re: [PATCH 2/2] powerpc/spufs: stop using access_ok

2020-04-29 Thread Jeremy Kerr
Hi Christophe, > > Just use the proper non __-prefixed get/put_user variants where > > that is not done yet. > > But it means you are doing the access_ok() check everytime, which is > what is to be avoided by doing the access_ok() once then using the > __-prefixed variant. 5 out of 8 of these

[PATCH 1/2] powerpc/spufs: fix copy_to_user while atomic

2020-04-29 Thread Jeremy Kerr
from Christoph Hellwig . Fixes: bf1ab978be23 ("[POWERPC] coredump: Add SPU elf notes to coredump.") Signed-off-by: Jeremy Kerr Reviewed-by: Arnd Bergmann Reviewed-by: Christoph Hellwig --- arch/powerpc/platforms/cell/spufs/file.c | 113 +++ 1 file changed, 75 in

[PATCH 2/2] powerpc/spufs: stop using access_ok

2020-04-29 Thread Jeremy Kerr
From: Christoph Hellwig Just use the proper non __-prefixed get/put_user variants where that is not done yet. Signed-off-by: Christoph Hellwig Signed-off-by: Jeremy Kerr --- arch/powerpc/platforms/cell/spufs/file.c | 42 +--- 1 file changed, 8 insertions(+), 34 deletions

Re: [RFC PATCH] powerpc/spufs: fix copy_to_user while atomic

2020-04-28 Thread Jeremy Kerr
Hi Christoph, > And another one that should go on top of this one to address Al's other > compaint: Yeah, I was pondering that one. The access_ok() is kinda redundant, but it does avoid forcing a SPU context save on those errors. However, it's not like we really need to optimise for the case of

Re: [RFC PATCH] powerpc/spufs: fix copy_to_user while atomic

2020-04-28 Thread Jeremy Kerr
Hi Christoph, > FYI, these little hunks reduce the difference to my version, maybe > you can fold them in? Sure, no problem. How do you want to coordinate these? I can submit mine through mpe, but that may make it tricky to synchronise with your changes. Or, you can include this change in your s

[RFC PATCH] powerpc/spufs: fix copy_to_user while atomic

2020-04-28 Thread Jeremy Kerr
Currently, we may perform a copy_to_user (through simple_read_from_buffer()) while holding a context's register_lock, while accessing the context save area. This change uses a temporary buffers for the context save area data, which we then pass to simple_read_from_buffer. Signed-off-by: J

Re: [PATCH 1/5] powerpc/spufs: simplify spufs core dumping

2020-04-27 Thread Jeremy Kerr
Hi Al & Christoph, > Again, this really needs fixing. Preferably - as a separate commit > preceding this series, so that it could be > backported. simple_read_from_buffer() is a blocking operation. I'll put together a patch that fixes this. Christoph: I'll do it in a way that matches your chan

Re: [PATCH 1/6] powerpc/spufs: simplify spufs core dumping

2020-04-06 Thread Jeremy Kerr
small on-stack buffer and avoid > memory allocations as well. That looks much better, thanks! Reviewed-by: Jeremy Kerr However, I no longer have access to hardware to test this on. Michael, are the coredump tests in spufs-testsuite still alive? Cheers, Jeremy

Re: [PATCH] powerpc/powernv/prd: Validate whether address to be mapped is part of system RAM

2019-10-03 Thread Jeremy Kerr
Hi Vaidy, The current topic is who owns setting up the ATT bits for that piece of memory. It is the kernel today. Kernel decides to set this up as normal memory or I/O memory and sets the bits in page table entry. Or, what if there's a range of address-space that isn't backed by system RAM (

Re: [PATCH] powerpc/powernv/prd: Validate whether address to be mapped is part of system RAM

2019-10-03 Thread Jeremy Kerr
Hi Vasant, > > OK. How about we just don't do that? > > Yes. Hostboot will fix that. It will make sure that HBRT is loaded > into regular memory. Super. > > It sounds like we're just trying to work around an invalid > > representation of the mappings. > > Its not workaround. Its additional che

Re: [PATCH] powerpc/powernv/prd: Validate whether address to be mapped is part of system RAM

2019-10-02 Thread Jeremy Kerr
Hi Vasant, > Correct. We will have `ibm,prd-label` property. That's not the issue. It sure sounds like the issue - someone has represented a range that should be mapped by HBRT, but isn't appropriate for mapping by HBRT. > Here issueis HBRT is loaded into NVDIMM memory. OK. How about we just d

Re: [PATCH] powerpc/powernv/prd: Validate whether address to be mapped is part of system RAM

2019-10-02 Thread Jeremy Kerr
Hi Vasant, > Add check to validate whether requested page is part of system RAM > or not before mmap() and error out if its not part of system RAM. opal_prd_range_is_valid() will return false if the reserved memory range does not have an ibm,prd-label property. If this you're getting invalid memo

Re: [PATCH] Documentation: fix spelling mistake, EACCESS -> EACCES

2018-11-07 Thread Jeremy Kerr
Hi Jon, Signed-off-by: Colin Ian King --- Documentation/filesystems/spufs.txt | 2 +- Documentation/gpu/drm-uapi.rst | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) Applied, thanks. This is the first patch to spufs.txt since 2006...I wonder if that stuff is being used by

Re: [PATCH] ipmi/powernv: Fix spurious warnings at boot

2018-07-18 Thread Jeremy Kerr
Hi William, Sometimes we have stale messages showing up in the recv queue that are being processed by the pollers. We don't want to print out warnings for these messages not having an outstanding request as they can be expected when doing a kexec from the petitroot environment or from another ru

Re: [PATCH] [RESEND] spufs: use timespec64 for timestamps

2018-01-16 Thread Jeremy Kerr
c64 and printing a 64-bit number for > consistency. If we still have spufs in the tree in 2038 I'd be worried :) But good to keep things consistent. Acked-by: Jeremy Kerr Michael: want to take this directly through your tree? Cheers, Jeremy

[PATCH] powerpc: Fix action argument for cpufeatures-based TLB flush

2017-09-26 Thread Jeremy Kerr
. This change passes TLB_INVAL_SCOPE_GLOBAL instead. Signed-off-by: Jeremy Kerr CC: Nicholas Piggin --- arch/powerpc/kernel/dt_cpu_ftrs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/kernel/dt_cpu_ftrs.c b/arch/powerpc/kernel/dt_cpu_ftrs.c index 1df770e

Re: [PATCH] powerpc/spufs: Fix coredump of SPU contexts

2017-05-29 Thread Jeremy Kerr
Hi Michael, > So fix it by setting rc = 0 before returning in the success case. > > Fixes: 7b1f4020d0d1 ("spufs: get rid of dump_emit() wrappers") > Signed-off-by: Michael Ellerman Looks good to me, thanks. Acked-by: Jeremy Kerr Cheers, Jeremy

[PATCH] powerpc/spufs: Fix hash faults for kernel regions

2017-05-23 Thread Jeremy Kerr
memory no longer work. This change adds _PAGE_PRIVILEGED to the hash fault handler for kernel accesses. Signed-off-by: Jeremy Kerr Reported-by: Sombat Tragolgosol CC: Aneesh Kumar K.V --- arch/powerpc/platforms/cell/spu_base.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --

Re: SPU not working for kernel 4.9, 4.10, 4.11a and 4.12

2017-05-23 Thread Jeremy Kerr
Hi all, > Looks like this also happens with the simple spu_run test: > > > https://github.com/jk-ozlabs/spufs-testsuite/blob/master/tests/03-spu_run/01-spu_run.c > > ... might need some debugging here, I'll update if I find anything. And it appears we're stuck in the POLL_WHILE_FALSE() loop

Re: SPU not working for kernel 4.9, 4.10, 4.11a and 4.12

2017-05-22 Thread Jeremy Kerr
Hi all, > I'd like to report that SPU not working for kernel > 4.9. The problem > caused by spe_context_run function, when call , it never return back. Looks like this also happens with the simple spu_run test: https://github.com/jk-ozlabs/spufs-testsuite/blob/master/tests/03-spu_run/01-spu_ru

Re: [PATCH 2/2] powerpc/powernv/opal-dump : Use IRQ_HANDLED instead of numbers in interrupt handler

2017-02-26 Thread Jeremy Kerr
Hi Mukesh, >> Unless I'm mistaken, there are two things I can see happening with the >> old code: if we hit the IRQ_NONE path enough, we'll report a "nobody >> cared" error (see __report_bad_irq) and disable the interrupt, or for >> the -1 case, we'll immediately log a "bogus return value" error (

Re: [PATCH 2/2] powerpc/powernv/opal-dump : Use IRQ_HANDLED instead of numbers in interrupt handler

2017-02-15 Thread Jeremy Kerr
Hi Mukesh, > The return value of an interrupt handler is the special type > irqreturn_t. An interrupt handler can return two special values, > IRQ_NONE or IRQ_HANDLED. Yep, you can assume that the reader knows that level of the interrupt handler API :) What we want to know is how that change in b

Re: [PATCH 2/2] powerpc/powernv/opal-dump : Use IRQ_HANDLED instead of numbers in interrupt handler

2017-02-14 Thread Jeremy Kerr
Hi Mukesh, > Converts all the return explicit number to a more proper IRQ_HANDLED, > which looks proper incase of interrupt handler returning case. This looks good to me, but can you describe the effects of those changes to the interrupt handler's return code? ie, what happened in the erroneous

Re: [PATCH 1/2] powerpc/powernv/opal-dump : Handles opal_dump_info properly

2017-02-14 Thread Jeremy Kerr
Hi Mukesh, > Moves the return value check of 'opal_dump_info' to a proper place which > was previously unnecessarily filling all the dump info even on failure. Acked-by: Jeremy Kerr Thanks! Jeremy

Re: [Openipmi-developer] [PATCH 3/4] ipmi: allow dynamic BMC version information

2016-09-13 Thread Jeremy Kerr
Hi Corey, > In all, this looks good. I have two minor nits inline below, and > two more major comments here: > > I would prefer if it always queried the data, even if the device id > information is provided by the lower level driver. OK, can do. I was concerned that the SMI driver may want to p

[PATCH 0/4] ipmi: Allow dynamic device IDs

2016-09-12 Thread Jeremy Kerr
r to use the dynamic IDs; the behaviour of the other SMIs is not changed by this series. However, if there's interest, I'm happy to alter the existing SMIs too, in a follow-up series. Questions & comments most welcome. Cheers, Jeremy --- Jeremy Kerr (4): ipmi: Add a refe

[PATCH 4/4] ipmi/powernv: Use dynamic device ids

2016-09-12 Thread Jeremy Kerr
We don't currently provide any device ID info in the SMI registration, so use the dynamic ID infrastructure instead. Signed-off-by: Jeremy Kerr --- drivers/char/ipmi/ipmi_powernv.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/char/ipmi/ipmi_powernv.c b/dr

[PATCH 3/4] ipmi: allow dynamic BMC version information

2016-09-12 Thread Jeremy Kerr
query the version information when needed. We keep a short-term cache of this information so we don't need to re-query for every attribute access. Signed-off-by: Jeremy Kerr --- drivers/char/ipmi/ipmi_msghandler.c | 158 ++-- 1 file changed, 153 insertions(+), 5 del

[PATCH 2/4] ipmi: Make ipmi_demangle_device_id more generic

2016-09-12 Thread Jeremy Kerr
split netfn, cmd and data buffer, so it can be used with non-sequential responses. Signed-off-by: Jeremy Kerr --- drivers/char/ipmi/ipmi_si_intf.c | 3 ++- drivers/char/ipmi/ipmi_ssif.c| 3 ++- include/linux/ipmi_smi.h | 16 +--- 3 files changed, 13 insertions(+), 9 dele

[PATCH 1/4] ipmi: Add a reference from BMC devices to their interfaces

2016-09-12 Thread Jeremy Kerr
In an upcoming change, we'll want to grab a reference to the ipmi_smi_t from a struct bmc_device. This change adds a pointer to allow this. Signed-off-by: Jeremy Kerr --- drivers/char/ipmi/ipmi_msghandler.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/char

Re: [RFC 0/3] extend kexec_file_load system call

2016-07-21 Thread Jeremy Kerr
Hi Thiago, > So even if not ideal, the solution above is desirable for powerpc. We would > like to preserve the ability of allowing userspace to pass parameters to the > OS via the DTB, even if secure boot is enabled. > > I would like to turn the above into a proposal: > > Extend the syscall a

Re: mailman From rewriting [was perf jit: genelf makes assumptions about endian]

2016-03-29 Thread Jeremy Kerr
Hi Stephen, >> Do you know why mailman would be re-writing From: there? It's confusing >> patchwork, as multiple mails are now coming from that address. > > Yep, Anton posts from samba.org. They publish a DMARC policy that > breaks mailing lists. (╯°□°)╯︵ ┻━━┻ This also breaks git-am: [jk@p

mailman From rewriting [was perf jit: genelf makes assumptions about endian]

2016-03-29 Thread Jeremy Kerr
Hi Stephen, From the mail that Anton has sent to linuxppc-dev: > From: Anton Blanchard via Linuxppc-dev > Reply-To: Anton Blanchard Do you know why mailman would be re-writing From: there? It's confusing patchwork, as multiple mails are now coming from that address. Anton: did you do anything

[PATCH 2/3] powerpc/powernv: Add powernv firmware interface drivers to powernv_defconfig

2016-02-23 Thread Jeremy Kerr
There are a few firmware-provided interfaces for OpenPOWER platforms: the PRD infrastructure, IPMI support, and MTD access to the PNOR flash. This change adds these to powernv_defconfig Signed-off-by: Jeremy Kerr --- arch/powerpc/configs/powernv_defconfig | 6 ++ 1 file changed, 6

[PATCH 3/3] powerpc/powernv: Add AST graphics driver to powernv_defconfig

2016-02-23 Thread Jeremy Kerr
Most current OpenPOWER platforms have an AST BMC, so add graphics support via the AST DRM driver. Signed-off-by: Jeremy Kerr --- arch/powerpc/configs/powernv_defconfig | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/configs/powernv_defconfig b/arch/powerpc

[PATCH 1/3] powerpc/powernv: Add powernv_defconfig

2016-02-23 Thread Jeremy Kerr
This change adds a defconfig for the non-virtualised power platforms, based on pseries_defconfig, but without pseries, and little-endian, and no OF trampoline. Signed-off-by: Jeremy Kerr --- arch/powerpc/configs/powernv_defconfig | 307 + 1 file changed, 307

Re: [PATCH 2/5] powerpc/cell: Drop support for 64K local store on 4K kernels

2015-08-10 Thread Jeremy Kerr
7;d be happy to drop this - impact should be little to none. Acked-by: Jeremy Kerr Cheers, Jeremy ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH] Fix crash due to processing "memory-controller" nodes as "memory"

2015-07-21 Thread Jeremy Kerr
Hi Anton, > [PATCH] Fix crash due to processing "memory-controller" nodes as "memory" Looks good to me. If you apply this to your kexec-lite repo, I'll update op-build to use the new version, and send the merge requests for op-build. I'd expect we'd have those changes upstream in the next couple

Re: [RESEND PATCH] powerpc/powernv: Fix vma page prot flags in opal-prd driver

2015-06-28 Thread Jeremy Kerr
> being set for the normal kernel COW pages. > > Signed-off-by: Vaidyanathan Srinivasan Acked-by: Jeremy Kerr Cheers, Jeremy ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH] powerpc/include: Add opal-prd to installed uapi headers

2015-06-16 Thread Jeremy Kerr
We'll want to build the opal-prd daemon with the prd headers, so include this in the uapi headers list. Signed-off-by: Jeremy Kerr --- arch/powerpc/include/uapi/asm/Kbuild |1 + 1 file changed, 1 insertion(+) diff --git a/arch/powerpc/include/uapi/asm/Kbuild b/arch/powerpc/include

[PATCH] powerpc/powernv: fix construction of opal PRD messages

2015-06-15 Thread Jeremy Kerr
p the list_head. We also clarify the memcpy of the message, as we're copying unnecessary bytes at the end of the message data. Signed-off-by: Jeremy Kerr --- arch/powerpc/platforms/powernv/opal-prd.c | 19 --- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/ar

[PATCH 3/3 v4] powerpc/powernv: Add opal-prd channel

2015-06-04 Thread Jeremy Kerr
This change adds a char device to access the "PRD" (processor runtime diagnostics) channel to OPAL firmware. Includes contributions from Vaidyanathan Srinivasan, Neelesh Gupta & Vishal Kulkarni. Signed-off-by: Neelesh Gupta Signed-off-by: Jeremy Kerr Acked-by: Stewart S

Re: [3/3,v3] powerpc/powernv: Add opal-prd channel

2015-06-04 Thread Jeremy Kerr
+ * Supported on POWERNV platform >> + * >> + * (C) Copyright IBM 2015 > > Usual syntax is: "Copyright IBM Corporation 2015" OK, fixed. >> + * >> + * Author: Vaidyanathan Srinivasan >> + * Author: Jeremy Kerr > > I'd rather you dropped

[PATCH 3/3 v3] powerpc/powernv: Add opal-prd channel

2015-05-28 Thread Jeremy Kerr
This change adds a char device to access the "PRD" (processor runtime diagnostics) channel to OPAL firmware. Includes contributions from Vaidyanathan Srinivasan, Neelesh Gupta & Vishal Kulkarni. Signed-off-by: Neelesh Gupta Signed-off-by: Jeremy Kerr Acked-by: Stewart Smith -

Re: [PATCH] drivers/mtd: add powernv flash MTD abstraction driver

2015-05-20 Thread Jeremy Kerr
Hi Brian, > While I have Jeremy's attention, let me plug a friendly reminder for > this unrelated comment: > > http://patchwork.ozlabs.org/patch/413355/ > > Jeremy, you still haven't updated patchwork.git for your last round of > supposed "merges". Ah, thanks for the reminder - I've just done t

[PATCH 2/3 v2] powerpc/powernv: Expose OPAL APIs required by PRD interface

2015-05-19 Thread Jeremy Kerr
The (upcoming) opal-prd driver needs to access the message notifier and xscom code, so add EXPORT_SYMBOL_GPL macros for these. Signed-off-by: Jeremy Kerr --- arch/powerpc/platforms/powernv/opal.c |4 1 file changed, 4 insertions(+) diff --git a/arch/powerpc/platforms/powernv/opal.c

[PATCH 3/3 v2] powerpc/powernv: Add opal-prd channel

2015-05-19 Thread Jeremy Kerr
This change adds a char device to access the "PRD" (processor runtime diagnostics) channel to OPAL firmware. Includes contributions from Vaidyanathan Srinivasan, Neelesh Gupta & Vishal Kulkarni. Signed-off-by: Neelesh Gupta Signed-off-by: Jeremy Kerr --- arch/powerpc/include/

[PATCH 1/3 v2] powerpc/powernv: Merge common platform device initialisation

2015-05-19 Thread Jeremy Kerr
opal_ipmi_init and opal_flash_init are equivalent, except for the compatbile string. Merge these two into a common opal_pdev_init function. Signed-off-by: Jeremy Kerr --- arch/powerpc/platforms/powernv/opal.c | 21 ++--- 1 file changed, 6 insertions(+), 15 deletions(-) diff

Re: [PATCH] powerpc/powernv: Add opal-prd channel

2015-04-30 Thread Jeremy Kerr
Hi Ben, >> +static LIST_HEAD(opal_prd_msg_queue); >> +static DEFINE_SPINLOCK(opal_prd_msg_queue_lock); >> +static DECLARE_WAIT_QUEUE_HEAD(opal_prd_msg_wait); >> +static atomic_t usage; > > opal_prd_usage ... otherwise it's a mess in the symbols map OK, I'll change this. > Also why limit the nu

Re: [PATCH] powerpc/cell: Drop cbe-oss-dev mailing list from MAINTAINERS

2015-04-30 Thread Jeremy Kerr
Hi Michael, > Traffic on the cbe-oss-dev list is more or less non-existent, other than > CC's from linuxppc. Plus all that spam that never makes it out of the moderation queue. > It's seems like we may as well just send everyone to linuxppc and > archive the list.

Re: [PATCH 0/2] Add a MTD driver for OpenPower PNOR flash

2015-04-06 Thread Jeremy Kerr
Hi all, > If the mtd maintainers are happy with it this would be the easier approach in > this instance. OK, sounds good to me. MTD folks: any objections to this process? If not, is this patch good to go in, or any changes we should make? Cheers, Jeremy __

Re: [PATCH] powerpc: Remove the celleb support

2015-04-01 Thread Jeremy Kerr
o longer available, and no one on the list has shown any > interest in it over the years. > > So remove it. If anyone has one and cares please speak up. I've never had any spufs-related feedback from celleb platforms either. Acked-by: Jeremy Kerr Cheers, Jeremy

Re: [PATCH] powerpc: Reword the "returning from prom_init" message

2015-03-31 Thread Jeremy Kerr
ssage a Viking funeral and pick a date for a minute of silence. (Wistfully-)Acked-by: Jeremy Kerr Cheers, Jeremy ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH] powerpc/powernv: Add opal-prd channel

2015-03-31 Thread Jeremy Kerr
This change adds a char device to access the "PRD" (processor runtime diagnostics) channel to OPAL firmware. Includes contributions from Vaidyanathan Srinivasan, Neelesh Gupta & Vishal Kulkarni. Signed-off-by: Neelesh Gupta Signed-off-by: Jeremy Kerr --- arch/powerpc/include/

[PATCH 2/2] drivers/mtd: add powernv flash MTD abstraction driver

2015-03-31 Thread Jeremy Kerr
From: Cyril Bur Powerpc powernv platforms allow access to certain system flash devices through a firmwarwe interface. This change adds an mtd driver for these flash devices. Minor updates from Jeremy Kerr and Joel Stanley. Signed-off-by: Cyril Bur Signed-off-by: Joel Stanley Signed-off-by

[PATCH 1/2] powerpc/powernv: Add interfaces for flash device access

2015-03-31 Thread Jeremy Kerr
update functionality, so we rename that initcall to opal_flash_update_init(). A future change will add an mtd driver that uses this interface. Changes from Joel Stanley and Jeremy Kerr. Signed-off-by: Cyril Bur Signed-off-by: Jeremy Kerr Signed-off-by: Joel Stanley --- arch/powerpc/include/asm

[PATCH 0/2] Add a MTD driver for OpenPower PNOR flash

2015-03-31 Thread Jeremy Kerr
Hi all, This series implements a simple mtd device to allow access to the PNOR flash on OpenPower machines. The flash is accessed through firmware calls. Patch 1/2 adds the Linux interface to these calls. Patch 2/2 adds a mtd driver that uses these calls. Because there's two subsystems involved

[PATCH 5/5 v2] powerpc/boot: don't clobber r6 and r7 in epapr boot

2015-02-10 Thread Jeremy Kerr
We use r6 and r7 for epapr boot, but the current pre-C init will clobber both of these. This change does a simple replacement, of r6 -> r12 and r7 -> r13, so that we hit platform init with these registers intact. Signed-off-by: Jeremy Kerr --- arch/powerpc/boot/crt0.S

[PATCH 4/5 v2] powerpc/boot: Fix stack corruption in epapr entry point

2015-02-10 Thread Jeremy Kerr
8 fdt We'll need at least 32 bytes in the initial stack frame, to handle the LR save area. We bump this to 112 bytes, as that'll be the max required by ABIv1. Thanks to Alistair Popple for debugging help. Signed-off-by: Jeremy Kerr --- arch/powerpc/boot/crt0.S |2 +- 1 file chang

[PATCH 1/5 v2] powerpc/boot/fdt: Use unsigned long for pointer casts

2015-02-10 Thread Jeremy Kerr
Now that the wrapper supports 64-bit builds, we see warnings when attempting to cast pointers to int. Use unsigned long instead. Signed-off-by: Jeremy Kerr --- arch/powerpc/boot/libfdt-wrapper.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/powerpc/boot

[PATCH 3/5 v2] powerpc/boot/wrapper: use the pseries wrapper for zImage.epapr

2015-02-10 Thread Jeremy Kerr
n't be breaking anything. Signed-off-by: Jeremy Kerr --- arch/powerpc/boot/wrapper |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/boot/wrapper b/arch/powerpc/boot/wrapper index ae0f88e..3f50c27 100755 --- a/arch/powerpc/boot/wrapper +++ b/arch/powerpc/bo

[PATCH 2/5 v2] powerpc/boot/fdt: Add little-endian support to libfdt wrappers

2015-02-10 Thread Jeremy Kerr
For epapr-style boot, we may be little-endian. This change implements the proper conversion for fdt*_to_cpu and cpu_to_fdt*. We also need the full cpu_to_* and *_to_cpu macros for this. Signed-off-by: Jeremy Kerr Acked-by: Benjamin Herrenschmidt --- arch/powerpc/boot/libfdt_env.h | 14

Re: [PATCH 4/4] powerpc/boot: don't clobber r6 and r7 in epapr boot

2015-02-10 Thread Jeremy Kerr
Hi Ben, > You might want to also remove the use of r8 and r9 which are supposed > to contain the OPAL entry and base for debugging purposes. We would also > have to restore them before booting the kernel. I'd prefer that to be a separate change, if that's okay - it'd require us to change the kent

Re: [PATCH 2/4] powerpc/boot/wrapper: use the pseries wrapper for zImage.epapr

2015-02-10 Thread Jeremy Kerr
Hi Ben, >> We'll likely be entering the zImage.epapr as BE, so include the pseries >> implementation of _zimage_start, which adds the endian fixup magic. > > Wont that break embedded epapr stuff ? How so? This will just give us the endian fixup trampoline (followed by a branch to _zimage_start_l

[PATCH 4/4] powerpc/boot: don't clobber r6 and r7 in epapr boot

2015-02-10 Thread Jeremy Kerr
We use r6 and r7 for epapr boot, but the current pre-C init will clobber both of these. This change does a simple replacement, of r6 -> r12 and r7 -> r13, so that we hit platform init with these registers intact. Signed-off-by: Jeremy Kerr --- arch/powerpc/boot/crt0.S

[PATCH 3/4] powerpc/boot: Fix stack corruption in epapr entry point

2015-02-10 Thread Jeremy Kerr
008 fdt This change allocates the minimum of 32 bytes for the base of the stack instead. Signed-off-by: Jeremy Kerr --- arch/powerpc/boot/crt0.S |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/boot/crt0.S b/arch/powerpc/boot/crt0.S index 14de4f8..bf8 100644 ---

[PATCH 1/4] powerpc/boot/fdt: Add little-endian support to libfdt wrappers

2015-02-10 Thread Jeremy Kerr
For epapr-style boot, we may be little-endian. This change implements the proper conversion for fdt*_to_cpu and cpu_to_fdt*. We also need the full cpu_to_* and *_to_cpu macros for this. Signed-off-by: Jeremy Kerr --- arch/powerpc/boot/libfdt_env.h | 14 -- arch/powerpc/boot/of.h

[PATCH 2/4] powerpc/boot/wrapper: use the pseries wrapper for zImage.epapr

2015-02-10 Thread Jeremy Kerr
We'll likely be entering the zImage.epapr as BE, so include the pseries implementation of _zimage_start, which adds the endian fixup magic. Signed-off-by: Jeremy Kerr --- arch/powerpc/boot/wrapper |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/boot/wr

Re: [PATCH V3] powerpc, powernv: Add OPAL platform event driver

2014-12-18 Thread Jeremy Kerr
Hi Anshuman, > This patch creates a new OPAL platform event character driver > which will give userspace clients the access to these events > and process them effectively. Following platforms events are > currently supported with this platform driver. > > (1) Environmental and Power Warning

Re: [PATCH v2] drivers/char/ipmi: Add powernv IPMI driver

2014-11-13 Thread Jeremy Kerr
Hi Corey, > This looks good. Can this go into the IPMI tree now, or does it need > work done in the PowerPC tree first? The 1/2 patch is already in the powerpc tree - but to ensure that we hit Linus' tree in the right order, Michael has suggested that you merge his topic branch (which only has t

Re: powerpc/powernv: Support OPAL requested heartbeat

2014-11-12 Thread Jeremy Kerr
Hi Ben, >> I'd assume "freq" with no units is in HZ, but looks like it's >> milliseconds. I guess it's too late to rename it. > > We still can, we haven't cut an official build with that fw, jk ? Not yet. You got in by a matter of minutes :) Can you send me an updated skiboot patch too then? C

[PATCH v2] drivers/char/ipmi: Add powernv IPMI driver

2014-11-11 Thread Jeremy Kerr
This change adds an initial IPMI driver for powerpc OPAL firmware. The interface is exposed entirely through firmware: we have two functions to send and receive IPMI messages, and an interrupt notification from the firmware to signify that a message is available. Signed-off-by: Jeremy Kerr

Re: [PATCH 0/2] Add IPMI support for powernv powerpc machines

2014-11-11 Thread Jeremy Kerr
Hi Corey, >>> Alternatively, they could be merged by one maintainer, pending an ack >>> from the other. >> >> I'm fine either way. > > How about the third option? :) > > I've put patch 1 in a topic branch: > > > https://git.kernel.org/cgit/linux/kernel/git/mpe/linux.git/log/?h=topic/opal-ipm

Re: [PATCH 0/2] Add IPMI support for powernv powerpc machines

2014-11-09 Thread Jeremy Kerr
Hi Ben, > Our OPAL interface can only do one at a time ? Because our underlying > FW driver already has a queue .. The Linux driver needs to match responses to their requests, so the driver needs to keep a reference to the requests that we've sent. The current implementation is very simple: we ju

Re: [PATCH 0/2] Add IPMI support for powernv powerpc machines

2014-11-09 Thread Jeremy Kerr
Hi Corey, Thanks for the review. >> IPMI folks: the IPMI driver could do with a little review, as it's >> not a conventional BT/KCS/SMI SI, in that the low-level send/recv >> interface will handle the entire message at once. > > Handling the entire message at once should be fine, as that's what

[PATCH 2/2] drivers/char/ipmi: Add powernv IPMI driver

2014-11-05 Thread Jeremy Kerr
This change adds an initial IPMI driver for powerpc OPAL firmware. The interface is exposed entirely through firmware: we have two functions to send and receive IPMI messages, and an interrupt notification from the firmware to signify that a message is available. Signed-off-by: Jeremy Kerr

[PATCH 0/2] Add IPMI support for powernv powerpc machines

2014-11-05 Thread Jeremy Kerr
which provides the structure & function definitions. This'll break the build if only 2/2 is in the tree, and CONFIG_IPMI_POWERNV is set. Alternatively, they could be merged by one maintainer, pending an ack from the other. Comments / queries / etc welcome. Cheers, Jeremy --- Jeremy Kerr (2)

[PATCH 1/2] powerpc/powernv: Add OPAL IPMI interface

2014-11-05 Thread Jeremy Kerr
Recent OPAL firmare adds a couple of functions to send and receive IPMI messages: https://github.com/open-power/skiboot/commit/b2a374da This change updates the token list and wrappers to suit, and adds the platform devices for any IPMI interfaces. Signed-off-by: Jeremy Kerr --- arch

Re: [PATCH] powernv: Use _GLOBAL_TOC for opal wrappers

2014-10-22 Thread Jeremy Kerr
Hi Michael, >> Currently, we can't call opal wrappers from modules when using the LE >> ABIv2, which requires a TOC init. > > What happens if we try? Build failure or run time bug? We'll get an arbitrary memory dereference (two, actually) in the opal wrappers, when we try to load the opal entry

[PATCH] powernv: Use _GLOBAL_TOC for opal wrappers

2014-10-21 Thread Jeremy Kerr
Currently, we can't call opal wrappers from modules when using the LE ABIv2, which requires a TOC init. This change uses the _GLOBAL_TOC() macro (rather than _GLOBAL) for the opal wrappers, so that we can do non-local calls to them. Signed-off-by: Jeremy Kerr --- arch/powerpc/plat

Re: [PATCH 02/15] powerpc/cell: Move data segment faulting code out of cell platform

2014-09-18 Thread Jeremy Kerr
Hi Mikey & Ian, > __spu_trap_data_seg() currently contains code to determine the VSID and ESID > required for a particular EA and mm struct. > > This code is generically useful for other co-processors. This moves the code > of the cell platform so it can be used by other powerpc code. OK, nice.

Re: [PATCH 01/15] powerpc/cell: Move spu_handle_mm_fault() out of cell platform

2014-09-18 Thread Jeremy Kerr
. Makes sense. Acked-by: Jeremy Kerr > @@ -58,12 +56,12 @@ int spu_handle_mm_fault(struct mm_struct *mm, unsigned > long ea, > goto out_unlock; > } > > - is_write = dsisr & MFC_DSISR_ACCESS_PUT; > + is_write = dsisr & DSISR_ISST

Re: [patch] powerpc/spufs: remove duplicate SPUFS_CNTL_MAP_SIZE define

2014-06-09 Thread Jeremy Kerr
Hi Dan, > The SPUFS_CNTL_MAP_SIZE define is cut and pasted twice so we can delete > the second instance. Looks good to me. Acked-by: Jeremy Kerr Cheers, Jeremy ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozla

[PATCH] powerpc/spufs: Remove MAX_USER_PRIO define

2014-02-10 Thread Jeremy Kerr
ead of our own. Signed-off-by: Jeremy Kerr --- Ingo: 6b6350f1 is currently in tip; this fixes a build breakage for spufs --- arch/powerpc/platforms/cell/spufs/sched.c |1 - 1 file changed, 1 deletion(-) diff --git a/arch/powerpc/platforms/cell/spufs/sched.c b/arch/powerpc/platforms/

[PATCH v2] arch/powerpc: Dynamically allocate slb_shadow from memblock

2013-12-04 Thread Jeremy Kerr
4982532 1169576 15355007 ea4c7f obj/vmlinux Tested on pseries. Signed-off-by: Jeremy Kerr --- v2: don't drop the endian conversion, use SLB_NUM_BOLTED --- arch/powerpc/kernel/paca.c | 32 +--- 1 file changed, 25 insertions(+), 7 deletions(-) diff --git a

[PATCH 1/2] arch/powerpc: Make slb_shadow a local

2013-12-04 Thread Jeremy Kerr
The only external user of slb_shadow is the pseries lpar code, and it can access through the paca array instead. Signed-off-by: Jeremy Kerr --- arch/powerpc/include/asm/lppaca.h |2 -- arch/powerpc/kernel/paca.c|2 +- arch/powerpc/platforms/pseries/lpar.c |2 +- 3

[PATCH 2/2] arch/powerpc: Dynamically allocate slb_shadow from memblock

2013-12-04 Thread Jeremy Kerr
4982532 1169576 15355007 ea4c7f obj/vmlinux Tested on pseries. Signed-off-by: Jeremy Kerr --- arch/powerpc/kernel/paca.c | 32 +--- 1 file changed, 25 insertions(+), 7 deletions(-) diff --git a/arch/powerpc/kernel/paca.c b/arch/powerpc/kernel/paca.c index

Re: [PATCH v9] PPC: POWERNV: move iommu_add_device earlier

2013-11-20 Thread Jeremy Kerr
Hi Alexey, > This patch does 2 things: > 1. removes the loop in which PCI devices were added to groups and > adds explicit iommu_add_device() calls to add devices as soon as they get > the iommu_table pointer assigned to them. > 2. moves a bus notifier to powernv code in order to avoid conflict wi

Re: linux-next: build failure after merge of the final tree

2013-08-20 Thread Jeremy Kerr
nd mounts as expected. Acked-by: Jeremy Kerr Cheers, Jeremy ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [git pull] Please pull powerpc.git merge branch

2013-06-09 Thread Jeremy Kerr
Hi Linus, > No. The date from the email was > > Date: Fri, 7 Jun 2013 15:42:54 +1000 > > and we want *that* date. Ah, gotchya. So, we now use the original date header (if present) in the mbox views: $ wget -qO - http://patchwork.ozlabs.org/patch/249598/mbox/ | grep ^Date Date: Fri, 7 Jun

Re: [git pull] Please pull powerpc.git merge branch

2013-06-09 Thread Jeremy Kerr
Hi Linus, > Is Jeremy the patchwork maintainer? Yep, that's me. > and it turns out that apparently 'patchwork' is just making up random > times, because when you download the email as an mbox, it will turn > this into that corrupt and incorrect > > Date: Thu, 06 Jun 2013 19:42:54 - > >

[PATCH] powerpc/powernv: defer OPAL exception handler registration

2013-05-02 Thread Jeremy Kerr
-off-by: Jeremy Kerr --- arch/powerpc/platforms/powernv/opal.c | 15 +-- 1 file changed, 13 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/platforms/powernv/opal.c b/arch/powerpc/platforms/powernv/opal.c index aaa0dba..ade4463 100644 --- a/arch/powerpc/platforms/powernv

[RFC, PATCH v3] powerpc/prom: Scan reserved-ranges node for memory reservations

2013-04-23 Thread Jeremy Kerr
Based on benh's proposal at https://lists.ozlabs.org/pipermail/linuxppc-dev/2012-September/101237.html, this change provides support for reserving memory from the reserved-ranges node at the root of the device tree. We just call memblock_reserve on these ranges for now. Signed-off-by: J

  1   2   3   >