diff mbox series

[v2,2/2] Organize declaration variables

Message ID 20220608170220.5751-3-jorge.lopez2@hp.com (mailing list archive)
State Superseded, archived
Headers show
Series Resolve-WMI-query-failures-on-some-devices | expand

Commit Message

Jorge Lopez June 8, 2022, 5:02 p.m. UTC 
This patch organizes declaration variables in method
hp_wmi_perform_query as requested.

Signed-off-by: Jorge Lopez <jorge.lopez2@hp.com>

---
Based on the latest platform-drivers-x86.git/for-next

V1 - Original submission
V2 - Organize declaration variables
---
 drivers/platform/x86/hp-wmi.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Andy Shevchenko June 8, 2022, 6:45 p.m. UTC | #1 
On Wed, Jun 8, 2022 at 7:20 PM Jorge Lopez <jorgealtxwork@gmail.com> wrote:
>
> This patch organizes declaration variables in method
> hp_wmi_perform_query as requested.

Ah, I see now. Please squash these changes in one patch.
Jorge Lopez June 8, 2022, 7:31 p.m. UTC | #2 
Will do. Thank you

On Wed, Jun 8, 2022 at 1:45 PM Andy Shevchenko
<andy.shevchenko@gmail.com> wrote:
>
> On Wed, Jun 8, 2022 at 7:20 PM Jorge Lopez <jorgealtxwork@gmail.com> wrote:
> >
> > This patch organizes declaration variables in method
> > hp_wmi_perform_query as requested.
>
> Ah, I see now. Please squash these changes in one patch.
>
> --
> With Best Regards,
> Andy Shevchenko
diff mbox series

Patch

diff --git a/drivers/platform/x86/hp-wmi.c b/drivers/platform/x86/hp-wmi.c
index 7bcfa07cc6ab..d3540dd62d06 100644
--- a/drivers/platform/x86/hp-wmi.c
+++ b/drivers/platform/x86/hp-wmi.c
@@ -290,9 +290,9 @@  static int hp_wmi_perform_query(int query, enum hp_wmi_command command,
 	struct bios_return *bios_return;
 	union acpi_object *obj = NULL;
 	struct bios_args *args = NULL;
-	int mid, actual_outsize, ret;
+	int mid, actual_insize, actual_outsize;
 	size_t bios_args_size;
-	int actual_insize;
+	int ret;
 
 	mid = encode_outsize_for_pvsz(outsize);
 	if (WARN_ON(mid < 0))