Re: [PATCH 14/64] i2c: cpm: reword according to newest specification

2024-04-11 Thread Jochen Friedrich
out_8(&cpm->i2c_reg->i2mod, 0x00); - out_8(&cpm->i2c_reg->i2com, I2COM_MASTER);/* Master mode */ + out_8(&cpm->i2c_reg->i2com, I2COM_MASTER);/* Host mode */ I2COM_MASTER might be coming from the datasheet. Maybe we can just drop the comment? The value we write is pret

Re: [PATCH v2] i2c: cpm: Remove linux,i2c-index conversion from be32

2023-12-08 Thread Jochen Friedrich
Acked-By: Jochen Friedrich Am 06.12.2023 um 23:24 schrieb Christophe Leroy: sparse reports an error on some data that gets converted from be32. That's because that data is typed u32 instead of __be32. The type is correct, the be32_to_cpu() conversion is not. Remove the conversion. Rep

Re: [PATCH 14/89] i2c: cpm: Convert to platform remove callback returning void

2023-05-16 Thread Jochen Friedrich
Acked-by: Jochen Friedrich Am 08.05.2023 um 22:51 schrieb Uwe Kleine-König: The .remove() callback for a platform driver returns an int which makes many driver authors wrongly assume it's possible to do error handling by returning an error code. However the value returned is (mostly) ig

Re: [PATCH v2] i2c: cpm: Fix i2c_ram structure

2020-09-23 Thread Jochen Friedrich
Acked-by: Jochen Friedrich Am 23.09.2020 um 16:08 schrieb nico.vi...@gmail.com: From: Nicolas VINCENT the i2c_ram structure is missing the sdmatmp field mentionned in datasheet for MPC8272 at paragraph 36.5. With this field missing, the hardware would write past the allocated memory done

Re: [PATCH] i2c: remove casting dma_alloc

2019-07-02 Thread Jochen Friedrich
From: Vasyl Generated by: alloc_cast.cocci Signed-off-by: Vasyl Acked-by: Jochen Friedrich --- drivers/i2c/busses/i2c-cpm.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/i2c/busses/i2c-cpm.c b/drivers/i2c/busses/i2c-cpm.c index 187900594e3d

Re: [PATCH] i2c: remove casting dma_alloc

2019-06-29 Thread Jochen Friedrich
Hi Wolfram,yes, I'm still here. However, I don't have the hardware anymore to test the patch.Thanks, Jochen Am 29.06.2019 13:34 schrieb Wolfram Sang :On Sun, Jun 23, 2019 at 11:13:53PM +0200, Vasyl Gomonovych wrote: > From: Vasyl > > Generated by:  alloc_cast.cocci > > Signed-off-by: Vasyl

Re: [PATCH] i2c-cpm: Fix to takeback i2c bus master-ship after a collision

2012-10-31 Thread Jochen Friedrich
Hi Sachin, looks good to me. Acked-By: Jochen Friedrich Signed-off-by: Sachin Surendran mailto:sachin.surend...@alliedtelesis.co.nz>> --- drivers/i2c/busses/i2c-cpm.c |8 1 file changed, 8 insertions(+) diff --git a/drivers/i2c/busses/i2c-cpm.c b/drivers/i2c/busses/i2c

Re: [PATCH RESEND 2/3] i2c/cpm: drop NO_IRQ

2010-04-02 Thread Jochen Friedrich
Wolfram Sang wrote: Drop NO_IRQ as 0 is the preferred way to describe 'no irq' Signed-off-by: Wolfram Sang Acked-by: Grant Likely Acked-by: Jochen Friedrich Thanks, Jochen ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs

Re: [V2 PATCH 06/10] of/drivers: Always use struct device.of_node to get node pointer

2010-03-18 Thread Jochen Friedrich
Hi Grant, The following structure elements duplicate the information in 'struct device.of_node' and so are being eliminated. This patches makes all readers of the following elements use device.of_node instead. struct dev_archdata.prom_node (sparc) struct dev_archdata.of_node (powerpc) struct o

Re: [RFC] Fix stuck on cpm-i2c driver

2009-06-20 Thread Jochen Friedrich
t; on the upper layers. Makes sense. I2C transactions better shouldn't be interrupted or data loss might happen. > Signed-off-by: Bruno Morelli > Signed-off-by: Michael Trimarchi Acked-by: Jochen Friedrich Thanks, Jochen ___ Linuxppc-dev

Re: [PATCH] i2c-cpm: Pass dev ptr to dma_*_coherent rather than NULL

2009-04-21 Thread Jochen Friedrich
Mark Ware schrieb: > Recent DMA changes result in a BUG() when NULL is passed to > dma_alloc_coherent in place of a device. This seems to have happened in 4ae0ff606e848fa4957ebf8f97e5db5fdeec27be. > Signed-off-by: Mark Ware Acked-by: Jochen Friedrich Thank

Re: Please pull bjdoo ks' i2c-fixes

2008-12-18 Thread Jochen Friedrich
Ben Dooks schrieb: > Ok, but this isn't really a fix, so I'll look at putting it into my > i2c-next tree, not i2c-fixes. That's OK for me. Thanks, Jochen ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc

Re: Please pull bjdoo ks' i2c-fixes

