Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-16 Thread Mateusz Zalega
On 09/15/13 16:21, Marek Vasut wrote: > I suppose this thread can be concluded by droping the INIT_ALL stuff > entirely. > Afterall, we do not want to init _ALL_ ports at once, but we want to init > them > selectively. > > Best regards, > Marek Vasut +1 -- Mateusz Zalega Samsung R&D Institu

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-15 Thread Marek Vasut
Dear Mateusz Zalega, > On 09/06/13 13:40, Marek Vasut wrote: > >>> Moreover, the 'int index' should likely be unsigned int and the > >>> special value to init all controllers at once should probably then > >>> be 0x > >> > >> Despite our greatest ambitions, I don't thi

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-06 Thread Mateusz Zalega
On 09/06/13 13:40, Marek Vasut wrote: >>> Moreover, the 'int index' should likely be unsigned int and the >>> special value to init all controllers at once should probably then >>> be 0x >> >> Despite our greatest ambitions, I don't think we're likely to use more >>

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-06 Thread Marek Vasut
Dear Mateusz Zalega, > On 09/06/13 13:24, Marek Vasut wrote: > > Moreover, the 'int index' should likely be unsigned int and the > > special value to init all controllers at once should probably then > > be 0x > > Despite our greatest ambitions, I don't think we're l

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-06 Thread Mateusz Zalega
On 09/06/13 13:24, Marek Vasut wrote: > Moreover, the 'int index' should likely be unsigned int and the special > value to init all controllers at once should probably then be > 0x Despite our greatest ambitions, I don't think we're likely to use more than 2^31-1

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-06 Thread Marek Vasut
Dear Mateusz Zalega, > On 09/05/13 19:51, Marek Vasut wrote: > >>> Why not wrap board_usb_init() and board_usb_init_fail() into single > >>> call. You now pass some flags to board_usb_init() already, so just add > >>> another for the fail case. How does it sound to you? > >> > >> Like overenginee

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-06 Thread Mateusz Zalega
On 09/05/13 19:51, Marek Vasut wrote: >>> Why not wrap board_usb_init() and board_usb_init_fail() into single call. >>> You now pass some flags to board_usb_init() already, so just add another >>> for the fail case. How does it sound to you? >> >> Like overengineering. It would lead to "board_usb_i

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-05 Thread Marek Vasut
Dear Mateusz Zalega, > This commit unifies board-specific USB initialization implementations > under one symbol (usb_board_init), declaration of which is available in > usb.h. > > Signed-off-by: Mateusz Zalega > Signed-off-by: Kyungmin Park > Reviewed-by: Lukasz Majewski > Cc: Minkyu Kang > C

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-05 Thread Marek Vasut
Dear Mateusz Zalega, > On 09/05/13 17:50, Marek Vasut wrote: > >> v3 changes: > >> - added 'index' argument to perform selective port initialization > > > > OK, a few general ideas again: > > > > Why not wrap board_usb_init() and board_usb_init_fail() into single call. > > You now pass some flag

Re: [U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-05 Thread Mateusz Zalega
On 09/05/13 17:50, Marek Vasut wrote: >> v3 changes: >> - added 'index' argument to perform selective port initialization > > OK, a few general ideas again: > > Why not wrap board_usb_init() and board_usb_init_fail() into single call. You > now pass some flags to board_usb_init() already, so jus

[U-Boot] [PATCH v3] usb: new board-specific USB init interface

2013-09-03 Thread Mateusz Zalega
This commit unifies board-specific USB initialization implementations under one symbol (usb_board_init), declaration of which is available in usb.h. Signed-off-by: Mateusz Zalega Signed-off-by: Kyungmin Park Reviewed-by: Lukasz Majewski Cc: Minkyu Kang Cc: Marek Vasut --- Changes since RFC (v