Change in flashrom[master]: layout: Add -i <region>[:<file>] support

[Daniel Campello (Code Review)]

Attention is currently required from: Sam McNally, Nico Huber, Nicola Corna, Paul Menzel, Stefan Reinauer, David Hendricks, Edward O'Callaghan, Arthur Heymans, Carl-Daniel Hailfinger.
Daniel Campello has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/23021 )

Change subject: layout: Add -i <region>[:<file>] support
......................................................................

Patch Set 26:

(2 comments)

File cli_classic.c:

https://review.coreboot.org/c/flashrom/+/23021/comment/14b9daa6_fd72825e 
PS25, Line 140: static char *get_optional_filename(char *argv[])
              : {
              : 	char *filename = NULL;
              : 
              : 	if (optarg == NULL && argv[optind] != NULL && argv[optind][0] != '-')
              : 		filename = strdup(argv[optind++]);
              : 	else if (optarg != NULL)
              : 		filename = strdup(optarg);
              : 
              : 	return filename;
              : }
> Not a overtly strong feeling here however should this part of the story perhaps be a separate patch? […]

This takes care of the "optional" nature of the -r/-w/-v options. We could separate on another patch by removing the `if (filename)` of read_by_layout, do_write and do_verify. Not sure if it is worth it...

https://review.coreboot.org/c/flashrom/+/23021/comment/e7af1234_f69fe0a6 
PS25, Line 203: 'i'}
> I guess this would be a breaking change, I would defer to Nico and Angels judgement here. […]

it was an oversight on my part... I will move back to image. As it stands we want the meaning to be the same...