Simplify the code by dropping unnecessary .owner initialization in the
driver.
Best regards,
Krzysztof
---
Krzysztof Kozlowski (11):
drm/mediatek: aal: drop driver owner initialization
drm/mediatek: ccorr: drop driver owner initialization
drm/mediatek: color: drop driver owner initialization
drm/mediatek: gamma: drop driver owner initialization
drm/mediatek: merge: drop driver owner initialization
drm/mediatek: ovl: drop driver owner initialization
drm/mediatek: ovl_adaptor: drop driver owner initialization
drm/mediatek: rdma: drop driver owner initialization
drm/mediatek: ethdr: drop driver owner initialization
drm/mediatek: mdp_rdma: drop driver owner initialization
drm/mediatek: padding: drop driver owner initialization
drivers/gpu/drm/mediatek/mtk_disp_aal.c | 1 -
drivers/gpu/drm/mediatek/mtk_disp_ccorr.c | 1 -
drivers/gpu/drm/mediatek/mtk_disp_color.c | 1 -
drivers/gpu/drm/mediatek/mtk_disp_gamma.c | 1 -
drivers/gpu/drm/mediatek/mtk_disp_merge.c | 1 -
drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 1 -
drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c | 1 -
drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 1 -
drivers/gpu/drm/mediatek/mtk_ethdr.c | 1 -
drivers/gpu/drm/mediatek/mtk_mdp_rdma.c | 1 -
drivers/gpu/drm/mediatek/mtk_padding.c | 1 -
11 files changed, 11 deletions(-)
---
base-commit: 7fdcff3312e16ba8d1419f8a18f465c5cc235ecf
change-id: 20240330-b4-module-owner-drm-mediatek-aa525b70f033
Best regards,
--
Krzysztof Kozlowski <[email protected]>
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_disp_aal.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_aal.c b/drivers/gpu/drm/mediatek/mtk_disp_aal.c
index 40fe403086c3..f6f2c24abc93 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_aal.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_aal.c
@@ -223,7 +223,6 @@ struct platform_driver mtk_disp_aal_driver = {
.remove_new = mtk_disp_aal_remove,
.driver = {
.name = "mediatek-disp-aal",
- .owner = THIS_MODULE,
.of_match_table = mtk_disp_aal_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_disp_ccorr.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ccorr.c b/drivers/gpu/drm/mediatek/mtk_disp_ccorr.c
index 465cddce0d32..ff56c20d4e21 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_ccorr.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_ccorr.c
@@ -214,7 +214,6 @@ struct platform_driver mtk_disp_ccorr_driver = {
.remove_new = mtk_disp_ccorr_remove,
.driver = {
.name = "mediatek-disp-ccorr",
- .owner = THIS_MODULE,
.of_match_table = mtk_disp_ccorr_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_disp_color.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_color.c b/drivers/gpu/drm/mediatek/mtk_disp_color.c
index 78ea99f1444f..b3117a1a6535 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_color.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_color.c
@@ -164,7 +164,6 @@ struct platform_driver mtk_disp_color_driver = {
.remove_new = mtk_disp_color_remove,
.driver = {
.name = "mediatek-disp-color",
- .owner = THIS_MODULE,
.of_match_table = mtk_disp_color_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_disp_merge.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_merge.c b/drivers/gpu/drm/mediatek/mtk_disp_merge.c
index 32a29924bd54..a04f76dbef73 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_merge.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_merge.c
@@ -376,7 +376,6 @@ struct platform_driver mtk_disp_merge_driver = {
.remove_new = mtk_disp_merge_remove,
.driver = {
.name = "mediatek-disp-merge",
- .owner = THIS_MODULE,
.of_match_table = mtk_disp_merge_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
index 2bffe4245466..c754dcab716c 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl.c
@@ -659,7 +659,6 @@ struct platform_driver mtk_disp_ovl_driver = {
.remove_new = mtk_disp_ovl_remove,
.driver = {
.name = "mediatek-disp-ovl",
- .owner = THIS_MODULE,
.of_match_table = mtk_disp_ovl_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_disp_gamma.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_gamma.c b/drivers/gpu/drm/mediatek/mtk_disp_gamma.c
index c1bc8b00d938..572f0716a0f1 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_gamma.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_gamma.c
@@ -334,7 +334,6 @@ struct platform_driver mtk_disp_gamma_driver = {
.remove_new = mtk_disp_gamma_remove,
.driver = {
.name = "mediatek-disp-gamma",
- .owner = THIS_MODULE,
.of_match_table = mtk_disp_gamma_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c b/drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c
index 034d31824d4d..72fd5a095960 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c
@@ -629,6 +629,5 @@ struct platform_driver mtk_disp_ovl_adaptor_driver = {
.remove_new = mtk_disp_ovl_adaptor_remove,
.driver = {
.name = "mediatek-disp-ovl-adaptor",
- .owner = THIS_MODULE,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_ethdr.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_ethdr.c b/drivers/gpu/drm/mediatek/mtk_ethdr.c
index 6a5d0c345aab..434ba8b055a4 100644
--- a/drivers/gpu/drm/mediatek/mtk_ethdr.c
+++ b/drivers/gpu/drm/mediatek/mtk_ethdr.c
@@ -363,7 +363,6 @@ struct platform_driver mtk_ethdr_driver = {
.remove_new = mtk_ethdr_remove,
.driver = {
.name = "mediatek-disp-ethdr",
- .owner = THIS_MODULE,
.of_match_table = mtk_ethdr_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_mdp_rdma.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_mdp_rdma.c b/drivers/gpu/drm/mediatek/mtk_mdp_rdma.c
index ee9ce9b6d078..925cbb7471ec 100644
--- a/drivers/gpu/drm/mediatek/mtk_mdp_rdma.c
+++ b/drivers/gpu/drm/mediatek/mtk_mdp_rdma.c
@@ -346,7 +346,6 @@ struct platform_driver mtk_mdp_rdma_driver = {
.remove_new = mtk_mdp_rdma_remove,
.driver = {
.name = "mediatek-mdp-rdma",
- .owner = THIS_MODULE,
.of_match_table = mtk_mdp_rdma_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_padding.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_padding.c b/drivers/gpu/drm/mediatek/mtk_padding.c
index 0d6451c149b6..aed84af5b9ac 100644
--- a/drivers/gpu/drm/mediatek/mtk_padding.c
+++ b/drivers/gpu/drm/mediatek/mtk_padding.c
@@ -154,7 +154,6 @@ struct platform_driver mtk_padding_driver = {
.remove = mtk_padding_remove,
.driver = {
.name = "mediatek-disp-padding",
- .owner = THIS_MODULE,
.of_match_table = mtk_padding_driver_dt_match,
},
};
--
2.34.1
Core in platform_driver_register() already sets the .owner, so driver
does not need to. Whatever is set here will be anyway overwritten by
main driver calling platform_driver_register().
Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/gpu/drm/mediatek/mtk_disp_rdma.c b/drivers/gpu/drm/mediatek/mtk_disp_rdma.c
index faa907f2f443..844f5c2b56b6 100644
--- a/drivers/gpu/drm/mediatek/mtk_disp_rdma.c
+++ b/drivers/gpu/drm/mediatek/mtk_disp_rdma.c
@@ -428,7 +428,6 @@ struct platform_driver mtk_disp_rdma_driver = {
.remove_new = mtk_disp_rdma_remove,
.driver = {
.name = "mediatek-disp-rdma",
- .owner = THIS_MODULE,
.of_match_table = mtk_disp_rdma_driver_dt_match,
},
};
--
2.34.1
Hi, Krzysztof:
On Sat, 2024-03-30 at 21:43 +0100, Krzysztof Kozlowski wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> Core in platform_driver_register() already sets the .owner, so
> driver
> does not need to. Whatever is set here will be anyway overwritten by
> main driver calling platform_driver_register().
For the whole series,
Reviewed-by: CK Hu <[email protected]>
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> drivers/gpu/drm/mediatek/mtk_disp_aal.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/mediatek/mtk_disp_aal.c
> b/drivers/gpu/drm/mediatek/mtk_disp_aal.c
> index 40fe403086c3..f6f2c24abc93 100644
> --- a/drivers/gpu/drm/mediatek/mtk_disp_aal.c
> +++ b/drivers/gpu/drm/mediatek/mtk_disp_aal.c
> @@ -223,7 +223,6 @@ struct platform_driver mtk_disp_aal_driver = {
> .remove_new= mtk_disp_aal_remove,
> .driver= {
> .name= "mediatek-disp-aal",
> -.owner= THIS_MODULE,
> .of_match_table = mtk_disp_aal_driver_dt_match,
> },
> };
>
> --
> 2.34.1
>
Il 30/03/24 21:43, Krzysztof Kozlowski ha scritto:
> Simplify the code by dropping unnecessary .owner initialization in the
> driver.
>
For the entire series:
Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
> Best regards,
> Krzysztof
>
> ---
> Krzysztof Kozlowski (11):
> drm/mediatek: aal: drop driver owner initialization
> drm/mediatek: ccorr: drop driver owner initialization
> drm/mediatek: color: drop driver owner initialization
> drm/mediatek: gamma: drop driver owner initialization
> drm/mediatek: merge: drop driver owner initialization
> drm/mediatek: ovl: drop driver owner initialization
> drm/mediatek: ovl_adaptor: drop driver owner initialization
> drm/mediatek: rdma: drop driver owner initialization
> drm/mediatek: ethdr: drop driver owner initialization
> drm/mediatek: mdp_rdma: drop driver owner initialization
> drm/mediatek: padding: drop driver owner initialization
>
> drivers/gpu/drm/mediatek/mtk_disp_aal.c | 1 -
> drivers/gpu/drm/mediatek/mtk_disp_ccorr.c | 1 -
> drivers/gpu/drm/mediatek/mtk_disp_color.c | 1 -
> drivers/gpu/drm/mediatek/mtk_disp_gamma.c | 1 -
> drivers/gpu/drm/mediatek/mtk_disp_merge.c | 1 -
> drivers/gpu/drm/mediatek/mtk_disp_ovl.c | 1 -
> drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c | 1 -
> drivers/gpu/drm/mediatek/mtk_disp_rdma.c | 1 -
> drivers/gpu/drm/mediatek/mtk_ethdr.c | 1 -
> drivers/gpu/drm/mediatek/mtk_mdp_rdma.c | 1 -
> drivers/gpu/drm/mediatek/mtk_padding.c | 1 -
> 11 files changed, 11 deletions(-)
> ---
> base-commit: 7fdcff3312e16ba8d1419f8a18f465c5cc235ecf
> change-id: 20240330-b4-module-owner-drm-mediatek-aa525b70f033
>
> Best regards,
On 01/04/2024 05:44, CK Hu (胡俊光) wrote:
> Hi, Krzysztof:
>
> On Sat, 2024-03-30 at 21:43 +0100, Krzysztof Kozlowski wrote:
>>
>> External email : Please do not click links or open attachments until
>> you have verified the sender or the content.
>> Core in platform_driver_register() already sets the .owner, so
>> driver
>> does not need to. Whatever is set here will be anyway overwritten by
>> main driver calling platform_driver_register().
>
> For the whole series,
>
> Reviewed-by: CK Hu <[email protected]>
For the tag to work on entire series, you need to provide it to the
cover letter.
Anyway, Philipp,
Everything good here from your side? Will you be able to take everything
to mediatek/drm? I think drm window is slowly closing.
Best regards,
Krzysztof
Hi, Krzysztof:
On Wed, 2024-04-10 at 09:43 +0200, Krzysztof Kozlowski wrote:
>
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
> On 01/04/2024 05:44, CK Hu (胡俊光) wrote:
> > Hi, Krzysztof:
> >
> > On Sat, 2024-03-30 at 21:43 +0100, Krzysztof Kozlowski wrote:
> >>
> >> External email : Please do not click links or open attachments
> until
> >> you have verified the sender or the content.
> >> Core in platform_driver_register() already sets the .owner, so
> >> driver
> >> does not need to. Whatever is set here will be anyway overwritten
> by
> >> main driver calling platform_driver_register().
> >
> > For the whole series,
> >
> > Reviewed-by: CK Hu <[email protected]>
>
> For the tag to work on entire series, you need to provide it to the
> cover letter.
>
> Anyway, Philipp,
>
> Everything good here from your side? Will you be able to take
> everything
> to mediatek/drm? I think drm window is slowly closing.
I've applied to mediatek_drm_next [1], thanks.
[1]
https://git.kernel.org/pub/scm/linux/kernel/git/chunkuang.hu/linux.git/log/?h=mediatek-drm-next
Regards,
CK
>
> Best regards,
> Krzysztof
>