gmane.linux.drivers.i2c

(unknown)

robothroli company — 2012-05-25T13:45:54

 i am robothroli, Purchase manager from roli Merchant Ltd. We are
Import/export Company based in taiwan. We are interested in purchasing
your product and I would like to make an inquiry. Please inform me on:

Sample availability and price
Minimum order quantity
FOB Prices

Sincerely
Purchase Manager
robothroli

Re: [PATCHv9 00/10] I2C fixes

Kevin Hilman — 2012-05-25T22:13:36


Can you also describe how it was tested?  

With the runtime PM changes, does it still hit full-chip retention in
idle and suspend after these changes?

I had a few minor comments on this version, otherwise feel free add

Reviewed-by: Kevin Hilman <khilman-l0cyMroinI0< at >public.gmane.org>

That being said, before this is merged, I woudl like to see some more
non-author Tested-bys.  We've been having lots of regressions of late
from OMAP drivers that are not being sufficiently tested before
merging.  We need to ensure proper testing before merge.

Other testers should also report what platforms they tested on, and how
it was tested.

Thanks,

Kevin

Re: [PATCHv9 07/10] I2C: OMAP: Handle error check for pm runtime

Kevin Hilman — 2012-05-25T22:06:04


nit: please use IS_ERR_VALUE() 

Kevin

Re: [PATCHv9 06/10] I2C: OMAP: Fix the crash in i2c remove

Kevin Hilman — 2012-05-25T21:51:34

Hi Wolfram,

Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ< at >public.gmane.org> writes:


[...]


Yes, this is correct.

In fact, this is the goal of runtime PM.  The driver itself tells the PM
core (using runtime PM) when the device needs to be accessible and when
it doesn't.

Technically speaking, the it's up to the platform-specific runtime PM
implementation to decide whether or not the clocks are actually disable
or not  (e.g. due to wakeup latency requirements, it might decide not to
cut clocks.)

Because of that, the changelog should be reworded to say something like
"ensure device is accessible" instead of "enable the clocks", because
the runtime PM implementation does more than just manage clocks.

Kevin

P.S. It's great to see you helping out maintaining i2c drivers.  Thanks!

P.P.S. Before you merge this, I would strongly recommend we wait for a
       few more Tested-bys, and a bit more description about how this
       was tested.  We've been having quite a few problems with
       regressions introdu

Re: [PATCH 8/8] i2c: i2c-bfin-twi: Move blackfin TWI register access Macro to head file.

Shubhrajyoti Datta — 2012-05-25T10:16:06

Great,

Apologies for the noise.

Re: [PATCH 8/8] i2c: i2c-bfin-twi: Move blackfin TWI register access Macro to head file.

Shubhrajyoti Datta — 2012-05-25T10:10:16

Hi ,
On Fri, May 25, 2012 at 3:24 PM, Zhang, Sonic <Sonic.Zhang-OyLXuOCK7orQT0dZR+AlfA< at >public.gmane.org> wrote:

Got it thanks.

You may want to add a dependency otherwise it might lead to a compilation break?

RE: [PATCH 8/8] i2c: i2c-bfin-twi: Move blackfin TWI register access Macro to head file.

Zhang, Sonic — 2012-05-25T09:54:13



The subject says these definitions are moved to a head file by the other patch for Blackfin architecture.

Regards,

Sonic

Re: [PATCH 8/8] i2c: i2c-bfin-twi: Move blackfin TWI register access Macro to head file.

Shubhrajyoti Datta — 2012-05-25T09:48:19

Hi ,
Some minor comments/ doubts.
On Wed, May 16, 2012 at 12:50 PM,  <sonic.zhang-OyLXuOCK7orQT0dZR+AlfA< at >public.gmane.org> wrote:

A small description may be helpful.
Also the $SUBJECT says move the patch looks more like the remove.
Am I missing something.

Re: [PATCH 8/8] i2c: i2c-bfin-twi: Move blackfin TWI register access Macro to head file.

Sonic Zhang — 2012-05-24T09:19:08

PING

On Wed, May 16, 2012 at 3:20 PM,  <sonic.zhang-OyLXuOCK7orQT0dZR+AlfA< at >public.gmane.org> wrote:

Re: [PATCH 6/8] i2c:i2c-bfin-twi: include twi head file

Sonic Zhang — 2012-05-24T09:18:32

PING

On Wed, May 16, 2012 at 3:20 PM,  <sonic.zhang-OyLXuOCK7orQT0dZR+AlfA< at >public.gmane.org> wrote:

GET BACK TO ME ASAP.

