All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/3] drm/radeon: add missing header dependencies
@ 2016-09-30  8:13 Baoyou Xie
  2016-09-30  8:13 ` [PATCH 3/3] drm/radeon: mark symbols static where possible Baoyou Xie
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Baoyou Xie @ 2016-09-30  8:13 UTC (permalink / raw)
  To: alexander.deucher, christian.koenig, airlied
  Cc: dri-devel, linux-kernel, arnd, baoyou.xie, xie.baoyou, han.fei,
	tang.qiang007

We get a few warnings when building kernel with W=1:
drivers/gpu/drm/radeon/radeon_clocks.c:35:10: warning: no previous prototype for 'radeon_legacy_get_engine_clock' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/atombios_encoders.c:75:1: warning: no previous prototype for 'atombios_get_backlight_level' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/r600_cs.c:2268:5: warning: no previous prototype for 'r600_cs_parse' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/evergreen_cs.c:2671:5: warning: no previous prototype for 'evergreen_cs_parse' [-Wmissing-prototypes]
....

In fact, these functions are declared
in drivers/gpu/drm/radeon/radeon_asic.h,
so this patch adds missing header dependencies.

Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
---
 drivers/gpu/drm/radeon/atombios_encoders.c      | 1 +
 drivers/gpu/drm/radeon/evergreen_cs.c           | 1 +
 drivers/gpu/drm/radeon/r600_cs.c                | 1 +
 drivers/gpu/drm/radeon/radeon_atombios.c        | 1 +
 drivers/gpu/drm/radeon/radeon_clocks.c          | 1 +
 drivers/gpu/drm/radeon/radeon_legacy_encoders.c | 1 +
 6 files changed, 6 insertions(+)

diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c b/drivers/gpu/drm/radeon/atombios_encoders.c
index 56bb758..fa4f8f0 100644
--- a/drivers/gpu/drm/radeon/atombios_encoders.c
+++ b/drivers/gpu/drm/radeon/atombios_encoders.c
@@ -28,6 +28,7 @@
 #include <drm/radeon_drm.h>
 #include "radeon.h"
 #include "radeon_audio.h"
+#include "radeon_asic.h"
 #include "atom.h"
 #include <linux/backlight.h>
 
diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c b/drivers/gpu/drm/radeon/evergreen_cs.c
index d960d39..f8b0509 100644
--- a/drivers/gpu/drm/radeon/evergreen_cs.c
+++ b/drivers/gpu/drm/radeon/evergreen_cs.c
@@ -27,6 +27,7 @@
  */
 #include <drm/drmP.h>
 #include "radeon.h"
+#include "radeon_asic.h"
 #include "evergreend.h"
 #include "evergreen_reg_safe.h"
 #include "cayman_reg_safe.h"
diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c
index b69c8de..595a197 100644
--- a/drivers/gpu/drm/radeon/r600_cs.c
+++ b/drivers/gpu/drm/radeon/r600_cs.c
@@ -28,6 +28,7 @@
 #include <linux/kernel.h>
 #include <drm/drmP.h>
 #include "radeon.h"
+#include "radeon_asic.h"
 #include "r600d.h"
 #include "r600_reg_safe.h"
 
diff --git a/drivers/gpu/drm/radeon/radeon_atombios.c b/drivers/gpu/drm/radeon/radeon_atombios.c
index 5df3ec7..4134759 100644
--- a/drivers/gpu/drm/radeon/radeon_atombios.c
+++ b/drivers/gpu/drm/radeon/radeon_atombios.c
@@ -29,6 +29,7 @@
 
 #include "atom.h"
 #include "atom-bits.h"
+#include "radeon_asic.h"
 
 extern void
 radeon_add_atom_encoder(struct drm_device *dev, uint32_t encoder_enum,
diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c b/drivers/gpu/drm/radeon/radeon_clocks.c
index 38e396d..c1135fe 100644
--- a/drivers/gpu/drm/radeon/radeon_clocks.c
+++ b/drivers/gpu/drm/radeon/radeon_clocks.c
@@ -29,6 +29,7 @@
 #include <drm/radeon_drm.h>
 #include "radeon_reg.h"
 #include "radeon.h"
+#include "radeon_asic.h"
 #include "atom.h"
 
 /* 10 khz */
diff --git a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
index 868c3ba..222a1fa 100644
--- a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
+++ b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
@@ -27,6 +27,7 @@
 #include <drm/drm_crtc_helper.h>
 #include <drm/radeon_drm.h>
 #include "radeon.h"
+#include "radeon_asic.h"
 #include "atom.h"
 #include <linux/backlight.h>
 #ifdef CONFIG_PMAC_BACKLIGHT
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* [PATCH 3/3] drm/radeon: mark symbols static where possible
  2016-09-30  8:13 [PATCH 1/3] drm/radeon: add missing header dependencies Baoyou Xie
@ 2016-09-30  8:13 ` Baoyou Xie
  2016-09-30  8:13 ` Baoyou Xie
  2016-09-30 11:52   ` Christian König
  2 siblings, 0 replies; 7+ messages in thread
