Re: [PATCH 1/4] component: Add documentation

2019-02-18 Thread Randy Dunlap
On 2/7/19 3:27 PM, Daniel Vetter wrote: Hi Daniel, I have a few possible changes for this documentation (see below). > --- > Documentation/driver-api/component.rst | 17 > Documentation/driver-api/device_link.rst | 3 + > Documentation/driver-api/index.rst | 1 + >

Re: [PATCH 1/4] component: Add documentation

2019-02-08 Thread Daniel Vetter
On Fri, Feb 8, 2019 at 4:29 PM Daniel Vetter wrote: > > On Fri, Feb 08, 2019 at 12:51:23PM +0100, Greg Kroah-Hartman wrote: > > On Fri, Feb 08, 2019 at 12:27:56AM +0100, Daniel Vetter wrote: > > > While typing these I think doing an s/component_master/aggregate/ > > > would be useful: > > > -

Re: [PATCH 1/4] component: Add documentation

2019-02-08 Thread Daniel Vetter
On Fri, Feb 08, 2019 at 12:51:23PM +0100, Greg Kroah-Hartman wrote: > On Fri, Feb 08, 2019 at 12:27:56AM +0100, Daniel Vetter wrote: > > While typing these I think doing an s/component_master/aggregate/ > > would be useful: > > - it's shorter :-) > > - I think component/aggregate is much more

Re: [PATCH 1/4] component: Add documentation

2019-02-08 Thread Greg Kroah-Hartman
On Fri, Feb 08, 2019 at 12:27:56AM +0100, Daniel Vetter wrote: > While typing these I think doing an s/component_master/aggregate/ > would be useful: > - it's shorter :-) > - I think component/aggregate is much more meaningful naming than > component/puppetmaster or something like that. At least

[PATCH 1/4] component: Add documentation

2019-02-07 Thread Daniel Vetter
While typing these I think doing an s/component_master/aggregate/ would be useful: - it's shorter :-) - I think component/aggregate is much more meaningful naming than component/puppetmaster or something like that. At least to my English ear "aggregate" emphasizes much more the "assemble a