2024-05-11 22:33:01

by Dmitry Baryshkov

[permalink] [raw]
Subject: [PATCH v2] docs: document python version used for compilation

The drm/msm driver had adopted using Python3 script to generate register
header files instead of shipping pre-generated header files. Document
the minimal Python version supported by the script. Per request by Jon
Hunter, the script is required to be compatible with Python 3.5.

Python is documented as an optional dependency, as it is required only
in a limited set of kernel configurations (following the example of
other optional dependencies).

Cc: Jon Hunter <[email protected]>
Signed-off-by: Dmitry Baryshkov <[email protected]>
---
Depends: https://lore.kernel.org/dri-devel/[email protected]/
---
Changes in v2:
- Expanded documentation for the Python usage.
- Link to v1: https://lore.kernel.org/r/[email protected]
---
Documentation/process/changes.rst | 8 ++++++++
1 file changed, 8 insertions(+)

diff --git a/Documentation/process/changes.rst b/Documentation/process/changes.rst
index 5685d7bfe4d0..30f17e3f954f 100644
--- a/Documentation/process/changes.rst
+++ b/Documentation/process/changes.rst
@@ -63,6 +63,7 @@ cpio any cpio --version
GNU tar 1.28 tar --version
gtags (optional) 6.6.5 gtags --version
mkimage (optional) 2017.01 mkimage --version
+Python (optional) 3.5.x python3 --version
====================== =============== ========================================

.. [#f1] Sphinx is needed only to build the Kernel documentation
@@ -198,6 +199,13 @@ platforms. The tool is available via the ``u-boot-tools`` package or can be
built from the U-Boot source code. See the instructions at
https://docs.u-boot.org/en/latest/build/tools.html#building-tools-for-linux

+Python
+------
+
+Python is required to generate register headers for the drm/msm driver or to
+build the linker script if the kernel is built using Clang compiler if LTO
+(Link Time Optimisation) is enabled.
+
System utilities
****************


---
base-commit: e8bd80985f8ed4c04e36209228163fcf06496763
change-id: 20240509-python-version-a8b6ca2125ff

Best regards,
--
Dmitry Baryshkov <[email protected]>



2024-05-11 23:53:02

by Abhinav Kumar

[permalink] [raw]
Subject: Re: [PATCH v2] docs: document python version used for compilation



On 5/11/2024 3:32 PM, Dmitry Baryshkov wrote:
> The drm/msm driver had adopted using Python3 script to generate register
> header files instead of shipping pre-generated header files. Document
> the minimal Python version supported by the script. Per request by Jon
> Hunter, the script is required to be compatible with Python 3.5.
>
> Python is documented as an optional dependency, as it is required only
> in a limited set of kernel configurations (following the example of
> other optional dependencies).
>
> Cc: Jon Hunter <[email protected]>
> Signed-off-by: Dmitry Baryshkov <[email protected]>
> ---
> Depends: https://lore.kernel.org/dri-devel/[email protected]/
> ---
> Changes in v2:
> - Expanded documentation for the Python usage.
> - Link to v1: https://lore.kernel.org/r/[email protected]
> ---
> Documentation/process/changes.rst | 8 ++++++++
> 1 file changed, 8 insertions(+)
>


Reviewed-by: Abhinav Kumar <[email protected]>

2024-05-13 08:50:27

by Jon Hunter

[permalink] [raw]
Subject: Re: [PATCH v2] docs: document python version used for compilation


On 12/05/2024 00:52, Abhinav Kumar wrote:
>
>
> On 5/11/2024 3:32 PM, Dmitry Baryshkov wrote:
>> The drm/msm driver had adopted using Python3 script to generate register
>> header files instead of shipping pre-generated header files. Document
>> the minimal Python version supported by the script. Per request by Jon
>> Hunter, the script is required to be compatible with Python 3.5.
>>
>> Python is documented as an optional dependency, as it is required only
>> in a limited set of kernel configurations (following the example of
>> other optional dependencies).
>>
>> Cc: Jon Hunter <[email protected]>
>> Signed-off-by: Dmitry Baryshkov <[email protected]>
>> ---
>> Depends:
>> https://lore.kernel.org/dri-devel/[email protected]/
>> ---
>> Changes in v2:
>> - Expanded documentation for the Python usage.
>> - Link to v1:
>> https://lore.kernel.org/r/[email protected]
>> ---
>>   Documentation/process/changes.rst | 8 ++++++++
>>   1 file changed, 8 insertions(+)
>>
>
>
> Reviewed-by: Abhinav Kumar <[email protected]>


Thanks! I made a request to update some of our legacy builders and so
hopefully they might be upgraded to a newer version. Anyway ...

Reviewed-by: Jon Hunter <[email protected]>

Cheers
Jon

--
nvpublic