Re: [PATCH] init: make init failures more explicit

2013-10-18 Thread Kieran Bingham
Fantastic I've been hurt by this in the past - and this patch would certainly would have helped save some time! -- Kieran On 18 October 2013 09:47, Michael Opdenacker michael.opdenac...@free-electrons.com wrote: This patch proposes to make init failures more explicit. Before this, the No

[PATCH] i2c: return probe deferred status on dev_pm_domain_attach

2015-10-11 Thread Kieran Bingham
this back to the original return status; Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- drivers/i2c/i2c-core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c index 5f89f1e..df83015 100644 --- a/drivers/i2c/i2c-core

Re: [PATCH] i2c: return probe deferred status on dev_pm_domain_attach

2015-10-12 Thread Kieran Bingham
On 12 October 2015 at 08:24, Wolfram Sang wrote: > > And add the original patch author to CC when resending. > Bah - I was sure I'd added him ... must have slipped. Should I add Cc: stable? or are they notified from the Fixes: tag? -- Regards Kieran -- To unsubscribe from

Re: [PATCH] i2c: return probe deferred status on dev_pm_domain_attach

2015-10-12 Thread Kieran Bingham
Hi Wolfram, On 12 October 2015 at 08:23, Wolfram Sang <w...@the-dreams.de> wrote: > Hi Kiera, > > On Sun, Oct 11, 2015 at 12:39:31PM +0100, Kieran Bingham wrote: >> A change of return status was introduced in commit 3fffd1283927 >> ("i2c: allow specifying separate

Re: [RESEND PATCH v4 0/8] i2c: Relax mandatory I2C ID table passing

2015-10-12 Thread Kieran Bingham
Hi Wolfram, On 9 October 2015 at 22:16, Wolfram Sang wrote: > > As said to Kieran personally in Dublin, I want a verification that all > binding methods still work, especially runtime instantiation for drivers > without i2c_device_ids. Ok, I should be able to find some time

Re: [PATCH] i2c: return probe deferred status on dev_pm_domain_attach

2015-10-12 Thread Kieran Bingham
On 12 October 2015 at 20:40, Wolfram Sang wrote: > >> Should I add Cc: stable? or are they notified from the Fixes: tag? > > My preference is: I'll add stable when I commit to my tree. I am happy > if people tell me when they think this should be done. > > In this case, it

[PATCH v2] i2c: return probe deferred status on dev_pm_domain_attach

2015-10-12 Thread Kieran Bingham
d clear up the IRQ wakeup status Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> Fixes: 3fffd1283927 ("i2c: allow specifying separate wakeup interrupt in device tree") --- V2: Fall through the error path before returning -EPROBE_DEFER drivers/i2c/i2c-core.c | 2 ++ 1

Re: [PATCH v2] i2c: return probe deferred status on dev_pm_domain_attach

2015-10-12 Thread Kieran Bingham
On 12 October 2015 at 21:17, Dmitry Torokhov <dmitry.torok...@gmail.com> wrote: > On Mon, Oct 12, 2015 at 12:49 PM, Kieran Bingham > <kieranbing...@gmail.com> wrote: >> A change of return status was introduced in commit 3fffd1283927 >> ("i2c: allow specifying

[PATCH v3] i2c: return probe deferred status on dev_pm_domain_attach

2015-10-12 Thread Kieran Bingham
d clear up the IRQ wakeup status Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> Fixes: 3fffd1283927 ("i2c: allow specifying separate wakeup interrupt in device tree") --- V2: Fall through the error path before returning -EPROBE_DEFER V3: Drop the if/else for cleaner c

Re: [PATCH 4/6] hwrng: st: Add support for ST's HW Random Number Generator

2015-09-12 Thread Kieran Bingham
Hi Lee, On 11 September 2015 at 21:08, Lee Jones wrote: > Signed-off-by: Pankaj Dev > Signed-off-by: Lee Jones > --- > drivers/char/hw_random/Kconfig | 10 +++ > drivers/char/hw_random/Makefile | 1 + >

Re: [PATCH 1/6] Documentation: hw_random: Fix device node name /dev/hw_random => /dev/hwrng

2015-09-14 Thread Kieran Bingham
Hi Lee, On 11 September 2015 at 21:08, Lee Jones wrote: > In April 2099, commit d405640 That's impressive ... :D > ("Driver Core: misc: add node name support > for misc devices.") inadvertently changed the device node name from > /dev/hw_random to /dev/hwrng. Since 6

Re: [PATCH v4 1/8] i2c: Add pointer dereference protection to i2c_match_id()

2015-09-11 Thread Kieran Bingham
Hi Lee, On 11 September 2015 at 10:31, Lee Jones <lee.jo...@linaro.org> wrote: > On Wed, 09 Sep 2015, Kieran Bingham wrote: > >> From: Lee Jones <lee.jo...@linaro.org> >> >> Here we're providing dereference protection for i2c_match_id(), which >> save

[RESEND PATCH v4 4/8] i2c: Make I2C ID tables non-mandatory for DT'ed devices

2015-09-11 Thread Kieran Bingham
match on DT match tables instead. Acked-by: Grant Likely <grant.lik...@linaro.org> Signed-off-by: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- drivers/i2c/i2c-core.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-)

[RESEND PATCH v4 3/8] i2c: Match using traditional OF methods, then by vendor-less compatible strings

2015-09-11 Thread Kieran Bingham
trings with the ',' string removed. The latter is required due to the unruly naming conventions used currently by I2C devices. Acked-by: Grant Likely <grant.lik...@linaro.org> Signed-off-by: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- dri

[RESEND PATCH v4 0/8] i2c: Relax mandatory I2C ID table passing

2015-09-11 Thread Kieran Bingham
method and issues an of_match_device() would also fail their probe(). Bolted on to this set is a new, more generic way for these devices to match against either of the I2C/OF tables. I hope this ticks all of your boxes. v4: [Kieran Bingham] - Rebase to v4.2 - adapt for dev_pm_domain_{attach,detach

[RESEND PATCH v4 6/8] i2c: Provide a temporary .probe2() call-back type

2015-09-11 Thread Kieran Bingham
Kieran: fix rebase conflicts and adapt for dev_pm_domain_{attach,detach}] Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- drivers/i2c/i2c-core.c | 22 ++ include/linux/i2c.h| 8 +++- 2 files changed, 21 insertions(+), 9 deletions(-) diff --git a/

[RESEND PATCH v4 7/8] mfd: 88pm860x: Move over to new I2C device .probe() call

2015-09-11 Thread Kieran Bingham
inaro.org> Signed-off-by: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- drivers/mfd/88pm860x-core.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/drivers/mfd/88pm860x-core.c b/drivers/mfd/88pm860x-core.c index

[RESEND PATCH v4 8/8] mfd: as3722: Rid driver of superfluous I2C device ID structure

2015-09-11 Thread Kieran Bingham
From: Lee Jones <lee.jo...@linaro.org> Also remove unused second probe() parameter 'i2c_device_id'. Acked-by: Grant Likely <grant.lik...@linaro.org> Signed-off-by: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- d

[RESEND PATCH v4 1/8] i2c: Add pointer dereference protection to i2c_match_id()

2015-09-11 Thread Kieran Bingham
imming. Acked-by: Grant Likely <grant.lik...@linaro.org> Signed-off-by: Lee Jones <lee.jo...@linaro.org> Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- drivers/i2c/i2c-core.c | 12 +++- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/drivers/i2c/i2c-

[RESEND PATCH v4 2/8] i2c: Add the ability to match device to compatible string without an of_node

2015-09-11 Thread Kieran Bingham
ce all of the compatible strings for each device have been brought into line. Acked-by: Grant Likely <grant.lik...@linaro.org> Signed-off-by: Lee Jones <lee.jo...@linaro.org> [Kieran: strnicmp to strncasecmp] Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- dri

[RESEND PATCH v4 5/8] i2c: Export i2c_match_id() for direct use by device drivers

2015-09-11 Thread Kieran Bingham
o...@linaro.org> Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- drivers/i2c/i2c-core.c | 3 ++- include/linux/i2c.h| 2 ++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/drivers/i2c/i2c-core.c b/drivers/i2c/i2c-core.c index 2ebc64d..0e40136 100644 --- a/driv

Re: [RESEND PATCH v4 0/8] i2c: Relax mandatory I2C ID table passing

2015-10-01 Thread Kieran Bingham
Hi Wolfram, On 1 October 2015 at 21:50, Wolfram Sang wrote: > >> > Yes but that is not true for drivers that support both OF and legacy board >> > files. For those drivers, there will be a lot of boiler plate code >> > duplicated >> > that would look something like: >> > >>

[PATCH v4 0/8] i2c: Relax mandatory I2C ID table passing

2015-09-09 Thread Kieran Bingham
boxes. v4: [Kieran Bingham] - Rebase to v4.2 - adapt for dev_pm_domain_{attach,detach} - strnicmp to strncasecmp v3: [Lee Jones] - Insist on passing 'struct i2c_client' instead of 'struct device' - Remove hook from of_match_device() v2: [Lee Jones] - Removal of ACPI support

[PATCH v4 3/8] i2c: Match using traditional OF methods, then by vendor-less compatible strings

2015-09-09 Thread Kieran Bingham
From: Lee Jones This function provides a single call for all I2C devices which need to match firstly using traditional OF means i.e by of_node, then if that fails we attempt to match using the supplied I2C client name with a list of supplied compatible strings with the ','

[PATCH v4 7/8] mfd: 88pm860x: Move over to new I2C device .probe() call

2015-09-09 Thread Kieran Bingham
From: Lee Jones As part of an effort to rid the mostly unused second parameter for I2C related .probe() functions and to conform to other existing frameworks we're moving over to a temporary replacement .probe() call-back. Acked-by: Grant Likely

[PATCH v4 6/8] i2c: Provide a temporary .probe2() call-back type

2015-09-09 Thread Kieran Bingham
Kieran: fix rebase conflicts and adapt for dev_pm_domain_{attach,detach}] Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- drivers/i2c/i2c-core.c | 22 ++ include/linux/i2c.h| 8 +++- 2 files changed, 21 insertions(+), 9 deletions(-) diff --git a/

