[PATCH 0/3] Documentation fixes for MT6370 RGB

Bagas Sanjaya posted 3 patches 1 year, 6 months ago
Documentation/leds/index.rst           |  1 +
Documentation/leds/leds-mt6370-rgb.rst | 42 +++++++++++++-------------
2 files changed, 22 insertions(+), 21 deletions(-)
[PATCH 0/3] Documentation fixes for MT6370 RGB
Posted by Bagas Sanjaya 1 year, 6 months ago
kernel test robot recently reported htmldocs warnings on documentation
for MT6370 RGB LED. So here are the fixes.

Bagas Sanjaya (3):
  Documentation: leds: Add MT6370 doc to the toctree
  Documentation: leds: MT6370: Properly wrap hw_pattern chart
  Documentation: leds: MT6370: Use bullet lists for timing variables

 Documentation/leds/index.rst           |  1 +
 Documentation/leds/leds-mt6370-rgb.rst | 42 +++++++++++++-------------
 2 files changed, 22 insertions(+), 21 deletions(-)


base-commit: 4ba9df04b7ac66d2d000ed7ae2d8136302d99a57
-- 
An old man doll... just what I always wanted! - Clara
Re: [PATCH 0/3] Documentation fixes for MT6370 RGB
Posted by Pavel Machek 1 year, 6 months ago
On Sun 2023-03-19 14:49:00, Bagas Sanjaya wrote:
> kernel test robot recently reported htmldocs warnings on documentation
> for MT6370 RGB LED. So here are the fixes.
> 
> Bagas Sanjaya (3):
>   Documentation: leds: Add MT6370 doc to the toctree
>   Documentation: leds: MT6370: Properly wrap hw_pattern chart
>   Documentation: leds: MT6370: Use bullet lists for timing variables
> 
>  Documentation/leds/index.rst           |  1 +
>  Documentation/leds/leds-mt6370-rgb.rst | 42 +++++++++++++-------------
>  2 files changed, 22 insertions(+), 21 deletions(-)

Series looks good to me.

Acked-by: Pavel Machek <pavel@ucw.cz>

								Pavel
-- 
People of Russia, stop Putin before his war on Ukraine escalates.
Re: [PATCH 0/3] Documentation fixes for MT6370 RGB
Posted by Bagas Sanjaya 1 year, 6 months ago
On 3/19/23 14:49, Bagas Sanjaya wrote:
> kernel test robot recently reported htmldocs warnings on documentation
> for MT6370 RGB LED. So here are the fixes.
> 
> Bagas Sanjaya (3):
>   Documentation: leds: Add MT6370 doc to the toctree
>   Documentation: leds: MT6370: Properly wrap hw_pattern chart
>   Documentation: leds: MT6370: Use bullet lists for timing variables
> 
>  Documentation/leds/index.rst           |  1 +
>  Documentation/leds/leds-mt6370-rgb.rst | 42 +++++++++++++-------------
>  2 files changed, 22 insertions(+), 21 deletions(-)
> 
> 
> base-commit: 4ba9df04b7ac66d2d000ed7ae2d8136302d99a57

ping

-- 
An old man doll... just what I always wanted! - Clara
Re: [PATCH 0/3] Documentation fixes for MT6370 RGB
Posted by Lee Jones 1 year, 6 months ago
On Thu, 23 Mar 2023, Bagas Sanjaya wrote:

> On 3/19/23 14:49, Bagas Sanjaya wrote:
> > kernel test robot recently reported htmldocs warnings on documentation
> > for MT6370 RGB LED. So here are the fixes.
> >
> > Bagas Sanjaya (3):
> >   Documentation: leds: Add MT6370 doc to the toctree
> >   Documentation: leds: MT6370: Properly wrap hw_pattern chart
> >   Documentation: leds: MT6370: Use bullet lists for timing variables
> >
> >  Documentation/leds/index.rst           |  1 +
> >  Documentation/leds/leds-mt6370-rgb.rst | 42 +++++++++++++-------------
> >  2 files changed, 22 insertions(+), 21 deletions(-)
> >
> >
> > base-commit: 4ba9df04b7ac66d2d000ed7ae2d8136302d99a57
>
> ping

a) Don't do that!

b) Especually don't do that 4 days after submission!

The usual expectation is 2 full weeks before submitting a [RESEND].

Mark Brown says it best:

"
Please don't send content free pings and please allow a reasonable time
for review.  People get busy, go on holiday, attend conferences and so
on so unless there is some reason for urgency (like critical bug fixes)
please allow at least a couple of weeks for review.  If there have been
review comments then people may be waiting for those to be addressed.

Sending content free pings adds to the mail volume (if they are seen at
all) which is often the problem and since they can't be reviewed
directly if something has gone wrong you'll have to resend the patches
anyway, so sending again is generally a better approach though there are
some other maintainers who like them - if in doubt look at how patches
for the subsystem are normally handled.
"

--
Lee Jones [李琼斯]
Re: [PATCH 0/3] Documentation fixes for MT6370 RGB
Posted by Bagas Sanjaya 1 year, 6 months ago
On 3/23/23 19:31, Lee Jones wrote:
>> ping
> 
> a) Don't do that!
> 
> b) Especually don't do that 4 days after submission!
> 
> The usual expectation is 2 full weeks before submitting a [RESEND].
> 
> Mark Brown says it best:
> 
> "
> Please don't send content free pings and please allow a reasonable time
> for review.  People get busy, go on holiday, attend conferences and so
> on so unless there is some reason for urgency (like critical bug fixes)
> please allow at least a couple of weeks for review.  If there have been
> review comments then people may be waiting for those to be addressed.
> 
> Sending content free pings adds to the mail volume (if they are seen at
> all) which is often the problem and since they can't be reviewed
> directly if something has gone wrong you'll have to resend the patches
> anyway, so sending again is generally a better approach though there are
> some other maintainers who like them - if in doubt look at how patches
> for the subsystem are normally handled.
> "
> 

Ah! I triggered the wrong message at the wrong time. Thanks anyway!

-- 
An old man doll... just what I always wanted! - Clara