2008-12-17 Thread Jochen Friedrich
Hi Ben, > Mike Ditto (1): > i2c-cpm: Detect and report NAK right away instead of timing out Could you also have a look at http://patchwork.ozlabs.org/patch/7452/ ? (Discussion at http://patchwork.ozlabs.org/patch// ) IIRC, Kumar prefers to push this via your tree. Thanks, Jochen _

Re: Erratic MPC8248 CPM2 I2C behaviour

2008-12-04 Thread Jochen Friedrich
Hi Laurent, > The two messages making up the transaction are parsed. The driver fills a TX > buffer descriptor for the first one, and a TX and an RX buffer descriptor for > the second one. > >> rbase 0x01e0 tbase 0x01c0 rfcr 0x30 tfcr 0x30 mrblr 0x0201 >> rstate 0x rptr 0x rbptr

Re: [PATCH] i2c-cpm: Add flexibility for I2C clock frequency and filter.

2008-11-06 Thread Jochen Friedrich
wider range of I2C bus frequencies. > > Signed-off-by: Mike Ditto <[EMAIL PROTECTED]> Acked-by: Jochen Friedrich <[EMAIL PROTECTED]> ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH] i2c-cpm: Detect and report NAK right away instead of timing out.

2008-11-03 Thread Jochen Friedrich
Hi Ben, > On Fri, Oct 31, 2008 at 05:29:25PM -0700, Mike Ditto wrote: >> Make the driver report an ENXIO error immediately upon NAK instead of >> waiting for another interrupt and getting a timeout. >> >> Signed-off-by: Mike Ditto <[EMAIL PROTECTED]> Acked-by: J

Re: [PATCH] i2c-cpm: Add flexibility for I2C clock frequency and filter.

2008-10-31 Thread Jochen Friedrich
Hi Mike, > This patch adds the ability to enable the digital filter in the device > tree (with the "clock-filter" boolean property) and automates the > predivider selection according to the clock-frequency and clock-filter > properties. looks good. David, is "clock-filter" an appropriate dts pro

Re: [PATCH] i2c-cpm: Suppress autoprobing for devices

2008-10-16 Thread Jochen Friedrich
Wolfram Sang schrieb: > Similar to commit 618b26d52843c0f85b8eb143cf2695d7f6fd072d, also remove > automatic probing for this i2c controller. Might need updates to dts files > using it. > > Signed-off-by: Wolfram Sang <[EMAIL PROTECTED]> Acked-by: Jochen Friedr

Re: Please pull from 'for-2.6.27'

2008-09-25 Thread Jochen Friedrich
Hi Scott, >> This isn't a regression since the stuff is new in 2.6.27. IMHO this can >> wait until 2.6.28. > > It would be a bug in 2.6.27 that did not exist in 2.6.26... is there > anything that used to use another mechanism that now uses this? No, GPIO support on CPM1/CPM2 is new in 2.6.27. B

Re: Please pull from 'for-2.6.27'