[PATCH v4 5/8] i2c: Export i2c_match_id() for direct use by device drivers

2015-09-09 Thread Kieran Bingham
From: Lee Jones When there was no other way to match a I2C device to driver i2c_match_id() was exclusively used. However, now there are other types of tables which are commonly supplied, matching on an i2c_device_id table is used less frequently. Instead of _always_

[PATCH v4 2/8] i2c: Add the ability to match device to compatible string without an of_node

2015-09-09 Thread Kieran Bingham
ce all of the compatible strings for each device have been brought into line. Acked-by: Grant Likely <grant.lik...@linaro.org> Signed-off-by: Lee Jones <lee.jo...@linaro.org> [Kieran: strnicmp to strncasecmp] Signed-off-by: Kieran Bingham <kieranbing...@gmail.com> --- dri

[PATCH v4 8/8] mfd: as3722: Rid driver of superfluous I2C device ID structure

2015-09-09 Thread Kieran Bingham
From: Lee Jones Also remove unused second probe() parameter 'i2c_device_id'. Acked-by: Grant Likely Signed-off-by: Lee Jones --- drivers/mfd/as3722.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff

[PATCH v4 1/8] i2c: Add pointer dereference protection to i2c_match_id()

2015-09-09 Thread Kieran Bingham
From: Lee Jones Here we're providing dereference protection for i2c_match_id(), which saves us having to do it each time it's called. We're also stripping out the (now) needless checks in i2c_device_match(). This patch paves the way for other, similar code trimming.

