All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lubomir Popov <lpopov@mm-sol.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] U-Boot OMAP issues
Date: Wed, 29 May 2013 16:36:36 +0300	[thread overview]
Message-ID: <51A60464.7060808@mm-sol.com> (raw)
In-Reply-To: <51A601A8.5090704@ti.com>

Hi Tom,

On 29/05/13 16:24, Tom Rini wrote:
> On 05/29/2013 02:34 AM, Lubomir Popov wrote:
>> Hi Tom,
>>
>> On 29.05.2013 01:55, Tom Rini wrote:
>>> On 05/27/2013 02:44 PM, Lubomir Popov wrote:
>>>
>>>>>>> P.S. I have an updated version of the I2C driver patch, with some
>>>>>>> minor
>>>>>>> improvements (mainly on identification of unconfigured bus). Should I
>>>>>>> submit it, what are you plans in respect to I2C?
>>>>>> Yes, please submit the i2c driver changes, I shall take them in some
>>>>>> form or another soon.
>>>>> OK, can even try to do this now from here.
>>>> Submitted - http://patchwork.ozlabs.org/patch/246204/
>>>> Looking at it again, I tend to dislike this solution more and more.
>>>> Ugly ifdef-ed code. Couldn't we reconsider again adding this as a new
>>>> driver, built for those SoCs that are confirmed to work (OMAP4 and 5
>>>> for sure)?
>>> Lets take another swag at this.  Update the functions for everyone and
>>> I'll get some omap3 testing done.
>> OK, but you shall have to either entirely remove the ifdefs and the old
>> functions, or edit the
>> #if defined(CONFIG_OMAPxxxx)... conditions to include the SoC types you
>> are testing on.
> 
> Yeah.  If you don't repost the patch with just always using the new
> functions, I'll do some local yanking out.  I kinda hope to pencil that
> in as my task for tomorrow.  Thanks :)
> 
OK, I shall do it now. Does it matter if I base on u-boot-ti, or on mainline?
Guess not...

Regards,
Lubo

           reply	other threads:[~2013-05-29 13:36 UTC|newest]

Thread overview: expand[flat|nested]  mbox.gz  Atom feed
 [parent not found: <51A601A8.5090704@ti.com>]

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=51A60464.7060808@mm-sol.com \
    --to=lpopov@mm-sol.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.