2008-09-17 Thread Jochen Friedrich
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 Hi Paul, > This looks like a biggish change for after -rc6, and the commit > message doesn't tell me what dire things will happen if the patch > doesn't go in, or what systems see this breakage, or whether this is a > regression since 2.6.26 (I assume

[PATCH] cpm1: Fix race condition in CPM1 GPIO library.

2008-08-27 Thread Jochen Friedrich
The CPM1 GPIO library code uses the non thread-safe clrbits32/setbits32 macros. This patch protects them with a spinlock. Based on the CPM2 patch from Laurent Pinchart <[EMAIL PROTECTED]>, commit 639d64456e20cbfc866b18dc03cf9f9babc9c7cd. Signed-off-by: Jochen Friedrich <[EMAIL

[PATCH] watchdog: delete unused driver mpc8xx_wdt.c

2008-08-08 Thread Jochen Friedrich
The watchdog driver mpc8xx_wdt.c was a device interface to arch/ppc/syslib/m8xx_wdt.c for MPC8xx hardware. Now that ARCH=ppc is gone, this driver is of no more use. For ARCH=powerpc, MPC8xx hardware is supported by mpc8xxx_wdt.c. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> Ac

Re: [PATCH] i2c-mpc: suppress I2C device probing

2008-08-07 Thread Jochen Friedrich
Hi Wolfgang, > $ grep I2C_CLASS_HWMON * > i2c-cpm.c: .class = I2C_CLASS_HWMON | I2C_CLASS_SPD, > i2c-mpc.c: .class = I2C_CLASS_HWMON | I2C_CLASS_SPD, > i2c-ibm_iic.c: adap->class = I2C_CLASS_HWMON | I2C_CLASS_SPD; > i2c-pasemi.c: smbus->adapter.class = I2C_CLASS_HWM

Re: I2C node in device tree breaks old-style drivers

2008-07-30 Thread Jochen Friedrich
This conclusion is correct. One possible way to fix this is to add support for blacklisting to drivers/of/base.c (untested): [RFC] of: Support blacklisting and blacklist cs4270. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- drivers/of/base.c |8 +++- 1 files changed, 7

Re: [PATCH] powerpc: i2c-mpc: make speed registers configurable via FDT

2008-07-25 Thread Jochen Friedrich
Hi Wolfgang, > The I2C driver for the MPC currently uses a fixed speed hard-coded into > the driver. This patch adds the FDT properties "fdr" and "dfsrr" for the > corresponding I2C registers to make the speed configurable via FDT, > e.g.: > > [EMAIL PROTECTED] { > compatible = "fsl-

Re: Getting GPIO to build again

2008-07-24 Thread Jochen Friedrich
Hi Anton, > Though, with this patch: > http://lkml.org/lkml/2008/7/10/269 > > We'll only have to select ARCH_WANT_OPTIONAL_GPIOLIB. Maybe for whole > powerpc. That would simplify the GPIO support patches for CPM1/2, as well. Thanks, Jochen ___ Linuxpp

[PATCH] watchdog: delete unused driver mpc8xx_wdt.c

2008-07-24 Thread Jochen Friedrich
The watchdog driver mpc8xx_wdt.c was a device interface to arch/ppc/syslib/m8xx_wdt.c for MPC8xx hardware. Now that ARCH=ppc is gone, this driver is of no more use. For ARCH=powerpc, MPC8xx hardware is supported by mpc8xxx_wdt.c. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- d

Re: [PATCHv3 1/2] [POWERPC] CPM2: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-07-18 Thread Jochen Friedrich
Hi Kumar, > but ports a-d are different on cpm1? I guess I'd like to see both > patches to understand the commonality and differences. Yes. Both patches are still in patchwork: http://patchwork.ozlabs.org/linuxppc/patch?id=19045 http://patchwork.ozlabs.org/linuxppc/patch?id=19386 Thanks, Joc

Re: [PATCHv3 1/2] [POWERPC] CPM2: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-07-18 Thread Jochen Friedrich
Hi Kumar, > On Jun 18, 2008, at 12:08 PM, Laurent Pinchart wrote: > >> +#if defined(CONFIG_CPM2) || defined(CONFIG_8xx_GPIO) >> + >> +struct cpm2_ioports { >> +u32 dir, par, sor, odr, dat; >> +u32 res[3]; >> +}; >> + > > is this really common for both CPM2 and 8xx? if so why the name?

Re: [RFC] I2C: fsl-i2c: make device probing configurable via FDT

2008-07-16 Thread Jochen Friedrich
Hi Grant, > On Wed, Jul 16, 2008 at 12:47:25PM +0200, Wolfgang Grandegger wrote: >> Currently, the I2C buses are probed for HWMON I2C devices, which might >> not be acceptable in same cases. This patch makes device probing >> configurable through the property "probe" of the FDT I2C device node: >>

Re: [PATCH] of: i2c: improve last resort compatible entry selection

2008-07-15 Thread Jochen Friedrich
Hi Jean, > Eeeek. The patch you mention here is only the conversion of ONE driver. > It is absolutely not relevant as to what the general rule is. Sorry, i must have misunderstood you then. http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=af294867a52bf718df835a688e

Re: [PATCH] of: i2c: improve last resort compatible entry selection

2008-07-15 Thread Jochen Friedrich
9ed300;hb=ae429083efe996ca2c569c44fd6fea440676dc33;hpb=60b129d7bfa3e20450816983bd52c49bb0bc1c21) So your patch is correct. > Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> Acked-by: Jochen Friedrich <[EMAIL PROTECTED]> Thanks, Jochen ___ Linuxppc-d

Re: [PATCH] of/i2c: don't pass -1 to irq_dispose_mapping, otherwise kernel will oops

2008-07-11 Thread Jochen Friedrich
Hi Anton, > diff --git a/drivers/of/of_i2c.c b/drivers/of/of_i2c.c > index b2ccdcb..95a24de 100644 > --- a/drivers/of/of_i2c.c > +++ b/drivers/of/of_i2c.c > @@ -93,10 +93,8 @@ void of_register_i2c_devices(struct i2c_adapter *adap, > if (info.irq == NO_IRQ) > inf

Re: [PATCH] of/i2c: don't pass -1 to irq_dispose_mapping, otherwise kernel will oops

2008-07-11 Thread Jochen Friedrich
Hi Grant, > What is the reason that info.irq is set to -1 in the first place? This > looks like another bug to me. Does something in the i2c layer depend on > the -1 value? Nope, it was a bug in the i2c documentation fixed recently: http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.

Re: New fsl device bindings file

2008-07-05 Thread Jochen Friedrich
Hi Scott, >> I'm going to move all the Freescale SoC related bindings into this new >> file. One of the aspects of the new file is we will NOT having an >> ordinal heading index. > > How about splitting up like this: > > Documentation/powerpc/device-tree/fsl/cpm.txt > Documentation/powerpc/de

[PATCH] powerpc: Add documentation for CPM GPIO banks

2008-07-05 Thread Jochen Friedrich
Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- Documentation/powerpc/device-tree/fsl/cpm/gpio.txt | 38 1 files changed, 38 insertions(+), 0 deletions(-) create mode 100644 Documentation/powerpc/device-tree/fsl/cpm/gpio.txt diff --git a/Documentation/p

[PATCH] powerpc: implement GPIO LIB API on CPM1 Freescale SoC.

2008-07-02 Thread Jochen Friedrich
This patch implement GPIO LIB support for the CPM1 GPIOs. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- This patch depends on the GPIO LIB support for the CPM2 GPIOs patch from Laurent Pinchart. Changes since last submission: - of_gc->gpio_cells = 2 to allow passing port o

[PATCH] powerpc: Add documentation for CPM GPIO banks to booting-without-of

2008-07-02 Thread Jochen Friedrich
Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- Documentation/powerpc/booting-without-of.txt | 39 ++ 1 files changed, 39 insertions(+), 0 deletions(-) diff --git a/Documentation/powerpc/booting-without-of.txt b/Documentation/powerpc/booting-without-

[PATCH] powerpc: Add i2c pins to dts and board setup

2008-07-02 Thread Jochen Friedrich
d-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- Documentation/powerpc/booting-without-of.txt | 42 ++ arch/powerpc/boot/dts/mpc8272ads.dts | 11 +++ arch/powerpc/boot/dts/mpc866ads.dts | 11 +++ arch/powerpc/boot/dts/mpc885ads.dts

[PATCH] powerpc: fix section numbering of CPM chapter in booting-without-of.

2008-07-02 Thread Jochen Friedrich
Within the Freescale Communications Processor Module chapter, both Serial and Network have the same section number iii). Renumber the sections to make them unique again. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- Documentation/powerpc/booting-without-of.txt | 12 ++---

Re: Commits added to powerpc.git master and powerpc-next branches

2008-07-02 Thread Jochen Friedrich
Hi Kumar, >> any chance to get this one into 2.6.27? >> >> [POWERPC] Add i2c pins to dts and board setup >> http://patchwork.ozlabs.org/linuxppc/patch?person=1023&id=18603 > > Will we always want defconfig updates to enable the driver? Good question. Maybe not, as the i2c bus seems to be unused

Re: Commits added to powerpc.git master and powerpc-next branches

2008-07-01 Thread Jochen Friedrich
Hi Paul, On Tuesday 01 July 2008, Paul Mackerras wrote: > The following commits have been added to the master and powerpc-next > branches of the powerpc.git repository. This includes patches pulled > from Kumar's and Josh's trees. Now that the I2C driver for CPM1/CPM2 boards is in the i2c tree,

Re: [PATCH 2/2] Add the of_find_i2c_device_by_node function, V4

2008-07-01 Thread Jochen Friedrich
Hi Jean, > I'm not the one to push this upstream though, as the patch is > essentially an openfirmware patch. That would be something for Jochen > Friedrich and Paul Mackerras I guess. Would be nice to have a > MAINTAINERS entry for OF... Nope. I only did a small contribution t

Re: [PATCH2/2] [POWERPC] CPM1: implement GPIO LIB API on CPM1 Freescale SoC.

2008-07-01 Thread Jochen Friedrich
Hi Grant, sorry for the late response on this one. > 2. You need to specifiy exact chip names in your compatible string. > "fsl,cpm1-pario-" is a made up thing. >> + for_each_compatible_node(np, NULL, "fsl,cpm1-pario-bank16") >> + cpm1_gpiochip_add16(np); >> + >> +

Re: [PATCH] Convert i2c-mpc from a platform driver into a of_platform driver, V2

2008-06-29 Thread Jochen Friedrich
Hi Jon, > This version adds of_find_i2c_device_by_node() which is needed by ASOC > drivers. Cool. I was about to do something similar (of_get_adapter_nr_by_node). IMHO, the patch should be splitted into the addition of of_find_i2c_device_by_node() and the actual patch of i2c-mpc. Thanks, Joch

Re: [PATCHv3 1/2] [POWERPC] CPM2: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-06-27 Thread Jochen Friedrich
Hi Laurent, > Is there any pending issue or can this be applied to powerpc-next ? Looks OK to me. >> Signed-off-by: Laurent Pinchart <[EMAIL PROTECTED]> >> Cc: Jochen Friedrich <[EMAIL PROTECTED]> Acked-by: Jochen Friedrich <[EMA

Re: "cell-index" vs. "index" vs. no index in I2C device nodes

2008-06-27 Thread Jochen Friedrich
Hi Timur, > Anyone know when of_i2c.c is going to come "online" replace what's in > fsl_soc.c? There is a patch being prepared to convert i2c-mpc from a platform driver to an of_platform driver making the fsl_soc.c code obsolete. See: http://patchwork.ozlabs.org/linuxppc/patch?id=18898 Thanks

Re: [PATCHv2 1/2] [POWERPC] CPM2: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-06-18 Thread Jochen Friedrich
Hi Laurent, Agreed. Jochen, will you resubmit or should I do it ? Please do as i'm currently away and won't be back until next week. Thanks, Jochen ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxp

Re: arch/ppc is going away Real Soon Now

2008-06-08 Thread Jochen Friedrich
> >>> Acked-by: Arnd Bergmann <[EMAIL PROTECTED]> >> > > >> Acked-by: Olof Johansson <[EMAIL PROTECTED]> >> > > > Acked-by: Scott Wood <[EMAIL PROTECTED]> >> > > Acked-by: Becky Bruce <[EMAIL PROTECTED]> >&

Re: "cell-index" vs. "index" vs. no index in I2C device nodes

2008-06-05 Thread Jochen Friedrich
Hi Timur, > It's a little late for that. I'm okay with coming up with a new property to > provide system-level indexing, but it needs to be the same property name for > each type of device. I don't want linux,i2c-index and linux,dma-index and > linux,ssi-index, etc. I also don't understand why

Re: "cell-index" vs. "index" vs. no index in I2C device nodes

2008-06-05 Thread Jochen Friedrich
Hi Grant, > if you need explicit indexing then use an alias. My opinion however > is that explicit indexing is unnecessary and is just an artifact of > current i2c subsystem internals. There is already enough information > in the device tree to match i2c devices with i2c busses without > resorti

Re: "cell-index" vs. "index" vs. no index in I2C device nodes

2008-06-05 Thread Jochen Friedrich
Hi Timur, > In situations where it doesn't matter which I2C bus is #1 and which one is #2, > then I think the code should just initialize idx based on the order the nodes > are found in the tree. > > In situations where it does matter, then we should use cell-index. that's what I did in i2c-cpm,

[PATCH] i2c: Add MAINTAINER entry for i2c-cpm.

2008-06-03 Thread Jochen Friedrich
Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- MAINTAINERS |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/MAINTAINERS b/MAINTAINERS index 0a6d2ca..bdfc7bd 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -1659,6 +1659,13 @@ L: [EMAIL PRO

Re: [PATCH 3/4] spi: Add OF binding support for SPI busses

2008-05-24 Thread Jochen Friedrich
Grant Likely schrieb: > On Sat, May 24, 2008 at 12:26 AM, Grant Likely > <[EMAIL PROTECTED]> wrote: >> On Thu, May 22, 2008 at 8:05 PM, David Brownell <[EMAIL PROTECTED]> wrote: >>> On Wednesday 21 May 2008, Anton Vorontsov wrote: > +++ b/drivers/spi/spi_of.c I think better placement for t

Re: [RFC PATCH 2/2] mmc: add OpenFirmware bindings for the mmc_spi driver

2008-05-24 Thread Jochen Friedrich
Hi Grant, >> + - compatible : should be "linux,mmc-spi". >> + - linux,modalias - should be "of_mmc_spi". > > I'm not even sure if the whole linux,modalias is even a good idea. I > had kind of thrown it in there as a convenient way to override > compatible when needed, but I haven't re

[PATCHv2] i2c: adds support for i2c bus on Freescale CPM1/CPM2 controllers

2008-05-21 Thread Jochen Friedrich
dbox2 hardware) and CPM2 (MPC8272). Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- drivers/i2c/busses/Kconfig | 10 + drivers/i2c/busses/Makefile |1 + drivers/i2c/busses/i2c-cpm.c | 745 ++ 3 files changed, 756 insertions(+),

[PATCHv2] [POWERPC] Add i2c pins to dts and board setup

2008-05-21 Thread Jochen Friedrich
Initialize I2C pins on boards with CPM1/CPM2 controllers. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- Documentation/powerpc/booting-without-of.txt | 42 ++ arch/powerpc/boot/dts/mpc8272ads.dts | 11 +++ arch/powerpc/boot/dts/mpc866a

Re: [PATCH] [POWERPC] Add i2c pins to dts and board setup

2008-05-21 Thread Jochen Friedrich
Hi Scott, >> + - linux,i2c-index : Can be used to hard code an i2c bus number. >> + - linux,i2c-class : Can be used to override the i2c class. > > Why do we need this? There are still a bunch of i2c drivers using the old API (mainly v4l and dvb stuff) which are slowly being converted by thei

[PATCH] i2c: adds support for i2c bus on Freescale CPM1/CPM2 controllers

2008-05-21 Thread Jochen Friedrich
dbox2 hardware) and CPM2 (MPC8272). Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- drivers/i2c/busses/Kconfig | 10 + drivers/i2c/busses/Makefile |1 + drivers/i2c/busses/i2c-cpm.c | 745 ++ 3 files changed, 756 insertions(+),

[PATCH] [POWERPC] Add i2c pins to dts and board setup

2008-05-21 Thread Jochen Friedrich
Initialize I2C pins on boards with CPM1/CPM2 controllers. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- Documentation/powerpc/booting-without-of.txt | 39 ++ arch/powerpc/boot/dts/mpc8272ads.dts | 11 +++ arch/powerpc/boot/dts/mpc866a

Re: [PATCH 6/8] [WATCHDOG] mpc8xxx_wdt: add support for MPC8xx watchdogs

2008-05-16 Thread Jochen Friedrich
ed to work on the actual hardware. > > Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> Tested-by: Jochen Friedrich <[EMAIL PROTECTED]> The driver works OK on MPC823. One nit however ist the late initialisation of the wdt timer. I had to add two hardcoded wdt resets in the board se

Re: [PATCH 0/8 v3] mpc83xx_wdt rework, support for mpc8610 and mpc8xx

2008-05-15 Thread Jochen Friedrich
Hi Anton, > - New patch supporting MPC8xx watchdogs (ok to drop until tested); Unfortunately, current 2.6.26-rc2 doesn't boot at all on MPC823. I'll have to bisect which patch broke it. I hope to have some testing results, soon. Thanks, Jochen ___ Linu

[PATCHv2] [POWERPC] Add i2c pins to dts and board setup

2008-05-15 Thread Jochen Friedrich
Initialize I2C pins on boards with CPM1/CPM2 controllers. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- Documentation/powerpc/booting-without-of.txt | 39 ++ arch/powerpc/boot/dts/mpc8272ads.dts | 12 arch/powerpc/boot/dts/mpc866a

Re: [patch 2/4] powerpc: add i2c pins to dts and board setup

2008-05-15 Thread Jochen Friedrich
Hi David, >> As I pointed out earlier, this patch is sticking dts-v0 style constants >> into a dts-v1 file. It will not work. > > Enough of this. *Sends patch converting all remaining dts files to > v1*. With any luck we can merge that soon, and forget about the > mistakes of v0 forever. Ma

Re: [PATCH 1/3] [WATCHDOG] mpc83xx_wdt: add support for MPC86xx CPUs

2008-05-12 Thread Jochen Friedrich
Hi Kumar, >> Also move the probe code into subsys_initcall, because we want start >> pinging the watchdog ASAP, and misc devices are available in >> subsys_initcall. >> >> Signed-off-by: Anton Vorontsov <[EMAIL PROTECTED]> >> --- >> drivers/watchdog/Kconfig |4 +- >> drivers/watchdog/mpc8

[PATCH] i2c: adds support for i2c bus on Freescale CPM1/CPM2 controllers

2008-05-10 Thread Jochen Friedrich
dbox2 hardware) and CPM2 (MPC8272). Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- drivers/i2c/busses/Kconfig | 10 + drivers/i2c/busses/Makefile |1 + drivers/i2c/busses/i2c-cpm.c | 728 ++ 3 files changed, 739 insertions(+),

[PATCH] [POWERPC] Add i2c pins to dts and board setup

2008-05-10 Thread Jochen Friedrich
Initialize I2C pins on boards with CPM1/CPM2 controllers. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- arch/powerpc/boot/dts/mpc8272ads.dts | 10 ++ arch/powerpc/boot/dts/mpc866ads.dts | 10 ++ arch/powerpc/boot/dts/mpc885ads.dts

[PATCHv2 2.6.26-rc1] [POWERPC] Fix of_i2c include for module compilation

2008-05-06 Thread Jochen Friedrich
Remove #ifdef CONFIG_OF_I2C as this breaks module compilation. Drivers using this header should depend on OF_I2C anyways, so there's no need to make this conditional Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- include/linux/of_i2c.h |4 1 files changed, 0 inser

Re: [PATCH] [POWERPC] Fix of_i2c include for module compilation

2008-05-06 Thread Jochen Friedrich
Hi Paul, >> -#ifdef CONFIG_OF_I2C >> +#if defined(CONFIG_OF_I2C) || defined(CONFIG_OF_I2C_MODULE) >> >> void of_register_i2c_devices(struct i2c_adapter *adap, >> struct device_node *adap_node); > > Why do we have that ifdef there at all? There's only that one > exter

Re: [i2c] [PATCH6/7] i2c: adds support for i2c bus on Freescale CPM1/CPM2 controllers

2008-05-06 Thread Jochen Friedrich
Hi Wolfram, >> +/* Begin transmission */ >> +setbits8(&i2c_reg->i2com, 0x80); > Hardcoded value. (I also wonder if 0x81 might be more suitable, as it > keeps the "be-a-master"-bit set. Still, both values work with my setup.) >> +#ifdef I2C_CHIP_ERRATA >> +/* >> + * Chip errata, cl

[PATCH] [POWERPC] Fix of_i2c include for module compilation

2008-05-03 Thread Jochen Friedrich
Now that of_i2c may be compiled as module, the header must be exported for module compilation, as well. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- include/linux/of_i2c.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/linux/of_i2c.h b/include

Re: [i2c] [PATCH 0/2] i2c: Add support for device alias names

2008-04-28 Thread Jochen Friedrich
Hi Wolfram, > I tested on this hardware > > MPC8260 (powerpc) + PCF8575 (io expander) + LM84 (sensor) > + RS5C372 (rtc) + X24645 (eeprom) It's also OK on dbox2 hardware: MPC823 (powerpc) + saa7127 (patch needed to add id_table) + dbox frontprocessor (8051 controller with i2c interface). Thank

Re: [PATCH1/7] i2c: Add support for device alias names

2008-04-27 Thread Jochen Friedrich
Hi Jean, >> -if ((err = f75375_probe(client)) < 0) >> +err = f75375_probe(client, NULL); >> +if (err < 0) >> goto exit_detach; >> >> return 0; > > These coding style cleanups don't belong to this patch (if they belong > anywhere - I don't much see the point) I jus

