[U-Boot] [PATCHv2] bootp: add ntpserver option to bootp request
Chris Packham
judge.packham at gmail.com
Tue May 17 06:09:56 CEST 2011
Hi Sergei,
On Mon, May 16, 2011 at 11:57 PM, Sergei Shtylyov <sshtylyov at mvista.com> wrote:
> Hello.
>
> On 16-05-2011 6:24, Chris Packham wrote:
>
>> From: Luuk Paulussen <luuk.paulussen at alliedtelesis.co.nz>
>
>> Signed-off-by: Luuk Paulussen<luuk.paulussen at alliedtelesis.co.nz>
>> Acked-by: Chris Packham<chris.packham at alliedtelesis.co.nz>
>> Cc: Ben Warren<biggerbadderben at 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
More information about the U-Boot
mailing list