* [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" @ 2021-06-16 11:53 Zhenguo Zhao 2021-06-24 12:58 ` Greg KH ` (2 more replies) 0 siblings, 3 replies; 6+ messages in thread From: Zhenguo Zhao @ 2021-06-16 11:53 UTC (permalink / raw) To: zhenguo6858, gregkh, jirislaby; +Cc: linux-kernel From: Zhenguo Zhao <zhenguo.zhao1@unisoc.com> config "initiator=0",gsmld will receives dlci SABM/DISC control command frame,as slaver,the CR bit value check should be 1. if check "cr == 0",it will goto invalid,so it can't send UA response frame and open slaver dlci. Signed-off-by: Zhenguo Zhao <zhenguo.zhao1@unisoc.com> --- drivers/tty/n_gsm.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/tty/n_gsm.c b/drivers/tty/n_gsm.c index 5fea02c..becca2c 100644 --- a/drivers/tty/n_gsm.c +++ b/drivers/tty/n_gsm.c @@ -1779,7 +1779,7 @@ static void gsm_queue(struct gsm_mux *gsm) switch (gsm->control) { case SABM|PF: - if (cr == 0) + if (cr == 1) goto invalid; if (dlci == NULL) dlci = gsm_dlci_alloc(gsm, address); @@ -1793,7 +1793,7 @@ static void gsm_queue(struct gsm_mux *gsm) } break; case DISC|PF: - if (cr == 0) + if (cr == 1) goto invalid; if (dlci == NULL || dlci->state == DLCI_CLOSED) { gsm_response(gsm, address, DM); -- 1.9.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" 2021-06-16 11:53 [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" Zhenguo Zhao @ 2021-06-24 12:58 ` Greg KH [not found] ` <CAGGV+3LgLdohhYhH+qJTokeNU_WdV9oRNHMc9a_5YTTVA3U8ow@mail.gmail.com> 2021-07-07 6:12 ` 赵振国 2 siblings, 0 replies; 6+ messages in thread From: Greg KH @ 2021-06-24 12:58 UTC (permalink / raw) To: Zhenguo Zhao; +Cc: jirislaby, linux-kernel On Wed, Jun 16, 2021 at 07:53:22PM +0800, Zhenguo Zhao wrote: > From: Zhenguo Zhao <zhenguo.zhao1@unisoc.com> > > config "initiator=0",gsmld will receives dlci SABM/DISC control command > frame,as slaver,the CR bit value check should be 1. I am really sorry, but I do not understand this changelog comment. Can you try rewriting it again? > > if check "cr == 0",it will goto invalid,so it can't send UA response > frame and open slaver dlci. Same here, I do not understand. thanks, greg k-h ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <CAGGV+3LgLdohhYhH+qJTokeNU_WdV9oRNHMc9a_5YTTVA3U8ow@mail.gmail.com>]
* Re: [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" [not found] ` <CAGGV+3LgLdohhYhH+qJTokeNU_WdV9oRNHMc9a_5YTTVA3U8ow@mail.gmail.com> @ 2021-07-07 6:08 ` 赵振国 2021-07-07 6:10 ` Greg KH 1 sibling, 0 replies; 6+ messages in thread From: 赵振国 @ 2021-07-07 6:08 UTC (permalink / raw) To: 赵振国; +Cc: linux-kernel Dear Jiri,Greg 1: our development board uses linux kernel , uart dev node is "/dev/ttyGS2" 2: config uart "/dev/ttyGS2" ,we use ngsm ldisc,and config "c.initiator = 0;" code is as follows #include <stdio.h> #include <stdint.h> #include <linux/gsmmux.h> #include <linux/tty.h> #define DEFAULT_SPEED B115200 #define SERIAL_PORT /dev/ttyGS2 int ldisc = N_GSM0710; struct gsm_config c; struct termios configuration; uint32_t first; /* open the serial port */ fd = open(SERIAL_PORT, O_RDWR | O_NOCTTY | O_NDELAY); /* configure the serial port : speed, flow control ... */ /* use n_gsm line discipline */ ioctl(fd, TIOCSETD, &ldisc); /* get n_gsm configuration */ ioctl(fd, GSMIOC_GETCONF, &c); /* we are responter and need encoding 0 (basic) */ c.initiator = 0; c.encapsulation = 0; /* our modem defaults to a maximum size of 127 bytes */ c.mru = 127; c.mtu = 127; /* set the new configuration */ ioctl(fd, GSMIOC_SETCONF, &c); /* get first gsmtty device node */ ioctl(fd, GSMIOC_GETFIRST, &first); printf("first muxed line: /dev/gsmtty%i\n", first); /* and wait for ever to keep the line discipline enabled */ daemon(0,0); pause(); 3: connect to ubuntu by uart serial port cable,ubuntu uart dev node is /"dev/ttyUSB0" send DLC0 SABM command by "/dev/ttyUSB0",but linux development board can't response,code is as follows int main(int argc, char **argv) { int fd; fd = open("/dev/ttyUSB0,O_RDWR | O_NOCTTY | O_NDELAY); char buf[256]={0xf9,0x03,0x3f,0x01,0x1c,0xf9}; write(fd,buf,6); close(fd); } 4: linux development board receive data,by uart,gsm_queue will check CR,find CR=1.so go to invalid,pls check again. static void gsm_queue(struct gsm_mux *gsm) { cr ^= 1 - gsm->initiator; /* Flip so 1 always means command */ dlci = gsm->dlci[address]; switch (gsm->control) { case SABM|PF: if (cr == 0) goto invalid; if (dlci == NULL) dlci = gsm_dlci_alloc(gsm, address); if (dlci == NULL) return; if (dlci->dead) gsm_response(gsm, address, DM); else { gsm_response(gsm, address, UA); gsm_dlci_open(dlci); } break; ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" [not found] ` <CAGGV+3LgLdohhYhH+qJTokeNU_WdV9oRNHMc9a_5YTTVA3U8ow@mail.gmail.com> 2021-07-07 6:08 ` 赵振国 @ 2021-07-07 6:10 ` Greg KH 1 sibling, 0 replies; 6+ messages in thread From: Greg KH @ 2021-07-07 6:10 UTC (permalink / raw) To: 赵振国; +Cc: Jiri Slaby, linux-kernel On Wed, Jul 07, 2021 at 02:02:54PM +0800, 赵振国 wrote: > Dear Jiri,Greg <snip> still html :( Please fix your email client, there are instructions on how to do that in the Documentation/ directory. greg k-h ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" 2021-06-16 11:53 [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" Zhenguo Zhao 2021-06-24 12:58 ` Greg KH [not found] ` <CAGGV+3LgLdohhYhH+qJTokeNU_WdV9oRNHMc9a_5YTTVA3U8ow@mail.gmail.com> @ 2021-07-07 6:12 ` 赵振国 2021-07-15 2:29 ` 赵振国 2 siblings, 1 reply; 6+ messages in thread From: 赵振国 @ 2021-07-07 6:12 UTC (permalink / raw) To: 赵振国, Greg KH, Jiri Slaby; +Cc: linux-kernel Dear Jiri,Greg 1: our development board uses linux kernel , uart dev node is "/dev/ttyGS2" 2: config uart "/dev/ttyGS2" ,we use ngsm ldisc,and config "c.initiator = 0;" code is as follows #include <stdio.h> #include <stdint.h> #include <linux/gsmmux.h> #include <linux/tty.h> #define DEFAULT_SPEED B115200 #define SERIAL_PORT /dev/ttyGS2 int ldisc = N_GSM0710; struct gsm_config c; struct termios configuration; uint32_t first; /* open the serial port */ fd = open(SERIAL_PORT, O_RDWR | O_NOCTTY | O_NDELAY); /* configure the serial port : speed, flow control ... */ /* use n_gsm line discipline */ ioctl(fd, TIOCSETD, &ldisc); /* get n_gsm configuration */ ioctl(fd, GSMIOC_GETCONF, &c); /* we are responter and need encoding 0 (basic) */ c.initiator = 0; c.encapsulation = 0; /* our modem defaults to a maximum size of 127 bytes */ c.mru = 127; c.mtu = 127; /* set the new configuration */ ioctl(fd, GSMIOC_SETCONF, &c); /* get first gsmtty device node */ ioctl(fd, GSMIOC_GETFIRST, &first); printf("first muxed line: /dev/gsmtty%i\n", first); /* and wait for ever to keep the line discipline enabled */ daemon(0,0); pause(); 3: connect to ubuntu by uart serial port cable,ubuntu uart dev node is /"dev/ttyUSB0" send DLC0 SABM command by "/dev/ttyUSB0",but linux development board can't response,code is as follows int main(int argc, char **argv) { int fd; fd = open("/dev/ttyUSB0,O_RDWR | O_NOCTTY | O_NDELAY); char buf[256]={0xf9,0x03,0x3f,0x01,0x1c,0xf9}; write(fd,buf,6); close(fd); } 4: linux development board receive data,by uart,gsm_queue will check CR,find CR=1.so go to invalid,pls check again. static void gsm_queue(struct gsm_mux *gsm) { cr ^= 1 - gsm->initiator; /* Flip so 1 always means command */ dlci = gsm->dlci[address]; switch (gsm->control) { case SABM|PF: if (cr == 0) goto invalid; if (dlci == NULL) dlci = gsm_dlci_alloc(gsm, address); if (dlci == NULL) return; if (dlci->dead) gsm_response(gsm, address, DM); else { gsm_response(gsm, address, UA); gsm_dlci_open(dlci); } break; ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" 2021-07-07 6:12 ` 赵振国 @ 2021-07-15 2:29 ` 赵振国 0 siblings, 0 replies; 6+ messages in thread From: 赵振国 @ 2021-07-15 2:29 UTC (permalink / raw) To: 赵振国, Greg KH, Jiri Slaby; +Cc: linux-kernel Dear Jiri,Greg ngsm as responser,pls review the process again. because according to this process, it can't work. Thanks! 赵振国 <zhenguo6858@gmail.com> 于2021年7月7日周三 下午2:12写道: > > Dear Jiri,Greg > > 1: our development board uses linux kernel , uart dev node is "/dev/ttyGS2" > 2: config uart "/dev/ttyGS2" ,we use ngsm ldisc,and config > "c.initiator = 0;" code is as follows > > #include <stdio.h> > #include <stdint.h> > #include <linux/gsmmux.h> > #include <linux/tty.h> > #define DEFAULT_SPEED B115200 > #define SERIAL_PORT /dev/ttyGS2 > > int ldisc = N_GSM0710; > struct gsm_config c; > struct termios configuration; > uint32_t first; > > /* open the serial port */ > fd = open(SERIAL_PORT, O_RDWR | O_NOCTTY | O_NDELAY); > > /* configure the serial port : speed, flow control ... */ > > /* use n_gsm line discipline */ > ioctl(fd, TIOCSETD, &ldisc); > > /* get n_gsm configuration */ > ioctl(fd, GSMIOC_GETCONF, &c); > /* we are responter and need encoding 0 (basic) */ > c.initiator = 0; > c.encapsulation = 0; > /* our modem defaults to a maximum size of 127 bytes */ > c.mru = 127; > c.mtu = 127; > /* set the new configuration */ > ioctl(fd, GSMIOC_SETCONF, &c); > /* get first gsmtty device node */ > ioctl(fd, GSMIOC_GETFIRST, &first); > printf("first muxed line: /dev/gsmtty%i\n", first); > > /* and wait for ever to keep the line discipline enabled */ > daemon(0,0); > pause(); > > 3: connect to ubuntu by uart serial port cable,ubuntu uart dev node > is /"dev/ttyUSB0" > send DLC0 SABM command by "/dev/ttyUSB0",but linux development board > can't response,code is as follows > > int main(int argc, char **argv) > { > int fd; > fd = open("/dev/ttyUSB0,O_RDWR | O_NOCTTY | O_NDELAY); > char buf[256]={0xf9,0x03,0x3f,0x01,0x1c,0xf9}; > write(fd,buf,6); > close(fd); > } > > 4: linux development board receive data,by uart,gsm_queue will check > CR,find CR=1.so go to invalid,pls check again. > static void gsm_queue(struct gsm_mux *gsm) > { > cr ^= 1 - gsm->initiator; /* Flip so 1 always means command */ > dlci = gsm->dlci[address]; > > switch (gsm->control) { > case SABM|PF: > if (cr == 0) > goto invalid; > if (dlci == NULL) > dlci = gsm_dlci_alloc(gsm, address); > if (dlci == NULL) > return; > if (dlci->dead) > gsm_response(gsm, address, DM); > else { > gsm_response(gsm, address, UA); > gsm_dlci_open(dlci); > } > break; ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-07-15 2:30 UTC | newest] Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-06-16 11:53 [PATCH v2] tty: n_gsm: CR bit value should be 0 when config "initiator=0" Zhenguo Zhao 2021-06-24 12:58 ` Greg KH [not found] ` <CAGGV+3LgLdohhYhH+qJTokeNU_WdV9oRNHMc9a_5YTTVA3U8ow@mail.gmail.com> 2021-07-07 6:08 ` 赵振国 2021-07-07 6:10 ` Greg KH 2021-07-07 6:12 ` 赵振国 2021-07-15 2:29 ` 赵振国
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.