From: Florian Eckert <fe@dev.tdt.de>
To: Andy Shevchenko <andy.shevchenko@gmail.com>
Cc: Linus Walleij <linus.walleij@linaro.org>,
Bartosz Golaszewski <bgolaszewski@baylibre.com>,
Darren Hart <dvhart@infradead.org>,
Andy Shevchenko <andy@infradead.org>,
Eckert.Florian@googlemail.com,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
"open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>,
Platform Driver <platform-driver-x86@vger.kernel.org>
Subject: Re: [PATCH v5 0/2] Add device driver for APU2/APU3 GPIOs
Date: Tue, 04 Dec 2018 10:58:24 +0100 [thread overview]
Message-ID: <84a2396b0b5ba34a2339d35159a97a89@dev.tdt.de> (raw)
In-Reply-To: <CAHp75VdEgm-=jHgJXz_mM6DZMnFVEeZW_es3VyL4L2V+khYWtg@mail.gmail.com>
>> >> Yes i will fix your hints tomorrow and send a v6 of my patchset.
>> >> Thank you for your hints and time
>> >> It would be nice if you could fix ACPI problemmatik.
>> >
>> > I would like to see the ACPI dump for that...
>>
>> See
>> https://github.com/openwrt/openwrt/pull/1232#issuecomment-443224576
>> In this comment Michał Żygowski appended to this thread the missing
>> files you want to have.
> So, let me clarify what we have:
> - some platforms are in the wild with old BIOS with broken ACPI tables
correct
> - you still may fix the things for new BIOS version for all affected
> platforms
I have seen that this is a lot of work and I didn't think it was so
complicated!
To get the GPIO support for APU2/APU3 merged into the gpio subsystem.
I am little confused what i should do now. By the way I only have one
board (APU3).
> - you need to support both
That is not necessary from my point of view. I am fine if the driver
supports at least
the current BIOS version. And if the coreboot maintainer fixes the ACPI
problem
then we could extend the driver and add the ACPI stuff.
But this not in my hand, right?
> For broken firmware you need to do the following:
> - create an MFD driver, which would instantiate GPIO and GPIO keys
> support (at least)
> - create one of each above drivers w/o any DMI crap (should be done
> as a part of MFD driver)
I will have a look how to achieve this.
If we want to support all BIOS version.
And is a must have to get the driver into mainline.
> For fixed BIOS you need to add the following (example, not a fully
> correct solution) at the level behind SB:
I do not understand how I could fix this. I have no idea from ACPI.
I have not found any driver where I can inspire myself.
>
> Scope (SB)
> {
> Device(GPIO)
> {
> After updating firmware you would need just an ACPI ID table to be
> added to the GPIO driver. MFD driver should not be enumerated at all.
That's maybe coming next when the coreboot maintainers have their bios
fixed, right?
prev parent reply other threads:[~2018-12-04 9:58 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-11-27 13:25 [PATCH v5 0/2] Add device driver for APU2/APU3 GPIOs Florian Eckert
2018-11-27 13:25 ` [PATCH v5 1/2] gpio: Add driver for PC Engines APU boards Florian Eckert
2018-11-28 5:19 ` kbuild test robot
2018-11-28 12:00 ` Andy Shevchenko
2018-12-04 10:17 ` Florian Eckert
2018-11-27 13:25 ` [PATCH v5 2/2] platform: Add reset button device " Florian Eckert
2018-11-28 12:05 ` Andy Shevchenko
2018-11-28 17:06 ` kbuild test robot
2018-11-28 12:07 ` [PATCH v5 0/2] Add device driver for APU2/APU3 GPIOs Andy Shevchenko
2018-11-29 10:15 ` Florian Eckert
2018-11-29 13:44 ` Andy Shevchenko
2018-11-29 14:02 ` Florian Eckert
2018-11-29 15:24 ` Andy Shevchenko
2018-12-03 7:58 ` Florian Eckert
2018-12-03 15:43 ` Andy Shevchenko
2018-12-04 9:58 ` Florian Eckert [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=84a2396b0b5ba34a2339d35159a97a89@dev.tdt.de \
--to=fe@dev.tdt.de \
--cc=Eckert.Florian@googlemail.com \
--cc=andy.shevchenko@gmail.com \
--cc=andy@infradead.org \
--cc=bgolaszewski@baylibre.com \
--cc=dvhart@infradead.org \
--cc=linus.walleij@linaro.org \
--cc=linux-gpio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=platform-driver-x86@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).