public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v3 0/9] Bring in new I2C framework
Date: Thu, 20 Jun 2013 11:19:09 +0200	[thread overview]
Message-ID: <51C2C90D.7040203@denx.de> (raw)
In-Reply-To: <000501ce6d90$ef9683a0$cec38ae0$%wilczek@samsung.com>

Hello Piotr,

Am 20.06.2013 10:34, schrieb Piotr Wilczek:
> Dear Minkyu and Heiko,
> 
>> -----Original Message-----
>> From: Heiko Schocher [mailto:hs at denx.de]
>> Sent: Thursday, June 20, 2013 9:14 AM
>> To: Piotr Wilczek
>> Cc: 'Minkyu Kang'; 'Simon Glass'; 'U-Boot Mailing List'; 'Dirk Eibach';
>> 'Stefan Roese'; 'Holger Brunck'; 'Mike Frysinger'; 'Stephen Warren'
>> Subject: Re: [PATCH v3 0/9] Bring in new I2C framework
>>
>> Hello Piotr,
>>
>> Am 20.06.2013 08:41, schrieb Piotr Wilczek:
>>> Dear Heiko,
>>>
>>>> -----Original Message-----
>>>> From: Minkyu Kang [mailto:mk7.kang at samsung.com]
>>>> Sent: Thursday, June 20, 2013 7:51 AM
>>>> To: Heiko Schocher
>>>> Cc: Simon Glass; Piotr Wilczek; U-Boot Mailing List; Dirk Eibach;
>>>> Stefan Roese; Holger Brunck; Mike Frysinger; Stephen Warren
>>>> Subject: Re: [PATCH v3 0/9] Bring in new I2C framework
>>>>
>>>> Dear Heiko,
>>>>
>>>> On 20/06/13 12:38, Heiko Schocher wrote:
>>>>> Hello Simon,
>>>>>
>>>>> Am 20.06.2013 00:07, schrieb Simon Glass:
>>>>>> Hi Heiko,
>>>>>>
>>>>>> On Sun, May 19, 2013 at 11:13 PM, Piotr Wilczek
>>>> <p.wilczek at samsung.com <mailto:p.wilczek@samsung.com>> wrote:
>>>>>>
>>>>>>     Hi Simon,
>>>>>>
>>>>>>     > -----Original Message-----
>>>>>>     > From: sjg at google.com <mailto:sjg@google.com>
>>>> [mailto:sjg at google.com <mailto:sjg@google.com>] On Behalf Of Simon
>>>> Glass
>>>>>>     > Sent: Saturday, May 18, 2013 7:41 PM
>>>>>>     > To: Piotr Wilczek
>>>>>>     > Cc: Heiko Schocher; U-Boot Mailing List; Dirk Eibach; Stefan
>>>> Roese;
>>>>>>     > Holger Brunck; Mike Frysinger; Stephen Warren; Minkyu Kang
>>>>>>     > Subject: Re: [PATCH v3 0/9] Bring in new I2C framework
>>>>>>     >
>>>>>>     > Hi Piotr,
>>>>>>     >
>>>>>>     > On Fri, May 17, 2013 at 6:17 AM, Piotr Wilczek
>>>> <p.wilczek at samsung.com <mailto:p.wilczek@samsung.com>>
>>>>>>     > wrote:
>>>>>>     > > Dear Heiko,
>>>>>>     > >
>>>>>>     > >> -----Original Message-----
>>>>>>     > >> From: Heiko Schocher [mailto:hs at denx.de
>>>> <mailto:hs@denx.de>]
>>>>>>     > >> Sent: Saturday, May 04, 2013 2:02 PM
>>>>>>     > >> To: u-boot at lists.denx.de <mailto:u-boot@lists.denx.de>
>>>>>>     > >> Cc: Heiko Schocher; Dirk Eibach; Stefan Roese; Simon
>>>>>> Glass;
>>>> Piotr
>>>>>>     > >> Wilczek; Holger Brunck; Mike Frysinger; Stephen Warren
>>>>>>     > >> Subject: [PATCH v3 0/9] Bring in new I2C framework
>>>>>>     > >>
>>>>>>     > >> It seems the timing is coming, to bring this finally in
>>>> mainline and
>>>>>>     > >> move boards over to the new i2c framework.
>>>>>>     > >>
>>>>>>     > >> Changes for v3 series:
>>>>>>     > >>
>>>>>>     > >> - Compiler collects the used i2c adapters in
>>>> _u_boot_list_i2c,
>>>>>>     > >>   so no need for defining the used i2c adpaters in the
>>>> board config
>>>>>>     > >> file.
>>>>>>     > >>   -> drop CONFIG_SYS_NUM_I2C_ADAPTERS and
>>>> CONFIG_SYS_I2C_ADAPTERS
>>>>>>     > >> - Some adaptions for the keymile boards, so we can drop
>>>>>>     > CONFIG_I2C_MUX
>>>>>>     > >>   completely
>>>>>>     > >> - add ppc4xx_i2c driver, ported from Dirk Eibach
>>>>>>     > >>
>>>>>>     > >> Serie compiles clean on arm and powerpc
>>>>>>     > >>
>>>>>>     > >> Ported i2c drivers:
>>>>>>     > >> soft_i2c  : from me, tested on some arm and powerpc
>> boards
>>>>>>     > >> fsl_i2c   : from me, only compile tested
>>>>>>     > >> tegra_i2c : ported from Simon Glass
>>>>>>     > >> ppc4xx_i2c: adapted from Dirk Eibach
>> <dirk.eibach@gdsys.cc>
>>>>>>     > >>             tested on (which board?) with 4 soft_i2c
>>>> drivers
>>>>>>     > >>             and one ppc4x_i2c driver.
>>>>>>     > >>
>>>>>>     > >> Cc: Dirk Eibach <dirk.eibach@gdsys.cc>
>>>>>>     > >> Cc: Stefan Roese <sr at denx.de <mailto:sr@denx.de>>
>>>>>>     > >> Cc: Simon Glass <sjg@chromium.org
>>>> <mailto:sjg@chromium.org>>
>>>>>>     > >> Cc: Piotr Wilczek <p.wilczek@samsung.com
>>>> <mailto:p.wilczek@samsung.com>>
>>>>>>     > >> Cc: Holger Brunck <holger.brunck@keymile.com
>>>> <mailto:holger.brunck@keymile.com>>
>>>>>>     > >> Cc: Mike Frysinger <vapier@gentoo.org
>>>> <mailto:vapier@gentoo.org>>
>>>>>>     > >> Cc: Stephen Warren <swarren@wwwdotorg.org
>>>> <mailto:swarren@wwwdotorg.org>>
>>>>>>     > >>
>>>>>>     > >
>>>>>>     > > I ported s3c24x0_i2c driver and tested this framework on
>>>> Samsung tree
>>>>>>     > on Trats2 board.
>>>>>>     >
>>>>>>     > Please can you send a patch for this port?
>>>>>>
>>>>>>     I posted that patch and some others with series introducing
>>>> Trsts2, where I
>>>>>>     want to use this framework.
>>>>>>     I'm sorry I didn't mention it in my first mail.
>>>>>>     http://patchwork.ozlabs.org/patch/244645/
>>>>>>     http://patchwork.ozlabs.org/patch/244644/
>>>>>>     .
>>>>>>     http://patchwork.ozlabs.org/patch/244638/
>>>>>>     http://patchwork.ozlabs.org/patch/244637/
>>>>>>     http://patchwork.ozlabs.org/patch/244640/
>>>>>>     http://patchwork.ozlabs.org/patch/244636/
>>>>>
>>>>> Hups, missed this patches ...
>>>>>
>>>>>> What is the status of this series now please?
>>>>>
>>>
>>> If you take this series, please replace the following patch:
>>> http://patchwork.ozlabs.org/patch/244642/
>>>
>>> with the following one:
>>> http://patchwork.ozlabs.org/patch/248927/
>>
>> I am little confused now ... can you send a complete list which patches
>> have to be picked up... thanks!
>>
> 
> This series does not apply cleanly any more on u-boot-i2c. I would have to
> post an updated version.
> 
> In my opinion, It would be better that this series to be disregarded and the
> new board Trats2 goes through the Samsung tree (it's been already tested).
> Then later I should post new patches to adapt s3c24x0 i2c driver.

Ok from my side ...

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

  reply	other threads:[~2013-06-20  9:19 UTC|newest]

Thread overview: 104+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-04 12:01 [U-Boot] [PATCH v3 0/9] Bring in new I2C framework Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 1/9] i2c: add i2c_core and prepare for new multibus support Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 2/9] i2c: common changes for multibus/multiadapter support Heiko Schocher
2013-05-06 16:39   ` Daniel Schwierzeck
2013-05-07 13:05     ` Heiko Schocher
2013-05-11 21:17   ` Simon Glass
2013-05-13  4:47     ` Heiko Schocher
2013-05-11 21:33   ` Simon Glass
2013-05-13  5:41     ` Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 3/9] i2c, soft-i2c: switch to new " Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 4/9] i2c, fsl_i2c: " Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 5/9] i2c, multibus: get rid of CONFIG_I2C_MUX Heiko Schocher
2013-05-06 12:23   ` Holger Brunck
2013-05-06 13:57     ` Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 6/9] i2c, multibus, keymile: get rid of EEprom_ivm envvariable Heiko Schocher
2013-05-06 12:24   ` Holger Brunck
2013-05-06 13:58     ` Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 7/9] tegra: i2c: Add function to know about current bus Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 8/9] tegra: i2c: Enable new CONFIG_SYS_I2C framework Heiko Schocher
2013-05-06 19:08   ` Stephen Warren
2013-05-07  8:01     ` [U-Boot] [PATCH v3 8/9] tegra: i2c: Enable new CONFIG_SYS_I2Cframework Marc Dietrich
2013-05-07 14:55       ` Stephen Warren
2013-05-07 16:17         ` Simon Glass
2013-05-08  4:11         ` Heiko Schocher
2013-05-07 13:07     ` [U-Boot] [PATCH v3 8/9] tegra: i2c: Enable new CONFIG_SYS_I2C framework Heiko Schocher
     [not found]     ` <5FBF8E85CA34454794F0F7ECBA79798F37ACAFEF3F@HQMAIL04.nvidia.com>
2013-05-07 13:12       ` Heiko Schocher
2013-07-29 16:12   ` Stephen Warren
2013-07-30  4:28     ` Heiko Schocher
2013-07-30  4:34       ` Simon Glass
2013-07-30 18:56       ` Stephen Warren
2013-07-31  4:29         ` Heiko Schocher
2013-07-30 19:22       ` Stephen Warren
2013-07-30 20:00         ` Stephen Warren
2013-07-30 21:21           ` Simon Glass
2013-07-30 21:32             ` Stephen Warren
2013-07-30 21:46               ` Simon Glass
2013-07-30 21:51                 ` Stephen Warren
2013-07-30 22:05                   ` Simon Glass
2013-07-31  5:46                     ` Heiko Schocher
2013-07-31 19:31                       ` Stephen Warren
2013-08-01  4:32                         ` Heiko Schocher
2013-08-01  5:39                           ` Stephen Warren
2013-08-01  6:02                             ` Heiko Schocher
2013-08-01  6:53                               ` Albert ARIBAUD
2013-08-01  8:38                                 ` Heiko Schocher
2013-08-01 14:22                                   ` Simon Glass
2013-08-01 15:06                                     ` Heiko Schocher
2013-08-01 20:16                                     ` Albert ARIBAUD
2013-08-02 19:32                                       ` Simon Glass
2013-08-01 20:14                                   ` Albert ARIBAUD
2013-08-01 20:34                                 ` Stephen Warren
2013-08-01 20:32                               ` Stephen Warren
2013-08-02  4:40                                 ` Heiko Schocher
2013-08-02 19:35                                   ` Simon Glass
2013-08-02 21:43                                   ` Stephen Warren
2013-08-03  3:55                                     ` Heiko Schocher
2013-08-05 15:40                                       ` Simon Glass
2013-08-05 17:28                                         ` Stephen Warren
2013-08-05 20:12                                           ` Simon Glass
2013-08-05 20:15                                             ` Stephen Warren
2013-08-05 17:59                                       ` Stephen Warren
2013-07-30 22:09                   ` Albert ARIBAUD
2013-07-30 22:11                     ` Simon Glass
2013-07-31  5:18                     ` Wolfgang Denk
2013-07-31  5:55                       ` Heiko Schocher
2013-07-31  7:06                       ` Albert ARIBAUD
2013-07-31  7:36                         ` Heiko Schocher
2013-07-31  8:16                           ` Albert ARIBAUD
2013-07-31  8:31                             ` Heiko Schocher
2013-07-31  9:38                               ` Albert ARIBAUD
2013-07-31 12:30                                 ` Simon Glass
2013-07-31 13:03                                   ` Heiko Schocher
2013-07-31 19:41                               ` Stephen Warren
2013-08-01  4:32                                 ` Heiko Schocher
2013-07-31 19:39                         ` Wolfgang Denk
2013-07-31  5:52                     ` Heiko Schocher
2013-07-31  5:03           ` Heiko Schocher
2013-08-05 19:21           ` Stephen Warren
2013-08-05 20:08             ` Tom Rini
2013-08-05 21:06               ` Stephen Warren
2013-08-05 23:18               ` Stephen Warren
2013-07-30 21:19         ` Simon Glass
2013-07-30 21:21           ` Stephen Warren
2013-07-30 21:45             ` Simon Glass
2013-07-31  5:01         ` Heiko Schocher
2013-05-04 12:01 ` [U-Boot] [PATCH v3 9/9] i2c, ppc4xx_i2c: switch to new multibus/multiadapter support Heiko Schocher
2013-05-06  6:52   ` Stefan Roese
2013-05-06  8:57 ` [U-Boot] [PATCH v3 0/9] Bring in new I2C framework Dirk Eibach
2013-05-06 14:11   ` Heiko Schocher
2013-05-17 13:17 ` Piotr Wilczek
2013-05-18 17:41   ` Simon Glass
2013-05-20  6:13     ` Piotr Wilczek
2013-06-19 22:07       ` Simon Glass
2013-06-20  3:38         ` Heiko Schocher
2013-06-20  5:50           ` Minkyu Kang
2013-06-20  6:41             ` Piotr Wilczek
2013-06-20  7:14               ` Heiko Schocher
2013-06-20  8:34                 ` Piotr Wilczek
2013-06-20  9:19                   ` Heiko Schocher [this message]
2013-06-20  5:52           ` Piotr Wilczek
2013-06-20  6:52           ` Dirk Eibach
2013-06-20  7:59             ` Heiko Schocher
2013-06-20  8:20               ` Dirk Eibach
2013-06-20  9:11                 ` Heiko Schocher

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=51C2C90D.7040203@denx.de \
    --to=hs@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox