Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-18 Thread Frank Rowand
On 7/16/19 3:56 PM, Rob Herring wrote: > On Mon, Jul 15, 2019 at 7:05 PM Frank Rowand wrote: >> >> On 7/15/19 11:40 AM, Saravana Kannan wrote: >>> Replying again because the previous email accidentally included HTML. >>> >>> Thanks for taking the time to reconsider the wording Frank. Your >>>

Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-16 Thread Saravana Kannan
Resending again due to HTML. Sorry about it, the darn thing keeps getting turned on for some reason! On Tue, Jul 16, 2019 at 3:57 PM Rob Herring wrote: > > On Mon, Jul 15, 2019 at 7:05 PM Frank Rowand wrote: > > > > On 7/15/19 11:40 AM, Saravana Kannan wrote: > > > Replying again because the

Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-16 Thread Rob Herring
On Mon, Jul 15, 2019 at 7:05 PM Frank Rowand wrote: > > On 7/15/19 11:40 AM, Saravana Kannan wrote: > > Replying again because the previous email accidentally included HTML. > > > > Thanks for taking the time to reconsider the wording Frank. Your > > intention was clear to me in the first email

Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-15 Thread Frank Rowand
On 7/15/19 11:40 AM, Saravana Kannan wrote: > Replying again because the previous email accidentally included HTML. > > Thanks for taking the time to reconsider the wording Frank. Your > intention was clear to me in the first email too. > > A kernel command line option can also completely

Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-15 Thread Saravana Kannan
Replying again because the previous email accidentally included HTML. Thanks for taking the time to reconsider the wording Frank. Your intention was clear to me in the first email too. A kernel command line option can also completely disable this functionality easily and cleanly. Can we pick

Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-15 Thread Frank Rowand
On 7/15/19 7:26 AM, Frank Rowand wrote: > HiRob, > > Sorry for such a late reply... > > > On 7/1/19 8:25 PM, Saravana Kannan wrote: >> On Mon, Jul 1, 2019 at 6:32 PM Rob Herring wrote: >>> >>> On Mon, Jul 1, 2019 at 6:48 PM Saravana Kannan wrote: Add device-links after the devices

Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-15 Thread Frank Rowand
HiRob, Sorry for such a late reply... On 7/1/19 8:25 PM, Saravana Kannan wrote: > On Mon, Jul 1, 2019 at 6:32 PM Rob Herring wrote: >> >> On Mon, Jul 1, 2019 at 6:48 PM Saravana Kannan wrote: >>> >>> Add device-links after the devices are created (but before they are >>> probed) by looking at

Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-01 Thread Saravana Kannan
On Mon, Jul 1, 2019 at 6:32 PM Rob Herring wrote: > > On Mon, Jul 1, 2019 at 6:48 PM Saravana Kannan wrote: > > > > Add device-links after the devices are created (but before they are > > probed) by looking at common DT bindings like clocks and > > interconnects. > > > > Automatically adding

Re: [PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-01 Thread Rob Herring
On Mon, Jul 1, 2019 at 6:48 PM Saravana Kannan wrote: > > Add device-links after the devices are created (but before they are > probed) by looking at common DT bindings like clocks and > interconnects. > > Automatically adding device-links for functional dependencies at the > framework level

[PATCH v3 2/4] of/platform: Add functional dependency link from DT bindings

2019-07-01 Thread Saravana Kannan
Add device-links after the devices are created (but before they are probed) by looking at common DT bindings like clocks and interconnects. Automatically adding device-links for functional dependencies at the framework level provides the following benefits: - Optimizes device probe order and