Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-24 Thread Saravana Kannan
On Mon, Jun 22, 2020 at 8:49 AM Geert Uytterhoeven wrote: > > Hi Saravana, > > On Sat, Jun 20, 2020 at 4:33 AM Saravana Kannan wrote: > > On Fri, Jun 19, 2020 at 1:07 PM Saravana Kannan > > wrote: > > > I think instead of deferred_probe_work_func() moving the device to the > > > end of the

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-22 Thread Geert Uytterhoeven
Hi Saravana, On Sat, Jun 20, 2020 at 4:33 AM Saravana Kannan wrote: > On Fri, Jun 19, 2020 at 1:07 PM Saravana Kannan wrote: > > I think instead of deferred_probe_work_func() moving the device to the > > end of the dpm_list, I think the device probing successfully is what > > should move it to

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-19 Thread Saravana Kannan
On Fri, Jun 19, 2020 at 1:07 PM Saravana Kannan wrote: > > I think instead of deferred_probe_work_func() moving the device to the > end of the dpm_list, I think the device probing successfully is what > should move it to the end of the dpm_list. That way, the dpm_list is > actually ordered by

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-19 Thread Saravana Kannan
Snipped a bunch of old context. On Fri, Jun 19, 2020 at 5:24 AM Geert Uytterhoeven wrote: > > On Fri, Jun 19, 2020 at 1:01 AM Saravana Kannan wrote: > > > > When you say the "The affected drivers are still probed in the same > > order", are you only referring to the devices that would have

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-19 Thread Geert Uytterhoeven
Hi Saravana, On Fri, Jun 19, 2020 at 1:01 AM Saravana Kannan wrote: > On Thu, Jun 18, 2020 at 12:32 AM Geert Uytterhoeven > wrote: > > On Wed, Jun 17, 2020 at 8:36 PM Saravana Kannan > > wrote: > > > On Wed, Jun 17, 2020 at 5:20 AM Geert Uytterhoeven > > > wrote: > > > > On Fri, May 15,

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-18 Thread Saravana Kannan
On Thu, Jun 18, 2020 at 12:32 AM Geert Uytterhoeven wrote: > > Hi Saravana, > > On Wed, Jun 17, 2020 at 8:36 PM Saravana Kannan wrote: > > On Wed, Jun 17, 2020 at 5:20 AM Geert Uytterhoeven > > wrote: > > > On Fri, May 15, 2020 at 7:38 AM Saravana Kannan > > > wrote: > > > > The

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-18 Thread Geert Uytterhoeven
Hi Saravana, On Wed, Jun 17, 2020 at 8:36 PM Saravana Kannan wrote: > On Wed, Jun 17, 2020 at 5:20 AM Geert Uytterhoeven > wrote: > > On Fri, May 15, 2020 at 7:38 AM Saravana Kannan > > wrote: > > > The fw_devlink_pause() and fw_devlink_resume() APIs allow batching the > > > parsing of the

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-17 Thread Saravana Kannan
Hi Geert, On Wed, Jun 17, 2020 at 5:20 AM Geert Uytterhoeven wrote: > > Hi Saravana, > > On Fri, May 15, 2020 at 7:38 AM Saravana Kannan wrote: > > The fw_devlink_pause() and fw_devlink_resume() APIs allow batching the > > parsing of the device tree nodes when a lot of devices are added. This >

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-06-17 Thread Geert Uytterhoeven
Hi Saravana, On Fri, May 15, 2020 at 7:38 AM Saravana Kannan wrote: > The fw_devlink_pause() and fw_devlink_resume() APIs allow batching the > parsing of the device tree nodes when a lot of devices are added. This > will significantly cut down parsing time (as much a 1 second on some > systems).

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-05-19 Thread Marek Szyprowski
Hi Saravana, On 19.05.2020 20:02, Saravana Kannan wrote: > On Tue, May 19, 2020 at 3:32 AM Marek Szyprowski > wrote: >> On 19.05.2020 09:11, Marek Szyprowski wrote: >>> On 19.05.2020 08:48, Saravana Kannan wrote: On Mon, May 18, 2020 at 11:25 PM Marek Szyprowski wrote: > On

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-05-19 Thread Saravana Kannan
On Tue, May 19, 2020 at 3:32 AM Marek Szyprowski wrote: > > Hi > > On 19.05.2020 09:11, Marek Szyprowski wrote: > > On 19.05.2020 08:48, Saravana Kannan wrote: > >> On Mon, May 18, 2020 at 11:25 PM Marek Szyprowski > >> wrote: > >>> On 15.05.2020 07:35, Saravana Kannan wrote: > The

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-05-19 Thread Marek Szyprowski
Hi On 19.05.2020 09:11, Marek Szyprowski wrote: > On 19.05.2020 08:48, Saravana Kannan wrote: >> On Mon, May 18, 2020 at 11:25 PM Marek Szyprowski >> wrote: >>> On 15.05.2020 07:35, Saravana Kannan wrote: The fw_devlink_pause() and fw_devlink_resume() APIs allow batching the parsing of

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-05-19 Thread Marek Szyprowski
Hi Saravana, On 19.05.2020 08:48, Saravana Kannan wrote: > On Mon, May 18, 2020 at 11:25 PM Marek Szyprowski > wrote: >> On 15.05.2020 07:35, Saravana Kannan wrote: >>> The fw_devlink_pause() and fw_devlink_resume() APIs allow batching the >>> parsing of the device tree nodes when a lot of

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-05-19 Thread Saravana Kannan
On Mon, May 18, 2020 at 11:25 PM Marek Szyprowski wrote: > > Hi Saravana, > > On 15.05.2020 07:35, Saravana Kannan wrote: > > The fw_devlink_pause() and fw_devlink_resume() APIs allow batching the > > parsing of the device tree nodes when a lot of devices are added. This > > will significantly

Re: [PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-05-19 Thread Marek Szyprowski
Hi Saravana, On 15.05.2020 07:35, Saravana Kannan wrote: > The fw_devlink_pause() and fw_devlink_resume() APIs allow batching the > parsing of the device tree nodes when a lot of devices are added. This > will significantly cut down parsing time (as much a 1 second on some > systems). So, use

[PATCH v1 4/4] of: platform: Batch fwnode parsing when adding all top level devices

2020-05-14 Thread Saravana Kannan
The fw_devlink_pause() and fw_devlink_resume() APIs allow batching the parsing of the device tree nodes when a lot of devices are added. This will significantly cut down parsing time (as much a 1 second on some systems). So, use them when adding devices for all the top level device tree nodes in a