Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230403211548.6253-1-jorge.lopez2@hp.com> <ZCuMkdb6jeL4S8hz@debian.me>
 <CAOOmCE-5NHkYcmBP1GGpr=sEoxF9jkjj1LevekEm8R0wbrM4xQ@mail.gmail.com> <565e6575-91f7-9ca2-1505-17527b95a816@redhat.com>
In-Reply-To: <565e6575-91f7-9ca2-1505-17527b95a816@redhat.com>
From:   Jorge Lopez <jorgealtxwork@gmail.com>
Date:   Fri, 14 Apr 2023 10:13:28 -0500
Message-ID: <CAOOmCE9HurxZf57CjUSxAF5TXvMGukyfP6RbaTbkEeWyeRqcsQ@mail.gmail.com>
Subject: Re: [PATCH v7] Introduction-of-HP-BIOSCFG-driver-documentation
To:     Hans de Goede <hdegoede@redhat.com>
Cc:     Bagas Sanjaya <bagasdotme@gmail.com>,
        Mark Pearson <mpearson-lenovo@squebb.ca>,
        =?UTF-8?Q?Thomas_Wei=C3=9Fschuh?= <thomas@t-8ch.de>,
        Linux x86 Platform Drivers 
        <platform-driver-x86@vger.kernel.org>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Documentation <linux-doc@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi there,

Version 9 review for the sysfs-class-firmware-attributes document was
submitted and I look forward  to getting your comments.   I understand
you have other things happening, but as soon as the document is
finalized, I can move to submit the rest of the files.

While the document review is ongoing,  I proceeded to address all
comments made by Thomas, Hans, Bagas, and Mark regarding both
documentation and source code changes.  The code compiles with W=3D1
flag without any issues and the  refactoring proposed by Thomas was
incorporated.  Testing on multiple platforms continues to validate the
driver proper operation between reviews.

Regards,

Jorge


On Thu, Apr 6, 2023 at 9:08=E2=80=AFAM Hans de Goede <hdegoede@redhat.com> =
wrote:
>
> Hi All,
>
> FWIW I have been reading along with both the v6 and
> this v7 posting.
>
> Thomas, Mark, thank you for your review / feedback
> on this series.
>
> On 4/4/23 15:37, Jorge Lopez wrote:
> > Hi Bagas,
> >
> >>
> >> Where is the rest of patches if this is a series? Had they been merged=
?
> >>
> >
> > There is only one change as requested by  Thomas Wei=C3=9Fschuh
> >
> >> Feel free to ONLY submit the patch with the documentation for the next
> >> revision. Then we can nail down the interface and initial functionalit=
y
> >> and you don't always have to adapt the code to the changing interface.
> >
> > Perhaps, I misunderstood Thomas request.   I will address a few other
> > comments and will submit all files again.
>
> I think that Thomas' suggestion to first focus on getting
> the userspace API right and then implement the agreed
> upon API is a good idea.
>
> So for the next version just post only the documentation
> patch again please.
>
> Note it is not the intention to merge just the documentation
> patch without merging the code first.
>
> The idea is to first nail the API down and then modify
> the code once to implement the agreed upon API.
>
> Then once the API has been agreed upon post newer
> versions which also include the code again.
>
> And then once people are also happy with the code
> we can merge both the code + documentation in one go.
>
> Regards,
>
> Hans
>
>
>
>
>
>
>
> >
> > Regards,
> >
> > Jorge
> >
>