* [PATCH] acpi: acpi_video: Fixed comment style code issues
@ 2021-06-12 18:40 Clayton Casciato
2021-06-14 14:50 ` Rafael J. Wysocki
0 siblings, 1 reply; 3+ messages in thread
From: Clayton Casciato @ 2021-06-12 18:40 UTC (permalink / raw)
To: rui.zhang; +Cc: lenb, linux-acpi, rjw, Clayton Casciato
Fixed coding style issues.
Signed-off-by: Clayton Casciato <majortomtosourcecontrol@gmail.com>
---
drivers/acpi/acpi_video.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/drivers/acpi/acpi_video.c b/drivers/acpi/acpi_video.c
index 0c884020f74b..61e52ab5aab6 100644
--- a/drivers/acpi/acpi_video.c
+++ b/drivers/acpi/acpi_video.c
@@ -134,8 +134,7 @@ struct acpi_video_device_attrib {
u32 display_type:4; /* Describe the specific type in use */
u32 vendor_specific:4; /* Chipset Vendor Specific */
u32 bios_can_detect:1; /* BIOS can detect the device */
- u32 depend_on_vga:1; /* Non-VGA output device whose power is related to
- the VGA device. */
+ u32 depend_on_vga:1; /* Non-VGA output device whose power is related to the VGA device */
u32 pipe_id:3; /* For VGA multiple-head devices. */
u32 reserved:10; /* Must be 0 */
@@ -1581,13 +1580,11 @@ static void acpi_video_bus_notify(struct acpi_device *device, u32 event)
input = video->input;
switch (event) {
- case ACPI_VIDEO_NOTIFY_SWITCH: /* User requested a switch,
- * most likely via hotkey. */
+ case ACPI_VIDEO_NOTIFY_SWITCH: /* User requested a switch, most likely via hotkey. */
keycode = KEY_SWITCHVIDEOMODE;
break;
- case ACPI_VIDEO_NOTIFY_PROBE: /* User plugged in or removed a video
- * connector. */
+ case ACPI_VIDEO_NOTIFY_PROBE: /* User plugged in or removed a video connector. */
acpi_video_device_enumerate(video);
acpi_video_device_rebind(video);
keycode = KEY_SWITCHVIDEOMODE;
--
2.31.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] acpi: acpi_video: Fixed comment style code issues
2021-06-12 18:40 [PATCH] acpi: acpi_video: Fixed comment style code issues Clayton Casciato
@ 2021-06-14 14:50 ` Rafael J. Wysocki
2021-06-15 5:43 ` Clayton Casciato
0 siblings, 1 reply; 3+ messages in thread
From: Rafael J. Wysocki @ 2021-06-14 14:50 UTC (permalink / raw)
To: Clayton Casciato
Cc: Zhang, Rui, Len Brown, ACPI Devel Maling List, Rafael J. Wysocki
On Sat, Jun 12, 2021 at 8:42 PM Clayton Casciato
<majortomtosourcecontrol@gmail.com> wrote:
>
> Fixed coding style issues.
The comments are broken the way they are to avoid going over 80
characters. What's wrong with them?
> Signed-off-by: Clayton Casciato <majortomtosourcecontrol@gmail.com>
> ---
> drivers/acpi/acpi_video.c | 9 +++------
> 1 file changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/acpi/acpi_video.c b/drivers/acpi/acpi_video.c
> index 0c884020f74b..61e52ab5aab6 100644
> --- a/drivers/acpi/acpi_video.c
> +++ b/drivers/acpi/acpi_video.c
> @@ -134,8 +134,7 @@ struct acpi_video_device_attrib {
> u32 display_type:4; /* Describe the specific type in use */
> u32 vendor_specific:4; /* Chipset Vendor Specific */
> u32 bios_can_detect:1; /* BIOS can detect the device */
> - u32 depend_on_vga:1; /* Non-VGA output device whose power is related to
> - the VGA device. */
> + u32 depend_on_vga:1; /* Non-VGA output device whose power is related to the VGA device */
> u32 pipe_id:3; /* For VGA multiple-head devices. */
> u32 reserved:10; /* Must be 0 */
>
> @@ -1581,13 +1580,11 @@ static void acpi_video_bus_notify(struct acpi_device *device, u32 event)
> input = video->input;
>
> switch (event) {
> - case ACPI_VIDEO_NOTIFY_SWITCH: /* User requested a switch,
> - * most likely via hotkey. */
> + case ACPI_VIDEO_NOTIFY_SWITCH: /* User requested a switch, most likely via hotkey. */
> keycode = KEY_SWITCHVIDEOMODE;
> break;
>
> - case ACPI_VIDEO_NOTIFY_PROBE: /* User plugged in or removed a video
> - * connector. */
> + case ACPI_VIDEO_NOTIFY_PROBE: /* User plugged in or removed a video connector. */
> acpi_video_device_enumerate(video);
> acpi_video_device_rebind(video);
> keycode = KEY_SWITCHVIDEOMODE;
> --
> 2.31.1
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] acpi: acpi_video: Fixed comment style code issues
2021-06-14 14:50 ` Rafael J. Wysocki
@ 2021-06-15 5:43 ` Clayton Casciato
0 siblings, 0 replies; 3+ messages in thread
From: Clayton Casciato @ 2021-06-15 5:43 UTC (permalink / raw)
To: Rafael J. Wysocki
Cc: Zhang, Rui, Len Brown, ACPI Devel Maling List, Rafael J. Wysocki
On Mon, Jun 14, 2021 at 04:50:05PM +0200, Rafael J. Wysocki wrote:
> On Sat, Jun 12, 2021 at 8:42 PM Clayton Casciato
> <majortomtosourcecontrol@gmail.com> wrote:
> >
> > Fixed coding style issues.
>
> The comments are broken the way they are to avoid going over 80
> characters. What's wrong with them?
>
Checkpatch warns that "block comments use a trailing */ on a separate line".
I believe the patch fixes the style issue and improves readability.
NB: the first comment exceeds 80 characters pre-patch.
From my understanding, the default limit is 100 characters as of bdc48fa11e46.
Thank you for your consideration!
> > Signed-off-by: Clayton Casciato <majortomtosourcecontrol@gmail.com>
> > ---
> > drivers/acpi/acpi_video.c | 9 +++------
> > 1 file changed, 3 insertions(+), 6 deletions(-)
> >
> > diff --git a/drivers/acpi/acpi_video.c b/drivers/acpi/acpi_video.c
> > index 0c884020f74b..61e52ab5aab6 100644
> > --- a/drivers/acpi/acpi_video.c
> > +++ b/drivers/acpi/acpi_video.c
> > @@ -134,8 +134,7 @@ struct acpi_video_device_attrib {
> > u32 display_type:4; /* Describe the specific type in use */
> > u32 vendor_specific:4; /* Chipset Vendor Specific */
> > u32 bios_can_detect:1; /* BIOS can detect the device */
> > - u32 depend_on_vga:1; /* Non-VGA output device whose power is related to
> > - the VGA device. */
> > + u32 depend_on_vga:1; /* Non-VGA output device whose power is related to the VGA device */
> > u32 pipe_id:3; /* For VGA multiple-head devices. */
> > u32 reserved:10; /* Must be 0 */
> >
> > @@ -1581,13 +1580,11 @@ static void acpi_video_bus_notify(struct acpi_device *device, u32 event)
> > input = video->input;
> >
> > switch (event) {
> > - case ACPI_VIDEO_NOTIFY_SWITCH: /* User requested a switch,
> > - * most likely via hotkey. */
> > + case ACPI_VIDEO_NOTIFY_SWITCH: /* User requested a switch, most likely via hotkey. */
> > keycode = KEY_SWITCHVIDEOMODE;
> > break;
> >
> > - case ACPI_VIDEO_NOTIFY_PROBE: /* User plugged in or removed a video
> > - * connector. */
> > + case ACPI_VIDEO_NOTIFY_PROBE: /* User plugged in or removed a video connector. */
> > acpi_video_device_enumerate(video);
> > acpi_video_device_rebind(video);
> > keycode = KEY_SWITCHVIDEOMODE;
> > --
> > 2.31.1
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-06-15 5:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-06-12 18:40 [PATCH] acpi: acpi_video: Fixed comment style code issues Clayton Casciato
2021-06-14 14:50 ` Rafael J. Wysocki
2021-06-15 5:43 ` Clayton Casciato
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.