[U-Boot] [PATCH 5/6] pxe: Ensure all memory access is to mapped memory
Sjoerd Simons
sjoerd.simons at collabora.co.uk
Sat Feb 28 15:12:13 CET 2015
On Fri, 2015-02-20 at 12:23 -0700, Simon Glass wrote:
> Hi Sjoerd,
>
> On 19 February 2015 at 15:41, Sjoerd Simons
> <sjoerd.simons at collabora.co.uk> wrote:
> > Properly map memory through map_sysmem so that pxe can be used from the
> > sandbox.
> >
> > Signed-off-by: Sjoerd Simons <sjoerd.simons at collabora.co.uk>
>
> Please run your patches through patman as you seem to have style
> violations. Also can you add some notes about how you have tested this
> on real hardware?
Will do for the next round together with addressing your comments. One
confused me tough, see below.
> > ---
> > common/cmd_pxe.c | 80 ++++++++++++++++++++++++++++++++------------------------
> > 1 file changed, 46 insertions(+), 34 deletions(-)
> > @@ -1281,13 +1288,14 @@ static int parse_label(char **c, struct pxe_menu *cfg)
> > *
> > * Returns 1 on success, < 0 on error.
> > */
> > -static int parse_pxefile_top(cmd_tbl_t *cmdtp, char *p, struct pxe_menu *cfg, int nest_level)
> > +static int parse_pxefile_top(cmd_tbl_t *cmdtp, char *p, unsigned long b,
> > + struct pxe_menu *cfg, int nest_level)
>
> s/b/base/ or something more meaningful (fix above/below also)
> > {
> > struct token t;
> > - char *s, *b, *label_name;
> > + char *s, *label_name, *base;
> > int err;
> >
> > - b = p;
> > + base = p;
>
> This worries me - assigning a pointer to a ulong.
base is a pointer here though. So this comment confuses me.
--
Sjoerd Simons <sjoerd.simons at collabora.co.uk>
Collabora Ltd.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 6170 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20150228/76d88f81/attachment.bin>
More information about the U-Boot
mailing list