[U-Boot] [PATCH v2 1/5] serial: Introduce ->getinfo() callback
Andy Shevchenko
andy.shevchenko at gmail.com
Tue Nov 20 18:27:28 UTC 2018
On Thu, Nov 15, 2018 at 09:09:55PM +0100, Alexander Graf wrote:
>
>
> On 15.11.18 20:31, Andy Shevchenko wrote:
> > On Thu, Nov 15, 2018 at 8:22 PM Alexander Graf <agraf at suse.de> wrote:
> >> On 15.11.18 18:58, Andy Shevchenko wrote:
> >>> New callback will give a necessary information to fill up ACPI SPCR table,
> >>> for example. Maybe used later for other purposes.
> >
> >>> +/* REVISIT: ACPI GAS specification implied */
> >>
> >> What does this REVISIT tag mean?
> >
> > Had you chance to read my cover letter?
> > There is a section called "Known issues", item 3 there might answer to
> > your question.
>
> The usual tag for "not finalized, please don't apply yet" in upstream
> patch submissions is "RFC". If you don't think your code is ready to be
> applied, just tag the patches with RFC, like this:
>
> [RFC] serial: Introduce ->getinfo() callback
>
> Then everyone knows that you don't think the patch is good enough yet.
> Otherwise, every submission really implies that you think it should get
> applied.
>
> As for the data structure itself, I think I gave you a bit of feedback.
> I wouldn't worry too much about getting everything right from the start,
> as this is a monolithic open source project. So changing the structure
> later on is easy.
Thanks, noted.
--
With Best Regards,
Andy Shevchenko
More information about the U-Boot
mailing list