public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Lukasz Majewski <l.majewski@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Start the deprecation process for generic board
Date: Mon, 07 Apr 2014 09:17:45 +0200	[thread overview]
Message-ID: <20140407091745.3265fe1e@amdc2363> (raw)
In-Reply-To: <CAPnjgZ1cgZhctZwurVqGnVxo9VWwEOAVzfnO=wtGJYgpkndOug@mail.gmail.com>

Hi Simon,

> Hi Lukasz,
> 
> On 4 April 2014 04:09, Lukasz Majewski <l.majewski@samsung.com> wrote:
> 
> > Hi Marek,
> >
> > > On Thursday, April 03, 2014 at 09:01:30 PM, Tom Rini wrote:
> > > > On Thu, Apr 03, 2014 at 11:35:10AM -0600, Simon Glass wrote:
> > > > > Hi Marek,
> > > > >
> > > > > On 3 April 2014 11:03, Marek Vasut <marex@denx.de> wrote:
> > > > > > On Friday, March 28, 2014 at 10:16:42 PM, Tom Rini wrote:
> > > > > > > On Sat, Mar 22, 2014 at 05:14:51PM -0600, Simon Glass
> > > > > > > wrote:
> > > > > > > > We should move forward to remove the old board init
> > > > > > > > code. Add a prominent message to encourage maintainers
> > > > > > > > to get started on this work.
> > > > > > > >
> > > > > > > > Signed-off-by: Simon Glass <sjg@chromium.org>
> > > > > > >
> > > > > > > Applied to u-boot/master, thanks!
> > > > > >
> > > > > > It's now -rc3 , MW is long closed and my board start
> > > > > > generating a warning. Even
> > > > > > worse, all boards now generate this warning and I'm worried
> > > > > > that this will cause
> > > > > > the more attentive maintainers to take action and produce
> > > > > > fast fixes which won't
> > > > > > receive sufficient testing until release is out.
> > > > > >
> > > > > > I seriously disagree with applying the patch for current
> > > > > > release.
> > > > >
> > > > > Despite the fact that it has been ~1 year and not a lot of
> > > > > people have taken action, I didn't actually expect it to go
> > > > > in right away, and I'm fine with a revert for this release if
> > > > > that's what Tom decides. But let's get it moving ASAP.
> > > > >
> > > > > Also let me know if there are any questions about how to
> > > > > test. I wrote up a README in the patch, but am happy to
> > > > > expand it as things come up.
> > > >
> > > > So, Marek and I were discussing this on IRC as well.  Given
> > > > about a year and nothing converted (except for new things) I
> > > > opted for the noisier approach.  It really is a simple switch
> >
> > Unfortunately, it isn't a simple switch in my case.
> >
> > The
> > +#define CONFIG_SYS_GENERIC_BOARD on the trats2 (and probably also
> > trats) board causes problem with power management initialization. I
> > will not be able to fix this and validate before v2014.04.
> >
> > This issue will be fixed after the release.
> >
> 
> FWIW this works OK on exynos5 which has used generic board for a
> while. I'm not sure what the problem is - I don't see a change in the
> ordering of the call to power_init().

I've observed strange behaviour when battery was removed. I need time
to investigate it.

Frankly, I plan to look closer on the power system after the release of
v2014.04.

> 
> Regards,
> Simon
> 


-- 
Best regards,

Lukasz Majewski

Samsung R&D Institute Poland (SRPOL) | Linux Platform Group

  reply	other threads:[~2014-04-07  7:17 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-03-22 23:14 [U-Boot] [PATCH] Start the deprecation process for generic board Simon Glass
2014-03-28 21:16 ` [U-Boot] " Tom Rini
2014-04-03 17:03   ` Marek Vasut
2014-04-03 17:35     ` Simon Glass
2014-04-03 17:48       ` Marek Vasut
2014-04-03 19:01       ` Tom Rini
2014-04-03 22:43         ` Marek Vasut
2014-04-04 10:09           ` Lukasz Majewski
2014-04-05  5:14             ` Simon Glass
2014-04-07  7:17               ` Lukasz Majewski [this message]
2014-04-25 22:29 ` [U-Boot] [PATCH] " York Sun
2014-04-26 20:25   ` Simon Glass

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=20140407091745.3265fe1e@amdc2363 \
    --to=l.majewski@samsung.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox