drm/msm: remove python 3.9 dependency for compiling msm
authorAbhinav Kumar <quic_abhinavk@quicinc.com>
Tue, 7 May 2024 23:04:40 +0000 (16:04 -0700)
committerThierry Reding <treding@nvidia.com>
Thu, 30 May 2024 16:49:23 +0000 (18:49 +0200)
Since commit 5acf49119630 ("drm/msm: import gen_header.py script from Mesa"),
compilation is broken on machines having python versions older than 3.9
due to dependency on argparse.BooleanOptionalAction.

Switch to use simple bool for the validate flag to remove the dependency.

Fixes: 5acf49119630 ("drm/msm: import gen_header.py script from Mesa")
Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
Tested-by: Douglas Anderson <dianders@chromium.org>
Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
Signed-off-by: Thierry Reding <treding@nvidia.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240507230440.3384949-1-quic_abhinavk@quicinc.com
drivers/gpu/drm/msm/registers/gen_header.py

index fc3bfdc..3926485 100644 (file)
@@ -538,7 +538,7 @@ class Parser(object):
                self.variants.add(reg.domain)
 
        def do_validate(self, schemafile):
-               if self.validate == False:
+               if not self.validate:
                        return
 
                try:
@@ -948,7 +948,8 @@ def main():
        parser = argparse.ArgumentParser()
        parser.add_argument('--rnn', type=str, required=True)
        parser.add_argument('--xml', type=str, required=True)
-       parser.add_argument('--validate', action=argparse.BooleanOptionalAction)
+       parser.add_argument('--validate', default=False, action='store_true')
+       parser.add_argument('--no-validate', dest='validate', action='store_false')
 
        subparsers = parser.add_subparsers()
        subparsers.required = True