Code review comment for lp:~salgado/linaro-image-tools/bug-702645

Revision history for this message
Loïc Minier (lool) wrote :

Looks good; you could mention the LP bug in the XXX comment, and ideally we would have an OmapConfig class which both Panda and Beagle would inherit from rather than the unobvious Panda inherits Beagle situation, but this is temporary anyway.

review: Approve

« Back to merge proposal