Re: [PATCH v7 1/9] of: property: add of_graph_get_next_port()

[Sakari Ailus <sakari.ailus@iki.fi>]

Dear Morimoto-san,

On Wed, Oct 23, 2024 at 04:40:45AM +0000, Kuninori Morimoto wrote:
> 
> Hi Sakari, again
> 
> > > diff --git a/drivers/of/property.c b/drivers/of/property.c
> > > index 11b922fde7af..6a5d27dd0c64 100644
> > > --- a/drivers/of/property.c
> > > +++ b/drivers/of/property.c
> > > @@ -630,6 +630,43 @@ struct device_node *of_graph_get_port_by_id(struct device_node *parent, u32 id)
> > >  }
> > >  EXPORT_SYMBOL(of_graph_get_port_by_id);
> > >  
> > > +/**
> > > + * of_graph_get_next_port() - get next port node.
> > > + * @parent: pointer to the parent device node, or parent ports node
> > > + * @prev: previous port node, or NULL to get first
> > > + *
> > > + * Parent device node can be used as @parent whether device node has ports node or not.
> > 
> > This line should be wrapped, no reason to have it longer than 80 chars.
> 
> We can use 100 char now on upstream ?
> 
> 	commit bdc48fa11e46f867ea4d75fa59ee87a7f48be144
> 	("checkpatch/coding-style: deprecate 80-column warning")

It's the checkpatch.pl warning that's gone, not the preference to have
lines shorter than that. This is reflected in
Documentation/process/coding-style.rst as well as the commit message of the
patch removing the warning.

> 
> Thank you for your help !!

You're welcome!

-- 
Kind regards,

Sakari Ailus