Comment 10 for bug 927781

Revision history for this message
John Rigby (jcrigby) wrote :

I did a quick review, I don't think adding this patch would break anything. The files need a proper u-boot style file header but that is easily fixed. Some of the ifdef's are a bit confusing to me but a better way to do it does not jump out at me. I can apply this to u-boot-linaro.

In the longer term it would be great if the Calxeda folks (Jason and Rob) could sign off on this and if Rob could send a patch upstream since he is the defacto u-boot pxe maintainer having sent the most recent fixed upstream.

Of course this is all based on if Justin and Dave still want this functionality in or are happy with the alternative method that Jason outlines in comment #2 above.