From: Lukasz Majewski <l.majewski@samsung.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] Start the deprecation process for generic board
Date: Fri, 04 Apr 2014 12:09:21 +0200 [thread overview]
Message-ID: <20140404120921.128bef54@amdc2363> (raw)
In-Reply-To: <201404040043.03031.marex@denx.de>
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.
>> and for the record,
> > if distro people would be so kind as to switch it on and report
> > back that nothing happens, that'd be really helpful too.
>
> Like I said already, it is fine for me to do this after the release,
> but not in -rc3 .
+1
>
> Best regards,
> Marek Vasut
--
Best regards,
Lukasz Majewski
Samsung R&D Institute Poland (SRPOL) | Linux Platform Group
next prev parent reply other threads:[~2014-04-04 10:09 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 [this message]
2014-04-05 5:14 ` Simon Glass
2014-04-07 7:17 ` Lukasz Majewski
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=20140404120921.128bef54@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