diff mbox series

[2/2] drm_print: Remove deprecated DRM_DEBUG_KMS_RATELIMITED()

Message ID 20230117180417.21066-2-nirmoy.das@intel.com (mailing list archive)
State New, archived
Headers show
Series [v2] drm/radeon: Do not use deprecated drm log API | expand

Commit Message

Nirmoy Das Jan. 17, 2023, 6:04 p.m. UTC 
There are no current users of DRM_DEBUG_KMS_RATELIMITED()
so remove it.

Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Maxime Ripard <mripard@kernel.org>
Cc: Thomas Zimmermann <tzimmermann@suse.de>
Cc: David Airlie <airlied@gmail.com>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: Sam Ravnborg <sam@ravnborg.org>

Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
---
 include/drm/drm_print.h | 3 ---
 1 file changed, 3 deletions(-)

Comments

Alex Deucher Jan. 17, 2023, 6:06 p.m. UTC | #1 
On Tue, Jan 17, 2023 at 1:05 PM Nirmoy Das <nirmoy.das@intel.com> wrote:
>
> There are no current users of DRM_DEBUG_KMS_RATELIMITED()
> so remove it.
>
> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
> Cc: Maxime Ripard <mripard@kernel.org>
> Cc: Thomas Zimmermann <tzimmermann@suse.de>
> Cc: David Airlie <airlied@gmail.com>
> Cc: Daniel Vetter <daniel@ffwll.ch>
> Cc: Sam Ravnborg <sam@ravnborg.org>
>
> Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
> Reviewed-by: Sam Ravnborg <sam@ravnborg.org>

Series is:
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>

Feel free to take the patches through whatever tree you want.

Alex

