6 Oct
2009
6 Oct
'09
7:11 p.m.
Hello again,
More patches - these two (combined) add support for mobile-originated
USSD. The demonstration application, implemented in ussd.c, is that
sending *#100# to the network will display your 5-digit extension. It's
not 100% finished - I've tested it on 6 handsets, 5 of them work
perfectly, but the 6th (Samsung i520) doesn't seem to receive the
response, so more experimenting to be done...
Best regards to all,
Mike H.
7 Oct
7 Oct
3:55 a.m.
On Tuesday 06 October 2009 19:11:52 Mike Haben wrote:
Hello again,
More patches - these two (combined) add support for mobile-originated
USSD. The demonstration application, implemented in ussd.c, is that
sending *#100# to the network will display your 5-digit extension. It's
not 100% finished - I've tested it on 6 handsets, 5 of them work
perfectly, but the 6th (Samsung i520) doesn't seem to receive the
response, so more experimenting to be done...
nice, please give us some time to review. Why didn't you add your copyright to
the new files?
holger
8:24 a.m.
Hi Mike,
On Tue, Oct 06, 2009 at 06:11:52PM +0100, Mike Haben wrote:
More patches - these two (combined) add support for
mobile-originated USSD. The demonstration application, implemented
in ussd.c, is that sending *#100# to the network will display your
5-digit extension. It's not 100% finished - I've tested it on 6
handsets, 5 of them work perfectly, but the 6th (Samsung i520)
doesn't seem to receive the response, so more experimenting to be
done...
thanks a lot.
some initial review comments
* please move the "#define GSM0480_" into the header file
* please pay attention to the coding style, we use
linux-2.6/Documentation/CodingStyle . This particularly relates to having
"do {" on one line and not using // style comments, and space at "if
("
* I don't like the ptr8[n] arrays. If there is a common/static header,
please define a structure for it. For the variable parts at the end,
things like msgb_tv_put() or msgb_tlv_put() should be used.
can you please take care of this and re-submit? thanks.
--
- Harald Welte <laforge(a)gnumonks.org> http://laforge.gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
(ETSI EN 300 175-7 Ch. A6)
9 Oct
9 Oct
7:52 p.m.
Hi Harald, thanks for the comments - update patch attached. USSD uses
TLVs inside TLVs inside other TLVs, so I've defined a couple of
msgb_push utility functions, this seemed the most elegant way to express
the construction of these messages.
Interested (and secretly pleased) to find that most of the Linux-2.6
coding style guidelines directly contradict the coding standard imposed
on me at work...
Best regards, Mike H.
Harald Welte wrote:
Hi Mike,
On Tue, Oct 06, 2009 at 06:11:52PM +0100, Mike Haben wrote:
More patches - these two (combined) add support
for
mobile-originated USSD. The demonstration application, implemented
in ussd.c, is that sending *#100# to the network will display your
5-digit extension. It's not 100% finished - I've tested it on 6
handsets, 5 of them work perfectly, but the 6th (Samsung i520)
doesn't seem to receive the response, so more experimenting to be
done...
thanks a lot.
some initial review comments
* please move the "#define GSM0480_" into the header file
* please pay attention to the coding style, we use
linux-2.6/Documentation/CodingStyle . This particularly relates to having
"do {" on one line and not using // style comments, and space at "if
("
* I don't like the ptr8[n] arrays. If there is a common/static header,
please define a structure for it. For the variable parts at the end,
things like msgb_tv_put() or msgb_tlv_put() should be used.
can you please take care of this and re-submit? thanks.
7:53 p.m.
New subject: Patches - add support for mobile-originated USSD - D'oh!
Now with patch...
Harald Welte wrote:
Hi Mike,
On Tue, Oct 06, 2009 at 06:11:52PM +0100, Mike Haben wrote:
More patches - these two (combined) add support
for
mobile-originated USSD. The demonstration application, implemented
in ussd.c, is that sending *#100# to the network will display your
5-digit extension. It's not 100% finished - I've tested it on 6
handsets, 5 of them work perfectly, but the 6th (Samsung i520)
doesn't seem to receive the response, so more experimenting to be
done...
thanks a lot.
some initial review comments
* please move the "#define GSM0480_" into the header file
* please pay attention to the coding style, we use
linux-2.6/Documentation/CodingStyle . This particularly relates to having
"do {" on one line and not using // style comments, and space at "if
("
* I don't like the ptr8[n] arrays. If there is a common/static header,
please define a structure for it. For the variable parts at the end,
things like msgb_tv_put() or msgb_tlv_put() should be used.
can you please take care of this and re-submit? thanks.
5319
days inactive
5322
days old
5 comments
3 participants
participants (3)
-
Harald Welte -
Holger Freyther -
Mike Haben