[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