From: Baoyou Xie @ 2016-09-30  8:13 UTC (permalink / raw)
  To: alexander.deucher, christian.koenig, airlied
  Cc: dri-devel, linux-kernel, arnd, baoyou.xie, xie.baoyou, han.fei,
	tang.qiang007

We get 4 warnings when building kernel with W=1:
drivers/gpu/drm/radeon/si.c:7850:5: warning: no previous prototype for 'si_vce_send_vcepll_ctlreq' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/radeon_dp_mst.c:226:21: warning: no previous prototype for 'radeon_mst_best_encoder' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/radeon_dp_mst.c:344:26: warning: no previous prototype for 'radeon_mst_find_connector' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/radeon_dp_mst.c:600:6: warning: no previous prototype for 'radeon_dp_mst_encoder_destroy' [-Wmissing-prototypes]

In fact, these functions are only used in the file in which they are
declared and don't need a declaration, but can be made static.
So this patch marks these functions with 'static'.

Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
---
 drivers/gpu/drm/radeon/radeon_dp_mst.c | 8 +++++---
 drivers/gpu/drm/radeon/si.c            | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_dp_mst.c b/drivers/gpu/drm/radeon/radeon_dp_mst.c
index de504ea..6d1237d 100644
--- a/drivers/gpu/drm/radeon/radeon_dp_mst.c
+++ b/drivers/gpu/drm/radeon/radeon_dp_mst.c
@@ -223,7 +223,8 @@ radeon_dp_mst_mode_valid(struct drm_connector *connector,
 	return MODE_OK;
 }
 