Re: [i2c] platform_data when using of?

2008-04-23 Thread Jochen Friedrich
Hi Wolfram, > Hello, > > I finally could get the work started with I2C on a MPC8260-based > platform. I applied Jochen's series on top of 2.6.25 and it seems I > could get the i2c-cpm and the rtc-rs5c372 driver working (except that it > doesn't autoload as a module, but I think this is my fault s

Re: [PATCH] rtc-pcf8563: Add device ids table

2008-04-23 Thread Jochen Friedrich
Hi Jean, > Jochen, I'm a bit confused by the dependencies that exist - or not - > between these 7 patches you sent at once. I thought they had to be > applied in sequence but it seems not? And some of them should > apparently go through me i2c tree but others (e.g. [7/7]) not? [1/7] and [2/7] are

Re: [PATCH] rtc-pcf8563: Add device ids table

2008-04-23 Thread Jochen Friedrich
Hi Stefan, > Please don't get we wrong, I didn't want to put additional pressure on you > here. I just wanted to express that I'm waiting for these patches to arrive > upstream quite some time now too. and unfortunately, the longer we wait the more drivers are ported to new-style and the bigger

Re: [2.6 patch] drivers/of/of_i2c.c: add MODULE_LICENSE

2008-04-23 Thread Jochen Friedrich
Adrian Bunk schrieb: > After commit 585468e5d5962660867c269e26f0a4b89a599473 > ([POWERPC] i2c: Fix build breakage introduced by OF helpers) > drivers/of/of_i2c.c needs a MODULE_LICENSE. > > Signed-off-by: Adrian Bunk <[EMAIL PROTECTED]> Acked-by: Jochen Friedrich <[E

Re: [PATCH] rtc-pcf8563: Add device ids table

2008-04-19 Thread Jochen Friedrich
Jean Delvare wrote: > On Sat, 19 Apr 2008 15:09:55 +0200, Jochen Friedrich wrote: >> Hi Jean, >> >>>>> Is the new-style driver conversion patch in 2.6.25-rc8-mm2 scheduled for >>>>> 2.6.26 ? >>>> hope so! :) >>> Note that this p