[PATCH v4 4/8] i2c: Make I2C ID tables non-mandatory for DT'ed devices

2015-09-09 Thread Kieran Bingham
From: Lee Jones Currently the I2C framework insists on devices supplying an I2C ID table. Many of the devices which do so unnecessarily adding quite a few wasted lines to kernel code. This patch allows drivers a means to 'not' supply the aforementioned table and match on

Re: [PATCH v2 1/7] Documentation: hw_random: Fix device node name reference /dev/hw_random => /dev/hwrng

2015-09-18 Thread Kieran Bingham
ff-by: Lee Jones <lee.jo...@linaro.org> I see the KConfig change went into a separate patch. That's fine by me: Acked-by: Kieran Bingham <kieranbing...@gmail.com> > --- > Documentation/hw_random.txt | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff

Re: [PATCH v2 2/7] hwrng: Kconfig: Fix device node name reference /dev/hw_random => /dev/hwrng

2015-09-18 Thread Kieran Bingham
s passed since the change > it seems unpractical to change it back, as this node name is probably > considered ABI by now. So instead, we'll just change the Kconfig help > to match the current situation. > > NB: It looks like rng-tools have already been updated. > > Signed-off-by: Lee

Re: [PATCH v2 5/7] hwrng: st: Add support for ST's HW Random Number Generator

2015-09-18 Thread Kieran Bingham
On 17 September 2015 at 14:45, Lee Jones <lee.jo...@linaro.org> wrote: > Signed-off-by: Pankaj Dev <pankaj@st.com> > Signed-off-by: Lee Jones <lee.jo...@linaro.org> Comments addressed, Also LGTM. Acked-by: Kieran Bingham <kieranbing...@gmail.com> > ---

[PATCH 3/3] dt-bindings: Add Renesas R-Car FDP1 bindings

2016-06-09 Thread Kieran Bingham
The FDP1 is a de-interlacing module which converts interlaced video to progressive video. It is also capable of performing pixel format conversion between YCbCr/YUV formats and RGB formats. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- .../devicetree/bindings/media/renesas,fd

[TEST PATCH] rtc: convert ds1307 to interim probe_new

2016-06-12 Thread Kieran Bingham
Just for testing, specify a ds device to identify the code path used when instantiating the driver from userspace. As we match on only the device, not the manufacturer, I've changed your sketch so that the test maxim line is on a compatible with name ds to make it unique. Otherwise, we

Re: [PATCHv5 0/8] 2c: Relax mandatory I2C ID table passing

2016-06-10 Thread Kieran Bingham
on. I've already switched my dev board environment back to the Salvator-X for $DAYJOB and switching back now will have to wait to the weekend. > Thanks, > >Wolfram > -- Regards Kieran Bingham

Re: [PATCH RFC 1/2] v4l: platform: Add Renesas R-Car FDP1 Driver

