|
From: | Philippe Mathieu-Daudé |
Subject: | Re: [PATCH] ppc/pegasos2: Improve readability of VIA south bridge creation |
Date: | Wed, 18 Jan 2023 10:53:33 +0100 |
User-agent: | Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 |
On 17/1/23 22:57, BALATON Zoltan wrote:
On Tue, 17 Jan 2023, BALATON Zoltan wrote:Slightly improve readability of creating the south btidge by cnamgingStill left a typo in "bridge" above...
Also "naming".
This is alternative, inspired by Phil's patches, maybe I'd also need to add Inspired-by: tag.
Not necessary.
type of a local variable to avoid some casts within function arguments which makes some lines shorter and easier to read. Also remove an unneded line break.and "unneeded" here. I wait what Phil says then correct these if needed.
Fine by me, so with the typos fixed: Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Signed-off-by: BALATON Zoltan <balaton@eik.bme.hu> --- hw/ppc/pegasos2.c | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-)
[Prev in Thread] | Current Thread | [Next in Thread] |