Re: [U-Boot] [u-boot-release] [PATCH 7/7] powerpc/8xxx: Add support for interactive DDR programming interface

York Sun wrote:

+/* Option parameter Structures */ +typedef struct {

• const char *option_name;
• size_t offset;
• unsigned int size;
• const char printHex;

+} options_strings_t;

Does it make sense for only printHex (which should be "print_hex") and option_name to be const, but nothing else?