2016-06-10 Thread Kieran Bingham
Today I learned about make C=1 So ... reviewing my own patch, consider the following sparse warnings 'fixed up' I'll run make C=1 before any future submissions from now on. On 09/06/16 18:37, Kieran Bingham wrote: > The FDP1 driver performs advanced de-interlacing on a memory 2 memory >

[PATCH RFC 2/2] MAINTAINERS: Add support for FDP driver

2016-06-09 Thread Kieran Bingham
Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- MAINTAINERS | 9 + 1 file changed, 9 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 66de4da2d244..bc083b58e478 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -7312,6 +7312,15 @@ S: Supported F: Documen

[PATCH RFC 0/2] v4l: platform: Add Renesas R-Car FDP1 Driver

2016-06-09 Thread Kieran Bingham
driver this seemed like an apt point to start the review process. Kieran Bingham (2): v4l: platform: Add Renesas R-Car FDP1 Driver MAINTAINERS: Add support for FDP driver MAINTAINERS|9 + drivers/media/platform/Kconfig | 13 + drivers/media/platform/Makefile

[PATCH RFC 1/2] v4l: platform: Add Renesas R-Car FDP1 Driver

2016-06-09 Thread Kieran Bingham
The FDP1 driver performs advanced de-interlacing on a memory 2 memory based video stream, and supports conversion from YCbCr/YUV to RGB pixel formats Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- drivers/media/platform/Kconfig | 13 + drivers/media/platform/Makefile

Re: [PATCHv5 0/8] 2c: Relax mandatory I2C ID table passing

2016-06-10 Thread Kieran Bingham
nted omap_rng3927 0 rng_core6410 1 omap_rng rtc_ds1307 12121 0 root@arm:~# cat /sys/class/rtc/rtc0/device/modalias i2c:ds1307 root@arm:~# cat /sys/class/rtc/rtc0/date 2016-06-10 * Module autoload successful, * i2c read successful. Code tested at repository https://github.com/kbingham/linux.git tag i2c-dt/v4.7-rc2-relax-conversion-zero-test Is this what you were looking for? > Thanks, > >Wolfram > -- Regards Kieran Bingham

[PATCH] clk: renesas: r8a7795: Provide FDP1 clocks

2016-06-09 Thread Kieran Bingham
Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be> Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com> Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- drivers/clk/renesas/r8a7795-cpg-mssr.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/d

[RESEND PATCH 0/2] arm64: dts: r8a7795: Add FCPF and FDP1 device nodes

2016-06-09 Thread Kieran Bingham
to those bindings from me [1], and of course the bindings for the FDP1 [2] which are all posted separately. [0] http://www.spinics.net/lists/devicetree/msg123968.html [1] https://lkml.org/lkml/2016/6/9/291 [2] https://lkml.org/lkml/2016/6/9/294 Kieran Bingham (2): arm64: dts: r8a7795: add FCPF

[PATCH] [media] v4l: vsp1: Fix format-info documentation

2016-06-09 Thread Kieran Bingham
Minor tweaks to document the swap register and make the documentation match the struct ordering Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- drivers/media/platform/vsp1/vsp1_pipe.h | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/platfor

[PATCH 2/3] dt-bindings: Document Renesas R-Car FCP power-domains usage

2016-06-09 Thread Kieran Bingham
The power domain must be specified to bring the device out of module standby. Document this in the example provided, so that new additions are not missed. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- Documentation/devicetree/bindings/media/renesas,fcp.txt | 1 + 1 file chan

[PATCH 1/3] dt-bindings: Update Renesas R-Car FCP DT binding

2016-06-09 Thread Kieran Bingham
The FCP driver, can also support the FCPF variant for FDP1 compatible processing. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- Documentation/devicetree/bindings/media/renesas,fcp.txt | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Documentation/devi

[PATCH 2/2] arm64: dts: r8a7795: add FDP1 device nodes

2016-06-09 Thread Kieran Bingham
Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- arch/arm64/boot/dts/renesas/r8a7795.dtsi | 27 +++ 1 file changed, 27 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi index e63e5f29cceb..0c0aa1

[PATCH 1/2] arm64: dts: r8a7795: add FCPF device nodes

2016-06-09 Thread Kieran Bingham
Provide nodes for the FCP devices dedicated to the FDP device channels. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- arch/arm64/boot/dts/renesas/r8a7795.dtsi | 21 + 1 file changed, 21 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi

[PATCH 1/2] arm64: dts: r8a7795: add FCPF device nodes

2016-06-09 Thread Kieran Bingham
Provide nodes for the FCP devices dedicated to the FDP device channels. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- arch/arm64/boot/dts/renesas/r8a7795.dtsi | 21 + 1 file changed, 21 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi

[PATCH] v4l: Extend FCP compatible list to support the FDP

2016-06-09 Thread Kieran Bingham
The FCP must be powered up for the FDP1 to function, even when the FDP1 does not make use of the FCNL features. Extend the compatible list to allow us to use the power domain and runtime-pm support. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- drivers/media/platform/rcar-fcp.c |

[PATCH] v4l: Extend FCP compatible list to support the FDP

