Received-SPF: softfail (google.com: domain of transitioning linux-kernel-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Date:   Fri, 6 May 2022 16:48:49 -0400
From:   Ian Cowan <ian@linux.cowan.aero>
To:     "Rafael J. Wysocki" <rafael@kernel.org>
Cc:     Len Brown <lenb@kernel.org>,
        ACPI Devel Maling List <linux-acpi@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] drivers: acpi: clean up spaces to be consistent
Message-ID: <YnWJsS8OaCJssz89@fedora>
References: <20220425021407.486916-1-ian@linux.cowan.aero>
 <CAJZ5v0hb9Goj9N+VCfAB9Fy+HA7EqG=Z+XJo1t53KcmF62-PbA@mail.gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <CAJZ5v0hb9Goj9N+VCfAB9Fy+HA7EqG=Z+XJo1t53KcmF62-PbA@mail.gmail.com>
Precedence: bulk

On Fri, May 06, 2022 at 08:34:09PM +0200, Rafael J. Wysocki wrote:
> On Mon, Apr 25, 2022 at 4:14 AM Ian Cowan <ian@linux.cowan.aero> wrote:
> > @@ -190,12 +190,14 @@ static int acpi_ac_battery_notify(struct notifier_block *nb,
> >  static int __init thinkpad_e530_quirk(const struct dmi_system_id *d)
> >  {
> >         ac_sleep_before_get_state_ms = 1000;
> > +
> >         return 0;
> >  }
> >
> >  static int __init ac_only_quirk(const struct dmi_system_id *d)
> >  {
> >         ac_only = 1;
> > +
> >         return 0;
> >  }
> 
> I don't really think that it is useful to add empty lines in the two
> cases above.
> 
> The rest of the patch is fine with me.
> 

I just submitted v2 removing these 2 changes and leaving the others.

Thanks!
Ian