-struct drm_encoder *radeon_mst_best_encoder(struct drm_connector *connector)
+static struct
+drm_encoder *radeon_mst_best_encoder(struct drm_connector *connector)
 {
 	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
 
@@ -341,7 +342,8 @@ const struct drm_dp_mst_topology_cbs mst_cbs = {
 	.hotplug = radeon_dp_mst_hotplug,
 };
 
-struct radeon_connector *radeon_mst_find_connector(struct drm_encoder *encoder)
+static struct
+radeon_connector *radeon_mst_find_connector(struct drm_encoder *encoder)
 {
 	struct drm_device *dev = encoder->dev;
 	struct drm_connector *connector;
@@ -597,7 +599,7 @@ static const struct drm_encoder_helper_funcs radeon_mst_helper_funcs = {
 	.commit = radeon_mst_encoder_commit,
 };
 
-void radeon_dp_mst_encoder_destroy(struct drm_encoder *encoder)
+static void radeon_dp_mst_encoder_destroy(struct drm_encoder *encoder)
 {
 	drm_encoder_cleanup(encoder);
 	kfree(encoder);
diff --git a/drivers/gpu/drm/radeon/si.c b/drivers/gpu/drm/radeon/si.c
index e5ceeb4..f73e441 100644
--- a/drivers/gpu/drm/radeon/si.c
+++ b/drivers/gpu/drm/radeon/si.c
@@ -7847,7 +7847,7 @@ static void si_program_aspm(struct radeon_device *rdev)
 	}
 }
 
-int si_vce_send_vcepll_ctlreq(struct radeon_device *rdev)
+static int si_vce_send_vcepll_ctlreq(struct radeon_device *rdev)
 {
 	unsigned i;
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* [PATCH 3/3] drm/radeon: mark symbols static where possible
  2016-09-30  8:13 [PATCH 1/3] drm/radeon: add missing header dependencies Baoyou Xie
  2016-09-30  8:13 ` [PATCH 3/3] drm/radeon: mark symbols static where possible Baoyou Xie
@ 2016-09-30  8:13 ` Baoyou Xie
  2016-09-30 11:52   ` Christian König
  2 siblings, 0 replies; 7+ messages in thread
From: Baoyou Xie @ 2016-09-30  8:13 UTC (permalink / raw)
  To: alexander.deucher, christian.koenig, airlied
  Cc: dri-devel, linux-kernel, arnd, baoyou.xie, xie.baoyou, han.fei,
	tang.qiang007

We get 4 warnings when building kernel with W=1:
drivers/gpu/drm/radeon/si.c:7850:5: warning: no previous prototype for 'si_vce_send_vcepll_ctlreq' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/radeon_dp_mst.c:226:21: warning: no previous prototype for 'radeon_mst_best_encoder' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/radeon_dp_mst.c:344:26: warning: no previous prototype for 'radeon_mst_find_connector' [-Wmissing-prototypes]
drivers/gpu/drm/radeon/radeon_dp_mst.c:600:6: warning: no previous prototype for 'radeon_dp_mst_encoder_destroy' [-Wmissing-prototypes]

In fact, these functions are only used in the file in which they are
declared and don't need a declaration, but can be made static.
So this patch marks these functions with 'static'.

Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
---
 drivers/gpu/drm/radeon/radeon_dp_mst.c | 8 +++++---
 drivers/gpu/drm/radeon/si.c            | 2 +-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_dp_mst.c b/drivers/gpu/drm/radeon/radeon_dp_mst.c
index de504ea..6d1237d 100644
--- a/drivers/gpu/drm/radeon/radeon_dp_mst.c
+++ b/drivers/gpu/drm/radeon/radeon_dp_mst.c
@@ -223,7 +223,8 @@ radeon_dp_mst_mode_valid(struct drm_connector *connector,
 	return MODE_OK;
 }
 
-struct drm_encoder *radeon_mst_best_encoder(struct drm_connector *connector)
+static struct
+drm_encoder *radeon_mst_best_encoder(struct drm_connector *connector)
 {
 	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
 
@@ -341,7 +342,8 @@ const struct drm_dp_mst_topology_cbs mst_cbs = {
 	.hotplug = radeon_dp_mst_hotplug,
 };
 
-struct radeon_connector *radeon_mst_find_connector(struct drm_encoder *encoder)
+static struct
+radeon_connector *radeon_mst_find_connector(struct drm_encoder *encoder)
 {
 	struct drm_device *dev = encoder->dev;
 	struct drm_connector *connector;
@@ -597,7 +599,7 @@ static const struct drm_encoder_helper_funcs radeon_mst_helper_funcs = {
 	.commit = radeon_mst_encoder_commit,
 };
 
-void radeon_dp_mst_encoder_destroy(struct drm_encoder *encoder)
+static void radeon_dp_mst_encoder_destroy(struct drm_encoder *encoder)
 {
 	drm_encoder_cleanup(encoder);
 	kfree(encoder);
diff --git a/drivers/gpu/drm/radeon/si.c b/drivers/gpu/drm/radeon/si.c
index e5ceeb4..f73e441 100644
--- a/drivers/gpu/drm/radeon/si.c
+++ b/drivers/gpu/drm/radeon/si.c
@@ -7847,7 +7847,7 @@ static void si_program_aspm(struct radeon_device *rdev)
 	}
 }
 
-int si_vce_send_vcepll_ctlreq(struct radeon_device *rdev)
+static int si_vce_send_vcepll_ctlreq(struct radeon_device *rdev)
 {
 	unsigned i;
 
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] drm/radeon: add missing header dependencies
  2016-09-30  8:13 [PATCH 1/3] drm/radeon: add missing header dependencies Baoyou Xie
@ 2016-09-30 11:52   ` Christian König
  2016-09-30  8:13 ` Baoyou Xie
  2016-09-30 11:52   ` Christian König
  2 siblings, 0 replies; 7+ messages in thread
From: Christian König @ 2016-09-30 11:52 UTC (permalink / raw)
  To: Baoyou Xie, alexander.deucher, airlied
  Cc: dri-devel, linux-kernel, arnd, xie.baoyou, han.fei, tang.qiang007

This one and patch #3 are Reviewed-by: Christian König 
<christian.koenig@amd.com>.

Where is patch #2? That never made it into my inbox.

Regards,
Christian.

Am 30.09.2016 um 10:13 schrieb Baoyou Xie:
> We get a few warnings when building kernel with W=1:
> drivers/gpu/drm/radeon/radeon_clocks.c:35:10: warning: no previous prototype for 'radeon_legacy_get_engine_clock' [-Wmissing-prototypes]
> drivers/gpu/drm/radeon/atombios_encoders.c:75:1: warning: no previous prototype for 'atombios_get_backlight_level' [-Wmissing-prototypes]
> drivers/gpu/drm/radeon/r600_cs.c:2268:5: warning: no previous prototype for 'r600_cs_parse' [-Wmissing-prototypes]
> drivers/gpu/drm/radeon/evergreen_cs.c:2671:5: warning: no previous prototype for 'evergreen_cs_parse' [-Wmissing-prototypes]
> ....
>
> In fact, these functions are declared
> in drivers/gpu/drm/radeon/radeon_asic.h,
> so this patch adds missing header dependencies.
>
> Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
> ---
>   drivers/gpu/drm/radeon/atombios_encoders.c      | 1 +
>   drivers/gpu/drm/radeon/evergreen_cs.c           | 1 +
>   drivers/gpu/drm/radeon/r600_cs.c                | 1 +
>   drivers/gpu/drm/radeon/radeon_atombios.c        | 1 +
>   drivers/gpu/drm/radeon/radeon_clocks.c          | 1 +
>   drivers/gpu/drm/radeon/radeon_legacy_encoders.c | 1 +
>   6 files changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c b/drivers/gpu/drm/radeon/atombios_encoders.c
> index 56bb758..fa4f8f0 100644
> --- a/drivers/gpu/drm/radeon/atombios_encoders.c
> +++ b/drivers/gpu/drm/radeon/atombios_encoders.c
> @@ -28,6 +28,7 @@
>   #include <drm/radeon_drm.h>
>   #include "radeon.h"
>   #include "radeon_audio.h"
> +#include "radeon_asic.h"
>   #include "atom.h"
>   #include <linux/backlight.h>
>   
> diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c b/drivers/gpu/drm/radeon/evergreen_cs.c
> index d960d39..f8b0509 100644
> --- a/drivers/gpu/drm/radeon/evergreen_cs.c
> +++ b/drivers/gpu/drm/radeon/evergreen_cs.c
> @@ -27,6 +27,7 @@
>    */
>   #include <drm/drmP.h>
>   #include "radeon.h"
> +#include "radeon_asic.h"
>   #include "evergreend.h"
>   #include "evergreen_reg_safe.h"
>   #include "cayman_reg_safe.h"
> diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c
> index b69c8de..595a197 100644
> --- a/drivers/gpu/drm/radeon/r600_cs.c
> +++ b/drivers/gpu/drm/radeon/r600_cs.c
> @@ -28,6 +28,7 @@
>   #include <linux/kernel.h>
>   #include <drm/drmP.h>
>   #include "radeon.h"
> +#include "radeon_asic.h"
>   #include "r600d.h"
>   #include "r600_reg_safe.h"
>   
> diff --git a/drivers/gpu/drm/radeon/radeon_atombios.c b/drivers/gpu/drm/radeon/radeon_atombios.c
> index 5df3ec7..4134759 100644
> --- a/drivers/gpu/drm/radeon/radeon_atombios.c
> +++ b/drivers/gpu/drm/radeon/radeon_atombios.c
> @@ -29,6 +29,7 @@
>   
>   #include "atom.h"
>   #include "atom-bits.h"
> +#include "radeon_asic.h"
>   
>   extern void
>   radeon_add_atom_encoder(struct drm_device *dev, uint32_t encoder_enum,
> diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c b/drivers/gpu/drm/radeon/radeon_clocks.c
> index 38e396d..c1135fe 100644
> --- a/drivers/gpu/drm/radeon/radeon_clocks.c
> +++ b/drivers/gpu/drm/radeon/radeon_clocks.c
> @@ -29,6 +29,7 @@
>   #include <drm/radeon_drm.h>
>   #include "radeon_reg.h"
>   #include "radeon.h"
> +#include "radeon_asic.h"
>   #include "atom.h"
>   
>   /* 10 khz */
> diff --git a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
> index 868c3ba..222a1fa 100644
> --- a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
> +++ b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
> @@ -27,6 +27,7 @@
>   #include <drm/drm_crtc_helper.h>
>   #include <drm/radeon_drm.h>
>   #include "radeon.h"
> +#include "radeon_asic.h"
>   #include "atom.h"
>   #include <linux/backlight.h>
>   #ifdef CONFIG_PMAC_BACKLIGHT

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] drm/radeon: add missing header dependencies
@ 2016-09-30 11:52   ` Christian König
  0 siblings, 0 replies; 7+ messages in thread
From: Christian König @ 2016-09-30 11:52 UTC (permalink / raw)
  To: Baoyou Xie, alexander.deucher, airlied
  Cc: dri-devel, linux-kernel, arnd, xie.baoyou, han.fei, tang.qiang007

This one and patch #3 are Reviewed-by: Christian König 
<christian.koenig@amd.com>.

Where is patch #2? That never made it into my inbox.

Regards,
Christian.

Am 30.09.2016 um 10:13 schrieb Baoyou Xie:
> We get a few warnings when building kernel with W=1:
> drivers/gpu/drm/radeon/radeon_clocks.c:35:10: warning: no previous prototype for 'radeon_legacy_get_engine_clock' [-Wmissing-prototypes]
> drivers/gpu/drm/radeon/atombios_encoders.c:75:1: warning: no previous prototype for 'atombios_get_backlight_level' [-Wmissing-prototypes]
> drivers/gpu/drm/radeon/r600_cs.c:2268:5: warning: no previous prototype for 'r600_cs_parse' [-Wmissing-prototypes]
> drivers/gpu/drm/radeon/evergreen_cs.c:2671:5: warning: no previous prototype for 'evergreen_cs_parse' [-Wmissing-prototypes]
> ....
>
> In fact, these functions are declared
> in drivers/gpu/drm/radeon/radeon_asic.h,
> so this patch adds missing header dependencies.
>
> Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
> ---
>   drivers/gpu/drm/radeon/atombios_encoders.c      | 1 +
>   drivers/gpu/drm/radeon/evergreen_cs.c           | 1 +
>   drivers/gpu/drm/radeon/r600_cs.c                | 1 +
>   drivers/gpu/drm/radeon/radeon_atombios.c        | 1 +
>   drivers/gpu/drm/radeon/radeon_clocks.c          | 1 +
>   drivers/gpu/drm/radeon/radeon_legacy_encoders.c | 1 +
>   6 files changed, 6 insertions(+)
>
> diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c b/drivers/gpu/drm/radeon/atombios_encoders.c
> index 56bb758..fa4f8f0 100644
> --- a/drivers/gpu/drm/radeon/atombios_encoders.c
> +++ b/drivers/gpu/drm/radeon/atombios_encoders.c
> @@ -28,6 +28,7 @@
>   #include <drm/radeon_drm.h>
>   #include "radeon.h"
>   #include "radeon_audio.h"
> +#include "radeon_asic.h"
>   #include "atom.h"
>   #include <linux/backlight.h>
>   
> diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c b/drivers/gpu/drm/radeon/evergreen_cs.c
> index d960d39..f8b0509 100644
> --- a/drivers/gpu/drm/radeon/evergreen_cs.c
> +++ b/drivers/gpu/drm/radeon/evergreen_cs.c
> @@ -27,6 +27,7 @@
>    */
>   #include <drm/drmP.h>
>   #include "radeon.h"
> +#include "radeon_asic.h"
>   #include "evergreend.h"
>   #include "evergreen_reg_safe.h"
>   #include "cayman_reg_safe.h"
> diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c
> index b69c8de..595a197 100644
> --- a/drivers/gpu/drm/radeon/r600_cs.c
> +++ b/drivers/gpu/drm/radeon/r600_cs.c
> @@ -28,6 +28,7 @@
>   #include <linux/kernel.h>
>   #include <drm/drmP.h>
>   #include "radeon.h"
> +#include "radeon_asic.h"
>   #include "r600d.h"
>   #include "r600_reg_safe.h"
>   
> diff --git a/drivers/gpu/drm/radeon/radeon_atombios.c b/drivers/gpu/drm/radeon/radeon_atombios.c
> index 5df3ec7..4134759 100644
> --- a/drivers/gpu/drm/radeon/radeon_atombios.c
> +++ b/drivers/gpu/drm/radeon/radeon_atombios.c
> @@ -29,6 +29,7 @@
>   
>   #include "atom.h"
>   #include "atom-bits.h"
> +#include "radeon_asic.h"
>   
>   extern void
>   radeon_add_atom_encoder(struct drm_device *dev, uint32_t encoder_enum,
> diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c b/drivers/gpu/drm/radeon/radeon_clocks.c
> index 38e396d..c1135fe 100644
> --- a/drivers/gpu/drm/radeon/radeon_clocks.c
> +++ b/drivers/gpu/drm/radeon/radeon_clocks.c
> @@ -29,6 +29,7 @@
>   #include <drm/radeon_drm.h>
>   #include "radeon_reg.h"
>   #include "radeon.h"
> +#include "radeon_asic.h"
>   #include "atom.h"
>   
>   /* 10 khz */
> diff --git a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
> index 868c3ba..222a1fa 100644
> --- a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
> +++ b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
> @@ -27,6 +27,7 @@
>   #include <drm/drm_crtc_helper.h>
>   #include <drm/radeon_drm.h>
>   #include "radeon.h"
> +#include "radeon_asic.h"
>   #include "atom.h"
>   #include <linux/backlight.h>
>   #ifdef CONFIG_PMAC_BACKLIGHT

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] drm/radeon: add missing header dependencies
  2016-09-30 11:52   ` Christian König
@ 2016-09-30 17:26     ` Alex Deucher
  -1 siblings, 0 replies; 7+ messages in thread
From: Alex Deucher @ 2016-09-30 17:26 UTC (permalink / raw)
  To: Christian König
  Cc: Baoyou Xie, Deucher, Alexander, Dave Airlie, Arnd Bergmann,
	tang.qiang007, xie.baoyou, LKML, Maling list - DRI developers,
	han.fei

On Fri, Sep 30, 2016 at 7:52 AM, Christian König
<christian.koenig@amd.com> wrote:
> This one and patch #3 are Reviewed-by: Christian König
> <christian.koenig@amd.com>.
>

Applied 1 and 3, thanks!

Alex

> Where is patch #2? That never made it into my inbox.
>
> Regards,
> Christian.
>
>
> Am 30.09.2016 um 10:13 schrieb Baoyou Xie:
>>
>> We get a few warnings when building kernel with W=1:
>> drivers/gpu/drm/radeon/radeon_clocks.c:35:10: warning: no previous
>> prototype for 'radeon_legacy_get_engine_clock' [-Wmissing-prototypes]
>> drivers/gpu/drm/radeon/atombios_encoders.c:75:1: warning: no previous
>> prototype for 'atombios_get_backlight_level' [-Wmissing-prototypes]
>> drivers/gpu/drm/radeon/r600_cs.c:2268:5: warning: no previous prototype
>> for 'r600_cs_parse' [-Wmissing-prototypes]
>> drivers/gpu/drm/radeon/evergreen_cs.c:2671:5: warning: no previous
>> prototype for 'evergreen_cs_parse' [-Wmissing-prototypes]
>> ....
>>
>> In fact, these functions are declared
>> in drivers/gpu/drm/radeon/radeon_asic.h,
>> so this patch adds missing header dependencies.
>>
>> Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
>> ---
>>   drivers/gpu/drm/radeon/atombios_encoders.c      | 1 +
>>   drivers/gpu/drm/radeon/evergreen_cs.c           | 1 +
>>   drivers/gpu/drm/radeon/r600_cs.c                | 1 +
>>   drivers/gpu/drm/radeon/radeon_atombios.c        | 1 +
>>   drivers/gpu/drm/radeon/radeon_clocks.c          | 1 +
>>   drivers/gpu/drm/radeon/radeon_legacy_encoders.c | 1 +
>>   6 files changed, 6 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c
>> b/drivers/gpu/drm/radeon/atombios_encoders.c
>> index 56bb758..fa4f8f0 100644
>> --- a/drivers/gpu/drm/radeon/atombios_encoders.c
>> +++ b/drivers/gpu/drm/radeon/atombios_encoders.c
>> @@ -28,6 +28,7 @@
>>   #include <drm/radeon_drm.h>
>>   #include "radeon.h"
>>   #include "radeon_audio.h"
>> +#include "radeon_asic.h"
>>   #include "atom.h"
>>   #include <linux/backlight.h>
>>   diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c
>> b/drivers/gpu/drm/radeon/evergreen_cs.c
>> index d960d39..f8b0509 100644
>> --- a/drivers/gpu/drm/radeon/evergreen_cs.c
>> +++ b/drivers/gpu/drm/radeon/evergreen_cs.c
>> @@ -27,6 +27,7 @@
>>    */
>>   #include <drm/drmP.h>
>>   #include "radeon.h"
>> +#include "radeon_asic.h"
>>   #include "evergreend.h"
>>   #include "evergreen_reg_safe.h"
>>   #include "cayman_reg_safe.h"
>> diff --git a/drivers/gpu/drm/radeon/r600_cs.c
>> b/drivers/gpu/drm/radeon/r600_cs.c
>> index b69c8de..595a197 100644
>> --- a/drivers/gpu/drm/radeon/r600_cs.c
>> +++ b/drivers/gpu/drm/radeon/r600_cs.c
>> @@ -28,6 +28,7 @@
>>   #include <linux/kernel.h>
>>   #include <drm/drmP.h>
>>   #include "radeon.h"
>> +#include "radeon_asic.h"
>>   #include "r600d.h"
>>   #include "r600_reg_safe.h"
>>   diff --git a/drivers/gpu/drm/radeon/radeon_atombios.c
>> b/drivers/gpu/drm/radeon/radeon_atombios.c
>> index 5df3ec7..4134759 100644
>> --- a/drivers/gpu/drm/radeon/radeon_atombios.c
>> +++ b/drivers/gpu/drm/radeon/radeon_atombios.c
>> @@ -29,6 +29,7 @@
>>     #include "atom.h"
>>   #include "atom-bits.h"
>> +#include "radeon_asic.h"
>>     extern void
>>   radeon_add_atom_encoder(struct drm_device *dev, uint32_t encoder_enum,
>> diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c
>> b/drivers/gpu/drm/radeon/radeon_clocks.c
>> index 38e396d..c1135fe 100644
>> --- a/drivers/gpu/drm/radeon/radeon_clocks.c
>> +++ b/drivers/gpu/drm/radeon/radeon_clocks.c
>> @@ -29,6 +29,7 @@
>>   #include <drm/radeon_drm.h>
>>   #include "radeon_reg.h"
>>   #include "radeon.h"
>> +#include "radeon_asic.h"
>>   #include "atom.h"
>>     /* 10 khz */
>> diff --git a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
>> b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
>> index 868c3ba..222a1fa 100644
>> --- a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
>> +++ b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
>> @@ -27,6 +27,7 @@
>>   #include <drm/drm_crtc_helper.h>
>>   #include <drm/radeon_drm.h>
>>   #include "radeon.h"
>> +#include "radeon_asic.h"
>>   #include "atom.h"
>>   #include <linux/backlight.h>
>>   #ifdef CONFIG_PMAC_BACKLIGHT
>
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread
* Re: [PATCH 1/3] drm/radeon: add missing header dependencies
@ 2016-09-30 17:26     ` Alex Deucher
  0 siblings, 0 replies; 7+ messages in thread
From: Alex Deucher @ 2016-09-30 17:26 UTC (permalink / raw)
  To: Christian König
  Cc: Arnd Bergmann, tang.qiang007, xie.baoyou, LKML,
	Maling list - DRI developers, han.fei, Deucher, Alexander,
	Baoyou Xie

On Fri, Sep 30, 2016 at 7:52 AM, Christian König
<christian.koenig@amd.com> wrote:
> This one and patch #3 are Reviewed-by: Christian König
> <christian.koenig@amd.com>.
>

Applied 1 and 3, thanks!

Alex

> Where is patch #2? That never made it into my inbox.
>
> Regards,
> Christian.
>
>
> Am 30.09.2016 um 10:13 schrieb Baoyou Xie:
>>
>> We get a few warnings when building kernel with W=1:
>> drivers/gpu/drm/radeon/radeon_clocks.c:35:10: warning: no previous
>> prototype for 'radeon_legacy_get_engine_clock' [-Wmissing-prototypes]
>> drivers/gpu/drm/radeon/atombios_encoders.c:75:1: warning: no previous
>> prototype for 'atombios_get_backlight_level' [-Wmissing-prototypes]
>> drivers/gpu/drm/radeon/r600_cs.c:2268:5: warning: no previous prototype
>> for 'r600_cs_parse' [-Wmissing-prototypes]
>> drivers/gpu/drm/radeon/evergreen_cs.c:2671:5: warning: no previous
>> prototype for 'evergreen_cs_parse' [-Wmissing-prototypes]
>> ....
>>
>> In fact, these functions are declared
>> in drivers/gpu/drm/radeon/radeon_asic.h,
>> so this patch adds missing header dependencies.
>>
>> Signed-off-by: Baoyou Xie <baoyou.xie@linaro.org>
>> ---
>>   drivers/gpu/drm/radeon/atombios_encoders.c      | 1 +
>>   drivers/gpu/drm/radeon/evergreen_cs.c           | 1 +
>>   drivers/gpu/drm/radeon/r600_cs.c                | 1 +
>>   drivers/gpu/drm/radeon/radeon_atombios.c        | 1 +
>>   drivers/gpu/drm/radeon/radeon_clocks.c          | 1 +
>>   drivers/gpu/drm/radeon/radeon_legacy_encoders.c | 1 +
>>   6 files changed, 6 insertions(+)
>>
>> diff --git a/drivers/gpu/drm/radeon/atombios_encoders.c
>> b/drivers/gpu/drm/radeon/atombios_encoders.c
>> index 56bb758..fa4f8f0 100644
>> --- a/drivers/gpu/drm/radeon/atombios_encoders.c
>> +++ b/drivers/gpu/drm/radeon/atombios_encoders.c
>> @@ -28,6 +28,7 @@
>>   #include <drm/radeon_drm.h>
>>   #include "radeon.h"
>>   #include "radeon_audio.h"
>> +#include "radeon_asic.h"
>>   #include "atom.h"
>>   #include <linux/backlight.h>
>>   diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c
>> b/drivers/gpu/drm/radeon/evergreen_cs.c
>> index d960d39..f8b0509 100644
>> --- a/drivers/gpu/drm/radeon/evergreen_cs.c
>> +++ b/drivers/gpu/drm/radeon/evergreen_cs.c
>> @@ -27,6 +27,7 @@
>>    */
>>   #include <drm/drmP.h>
>>   #include "radeon.h"
>> +#include "radeon_asic.h"
>>   #include "evergreend.h"
>>   #include "evergreen_reg_safe.h"
>>   #include "cayman_reg_safe.h"
>> diff --git a/drivers/gpu/drm/radeon/r600_cs.c
>> b/drivers/gpu/drm/radeon/r600_cs.c
>> index b69c8de..595a197 100644
>> --- a/drivers/gpu/drm/radeon/r600_cs.c
>> +++ b/drivers/gpu/drm/radeon/r600_cs.c
>> @@ -28,6 +28,7 @@
>>   #include <linux/kernel.h>
>>   #include <drm/drmP.h>
>>   #include "radeon.h"
>> +#include "radeon_asic.h"
>>   #include "r600d.h"
>>   #include "r600_reg_safe.h"
>>   diff --git a/drivers/gpu/drm/radeon/radeon_atombios.c
>> b/drivers/gpu/drm/radeon/radeon_atombios.c
>> index 5df3ec7..4134759 100644
>> --- a/drivers/gpu/drm/radeon/radeon_atombios.c
>> +++ b/drivers/gpu/drm/radeon/radeon_atombios.c
>> @@ -29,6 +29,7 @@
>>     #include "atom.h"
>>   #include "atom-bits.h"
>> +#include "radeon_asic.h"
>>     extern void
>>   radeon_add_atom_encoder(struct drm_device *dev, uint32_t encoder_enum,
>> diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c
>> b/drivers/gpu/drm/radeon/radeon_clocks.c
>> index 38e396d..c1135fe 100644
>> --- a/drivers/gpu/drm/radeon/radeon_clocks.c
>> +++ b/drivers/gpu/drm/radeon/radeon_clocks.c
>> @@ -29,6 +29,7 @@
>>   #include <drm/radeon_drm.h>
>>   #include "radeon_reg.h"
>>   #include "radeon.h"
>> +#include "radeon_asic.h"
>>   #include "atom.h"
>>     /* 10 khz */
>> diff --git a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
>> b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
>> index 868c3ba..222a1fa 100644
>> --- a/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
>> +++ b/drivers/gpu/drm/radeon/radeon_legacy_encoders.c
>> @@ -27,6 +27,7 @@
>>   #include <drm/drm_crtc_helper.h>
>>   #include <drm/radeon_drm.h>
>>   #include "radeon.h"
>> +#include "radeon_asic.h"
>>   #include "atom.h"
>>   #include <linux/backlight.h>
>>   #ifdef CONFIG_PMAC_BACKLIGHT
>
>
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-09-30 17:26 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-09-30  8:13 [PATCH 1/3] drm/radeon: add missing header dependencies Baoyou Xie
2016-09-30  8:13 ` [PATCH 3/3] drm/radeon: mark symbols static where possible Baoyou Xie
2016-09-30  8:13 ` Baoyou Xie
2016-09-30 11:52 ` [PATCH 1/3] drm/radeon: add missing header dependencies Christian König
2016-09-30 11:52   ` Christian König
2016-09-30 17:26   ` Alex Deucher
2016-09-30 17:26     ` Alex Deucher

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.