[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <YGWVpvEci16RpAOg@hovoldconsulting.com>
Date: Thu, 1 Apr 2021 11:43:02 +0200
From: Johan Hovold <johan@...nel.org>
To: Pavel Machek <pavel@....cz>
Cc: Rob Herring <robh@...nel.org>, Tony Lindgren <tony@...mide.com>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Alan Cox <gnomes@...rguk.ukuu.org.uk>,
Lee Jones <lee.jones@...aro.org>, Jiri Slaby <jslaby@...e.cz>,
Merlijn Wajer <merlijn@...zup.org>,
Peter Hurley <peter@...leysoftware.com>,
Sebastian Reichel <sre@...nel.org>,
linux-serial@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-omap@...r.kernel.org
Subject: Re: [PATCH 3/6] dt-bindings: serdev: ngsm: Add binding for GNSS
child node
On Fri, Mar 05, 2021 at 11:46:35AM +0100, Pavel Machek wrote:
> Hi!
>
> > > > For motorola modem case, we may have a GNSS device on channel 4.
> > > > Let's add that to the binding and example.
> > > >
> > > > Signed-off-by: Tony Lindgren <tony@...mide.com>
> > > > ---
> > > > .../devicetree/bindings/serdev/serdev-ngsm.yaml | 9 +++++++++
> > > > 1 file changed, 9 insertions(+)
>
> >
> > And since we're describing a mux, I think you need nodes for the virtual
> > ports rather than a reg property in what should be a serial client. That
> > is something like
> >
> > serial@nnn {
> > modem {
> > compatible = "etsi,ts27001-mux";
> >
> > serial@4 {
> > compatible = "etsi,ts27001-serial";
> > reg = <4>;
> >
> > gnss {
> > compatible = "motorola,motmdm-gnss";
> > };
> > };
> > };
> > };
> >
> > This way you can actually use serdev for the client drivers (e.g. for
> > gnss), and those drivers also be used for non-muxed ports if needed
> > (e.g. over USB).
>
> I have done changes you requested, and then hit "serdev is busy
> because it can have at most one child" limit in the code. You have
> pretty clean driver in your inbox, and no reply. No help with serdev
> core limitations, either. Can you start to communicate?
Heh, look at the devicetree example above that I gave you back in March.
It has the virtual serial ports described ("serial@4"), which you were
missing.
Johan
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists