| Message ID | 20210107132838.396641-1-djrscally@gmail.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | Add functionality to ipu3-cio2 driver allowing software_node connections to sensors on platforms designed for Windows | expand |
Hi Greg, On Thu, Jan 07, 2021 at 01:28:23PM +0000, Daniel Scally wrote: > > Hello all > > v4: > https://lore.kernel.org/linux-media/20210103231235.792999-1-djrscally@gmail.com/T/#m11b7cb977e1b73fba1e625c3d6a189e2943a7783 > v3: > https://lore.kernel.org/linux-media/20201224010907.263125-1-djrscally@gmail.com/T/#m37b831bb2b406917d6db5da9acf9ed35df65d72d > v2: > https://lore.kernel.org/linux-media/20201217234337.1983732-1-djrscally@gmail.com/T/#md93fd090009b42a6a98aed892aff0d38cf07e0cd > v1: > https://lore.kernel.org/linux-media/20201130133129.1024662-1-djrscally@gmail.com/T/#m91934e12e3d033da2e768e952ea3b4a125ee3e67 > > This series is to start adding support for webcams on laptops with ACPI tables > designed for use with CIO2 on Windows. This series extends the ipu3-cio2 > driver to allow for patching the firmware via software_nodes if endpoints > aren't defined by ACPI. > > I'm hopeful that most or all of this series could get picked up for 5.12. > We touch a few different areas (listed below), but I think the easiest > approach would be to merge everything through media tree. Rafael, Greg, > Mauro and Sergey; are you ok with that plan, or would you prefer a > different approach? Mauro; if that plan is ok (and of course assuming that > the rest of the patches are acked by their respective maintainers) could > we get a dedicated feature branch just in case the following series ends > up being ready in time too? > > lib > lib/test_printf.c: Use helper function to unwind array of > software_nodes > > base > software_node: Fix refcounts in software_node_get_next_child() > property: Return true in fwnode_device_is_available for NULL ops > property: Call fwnode_graph_get_endpoint_by_id() for fwnode->secondary > software_node: Enforce parent before child ordering of nodes arrays > software_node: unregister software_nodes in reverse order Could you please let us know with an Acked-by if these patches can be merged through the linux-media tree for v5.12 ? This is a cross-tree series and we would like to avoid topic branches if possible. > include: fwnode.h: Define format macros for ports and endpoints > > acpi > acpi: Add acpi_dev_get_next_match_dev() and helper macro > > media > media: v4l2-core: v4l2-async: Check sd->fwnode->secondary in > match_fwnode() > ipu3-cio2: Add T: entry to MAINTAINERS > ipu3-cio2: Rename ipu3-cio2.c > ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver > include: media: v4l2-fwnode: Include v4l2_fwnode_bus_type > > Series-level changelog: > - Rebased onto 5.11-rc1 > > Thanks > Dan > > Andy Shevchenko (1): > media: ipu3-cio2: Add headers that ipu3-cio2.h is direct user of > > Daniel Scally (13): > software_node: Fix refcounts in software_node_get_next_child() > device property: Return true in fwnode_device_is_available for NULL > ops > device property: Call fwnode_graph_get_endpoint_by_id() for > fwnode->secondary > software_node: Enforce parent before child ordering of nodes arrays > software_node: unregister software_nodes in reverse order > device property: Define format macros for ports and endpoints > lib/test_printf.c: Use helper function to unwind array of > software_nodes > ipu3-cio2: Add T: entry to MAINTAINERS > ipu3-cio2: Rename ipu3-cio2.c > media: v4l2-core: v4l2-async: Check sd->fwnode->secondary in > match_fwnode() > ACPI / bus: Add acpi_dev_get_next_match_dev() and helper macro > media: v4l2-fwnode: Include v4l2_fwnode_bus_type > ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver > > Heikki Krogerus (1): > software_node: Add support for fwnode_graph*() family of functions > > MAINTAINERS | 2 + > drivers/acpi/utils.c | 30 +- > drivers/base/property.c | 15 +- > drivers/base/swnode.c | 180 ++++++++-- > drivers/media/pci/intel/ipu3/Kconfig | 18 + > drivers/media/pci/intel/ipu3/Makefile | 3 + > drivers/media/pci/intel/ipu3/cio2-bridge.c | 311 ++++++++++++++++++ > drivers/media/pci/intel/ipu3/cio2-bridge.h | 125 +++++++ > .../ipu3/{ipu3-cio2.c => ipu3-cio2-main.c} | 34 ++ > drivers/media/pci/intel/ipu3/ipu3-cio2.h | 24 ++ > drivers/media/v4l2-core/v4l2-async.c | 8 + > drivers/media/v4l2-core/v4l2-fwnode.c | 11 - > include/acpi/acpi_bus.h | 7 + > include/linux/fwnode.h | 7 + > include/media/v4l2-fwnode.h | 22 ++ > lib/test_printf.c | 4 +- > 16 files changed, 763 insertions(+), 38 deletions(-) > create mode 100644 drivers/media/pci/intel/ipu3/cio2-bridge.c > create mode 100644 drivers/media/pci/intel/ipu3/cio2-bridge.h > rename drivers/media/pci/intel/ipu3/{ipu3-cio2.c => ipu3-cio2-main.c} (98%)
On Sat, Jan 09, 2021 at 04:08:04AM +0200, Laurent Pinchart wrote: > Hi Greg, > > On Thu, Jan 07, 2021 at 01:28:23PM +0000, Daniel Scally wrote: > > > > Hello all > > > > v4: > > https://lore.kernel.org/linux-media/20210103231235.792999-1-djrscally@gmail.com/T/#m11b7cb977e1b73fba1e625c3d6a189e2943a7783 > > v3: > > https://lore.kernel.org/linux-media/20201224010907.263125-1-djrscally@gmail.com/T/#m37b831bb2b406917d6db5da9acf9ed35df65d72d > > v2: > > https://lore.kernel.org/linux-media/20201217234337.1983732-1-djrscally@gmail.com/T/#md93fd090009b42a6a98aed892aff0d38cf07e0cd > > v1: > > https://lore.kernel.org/linux-media/20201130133129.1024662-1-djrscally@gmail.com/T/#m91934e12e3d033da2e768e952ea3b4a125ee3e67 > > > > This series is to start adding support for webcams on laptops with ACPI tables > > designed for use with CIO2 on Windows. This series extends the ipu3-cio2 > > driver to allow for patching the firmware via software_nodes if endpoints > > aren't defined by ACPI. > > > > I'm hopeful that most or all of this series could get picked up for 5.12. > > We touch a few different areas (listed below), but I think the easiest > > approach would be to merge everything through media tree. Rafael, Greg, > > Mauro and Sergey; are you ok with that plan, or would you prefer a > > different approach? Mauro; if that plan is ok (and of course assuming that > > the rest of the patches are acked by their respective maintainers) could > > we get a dedicated feature branch just in case the following series ends > > up being ready in time too? > > > > lib > > lib/test_printf.c: Use helper function to unwind array of > > software_nodes > > > > base > > software_node: Fix refcounts in software_node_get_next_child() > > property: Return true in fwnode_device_is_available for NULL ops > > property: Call fwnode_graph_get_endpoint_by_id() for fwnode->secondary > > software_node: Enforce parent before child ordering of nodes arrays > > software_node: unregister software_nodes in reverse order > > Could you please let us know with an Acked-by if these patches can be > merged through the linux-media tree for v5.12 ? This is a cross-tree > series and we would like to avoid topic branches if possible. Yes, they are all fine with me, will go provide my ack, sorry for the delay. greg k-h
On Thu, Jan 7, 2021 at 2:30 PM Daniel Scally <djrscally@gmail.com> wrote: > > > Hello all > > v4: > https://lore.kernel.org/linux-media/20210103231235.792999-1-djrscally@gmail.com/T/#m11b7cb977e1b73fba1e625c3d6a189e2943a7783 > v3: > https://lore.kernel.org/linux-media/20201224010907.263125-1-djrscally@gmail.com/T/#m37b831bb2b406917d6db5da9acf9ed35df65d72d > v2: > https://lore.kernel.org/linux-media/20201217234337.1983732-1-djrscally@gmail.com/T/#md93fd090009b42a6a98aed892aff0d38cf07e0cd > v1: > https://lore.kernel.org/linux-media/20201130133129.1024662-1-djrscally@gmail.com/T/#m91934e12e3d033da2e768e952ea3b4a125ee3e67 > > This series is to start adding support for webcams on laptops with ACPI tables > designed for use with CIO2 on Windows. This series extends the ipu3-cio2 > driver to allow for patching the firmware via software_nodes if endpoints > aren't defined by ACPI. > > I'm hopeful that most or all of this series could get picked up for 5.12. > We touch a few different areas (listed below), but I think the easiest > approach would be to merge everything through media tree. Rafael, Greg, > Mauro and Sergey; are you ok with that plan, or would you prefer a > different approach? Mauro; if that plan is ok (and of course assuming that > the rest of the patches are acked by their respective maintainers) could > we get a dedicated feature branch just in case the following series ends > up being ready in time too? > > lib > lib/test_printf.c: Use helper function to unwind array of > software_nodes > > base > software_node: Fix refcounts in software_node_get_next_child() > property: Return true in fwnode_device_is_available for NULL ops > property: Call fwnode_graph_get_endpoint_by_id() for fwnode->secondary > software_node: Enforce parent before child ordering of nodes arrays > software_node: unregister software_nodes in reverse order > include: fwnode.h: Define format macros for ports and endpoints > > acpi > acpi: Add acpi_dev_get_next_match_dev() and helper macro > > media > media: v4l2-core: v4l2-async: Check sd->fwnode->secondary in > match_fwnode() > ipu3-cio2: Add T: entry to MAINTAINERS > ipu3-cio2: Rename ipu3-cio2.c > ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver > include: media: v4l2-fwnode: Include v4l2_fwnode_bus_type > > Series-level changelog: > - Rebased onto 5.11-rc1 > > Thanks > Dan > > Andy Shevchenko (1): > media: ipu3-cio2: Add headers that ipu3-cio2.h is direct user of > > Daniel Scally (13): > software_node: Fix refcounts in software_node_get_next_child() > device property: Return true in fwnode_device_is_available for NULL > ops > device property: Call fwnode_graph_get_endpoint_by_id() for > fwnode->secondary > software_node: Enforce parent before child ordering of nodes arrays > software_node: unregister software_nodes in reverse order > device property: Define format macros for ports and endpoints > lib/test_printf.c: Use helper function to unwind array of > software_nodes > ipu3-cio2: Add T: entry to MAINTAINERS > ipu3-cio2: Rename ipu3-cio2.c > media: v4l2-core: v4l2-async: Check sd->fwnode->secondary in > match_fwnode() > ACPI / bus: Add acpi_dev_get_next_match_dev() and helper macro > media: v4l2-fwnode: Include v4l2_fwnode_bus_type > ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver > > Heikki Krogerus (1): > software_node: Add support for fwnode_graph*() family of functions Please feel free to add Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> to all of the device properties patches in this series if that helps. Thanks!
Hi Rafael, Sakari On 12/01/2021 19:34, Rafael J. Wysocki wrote: > <snip> >> I'm hopeful that most or all of this series could get picked up for 5.12. >> We touch a few different areas (listed below), but I think the easiest >> approach would be to merge everything through media tree. Rafael, Greg, >> Mauro and Sergey; are you ok with that plan, or would you prefer a >> different approach? Mauro; if that plan is ok (and of course assuming that >> the rest of the patches are acked by their respective maintainers) could >> we get a dedicated feature branch just in case the following series ends >> up being ready in time too? >> >> <snip> > Please feel free to add > > Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > to all of the device properties patches in this series if that helps. > > Thanks! Thanks very much (and Greg too). Sakari; unless I'm misunderstanding something, I think this series could be picked up now, right? Would it be ok to do that through your tree? I think the idea of a dedicated feature branch can be dropped, I won't have the second series ready in time for this round anyway. First time doing this, so if I've missed something please let me know!
Hi Daniel, On Tue, Jan 12, 2021 at 11:35:37PM +0000, Daniel Scally wrote: > Hi Rafael, Sakari > > On 12/01/2021 19:34, Rafael J. Wysocki wrote: > > <snip> > >> I'm hopeful that most or all of this series could get picked up for 5.12. > >> We touch a few different areas (listed below), but I think the easiest > >> approach would be to merge everything through media tree. Rafael, Greg, > >> Mauro and Sergey; are you ok with that plan, or would you prefer a > >> different approach? Mauro; if that plan is ok (and of course assuming that > >> the rest of the patches are acked by their respective maintainers) could > >> we get a dedicated feature branch just in case the following series ends > >> up being ready in time too? > >> > >> <snip> > > Please feel free to add > > > > Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com> > > > > to all of the device properties patches in this series if that helps. > > > > Thanks! > > Thanks very much (and Greg too). > > > Sakari; unless I'm misunderstanding something, I think this series could > be picked up now, right? Would it be ok to do that through your tree? I > think the idea of a dedicated feature branch can be dropped, I won't > have the second series ready in time for this round anyway. > > > First time doing this, so if I've missed something please let me know! I think it's ready, indeed. I'll let you know if there are any issues.
Hello all v4: https://lore.kernel.org/linux-media/20210103231235.792999-1-djrscally@gmail.com/T/#m11b7cb977e1b73fba1e625c3d6a189e2943a7783 v3: https://lore.kernel.org/linux-media/20201224010907.263125-1-djrscally@gmail.com/T/#m37b831bb2b406917d6db5da9acf9ed35df65d72d v2: https://lore.kernel.org/linux-media/20201217234337.1983732-1-djrscally@gmail.com/T/#md93fd090009b42a6a98aed892aff0d38cf07e0cd v1: https://lore.kernel.org/linux-media/20201130133129.1024662-1-djrscally@gmail.com/T/#m91934e12e3d033da2e768e952ea3b4a125ee3e67 This series is to start adding support for webcams on laptops with ACPI tables designed for use with CIO2 on Windows. This series extends the ipu3-cio2 driver to allow for patching the firmware via software_nodes if endpoints aren't defined by ACPI. I'm hopeful that most or all of this series could get picked up for 5.12. We touch a few different areas (listed below), but I think the easiest approach would be to merge everything through media tree. Rafael, Greg, Mauro and Sergey; are you ok with that plan, or would you prefer a different approach? Mauro; if that plan is ok (and of course assuming that the rest of the patches are acked by their respective maintainers) could we get a dedicated feature branch just in case the following series ends up being ready in time too? lib lib/test_printf.c: Use helper function to unwind array of software_nodes base software_node: Fix refcounts in software_node_get_next_child() property: Return true in fwnode_device_is_available for NULL ops property: Call fwnode_graph_get_endpoint_by_id() for fwnode->secondary software_node: Enforce parent before child ordering of nodes arrays software_node: unregister software_nodes in reverse order include: fwnode.h: Define format macros for ports and endpoints acpi acpi: Add acpi_dev_get_next_match_dev() and helper macro media media: v4l2-core: v4l2-async: Check sd->fwnode->secondary in match_fwnode() ipu3-cio2: Add T: entry to MAINTAINERS ipu3-cio2: Rename ipu3-cio2.c ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver include: media: v4l2-fwnode: Include v4l2_fwnode_bus_type Series-level changelog: - Rebased onto 5.11-rc1 Thanks Dan Andy Shevchenko (1): media: ipu3-cio2: Add headers that ipu3-cio2.h is direct user of Daniel Scally (13): software_node: Fix refcounts in software_node_get_next_child() device property: Return true in fwnode_device_is_available for NULL ops device property: Call fwnode_graph_get_endpoint_by_id() for fwnode->secondary software_node: Enforce parent before child ordering of nodes arrays software_node: unregister software_nodes in reverse order device property: Define format macros for ports and endpoints lib/test_printf.c: Use helper function to unwind array of software_nodes ipu3-cio2: Add T: entry to MAINTAINERS ipu3-cio2: Rename ipu3-cio2.c media: v4l2-core: v4l2-async: Check sd->fwnode->secondary in match_fwnode() ACPI / bus: Add acpi_dev_get_next_match_dev() and helper macro media: v4l2-fwnode: Include v4l2_fwnode_bus_type ipu3-cio2: Add cio2-bridge to ipu3-cio2 driver Heikki Krogerus (1): software_node: Add support for fwnode_graph*() family of functions MAINTAINERS | 2 + drivers/acpi/utils.c | 30 +- drivers/base/property.c | 15 +- drivers/base/swnode.c | 180 ++++++++-- drivers/media/pci/intel/ipu3/Kconfig | 18 + drivers/media/pci/intel/ipu3/Makefile | 3 + drivers/media/pci/intel/ipu3/cio2-bridge.c | 311 ++++++++++++++++++ drivers/media/pci/intel/ipu3/cio2-bridge.h | 125 +++++++ .../ipu3/{ipu3-cio2.c => ipu3-cio2-main.c} | 34 ++ drivers/media/pci/intel/ipu3/ipu3-cio2.h | 24 ++ drivers/media/v4l2-core/v4l2-async.c | 8 + drivers/media/v4l2-core/v4l2-fwnode.c | 11 - include/acpi/acpi_bus.h | 7 + include/linux/fwnode.h | 7 + include/media/v4l2-fwnode.h | 22 ++ lib/test_printf.c | 4 +- 16 files changed, 763 insertions(+), 38 deletions(-) create mode 100644 drivers/media/pci/intel/ipu3/cio2-bridge.c create mode 100644 drivers/media/pci/intel/ipu3/cio2-bridge.h rename drivers/media/pci/intel/ipu3/{ipu3-cio2.c => ipu3-cio2-main.c} (98%)