Thread (26 messages) 26 messages, 3 authors, 2020-05-12

Re: [PATCHv6 0/6] n_gsm serdev support and GNSS driver for droid4

From: Tony Lindgren <tony@atomide.com>
Date: 2020-05-01 14:20:26
Also in: linux-devicetree, linux-omap, lkml

* Pavel Machek [off-list ref] [200501 08:22]:
Hi!
quoted
Now without the chardev support, the /dev/gsmtty* using apps need
to use "U1234AT+CFUN?" format for the packets. The advantage is
less kernel code, and we keep the existing /dev/gsmtty* interface.
Actually... yes, this works. But no, this is not "existing" tty
interface.

ttys work per character, and this interface definitely does not... it
is "packet" based, write() syscalls need exactly right lengths. You
can't just open minicom, and type "U1234...". You can't paste it,
either (I tried). tty controls like start/stop bits and baud rate are
useless here. CR/LF conversions are unwanted/dangerous because it is
confusing hard to debug if you get them wrong.
Yes.. That's what n_gsm spins up.
Now, I don't see reason why this could not be made to work, and it may
be more important to have something in mainline and work with that. So
if you can make this into -next, I'll not complain too loudly. But it
is... still wrong and I liked motmdm* more :-).
Yes.. There are issues too with the motmdm* char dev interface too.

I don't think it would work as is for devices with network interfaces
on some channel, and continuation packets could need more handling
possibly.

But let's try to get the basics sorted out first and use the "raw"
gsmtty* interface. More stuff can always be added later as needed.

Regards,

Tony


Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help