[PATCHv2 1/2] [POWERPC] CPM2: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-04-18 Thread Jochen Friedrich
Based on earlier work by Laurent Pinchart. This patch implement GPIO LIB support for the CPM2 GPIOs. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> Cc: Laurent Pinchart <[EMAIL PROTECTED]> --- arch/powerpc/platforms/Kconfig |2 + arch/powerpc/sysdev/cpm2.c | 1

[PATCH2/2] [POWERPC] CPM1: implement GPIO LIB API on CPM1 Freescale SoC.

2008-04-18 Thread Jochen Friedrich
This patch implement GPIO LIB support for the CPM1 GPIOs. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- arch/powerpc/platforms/8xx/Kconfig | 10 ++ arch/powerpc/sysdev/cpm1.c | 261 +++- 2 files changed, 266 insertions(+), 5 deletions(-)

[PATCH1/2] [POWERPC] CPM2: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-04-18 Thread Jochen Friedrich
Based on earlier work by Laurent Pinchart. This patch implement GPIO LIB support for the CPM2 GPIOs. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> Cc: Laurent Pinchart <[EMAIL PROTECTED]> --- arch/powerpc/platforms/Kconfig |2 + arch/powerpc/sysdev/cpm2.c | 1

[PATCH] [POWERPC] i2c: fix build breakage introduced by OF helpers

2008-04-18 Thread Jochen Friedrich
Fix build breakage introduced in commit "[POWERPC] i2c: OF helpers for the i2c API". If i2c-core is compiled as module, the helper needs to be compiled as module, as well. Rename i2c.c to of_i2c.c to avoid name space conflict. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]>

[PATCHv3 3/7] i2c: OF helpers for the i2c API

2008-04-18 Thread Jochen Friedrich
This patch implements various helpers to support OF bindings for the i2c API. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> Acked-by: David S. Miller <[EMAIL PROTECTED]> Acked-by: Stephen Rothwell <[EMAIL PROTECTED]> --- drivers/of/Kconfig |6 +++ drivers/of/

Re: build breakage from of i2c helper patch

2008-04-18 Thread Jochen Friedrich
Hi Paul, > Which way around is this dependency? Do you mean that the patches you > listed depend on the "OF helpers for the i2c API" patch, or that the > OF helpers patch depends on these other ones? The OF helpers patch depends on the patches below to support autoloading i2c modules using the a

Re: build breakage from of i2c helper patch

2008-04-18 Thread Jochen Friedrich
Hi Jean, > (blind shot) Maybe this means that some code that is in i2c-core should > be moved to i2c-boardinfo instead? i2c-boardinfo is always built into > the kernel even when i2c-core is modular. unfortunately, i2c_new_device() tries to attach the module, so this function really needs full i2c

