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 v4] i2c, omap24xx: convert driver to new mutlibus/mutliadapter framework
Date: Mon, 04 Nov 2013 07:56:35 +0100	[thread overview]
Message-ID: <52774523.1080909@denx.de> (raw)
In-Reply-To: <526F9C44.30909@denx.de>

Hello Tom,

Am 29.10.2013 12:30, schrieb Heiko Schocher:
> Hello Igor,
>
> Am 29.10.2013 12:07, schrieb Igor Grinberg:
>> On 10/28/13 08:19, Heiko Schocher wrote:
>>> Hello Igor,
>>>
>>> Am 27.10.2013 08:30, schrieb Igor Grinberg:
>>>> Hi Heiko,
>>>>
>>>> On 10/22/13 11:03, Heiko Schocher wrote:
>>>>> - add omap24xx driver to new multibus/multiadpater support
>>>>> - adapted all config files, which uses this driver
>>>>>
>>>>> Tested on the am335x based siemens boards rut, dxr2 and pxm2
>>>>> posted here:
>>>>> http://patchwork.ozlabs.org/patch/263211/
>>>>>
>>>>> Signed-off-by: Heiko Schocher<hs@denx.de>
>>>>> Tested-by: Tom Rini<trini@ti.com>
>>>>> Cc: Lars Poeschel<poeschel@lemonage.de>
>>>>> Cc: Steve Sakoman<sakoman@gmail.com>
>>>>> Cc: Thomas Weber<weber@corscience.de>
>>>>> Cc: Tom Rix<Tom.Rix@windriver.com>
>>>>> Cc: Grazvydas Ignotas<notasas@gmail.com>
>>>>> Cc: Enric Balletbo i Serra<eballetbo@iseebcn.com>
>>>>> Cc: Luca Ceresoli<luca.ceresoli@comelit.it>
>>>>> Cc: Igor Grinberg<grinberg@compulab.co.il>
>>>>> Cc: Ilya Yanok<yanok@emcraft.com>
>>>>> Cc: Stefano Babic<sbabic@denx.de>
>>>>> Cc: Nishanth Menon<nm@ti.com>
>>>>> Cc: Pali Roh?r<pali.rohar@gmail.com>
>>>>> Cc: Peter Barada<peter.barada@logicpd.com>
>>>>> Cc: Nagendra T S<nagendra@mistralsolutions.com>
>>>>> Cc: Michael Jones<michael.jones@matrix-vision.de>
>>>>> Cc: Raphael Assenat<raph@8d.com>
>>>>
>>>> [...]
>>> [...]
>>>
>>>>> diff --git a/board/compulab/cm_t35/eeprom.h b/board/compulab/cm_t35/eeprom.h
>>>>> index 02ffbb1..a07559d 100644
>>>>> --- a/board/compulab/cm_t35/eeprom.h
>>>>> +++ b/board/compulab/cm_t35/eeprom.h
>>>>> @@ -10,7 +10,7 @@
>>>>> #ifndef _EEPROM_
>>>>> #define _EEPROM_
>>>>>
>>>>> -#ifdef CONFIG_DRIVER_OMAP34XX_I2C
>>>>> +#ifdef CONFIG_SYS_I2C_OMAP34XX
>>>>> int cm_t3x_eeprom_read_mac_addr(uchar *buf);
>>>>> u32 cm_t3x_eeprom_get_board_rev(void);
>>>>> #else
>>>>
>>>> Please, rebase on top of:
>>>> http://patchwork.ozlabs.org/patch/275284/
>>>> which should be applied any time soon.
>>>>
>>>> Otherwise,
>>>> Acked-by: Igor Grinberg<grinberg@compulab.co.il>
>>>
>>> Hmm.. I cannot see, that your patch is accepted ... but I looked in it,
>>> and it looks good to me. So, I propose to add this patch to the i2c tree,
>>> so I do the necessary changes in my patch when applying it to the
>>> i2c tree (as they are trivial changes) ...
>>
>> Well, I have no problem with that, but we need to take into account
>> that there are several more patches (unrelated to i2c) on the mailing
>> list waiting for approval/apply and depend on my series.
>
> Ok.
>
>>> @Tom: Is this OK for you?
>>
>> I think, it would be better to apply my series to u-boot-ti and you can
>> pull from it to the i2c tree?
>
> Hmm.. u-boot-i2c is based on u-boot.git not on u-boot-ti, and I did
> not want to change this. I can of course rebase my patch against u-boot-ti
> when Tom added your patch to u-boot-ti and resend it, and tom pick up my
> patch ...

Tom, what do you think?

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-11-04  6:56 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-10-22  9:03 [U-Boot] [PATCH v4] i2c, omap24xx: convert driver to new mutlibus/mutliadapter framework Heiko Schocher
2013-10-27  7:30 ` Igor Grinberg
2013-10-28  6:19   ` Heiko Schocher
2013-10-29 11:07     ` Igor Grinberg
2013-10-29 11:30       ` Heiko Schocher
2013-11-04  6:56         ` Heiko Schocher [this message]
2013-11-04 13:39           ` Tom Rini
2013-10-28  9:07 ` Stefano Babic
2013-11-13  6:33 ` [U-Boot] [U-Boot, " 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=52774523.1080909@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