Re: [U-Boot] [PATCH v5 1/1] cmd: part: Add 'number' sub-command

2019-07-24 Thread Tom Rini
On Fri, Jun 14, 2019 at 05:01:26PM +0300, Igor Opaniuk wrote: > From: Ruslan Trofymenko > > This sub-command serves for getting the partition index from > partition name. Also it can be used to test the existence of specified > partition. > > Use case: > For example, in most CI environments thi

Re: [U-Boot] [PATCH v5 1/1] cmd: part: Add 'number' sub-command

2019-07-15 Thread Tom Rini
On Fri, Jul 12, 2019 at 12:25:50PM +0300, Igor Opaniuk wrote: > Hi Tom, > > On Fri, Jun 14, 2019 at 5:22 PM Sam Protsenko > wrote: > > > > Hi Igor, > > > > Once again: > > > > Reviewed-by: Sam Protsenko > > > > Thanks! > > > > On Fri, Jun 14, 2019 at 5:01 PM Igor Opaniuk wrote: > > > > > > From

Re: [U-Boot] [PATCH v5 1/1] cmd: part: Add 'number' sub-command

2019-07-12 Thread Igor Opaniuk
Hi Tom, On Fri, Jun 14, 2019 at 5:22 PM Sam Protsenko wrote: > > Hi Igor, > > Once again: > > Reviewed-by: Sam Protsenko > > Thanks! > > On Fri, Jun 14, 2019 at 5:01 PM Igor Opaniuk wrote: > > > > From: Ruslan Trofymenko > > > > This sub-command serves for getting the partition index from > >

Re: [U-Boot] [PATCH v5 1/1] cmd: part: Add 'number' sub-command

2019-06-14 Thread Igor Opaniuk
Hi Eugeniu, On Fri, Jun 14, 2019 at 5:08 PM Eugeniu Rosca wrote: > > Hi Igor, > > I believe it's common sense, best practice and it is generally polite > to include in CC anybody who contributed with review comments during > the previous patch iterations. Otherwise, it raises questions, > mistrus

Re: [U-Boot] [PATCH v5 1/1] cmd: part: Add 'number' sub-command

2019-06-14 Thread Sam Protsenko
Hi Igor, Once again: Reviewed-by: Sam Protsenko Thanks! On Fri, Jun 14, 2019 at 5:01 PM Igor Opaniuk wrote: > > From: Ruslan Trofymenko > > This sub-command serves for getting the partition index from > partition name. Also it can be used to test the existence of specified > partition. > > U

Re: [U-Boot] [PATCH v5 1/1] cmd: part: Add 'number' sub-command

2019-06-14 Thread Eugeniu Rosca
Hi Igor, I believe it's common sense, best practice and it is generally polite to include in CC anybody who contributed with review comments during the previous patch iterations. Otherwise, it raises questions, mistrust and all kind of weird assumptions, which may hurt the community in the long ru

[U-Boot] [PATCH v5 1/1] cmd: part: Add 'number' sub-command

2019-06-14 Thread Igor Opaniuk
From: Ruslan Trofymenko This sub-command serves for getting the partition index from partition name. Also it can be used to test the existence of specified partition. Use case: For example, in most CI environments this U-Boot command for automatic testing of Linux rootfs is used: => setenv