Re: [PATCH] powerpc: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-04-17 Thread Jochen Friedrich
Hi Laurent, > Any preference for common function names ? cpm2_gpio32* might be too > CPM2-specific for cpm_common.c. cpm_gpio32* is a bit too generic as it > doesn't support port E on the CPM1. I would keep the cpm2 name. I'll just add a comment to cpm1.c that fsl,cpm1-pario-bank32e uses cpm2 l

Re: [PATCH] powerpc: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-04-17 Thread Jochen Friedrich
Hi Laurent, > BTW, what's the purpose of the shadow data register in the CPM GPIO chip > structure ? As the set operation is protected by a spinlock, the only use I > can think of is to prevent changing bits in the PDAT register for pins > configured as inputs. Is that correct ? If so this shou

Re: [PATCH] powerpc: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-04-17 Thread Jochen Friedrich
Hi Laurent, >> http://patchwork.ozlabs.org/linuxppc/patch?id=16282 > > The link doesn't seem to be related. http://patchwork.ozlabs.org/linuxppc/patch?id=17490 is the correct one. > I'll rework the patch to move CPM2 GPIO support to cpm_common.c. Have you > thought about a proper name for the

Re: [PATCH] powerpc: Implement GPIO LIB API on CPM2 Freescale SoC.

2008-04-17 Thread Jochen Friedrich
Hi Laurent, > This patch implement GPIO LIB support for the CPM2 GPIOs. > > The code is based on Jochen Friedrich's patch for the CPM1. I guess most of this should go to cpm_common.c and should be used for Port E on CPM1, as well. This port uses the same layout as CPM2 while Port B uses a diffe

