Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-10 Thread Marek Vasut
On Friday, October 10, 2014 at 04:35:11 AM, Simon Glass wrote: > Hi, Hi, > On 9 October 2014 20:26, Fabio Estevam wrote: > > On Thu, Oct 9, 2014 at 11:23 PM, Marek Vasut wrote: > >> What about [1], this is where we can source the more exotic toolchains > >> from, can we not? I think it was even

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Simon Glass
Hi, On 9 October 2014 20:26, Fabio Estevam wrote: > On Thu, Oct 9, 2014 at 11:23 PM, Marek Vasut wrote: > >> What about [1], this is where we can source the more exotic toolchains from, >> can we not? I think it was even you who pointed me to this site and it really >> is a nice one ;-) >> >> [1

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Fabio Estevam
On Thu, Oct 9, 2014 at 11:23 PM, Marek Vasut wrote: > What about [1], this is where we can source the more exotic toolchains from, > can we not? I think it was even you who pointed me to this site and it really > is a nice one ;-) > > [1] https://www.kernel.org/pub/tools/crosstool/ Exactly, the

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Marek Vasut
On Friday, October 10, 2014 at 04:00:00 AM, Simon Glass wrote: > Hi Marek, > > On 9 October 2014 17:59, Marek Vasut wrote: > > On Thursday, October 09, 2014 at 08:04:29 PM, Simon Glass wrote: > >> Hi Marek, > > > > Hi Simon, > > > > [..] > > > >> > I mean more continuous integration (build tes

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Fabio Estevam
On Thu, Oct 9, 2014 at 11:07 PM, Simon Glass wrote: > It's not really the builder that is the problem - we have one. It's > all the different toolchains that no one knows about. I still can't > build all the boards successfully. Aren't all the toolchains U-boot need included in the make.cross sc

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Simon Glass
Hi, On 9 October 2014 20:06, Fabio Estevam wrote: > Hi Marek, > > On Thu, Oct 9, 2014 at 8:59 PM, Marek Vasut wrote: > >> I agree that buildman solves the CI part nicely, but we also have the part >> where one has to install the myriad of toolchains for all the architectures >> to be able to do

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Fabio Estevam
Hi Marek, On Thu, Oct 9, 2014 at 8:59 PM, Marek Vasut wrote: > I agree that buildman solves the CI part nicely, but we also have the part > where one has to install the myriad of toolchains for all the architectures > to be able to do the compile testing. I wonder if this cannot be made easy > i

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Simon Glass
Hi Marek, On 9 October 2014 17:59, Marek Vasut wrote: > On Thursday, October 09, 2014 at 08:04:29 PM, Simon Glass wrote: >> Hi Marek, > > Hi Simon, > > [..] > >> > I mean more continuous integration (build testing) of the code before a >> > PR is submitted to the ML. Right now, we all do our own

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Marek Vasut
On Thursday, October 09, 2014 at 08:04:29 PM, Simon Glass wrote: > Hi Marek, Hi Simon, [..] > > I mean more continuous integration (build testing) of the code before a > > PR is submitted to the ML. Right now, we all do our own thing when it > > comes to testing before PR, but it would be nice t

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Simon Glass
Hi Marek, On 9 October 2014 11:32, Marek Vasut wrote: > On Thursday, October 09, 2014 at 07:03:42 PM, Simon Glass wrote: > > Hi Marek, > > > > On 9 October 2014 10:27, Marek Vasut wrote: > > > On Thursday, October 09, 2014 at 06:14:11 PM, Simon Glass wrote: > > >> Hi Marek, > > >> > > >> On 9 O

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Marek Vasut
On Thursday, October 09, 2014 at 07:03:42 PM, Simon Glass wrote: > Hi Marek, > > On 9 October 2014 10:27, Marek Vasut wrote: > > On Thursday, October 09, 2014 at 06:14:11 PM, Simon Glass wrote: > >> Hi Marek, > >> > >> On 9 October 2014 09:12, Marek Vasut wrote: > >> > On Thursday, October 09,

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Tom Rini
On Thu, Oct 09, 2014 at 11:03:02AM -0600, Simon Glass wrote: > Hi Tom, > > On 9 October 2014 10:23, Tom Rini wrote: > > On Thu, Oct 09, 2014 at 05:12:03PM +0200, Marek Vasut wrote: > >> On Thursday, October 09, 2014 at 08:18:14 AM, Simon Glass wrote: > >> > Hi, > >> > > >> > On 20 September 2014

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Simon Glass
Hi Marek, On 9 October 2014 10:27, Marek Vasut wrote: > On Thursday, October 09, 2014 at 06:14:11 PM, Simon Glass wrote: >> Hi Marek, >> >> On 9 October 2014 09:12, Marek Vasut wrote: >> > On Thursday, October 09, 2014 at 08:18:14 AM, Simon Glass wrote: >> > > Hi, >> > > >> > > On 20 September 2

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Simon Glass
Hi Tom, On 9 October 2014 10:23, Tom Rini wrote: > On Thu, Oct 09, 2014 at 05:12:03PM +0200, Marek Vasut wrote: >> On Thursday, October 09, 2014 at 08:18:14 AM, Simon Glass wrote: >> > Hi, >> > >> > On 20 September 2014 08:54, Hans de Goede wrote: >> > > In some cases we really want to move forw

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Marek Vasut
On Thursday, October 09, 2014 at 06:14:11 PM, Simon Glass wrote: > Hi Marek, > > On 9 October 2014 09:12, Marek Vasut wrote: > > On Thursday, October 09, 2014 at 08:18:14 AM, Simon Glass wrote: > > > Hi, > > > > > > On 20 September 2014 08:54, Hans de Goede wrote: > > > > In some cases we reall

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Tom Rini
On Thu, Oct 09, 2014 at 05:12:03PM +0200, Marek Vasut wrote: > On Thursday, October 09, 2014 at 08:18:14 AM, Simon Glass wrote: > > Hi, > > > > On 20 September 2014 08:54, Hans de Goede wrote: > > > In some cases we really want to move forward with a deregister, add a > > > force parameter to all

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Simon Glass
Hi Marek, On 9 October 2014 09:12, Marek Vasut wrote: > > On Thursday, October 09, 2014 at 08:18:14 AM, Simon Glass wrote: > > Hi, > > > > On 20 September 2014 08:54, Hans de Goede wrote: > > > In some cases we really want to move forward with a deregister, add a > > > force parameter to allow t

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-09 Thread Marek Vasut
On Thursday, October 09, 2014 at 08:18:14 AM, Simon Glass wrote: > Hi, > > On 20 September 2014 08:54, Hans de Goede wrote: > > In some cases we really want to move forward with a deregister, add a > > force parameter to allow this, and replace the dev with a nulldev in > > this case. > > > > Si

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-08 Thread Simon Glass
Hi, On 20 September 2014 08:54, Hans de Goede wrote: > In some cases we really want to move forward with a deregister, add a force > parameter to allow this, and replace the dev with a nulldev in this case. > > Signed-off-by: Hans de Goede > --- > common/stdio.c | 13 ++-

Re: [U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-10-08 Thread Rommel Custodio
Dear Hans de Goede, Typographical error here: > --- a/drivers/serial/serial-uclass.c > +++ b/drivers/serial/serial-uclass.c > -197,7 +197,7 static int serial_pre_remove(struct udevice *dev) > #ifdef CONFIG_SYS_STDIO_DEREGISTER > struct serial_dev_priv *upriv = dev->uclass_priv;

[U-Boot] [PATCH fix for v2014.10 4/5] stdio: Add force parameter to stdio_deregister

2014-09-20 Thread Hans de Goede
In some cases we really want to move forward with a deregister, add a force parameter to allow this, and replace the dev with a nulldev in this case. Signed-off-by: Hans de Goede --- common/stdio.c | 13 ++--- common/usb_kbd.c | 2 +- drivers/serial/serial-