Mrs Anna Kennedy — 2012-05-23T18:58:27

Good day my beloved friend,

How are you and your lovely family doing today,i hope all is well?if so glory 
be to God,i have an urgent purposal for you, if interested kindly contact me on 
this e-mail (anna_kennedy_hood-PkbjNfxxIAQqdlJmJB21zg< at >public.gmane.org)

[PATCH v4 7/7] ARM: davinci: add support for the am1808 based enbw_cmc board

Heiko Schocher — 2012-05-22T13:55:20

- AM1808 based board
- 64 MiB DDR ram
- 2 MiB Nor flash
- 128 MiB NAND flash
- use internal RTC
- I2C support
- hwmon lm75 support
- UBI/UBIFS support
- MMC support
- USB OTG support

Signed-off-by: Heiko Schocher <hs-ynQEQJNshbs< at >public.gmane.org>
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r< at >public.gmane.org
Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ< at >public.gmane.org
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/< at >public.gmane.org
Cc: linux-mtd-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r< at >public.gmane.org
Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA< at >public.gmane.org
Cc: netdev-u79uwXL29TY76Z2rM5mHXA< at >public.gmane.org
Cc: David Woodhouse <dwmw2-wEGCiKHe2LqWVfeAwA7xHQ< at >public.gmane.org>
Cc: Ben Dooks <ben-linux-elnMNo+KYs3YtjvyW6yDsg< at >public.gmane.org>
Cc: Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ< at >public.gmane.org>
Cc: Sekhar Nori <nsekhar-l0cyMroinI0< at >public.gmane.org>
Cc: Kevin Hilman <khilman-l0cyMroinI0< at >public.gmane.org>
Cc: Wolfgang Denk <wd-ynQEQJNshbs< at >public.gmane.org>
Cc: Scott Wood <scottwood-KZf

[PATCH v4 0/7] ARM: davinci: add support for the am1808 based enbw_cmc board

Heiko Schocher — 2012-05-22T13:55:13

this patchserie add support for the davinci am1808 based
enbw_cmc board.

changes for v2:
Post this patchserie now as v2, as reworked in the
comments I got for the RFC serie.

changes for v3:
- Interrupt Controller:
  - comment from Sergei Shtylyov:
    - rename compatible" prop to "ti,cp_intc"
    - cp_intc_init() is now also for the of case
      the name of the init function (it calls the
      "new" __cp_intc_init() function, which was
      the "old" cp_intc_init()). Through this
      rework the changes for OF is better visible.
      As the OF case uses the irq_domain rework from
      Grant Likely, maybe the none OF case can use
      this also, but this should be tested on a hw ...

changes for v4:
- Interrupt Controller:
  - split in two patches as Nori Sekhar suggested
    one for the irq_domain change
    one for DT support
  - add comment from Grant Likely for the DT part:
    remove if/else clause, not needed.
    Make use of DT runtime configurable
    The non OF case is not tested!
     
Got

[PATCH v4 5/7] ARM: davinci: i2c: add OF support

Heiko Schocher — 2012-05-22T13:55:18

add of support for the davinci i2c driver.

Signed-off-by: Heiko Schocher <hs-ynQEQJNshbs< at >public.gmane.org>
Cc: davinci-linux-open-source-VycZQUHpC/PFrsHnngEfi1aTQe2KTcn/< at >public.gmane.org
Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r< at >public.gmane.org
Cc: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ< at >public.gmane.org
Cc: linux-i2c-u79uwXL29TY76Z2rM5mHXA< at >public.gmane.org
Cc: Ben Dooks <ben-linux-elnMNo+KYs3YtjvyW6yDsg< at >public.gmane.org>
Cc: Wolfram Sang <w.sang-bIcnvbaLZ9MEGnE8C9+IrQ< at >public.gmane.org>
Cc: Grant Likely <grant.likely-s3s/WqlpOiPyB63q8FvJNQ< at >public.gmane.org>
Cc: Sekhar Nori <nsekhar-l0cyMroinI0< at >public.gmane.org>
Cc: Wolfgang Denk <wd-ynQEQJNshbs< at >public.gmane.org>
Cc: Sylwester Nawrocki <s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ< at >public.gmane.org>

---
- changes for v2:
- add comments from Sylwester Nawrocki <s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ< at >public.gmane.org>:
  - use "cell-index" instead "id"
  - OF_DEV_AUXDATA in the machine code, instead pre-define platform
    device name
- add comment from Grant L

Re: [PATCH V2 00/11] enable imx6q_sabrelite sgtl5000 audio support

Mark Brown — 2012-05-22T09:56:35



There was mention of some changes but only a vauge description of them,
I've no idea where they are or anything, initially people had mentioned
an already cross-merged point in Linus' tree but apparently that wasn't
sufficient due to these arch specific bits and I don't know where those
are.

We should really have got this resolved in -next...



Well, a lot of the time there's no actual dependency (eg, you add a
driver and add registration of that device so there's no need for them
to go in together since both are perfectly fine without the other) then
it is best to merge things separately.  If there are dependencies then
that needs to be handled differently but it needs communication and
ideally the dependencies would be on focused things that can just be
pulled in.  

In the case of things like the devm_clk_get() cleanup if there's any
complexity at all then it's usually best just to punt for a release.

Re: [RFC v2 5/5] drm: Add NVIDIA Tegra support

Terje Bergström — 2012-05-22T06:19:23



Hi,

There is a reason for existence of bus_type in Linux kernel. It exposes
the various busses to developers, and give a framework for drivers to
work in. It just makes the drivers easier to develop, and makes the big
picture easier to understand.

The problem is that bus_type is cumbersome to implement, and most
implementations seem to duplicate significant amount of code from
platform bus. This is the problem that we should tackle.

If I manage to get the boilerplate code in nvhost for bus_type small
enough, that's the structure we should use. If bus_type is just
inherently fat and broken, I'll need to migrate nvhost away from it.

Terje

RE: Howto compile py-smbus with python 3.2

Renz, Bernhard — 2012-05-21T15:40:07

Hi Jean,

At the moment, I comment the Python2 code lines and add the Python3 ones. Below is the diff -ruNp output.
To get the code work for both platforms in one file, we have to make an if statement like if platform.python_version() >= 3.


--- smbusmodule.c       2012-05-14 17:47:22.606768184 +0200
+++ smbusmodule_org.c   2009-01-22 10:55:33.000000000 +0100
< at >< at > -5,12 +5,12 < at >< at >
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; version 2 of the License.
- *
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- *
+ *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
  * Foundation, Inc., 59 Temple Plac

Re: Howto compile py-smbus with python 3.2

Jean Delvare — 2012-05-21T15:14:19

Hi Bernhard,

On Mon, 21 May 2012 15:10:38 +0000, Renz, Bernhard wrote:

I'll do if you provide your changes as a patch (use diff -ruNp). BTW I
hope that your changes do not break support for python version 2, a lot
of distributions are still shipping that so we can't break it.

Thanks,

RE: Howto compile py-smbus with python 3.2

Renz, Bernhard — 2012-05-21T15:10:38

Hello,

I got it to work!
With the following pages I did a conversion of py-smbus from Python2 to Python3:
http://docs.python.org/release/3.1.5/extending/building.html#building
http://python3porting.com/cextensions.html

Here is the smbusmodule.c, which is based on the i2c-tools-3.1.0 release, which compile with Python3:
Hope maybe Khali can provide it for other developers on his repository....


/*
 * smbusmodule.c - Python bindings for Linux SMBus access through i2c-dev
 * Copyright (C) 2005-2007 Mark M. Hoffman <mhoffman-xQSgfq/1h4JiLUuM0BA3LQ< at >public.gmane.org>
 *
 * This program is free software; you can redistribute it and/or modify
 * it under the terms of the GNU General Public License as published by
 * the Free Software Foundation; version 2 of the License.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU General Public License for more detail

Re: [PATCH V2 00/11] enable imx6q_sabrelite sgtl5000 audio support

Mark Brown — 2012-05-21T14:45:44



No, apart from anything else the second patch depends on clkdev which is
nothing to do with pinctrl!  For the first patch it wasn't clear if the
changes in pinctrl were the actual dependencies or if there were more
things needed from the ARM tree, and in any case until those patches are
actually in the ASoC tree we'll break the build if they get applied.

Guys, you really need to think about how you're organising what you're
doing more.  You need to split your work out into focused lines of
development rather than just having a single branch.  

This patch series contains a whole bunch of different changes (the
devm_clk_get() change is as far as I can tell completely unrelated to
the rest for example) with unclear dependencies on multiple external
trees.  You should be splitting unrelated changes out, trying to
minimise interdependencies, and clearly identifying the dependencies
that are there we can get things applied in a timely fashion.

Please resend these patches *after* their dependencies are in mai

Re: [PATCH V2 00/11] enable imx6q_sabrelite sgtl5000 audio support

Richard Zhao — 2012-05-21T12:59:31

Hi Mark,

Linus has sent out pull request for pinctrl. So is that ok for you
to pick up the above two patch?

Thanks
Richard