Re: build breakage from of i2c helper patch

2008-04-17 Thread Jochen Friedrich
Hi Kumar, > I'm guessing the "[POWERPC] i2c: OF helpers for the i2c API" > > http://git.kernel.org/?p=linux/kernel/git/paulus/powerpc.git;a=commitdiff;h=612212a3f2f053ea68ce9cd16d3deeca7754e8c9 > > is causing this build failure: > > http://kisskb.ellerman.id.au/kisskb/buildresult/24375/ It loo

[PATCHv2 7/7] [POWERPC] Add i2c pins to dts and board setup

2008-04-11 Thread Jochen Friedrich
Initialize I2C pins on boards with CPM1/CPM2 controllers. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- arch/powerpc/boot/dts/mpc8272ads.dts | 10 ++ arch/powerpc/boot/dts/mpc866ads.dts | 10 ++ arch/powerpc/boot/dts/mpc885ads.dts

Re: [PATCH3/7] i2c: OF helpers for the i2c API

2008-04-11 Thread Jochen Friedrich
David Miller schrieb: > I wouldn't mind trying to use this infrastructure on > sparc64, and I don't see any powerpc specific interfaces > being invoked, so if you could remove the PPC_OF > requirement I'd appreciate it. I've resenf the patch without the dependency on PPC_OF. Thanks for the comme

[PATCHv2 3/7] i2c: OF helpers for the i2c API

2008-04-11 Thread Jochen Friedrich
This patch implements various helpers to support OF bindings for the i2c API. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- drivers/of/Kconfig |6 +++ drivers/of/Makefile|1 + drivers/of/i2c.c | 115 include

[PATCH7/7] [POWERPC] Add i2c pins to dts and board setup

2008-04-11 Thread Jochen Friedrich
Initialize I2C pins on boards with CPM1/CPM2 controllers. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- arch/powerpc/boot/dts/mpc8272ads.dts | 10 ++ arch/powerpc/boot/dts/mpc866ads.dts | 10 ++ arch/powerpc/boot/dts/mpc885ads.dts

[PATCH6/7] i2c: adds support for i2c bus on Freescale CPM1/CPM2 controllers

2008-04-11 Thread Jochen Friedrich
dbox2 hardware) and CPM2 (MPC8272). Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- drivers/i2c/busses/Kconfig | 10 + drivers/i2c/busses/Makefile |1 + drivers/i2c/busses/i2c-cpm.c | 728 ++ 3 files changed, 739 insertions(+),

[PATCH5/7] i2c: Kill the old driver matching scheme

2008-04-11 Thread Jochen Friedrich
Based on earlier work by Jon Smirl and Jean Delvare. Remove the old driver_name/type scheme for i2c driver matching. Only the standard aliasing model will be used from now on. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> Cc: Jean Delvare <[EMAIL PROTECTED]> Cc: Jon Smirl <[

[PATCH4/7] i2c: Convert PowerPC MPC i2c to of_platform_driver from platform_driver

2008-04-11 Thread Jochen Friedrich
c_ to make them match the file name. Common code moved to of-common.* Orginal ppc driver left intact for deletion when ppc arch is removed. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> Cc: Jon Smirl <[EMAIL PROTECTED]> --- arch/powerpc/sysdev

[PATCH3/7] i2c: OF helpers for the i2c API

2008-04-11 Thread Jochen Friedrich
This patch implements various helpers to support OF bindings for the i2c API. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> --- drivers/of/Kconfig |6 +++ drivers/of/Makefile|1 + drivers/of/i2c.c | 115 include

[PATCH2/7] i2c: Convert all new-style drivers to use module aliasing

2008-04-11 Thread Jochen Friedrich
Based on earlier work by Jon Smirl and Jean Delvare. Update all the new-style i2c drivers to use standard module aliasing instead of the old driver_name/type driver matching scheme. Signed-off-by: Jochen Friedrich <[EMAIL PROTECTED]> Cc: Jean Delvare <[EMAIL PROTECTED]> Cc: Jon S

[PATCH1/7] i2c: Add support for device alias names

2008-04-11 Thread Jochen Friedrich
Based on earlier work by Jon Smirl and Jean Delvare. This patch allows new-style i2c chip drivers to have alias names using the official kernel aliasing system and MODULE_DEVICE_TABLE(). At this point, the old i2c driver binding scheme (driver_name/type) is still supported. Signed-off-by: Jochen

  1   2   3   >