mbox series

[v8,0/9] of: property: add of_graph_get_next_port/port_endpoint()

Message ID 87msiub5tt.wl-kuninori.morimoto.gx@renesas.com
Headers show
Series of: property: add of_graph_get_next_port/port_endpoint() | expand

Message

Kuninori Morimoto Oct. 24, 2024, 2:19 a.m. UTC
Hi Rob, Saravana, Tomi, Laurent, Sakari, Mark

This is v8 patch-set

Current Of-graph has "endpoint base" for loop, but doesn't have
"port base" loop. "endpoint base" loop only is not enough.
This patch-set add new "port base" for loop, and use it.

v7 -> v8
	- add Acked-by from Sakari/Helge
	- tidyup comment line

v6 -> v7
	- based on latest linus/master branch
	- remove "ports" base functions
	- use "port" base function on "endpoint" function ([3/9])
	- tidyup [1/9] explanation

v5 -> v6
	- based on latest linus/master branch
	- [9/9]: fixed compile warrning

v4 -> v5
	- tidyup comments
	- [8/9]: parent NULL check was removed
	- [9/9]: use for_each_of_graph_port()

v3 -> v4
	- new for_each loop includes __free()
	 - comment indicates to use return_ptr() or no_free_ptr() if
	   it need to continue to use node
	 - each driver based on it
	- care "prev" leak on of_graph_get_next_ports()
	- of_graph_get_next_port_endpoint() indicates WARN() if port
	  has non-endpoint node
	- tidyup each git-log

v2 -> v3
	- return NULL if it it doesn't have ports / port
	- add visible comment on of_graph_get_next_ports()

v1 -> v2
	- add each Reviewed-by / Acked-by
	- tidyup/update Kernel Docs
	- use prev as parameter
	- update git-log explanation
	- remove extra changes

Kuninori Morimoto (9):
  of: property: add of_graph_get_next_port()
  of: property: add of_graph_get_next_port_endpoint()
  of: property: use new of_graph functions
  ASoC: test-component: use new of_graph functions
  ASoC: audio-graph-card: use new of_graph functions
  ASoC: audio-graph-card2: use new of_graph functions
  gpu: drm: omapdrm: use new of_graph functions
  fbdev: omapfb: use new of_graph functions
  media: xilinx-tpg: use new of_graph functions

 drivers/gpu/drm/omapdrm/dss/dpi.c             |   3 +-
 drivers/gpu/drm/omapdrm/dss/sdi.c             |   3 +-
 drivers/media/platform/xilinx/xilinx-tpg.c    |  14 +--
 drivers/of/property.c                         |  99 ++++++++++++++---
 drivers/video/fbdev/omap2/omapfb/dss/dpi.c    |   3 +-
 drivers/video/fbdev/omap2/omapfb/dss/dss-of.c |  66 -----------
 drivers/video/fbdev/omap2/omapfb/dss/dss.c    |  20 ++--
 drivers/video/fbdev/omap2/omapfb/dss/sdi.c    |   3 +-
 include/linux/of_graph.h                      |  49 +++++++++
 include/video/omapfb_dss.h                    |   8 --
 sound/soc/generic/audio-graph-card.c          |   2 +-
 sound/soc/generic/audio-graph-card2.c         | 104 ++++++++----------
 sound/soc/generic/test-component.c            |   3 +-
 13 files changed, 205 insertions(+), 172 deletions(-)

Comments

Rob Herring Oct. 24, 2024, 10:03 p.m. UTC | #1
On Wed, Oct 23, 2024 at 9:19 PM Kuninori Morimoto
<kuninori.morimoto.gx@renesas.com> wrote:
>
>
> Hi Rob, Saravana, Tomi, Laurent, Sakari, Mark
>
> This is v8 patch-set
>
> Current Of-graph has "endpoint base" for loop, but doesn't have
> "port base" loop. "endpoint base" loop only is not enough.
> This patch-set add new "port base" for loop, and use it.
>
> v7 -> v8
>         - add Acked-by from Sakari/Helge
>         - tidyup comment line
>
> v6 -> v7
>         - based on latest linus/master branch
>         - remove "ports" base functions
>         - use "port" base function on "endpoint" function ([3/9])
>         - tidyup [1/9] explanation
>
> v5 -> v6
>         - based on latest linus/master branch
>         - [9/9]: fixed compile warrning
>
> v4 -> v5
>         - tidyup comments
>         - [8/9]: parent NULL check was removed
>         - [9/9]: use for_each_of_graph_port()
>
> v3 -> v4
>         - new for_each loop includes __free()
>          - comment indicates to use return_ptr() or no_free_ptr() if
>            it need to continue to use node
>          - each driver based on it
>         - care "prev" leak on of_graph_get_next_ports()
>         - of_graph_get_next_port_endpoint() indicates WARN() if port
>           has non-endpoint node
>         - tidyup each git-log
>
> v2 -> v3
>         - return NULL if it it doesn't have ports / port
>         - add visible comment on of_graph_get_next_ports()
>
> v1 -> v2
>         - add each Reviewed-by / Acked-by
>         - tidyup/update Kernel Docs
>         - use prev as parameter
>         - update git-log explanation
>         - remove extra changes
>
> Kuninori Morimoto (9):
>   of: property: add of_graph_get_next_port()
>   of: property: add of_graph_get_next_port_endpoint()
>   of: property: use new of_graph functions
>   ASoC: test-component: use new of_graph functions
>   ASoC: audio-graph-card: use new of_graph functions
>   ASoC: audio-graph-card2: use new of_graph functions
>   gpu: drm: omapdrm: use new of_graph functions
>   fbdev: omapfb: use new of_graph functions
>   media: xilinx-tpg: use new of_graph functions
>
>  drivers/gpu/drm/omapdrm/dss/dpi.c             |   3 +-
>  drivers/gpu/drm/omapdrm/dss/sdi.c             |   3 +-
>  drivers/media/platform/xilinx/xilinx-tpg.c    |  14 +--
>  drivers/of/property.c                         |  99 ++++++++++++++---
>  drivers/video/fbdev/omap2/omapfb/dss/dpi.c    |   3 +-
>  drivers/video/fbdev/omap2/omapfb/dss/dss-of.c |  66 -----------
>  drivers/video/fbdev/omap2/omapfb/dss/dss.c    |  20 ++--
>  drivers/video/fbdev/omap2/omapfb/dss/sdi.c    |   3 +-
>  include/linux/of_graph.h                      |  49 +++++++++
>  include/video/omapfb_dss.h                    |   8 --
>  sound/soc/generic/audio-graph-card.c          |   2 +-
>  sound/soc/generic/audio-graph-card2.c         | 104 ++++++++----------
>  sound/soc/generic/test-component.c            |   3 +-
>  13 files changed, 205 insertions(+), 172 deletions(-)

Applied, thanks.

Rob