On 22 April 2016 at 12:42, Sylvain Garrigues <sylv...@sylvaingarrigues.com> wrote: > As the framebuffer settings are copied into the result message before it is > reconfigured, inconsistent behavior can happen when, for instance, you set > with > a single message the width, height, and depth, and ask at the same time to > allocate the buffer and get the pitch and the size. > > In this case, the reported pitch and size would be incorrect as they were > computed with the initial values of width, height and depth, not the ones the > client requested. > > Signed-off-by: Sylvain Garrigues <sylv...@sylvaingarrigues.com> > Reviewed-by: Andrew Baumann <andrew.baum...@microsoft.com> > --- > hw/misc/bcm2835_property.c | 31 ++++++++++++++++++++++--------- > 1 file changed, 22 insertions(+), 9 deletions(-)
Applied to target-arm.next, thanks. (I folded a couple of long lines that checkpatch.pl complains about.) -- PMM