2016-06-09 Thread Kieran Bingham
.html Kieran Bingham (1): v4l: Extend FCP compatible list to support the FDP drivers/media/platform/rcar-fcp.c | 1 + 1 file changed, 1 insertion(+) -- 2.7.4

[PATCH 2/2] arm64: dts: r8a7795: add FDP1 device nodes

2016-06-09 Thread Kieran Bingham
Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- arch/arm64/boot/dts/renesas/r8a7795.dtsi | 27 +++ 1 file changed, 27 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi index e63e5f29cceb..0c0aa1

Re: [PATCH] scripts/gdb: Perform path expansion to lx-symbol's arguments

2016-06-13 Thread Kieran Bingham
pend(os.getcwd()) >> >> # enforce update >> > > Reviewed-by: Jan Kiszka <jan.kis...@siemens.com> > > Kieran, will you take this, as you are already preparing other patches? Yes, that should be fine. > Thanks, > Jan > -- Regards Kieran Bingham

Re: [PATCH 1/3] dt-bindings: Update Renesas R-Car FCP DT binding

2016-06-13 Thread Kieran Bingham
On 10/06/16 18:37, Rob Herring wrote: > On Thu, Jun 09, 2016 at 02:41:32PM +0100, Kieran Bingham wrote: >> The FCP driver, can also support the FCPF variant for FDP1 compatible > > Drop the comma. Ok >> processing. >> >> Signed-off-by: K

Re: [PATCH 3/3] dt-bindings: Add Renesas R-Car FDP1 bindings

2016-06-13 Thread Kieran Bingham
On 10/06/16 18:40, Rob Herring wrote: > On Thu, Jun 09, 2016 at 02:41:34PM +0100, Kieran Bingham wrote: >> The FDP1 is a de-interlacing module which converts interlaced video to >> progressive video. It is also capable of performing pixel format conversion >> between YCbC

Re: [PATCH 2/4] dt-bindings: Update Renesas R-Car FCP DT binding

2016-05-30 Thread Kieran Bingham
Hi Geert! On 28/05/16 20:06, Geert Uytterhoeven wrote: > Hi Kieran, > > On Fri, May 27, 2016 at 7:19 PM, Kieran Bingham <kie...@ksquared.org.uk> > wrote: >> The FCP driver, can also support the FCPF variant for FDP1 compatible >> processing. >> &

Re: [PATCH 3/4] dt-bindings: Document Renesas R-Car FCP power-domains usage

2016-05-30 Thread Kieran Bingham
Hi Geert, On 28/05/16 20:03, Geert Uytterhoeven wrote: > Hi Kieran, > > On Fri, May 27, 2016 at 7:19 PM, Kieran Bingham <kie...@ksquared.org.uk> > wrote: >> The example misses the power-domains usage, and documentation that the >> property is used by the nod

[PATCH 3/4] dt-bindings: Document Renesas R-Car FCP power-domains usage

2016-05-27 Thread Kieran Bingham
The example misses the power-domains usage, and documentation that the property is used by the node. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- Documentation/devicetree/bindings/media/renesas,fcp.txt | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Documentation/devi

[PATCH 1/4] fcp: Extend FCP compatible list to support the FDP

2016-05-27 Thread Kieran Bingham
The FCP must be powered up for the FDP1 to function, even when the FDP1 does not make use of the FCNL features. Extend the compatible list to allow us to use the power domain and runtime-pm support. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- drivers/media/platform/rcar-fcp.c |

[PATCH 4/4] arm64: dts: r8a7795: add FCPF device nodes

2016-05-27 Thread Kieran Bingham
Provide nodes for the FCP devices dedicated to the FDP device channels. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- arch/arm64/boot/dts/renesas/r8a7795.dtsi | 21 + 1 file changed, 21 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi

[PATCH 0/4] RCar r8a7795 FCPF support

2016-05-27 Thread Kieran Bingham
/devicetree/bindings/media/renesas,fcp.txt These patches are based on Geert's renesas-drivers tree, and are pushed to a branch at g...@github.com:kbingham/linux.git renesas/fcpf for convenience. Kieran Bingham (4): v4l: Extend FCP compatible list to support the FDP dt-bindings: Update Renesas R-Car FCP

[PATCH 1/4] v4l: Extend FCP compatible list to support the FDP

2016-05-27 Thread Kieran Bingham
The FCP must be powered up for the FDP1 to function, even when the FDP1 does not make use of the FCNL features. Extend the compatible list to allow us to use the power domain and runtime-pm support. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- drivers/media/platform/rcar-fcp.c |

[PATCH 2/4] dt-bindings: Update Renesas R-Car FCP DT binding

2016-05-27 Thread Kieran Bingham
The FCP driver, can also support the FCPF variant for FDP1 compatible processing. Signed-off-by: Kieran Bingham <kie...@bingham.xyz> --- Documentation/devicetree/bindings/media/renesas,fcp.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) Cc: devicet...@vger.kernel.org diff