> ---
>  include/drm/drm_print.h | 3 ---
>  1 file changed, 3 deletions(-)
>
> diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
> index a44fb7ef257f..c3753da97c4e 100644
> --- a/include/drm/drm_print.h
> +++ b/include/drm/drm_print.h
> @@ -605,9 +605,6 @@ void __drm_err(const char *format, ...);
>  #define drm_dbg_kms_ratelimited(drm, fmt, ...) \
>         __DRM_DEFINE_DBG_RATELIMITED(KMS, drm, fmt, ## __VA_ARGS__)
>
> -/* NOTE: this is deprecated in favor of drm_dbg_kms_ratelimited(NULL, ...). */
> -#define DRM_DEBUG_KMS_RATELIMITED(fmt, ...) drm_dbg_kms_ratelimited(NULL, fmt, ## __VA_ARGS__)
> -
>  /*
>   * struct drm_device based WARNs
>   *
> --
> 2.39.0
>
Nirmoy Das Jan. 17, 2023, 6:12 p.m. UTC | #2 
Hi Alex,

On 1/17/2023 7:06 PM, Alex Deucher wrote:
> On Tue, Jan 17, 2023 at 1:05 PM Nirmoy Das <nirmoy.das@intel.com> wrote:
>> There are no current users of DRM_DEBUG_KMS_RATELIMITED()
>> so remove it.
>>
>> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>> Cc: Maxime Ripard <mripard@kernel.org>
>> Cc: Thomas Zimmermann <tzimmermann@suse.de>
>> Cc: David Airlie <airlied@gmail.com>
>> Cc: Daniel Vetter <daniel@ffwll.ch>
>> Cc: Sam Ravnborg <sam@ravnborg.org>
>>
>> Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
>> Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
> Series is:
> Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
>
> Feel free to take the patches through whatever tree you want.


Please help me with this, I don't have committer rights for any tree.


Nirmoy


>
> Alex
>
>> ---
>>   include/drm/drm_print.h | 3 ---
>>   1 file changed, 3 deletions(-)
>>
>> diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
>> index a44fb7ef257f..c3753da97c4e 100644
>> --- a/include/drm/drm_print.h
>> +++ b/include/drm/drm_print.h
>> @@ -605,9 +605,6 @@ void __drm_err(const char *format, ...);
>>   #define drm_dbg_kms_ratelimited(drm, fmt, ...) \
>>          __DRM_DEFINE_DBG_RATELIMITED(KMS, drm, fmt, ## __VA_ARGS__)
>>
>> -/* NOTE: this is deprecated in favor of drm_dbg_kms_ratelimited(NULL, ...). */
>> -#define DRM_DEBUG_KMS_RATELIMITED(fmt, ...) drm_dbg_kms_ratelimited(NULL, fmt, ## __VA_ARGS__)
>> -
>>   /*
>>    * struct drm_device based WARNs
>>    *
>> --
>> 2.39.0
>>
Christian König Jan. 18, 2023, 6:27 a.m. UTC | #3 
Am 17.01.23 um 19:12 schrieb Das, Nirmoy:
> Hi Alex,
>
> On 1/17/2023 7:06 PM, Alex Deucher wrote:
>> On Tue, Jan 17, 2023 at 1:05 PM Nirmoy Das <nirmoy.das@intel.com> wrote:
>>> There are no current users of DRM_DEBUG_KMS_RATELIMITED()
>>> so remove it.
>>>
>>> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>>> Cc: Maxime Ripard <mripard@kernel.org>
>>> Cc: Thomas Zimmermann <tzimmermann@suse.de>
>>> Cc: David Airlie <airlied@gmail.com>
>>> Cc: Daniel Vetter <daniel@ffwll.ch>
>>> Cc: Sam Ravnborg <sam@ravnborg.org>
>>>
>>> Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
>>> Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
>> Series is:
>> Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
>>
>> Feel free to take the patches through whatever tree you want.
>
>
> Please help me with this, I don't have committer rights for any tree.

Going to push that into drm-misc-next later today.

Thanks,
Christian.

>
>
> Nirmoy
>
>
>>
>> Alex
>>
>>> ---
>>>   include/drm/drm_print.h | 3 ---
>>>   1 file changed, 3 deletions(-)
>>>
>>> diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
>>> index a44fb7ef257f..c3753da97c4e 100644
>>> --- a/include/drm/drm_print.h
>>> +++ b/include/drm/drm_print.h
>>> @@ -605,9 +605,6 @@ void __drm_err(const char *format, ...);
>>>   #define drm_dbg_kms_ratelimited(drm, fmt, ...) \
>>>          __DRM_DEFINE_DBG_RATELIMITED(KMS, drm, fmt, ## __VA_ARGS__)
>>>
>>> -/* NOTE: this is deprecated in favor of 
>>> drm_dbg_kms_ratelimited(NULL, ...). */
>>> -#define DRM_DEBUG_KMS_RATELIMITED(fmt, ...) 
>>> drm_dbg_kms_ratelimited(NULL, fmt, ## __VA_ARGS__)
>>> -
>>>   /*
>>>    * struct drm_device based WARNs
>>>    *
>>> -- 
>>> 2.39.0
>>>
Nirmoy Das Jan. 18, 2023, 9:34 a.m. UTC | #4 
On 1/18/2023 7:27 AM, Christian König wrote:
>
>
> Am 17.01.23 um 19:12 schrieb Das, Nirmoy:
>> Hi Alex,
>>
>> On 1/17/2023 7:06 PM, Alex Deucher wrote:
>>> On Tue, Jan 17, 2023 at 1:05 PM Nirmoy Das <nirmoy.das@intel.com> 
>>> wrote:
>>>> There are no current users of DRM_DEBUG_KMS_RATELIMITED()
>>>> so remove it.
>>>>
>>>> Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
>>>> Cc: Maxime Ripard <mripard@kernel.org>
>>>> Cc: Thomas Zimmermann <tzimmermann@suse.de>
>>>> Cc: David Airlie <airlied@gmail.com>
>>>> Cc: Daniel Vetter <daniel@ffwll.ch>
>>>> Cc: Sam Ravnborg <sam@ravnborg.org>
>>>>
>>>> Signed-off-by: Nirmoy Das <nirmoy.das@intel.com>
>>>> Reviewed-by: Sam Ravnborg <sam@ravnborg.org>
>>> Series is:
>>> Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
>>>
>>> Feel free to take the patches through whatever tree you want.
>>
>>
>> Please help me with this, I don't have committer rights for any tree.
>
> Going to push that into drm-misc-next later today.


Thanks, Christian.


>
> Thanks,
> Christian.
>
>>
>>
>> Nirmoy
>>
>>
>>>
>>> Alex
>>>
>>>> ---
>>>>   include/drm/drm_print.h | 3 ---
>>>>   1 file changed, 3 deletions(-)
>>>>
>>>> diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
>>>> index a44fb7ef257f..c3753da97c4e 100644
>>>> --- a/include/drm/drm_print.h
>>>> +++ b/include/drm/drm_print.h
>>>> @@ -605,9 +605,6 @@ void __drm_err(const char *format, ...);
>>>>   #define drm_dbg_kms_ratelimited(drm, fmt, ...) \
>>>>          __DRM_DEFINE_DBG_RATELIMITED(KMS, drm, fmt, ## __VA_ARGS__)
>>>>
>>>> -/* NOTE: this is deprecated in favor of 
>>>> drm_dbg_kms_ratelimited(NULL, ...). */
>>>> -#define DRM_DEBUG_KMS_RATELIMITED(fmt, ...) 
>>>> drm_dbg_kms_ratelimited(NULL, fmt, ## __VA_ARGS__)
>>>> -
>>>>   /*
>>>>    * struct drm_device based WARNs
>>>>    *
>>>> -- 
>>>> 2.39.0
>>>>
>
diff mbox series

Patch

diff --git a/include/drm/drm_print.h b/include/drm/drm_print.h
index a44fb7ef257f..c3753da97c4e 100644
--- a/include/drm/drm_print.h
+++ b/include/drm/drm_print.h
@@ -605,9 +605,6 @@  void __drm_err(const char *format, ...);
 #define drm_dbg_kms_ratelimited(drm, fmt, ...) \
 	__DRM_DEFINE_DBG_RATELIMITED(KMS, drm, fmt, ## __VA_ARGS__)
 
-/* NOTE: this is deprecated in favor of drm_dbg_kms_ratelimited(NULL, ...). */
-#define DRM_DEBUG_KMS_RATELIMITED(fmt, ...) drm_dbg_kms_ratelimited(NULL, fmt, ## __VA_ARGS__)
-
 /*
  * struct drm_device based WARNs
  *