Re: [PATCH v3 1/2] dw-xdata-pcie: Fix documentation build warns and update outdated info

From: Greg Kroah-Hartman
Date: Sat Apr 10 2021 - 05:01:03 EST


On Wed, Apr 07, 2021 at 10:31:48PM +0200, Gustavo Pimentel wrote:
> Fixes documentation build warnings related to indentation and text
> formatting, such as:
>
> Documentation/misc-devices/dw-xdata-pcie.rst:20: WARNING: Unexpected
> indentation.
> Documentation/misc-devices/dw-xdata-pcie.rst:24: WARNING: Unexpected
> indentation.
> Documentation/misc-devices/dw-xdata-pcie.rst:25: WARNING: Block quote
> ends without a blank line; unexpected unindent.
> Documentation/misc-devices/dw-xdata-pcie.rst:30: WARNING: Unexpected
> indentation.
> Documentation/misc-devices/dw-xdata-pcie.rst:34: WARNING: Unexpected
> indentation.
> Documentation/misc-devices/dw-xdata-pcie.rst:35: WARNING: Block quote
> ends without a blank line; unexpected unindent.
> Documentation/misc-devices/dw-xdata-pcie.rst:40: WARNING: Unexpected
> indentation.

In the future, there's no need to wrap error/warning lines like this in
a changelog text. Not a big deal, but the above is messy to read,
right?

>
> Also fixes some outdated information related to stop file interface in sysfs.

When you say "also", that means you need a separate patch usually. And
for this patch, that is exactly what you need. Please split this up
into one patch that fixes the reported problem, and another one that
adds the needed information.

thanks,

greg k-h