Re: [PATCH 1/4] fcp: Extend FCP compatible list to support the FDP

2016-05-27 Thread Kieran Bingham
My apologies - I had a stale file in my patches folder :( This one had the wrong commit-shortlog, please ignore. -- Kieran On 27/05/16 18:19, Kieran Bingham wrote: > The FCP must be powered up for the FDP1 to function, even when the FDP1 > does not make use of the FCNL features.

Re: [PATCH v2] scripts/gdb: add constants.py to .gitignore

2016-06-21 Thread Kieran Bingham
.gitignores don't use a leading / on their ignores If you're happy, I'd like to drop the leading '/'. (I think I can do this silently as I pick the patch if you approve, rather than send another patch for a one-byte change) Other than that: Signed-off-by: Kieran Bingham <kie...@bingham.xyz> -- Regards Kieran Bingham

Re: [PATCH] scripts/gdb: add constants.py to .gitignore

2016-06-21 Thread Kieran Bingham
onfig > > # Kdevelop4 > *.kdev4 > + > +# Generated for CONFIG_GDB_SCRIPTS > +/scripts/gdb/linux/constants.py > -- Regards Kieran Bingham

Re: [PATCH v2] scripts/gdb: add constants.py to .gitignore

2016-06-21 Thread Kieran Bingham
heir style. > > If you want to drop the /, I'm okay with it. Thanks for the testing, and checking. Patch picked, and '/' silently dropped. -- Regards Kieran Bingham

Re: [PATCHv2 0/8] gdb/scripts: Linux awareness debug commands

2016-02-29 Thread Kieran Bingham
On 29/02/16 16:36, Jan Kiszka wrote: > Hi Kieran, > > On 2016-02-22 16:24, Kieran Bingham wrote: >> Sending this revised series, which now provides ability to transfer >> configured >> constants from the kernel to the python layer through the constants.py >> in

[PATCH 1/8] scripts/gdb: Provide linux constants

2016-02-22 Thread Kieran Bingham
in and generate the constants file, while maintaining dependencies on autogenerated files in include/generated CC: mma...@suse.com CC: linux-kbu...@vger.kernel.org Signed-off-by: Kieran Bingham <kieran.bing...@linaro.org> --- Changes since v1: Kbuild: - Call to generate constants_py with depe

[PATCH 2/8] scripts/gdb: Provide a kernel list item generators

2016-02-22 Thread Kieran Bingham
Facilitate linked-list items by providing a generator to return the dereferenced, and type-cast objects from a kernel linked list CC: Jeff Mahoney <je...@suse.com> Signed-off-by: Kieran Bingham <kieran.bing...@linaro.org> --- Changes since v1: * items function removed, and replac

[PATCH 3/8] scripts/gdb: Convert modules usage to lists functions

2016-02-22 Thread Kieran Bingham
Simplify the module list functions with the new list_for_each_entry abstractions Signed-off-by: Kieran Bingham <kieran.bing...@linaro.org> --- scripts/gdb/linux/modules.py | 17 ++--- 1 file changed, 6 insertions(+), 11 deletions(-) diff --git a/scripts/gdb/linux/module

[PATCH 6/8] scripts/gdb: Add io resource readers

2016-02-22 Thread Kieran Bingham
been registered Signed-off-by: Kieran Bingham <kieran.bing...@linaro.org> --- Changes since v1: Permenant commit message updated --- scripts/gdb/linux/proc.py | 57 +++ 1 file changed, 57 insertions(+) diff --git a/scripts/gdb/linux/proc.py b/s

Re: [PATCH 8/8] scripts/gdb: Add meminfo command

2016-02-24 Thread Kieran Bingham
I had missed some PEP8 line length warnings on this patch - I've now fixed them up. And a spurious 'pass' statement appeared (highlighted inline), which I've removed. -- Kieran On 22/02/16 15:24, Kieran Bingham wrote: > Provide an equivalent of /proc/meminfo which should be available from >

[PATCH 8/8] scripts/gdb: Add meminfo command

2016-02-22 Thread Kieran Bingham
Provide an equivalent of /proc/meminfo which should be available from core dumps, or crashed kernels. This should allow a debugger to identify if memory pressures were applicable in the instance of their issue Signed-off-by: Kieran Bingham <kieran.bing...@linaro.org> --- Changes f

[PATCH 7/8] scripts/gdb: Add mount point list command

2016-02-22 Thread Kieran Bingham
lx-mounts will identify current mount points based on the 'init_task' namespace by default, as we do not yet have a kernel thread list implementation to select the current running thread. Optionally, a user can specify a PID to list from that process' namespace Signed-off-by: Kieran Bingham

[PATCH 5/8] scripts/gdb: Support !CONFIG_MODULES gracefully

2016-02-22 Thread Kieran Bingham
dule. Error occurred in Python command: No struct type named module. Catch the error and return an empty module_list() for a clean command output as follows: (gdb) lx-lsmod AddressModule Size Used by (gdb) Signed-off-by: Kieran Bingham <kieran.bing...@linaro.org> -

[PATCH 4/8] scripts/gdb: Provide exception catching parser

2016-02-22 Thread Kieran Bingham
an invalid value, without the exception taking our execution context away from us Signed-off-by: Kieran Bingham <kieran.bing...@linaro.org> --- scripts/gdb/linux/utils.py | 7 +++ 1 file changed, 7 insertions(+) diff --git a/scripts/gdb/linux/utils.py b/scripts/gdb/linux/utils.py

[PATCHv2 0/8] gdb/scripts: Linux awareness debug commands

2016-02-22 Thread Kieran Bingham
submission can be found at http://git.linaro.org/people/kieran.bingham/linux.git gdb-scripts-2016-02-22-lkml-submission Kieran Bingham (8): scripts/gdb: Provide linux constants scripts/gdb: Provide a kernel list item generators scripts/gdb: Convert modules usage to lists functions scripts/gdb

Re: Dependency issue through subdir's

2016-01-25 Thread Kieran Bingham
Hi Michal, Thanks for your fast response! On 25/01/16 13:22, Michal Marek wrote: > On 2016-01-25 12:33, Kieran Bingham wrote: >> This is functional on a make -j1 build, however at -j2 and above (from a >> clean build), kbuild attempts to generate my file before >> include/

Re: [PATCH 5/5] scripts/gdb: Add meminfo command

2016-01-23 Thread Kieran Bingham
On 23/01/16 15:21, Jan Kiszka wrote: > On 2016-01-20 12:15, Kieran Bingham wrote: >> Provide an equivalent of /proc/meminfo which should be available from >> core dumps, or crashed kernels. This should allow a debugger to identify >> if memory pressures were appli

Re: [PATCH 3/5] scripts/gdb: Add io resource readers

2016-01-23 Thread Kieran Bingham
On 23/01/16 15:12, Jan Kiszka wrote: > On 2016-01-20 12:15, Kieran Bingham wrote: >> Provide iomem_resource and ioports_resource printers and command hooks >> >> Signed-off-by: Kieran Bingham <kieran.bing...@linaro.org> >> >> --- >> >> These two

Re: [PATCH 2/5] scripts/gdb: Provide a kernel list item generator

2016-01-23 Thread Kieran Bingham
On 23/01/16 15:08, Jan Kiszka wrote: > On 2016-01-20 12:15, Kieran Bingham wrote: >> Facilitate linked-list items by providing a generator to return >> the dereferenced, and type-cast objects from a kernel linked list >> >> Signed-off-by: Kieran Bingham

Re: [PATCH 1/5] scripts/gdb: Provide linux constants

2016-01-23 Thread Kieran Bingham
On 23/01/16 15:05, Jan Kiszka wrote: > On 2016-01-20 12:15, Kieran Bingham wrote: >> Some macro's and defines are needed when parsing memory, and without >> compiling the kernel as -g3 they are not available in the debug-symbols. >> >> We use the pre-processo

Re: [PATCH 4/5] scripts/gdb: Add mount point list command

2016-01-23 Thread Kieran Bingham
On 23/01/16 15:27, Jan Kiszka wrote: > On 2016-01-20 12:15, Kieran Bingham wrote: >> lx-mounts will identify current mount points based on the 'init_task' >> namespace by default, as we do not yet have a kernel thread list >> implementation to select the current running thr

Dependency issue through subdir's

2016-01-25 Thread Kieran Bingham
all 3 of the options above ... I'd appreciate any thoughts here. -- Regards Kieran Bingham

Re: [PATCHv3 10/13] scripts/gdb: Add cpu iterators

2016-03-13 Thread Kieran Bingham
On 13/03/16 16:33, Jan Kiszka wrote: > On 2016-03-03 12:41, Kieran Bingham wrote: >> The linux kernel provides macro's for iterating against values from the >> cpu_list masks. By providing some commonly used masks, we can mirror the >> kernels helper macros with e

Re: [PATCHv3 09/13] scripts/gdb: Add meminfo command

2016-03-13 Thread Kieran Bingham
On 13/03/16 16:34, Jan Kiszka wrote: > On 2016-03-03 12:41, Kieran Bingham wrote: >> Provide an equivalent of /proc/meminfo which should be available from >> core dumps, or crashed kernels. This should allow a debugger to identify >> if memory pressures were appli

Re: [PATCH] drm/msm/dsi: fix definition of msm_dsi_pll_28nm_8960_init()

2016-03-13 Thread Kieran Bingham
On 04/02/16 05:09, Archit Taneja wrote: > > > On 02/03/2016 07:55 PM, Luis Henriques wrote: >> This fixes the following build failure: >> >> drivers/gpu/drm/msm/dsi/pll/dsi_pll_28nm.o: In function >> `msm_dsi_pll_28nm_8960_init': >> dsi_pll_28nm.c:(.text+0x1198): multiple definition of >>

Re: [PATCH] drm/msm/dsi: fix definition of msm_dsi_pll_28nm_8960_init()

2016-03-14 Thread Kieran Bingham
On 14/03/16 01:07, Rob Clark wrote: > On Sun, Mar 13, 2016 at 5:12 PM, Kieran Bingham > <kieran.bing...@linaro.org> wrote: >> On 04/02/16 05:09, Archit Taneja wrote: >>> >>> >>> On 02/03/2016 07:55 PM, Luis Henriques wrote: >>>> This fixes t

Re: [PATCH] scripts/gdb: Account for changes in module data structure

2016-03-14 Thread Kieran Bingham
gt; Testing this has shown me that the module ko search path can find old modules versions in preference to newer ones, but that does not block this patch and can be investigated separately : Reviewed-by: Kieran Bingham <kieran.bing...@linaro.org> Tested-by: Kieran Bingham <kieran.bing...@linaro.or

gdb/scripts: Module symbol search paths

2016-03-14 Thread Kieran Bingham
Hi Jan, Whilst testing the modules update patch you sent, I discovered (due to having rebased to v4.5) that the module search path will end up picking an incorrect version of the .ko file if an earlier version exists.: (gdb) lx-symbols /opt/root/ubuntu-vivid.x86_64 loading vmlinux (gdb) c

Re: gdb/scripts: Module symbol search paths

2016-03-14 Thread Kieran Bingham
On 14/03/16 10:36, Jan Kiszka wrote: > Hi Kieran, > > On 2016-03-14 11:20, Kieran Bingham wrote: >> Hi Jan, >> >> Whilst testing the modules update patch you sent, I discovered (due to >> having rebased to v4.5) that the module search path will end up picking &

Re: gdb/scripts: Module symbol search paths

2016-03-14 Thread Kieran Bingham
On 14/03/16 10:55, Jan Kiszka wrote: > On 2016-03-14 11:48, Kieran Bingham wrote: >> On 14/03/16 10:36, Jan Kiszka wrote: >>> Hi Kieran, >>> >>> On 2016-03-14 11:20, Kieran Bingham wrote: >>>> Hi Jan, >>>> >>>> Whilst tes

Re: [PATCHv3 09/13] scripts/gdb: Add meminfo command

2016-03-14 Thread Kieran Bingham
On 13/03/16 19:08, Jan Kiszka wrote: > On 2016-03-13 19:16, Kieran Bingham wrote: >> On 13/03/16 16:34, Jan Kiszka wrote: >>> On 2016-03-03 12:41, Kieran Bingham wrote: >>>> Provide an equivalent of /proc/meminfo which should be available from >>>> c

Re: [PATCHv3 00/13] scripts/gdb: Linux awareness debug commands

2016-03-14 Thread Kieran Bingham
On 14/03/16 15:09, Jan Kiszka wrote: > On 2016-03-14 15:40, Kieran Bingham wrote: >> On 13/03/16 16:35, Jan Kiszka wrote: >>> On 2016-03-03 12:40, Kieran Bingham wrote: >>>> Hi Jan, >>>> >>>> V3 of the patchset respun. Now finally adding the

Re: [PATCHv3 08/13] scripts/gdb: Add mount point list command

2016-03-15 Thread Kieran Bingham
On 14/03/16 15:05, Jan Kiszka wrote: > On 2016-03-14 15:39, Kieran Bingham wrote: >> On 13/03/16 16:34, Jan Kiszka wrote: >>> On 2016-03-03 12:41, Kieran Bingham wrote: >>>> lx-mounts will identify current mount points based on the 'init_task' >>>> n

Re: [PATCHv3 00/13] scripts/gdb: Linux awareness debug commands

2016-03-14 Thread Kieran Bingham
On 13/03/16 16:35, Jan Kiszka wrote: > On 2016-03-03 12:40, Kieran Bingham wrote: >> Hi Jan, >> >> V3 of the patchset respun. Now finally adding the lx-interrupts command >> after I resolved my issues with the Radix Tree parsing. >> >> This command only pro

Re: [PATCHv3 08/13] scripts/gdb: Add mount point list command

2016-03-14 Thread Kieran Bingham
On 13/03/16 16:34, Jan Kiszka wrote: > On 2016-03-03 12:41, Kieran Bingham wrote: >> lx-mounts will identify current mount points based on the 'init_task' >> namespace by default, as we do not yet have a kernel thread list >> implementation to select the current running thr

Re: [PATCH] Fix issue with dmesg.py and python 3.X

2016-04-07 Thread Kieran Bingham
Hi Dom, I've just tested your patch quickly, and it generates an error on Python 2.7 On 05/04/16 19:38, Dom Cote wrote: > When using GDB built with python 2.7, > > Inferior.read_memory (address, length) > > returns a buffer object. When using GDB built with python 3.X, > it returns a

  1   2   3   4   5   >