On 14:48 Wed 24 Mar 2021, Randy Dunlap wrote:

On 3/24/21 6:50 AM, Bhaskar Chowdhury wrote:

...

s/miror/mirror/ s/needind/needing/

Signed-off-by: Bhaskar Chowdhury unixbhaskar@gmail.com

drivers/gpu/drm/radeon/r600_cs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c index 34b7c6f16479..aded1f2264e0 100644 --- a/drivers/gpu/drm/radeon/r600_cs.c +++ b/drivers/gpu/drm/radeon/r600_cs.c @@ -38,7 +38,7 @@ extern void r600_cs_legacy_get_tiling_conf(struct drm_device *dev, u32 *npipes,

struct r600_cs_track {

• /* configuration we miror so that we use same code btw kms/ums */

• /* configuration we mirror so that we use same code btw kms/ums */ u32 group_size; u32 nbanks; u32 npipes;

@@ -963,7 +963,7 @@ static int r600_cs_parse_packet0(struct radeon_cs_parser *p,

• This function will test against r600_reg_safe_bm and return 0
• if register is safe. If register is not flag as safe this function

• • will test it against a list of register needind special handling.

• • will test it against a list of register needing special handling.
  */

static int r600_cs_check_reg(struct radeon_cs_parser *p, u32 reg, u32 idx) { --

Those 2 LGTM, but please fix this while you are touching this file:

drivers/gpu/drm/radeon/r600_cs.c:2339: informations ==> information

Thanks, I have sent a V2 with the change...

thanks.

~Randy