Hi Sergei,
On Mon, May 16, 2011 at 11:57 PM, Sergei Shtylyov sshtylyov@mvista.com wrote:
Hello.
On 16-05-2011 6:24, Chris Packham wrote:
From: Luuk Paulussen luuk.paulussen@alliedtelesis.co.nz
Signed-off-by: Luuk Paulussenluuk.paulussen@alliedtelesis.co.nz Acked-by: Chris Packhamchris.packham@alliedtelesis.co.nz Cc: Ben Warrenbiggerbadderben@gmail.com
[...]
diff --git a/net/bootp.c b/net/bootp.c index 4db63cb..18c743e 100644 --- a/net/bootp.c +++ b/net/bootp.c @@ -228,6 +228,11 @@ static void BootpVendorFieldProcess (u8 * ext) NetOurNISDomain[size] = 0; } break; +#if defined(CONFIG_CMD_SNTP)&& defined(CONFIG_BOOTP_NTPSERVER)
- case 42: /* NTP server IP */
- NetCopyIP (&NetNtpServerIP, (IPaddr_t *) (ext + 2));
There should be no space between function name and (. You're now supposed to run your patch thru scripts/checkpatch.pl (from Linux source tree).
Will fix (and run through checkpatch.pl) in next version.
- break;
+#endif /* Application layer fields */ case 43: /* Vendor specific info - Not yet supported */ /* @@ -538,6 +548,11 @@ static int BootpExtended (u8 * e) *e++ = 32; e += 32; #endif +#if defined(CONFIG_BOOTP_NTPSERVER)
- *e++ = 42;
Too many spaces...
Ditto.
- *e++ = 4;
- e += 4;
+#endif
WBR, Sergei
v3 coming shortly