From: Robert Jarzmik <robert.jarzmik@free.fr>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Antonio Ospite <ao2@ao2.it>,
Alexandre Belloni <alexandre.belloni@bootlin.com>,
Harald Welte <laforge@gnumonks.org>,
linux-rtc@vger.kernel.org,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: Removing pcap? (EZX support in mainline linux)
Date: Mon, 08 Apr 2019 21:14:44 +0200 [thread overview]
Message-ID: <87d0lwpavv.fsf@belgarion.home> (raw)
In-Reply-To: <CAK8P3a0TKUQC7GkYsWrJGFaRsvK5AQjPn74AnkRcPioXUFef-Q@mail.gmail.com> (Arnd Bergmann's message of "Mon, 8 Apr 2019 16:02:11 +0200")
Arnd Bergmann <arnd@arndb.de> writes:
> On Mon, Apr 8, 2019 at 8:58 AM Antonio Ospite <ao2@ao2.it> wrote:
>> On Sun, 7 Apr 2019 16:07:41 +0200
>> Alexandre Belloni <alexandre.belloni@bootlin.com> wrote:
>> adding Robert to CC as he is listed as the current maintainer of
>> ARM/EZX SMARTPHONES in the MAINTAINERS file.
>>
>> > I've had a look at the PCAP RTC driver because I'm removing a few
>> > deprecated APIs. While doing so, I've discovered that the driver has
>> > never worked properly because PCAP_RTC_TOD_MASK and PCAP_RTC_DAY_MASK
>> > are both missing a bit so every day would roll over at 18:12:15 and the
>> > whole the date would roll over in 2014.
>> >
>> > I discussed with Arnd about 59ee93a528b9 ("ARM: pxa: remove irq_to_gpio
>> > from ezx-pcap driver") and it seems the whole EZX support has been
>> > broken for a while.
>> >
>>
>> As Harald said in another message I tried taking over OpenEZX for a
>> while, but eventually lost interest.
>>
>> JFTR some parts of the project has been backed up here:
>> https://gitlab.com/openezx
>>
>> I should also have a git backup of the wiki somewhere.
>>
>> > Is it worth fixing the RTC driver or could I remove it?
>> >
>>
>> I am not aware of any active user but I'd leave the final word to
>> Robert, IIRC he received some EZX hardware "recently".
>>
>> FWIW I think EZX code can be removed if it is a maintenance burden for
>> anyone.
>
> I would prefer if we could either keep or remove all of the ezx code, but not
> just remove a particular driver. Removing it all now would let us skip that
> discussion the next time someone has a problem with another device driver
> for it.
I'm on the same page here. I'm not strongly either for or against EZX removal
(which would impact AFAIK 6 old Motorola smartphones).
If I can find the wiki archives to cross-check the mask issue (especially
PCAP_RTC_DAY_MASK), I wouldn't mind seeing a patch on that if it is kept.
If I can make the EZX boot (the one I got from Antonia), I could even test
it. As if it's worth it, I'm not personally an active user of this specific
platform, so if no one is, one less would be less maintainance burden.
Cheers.
--
Robert
next prev parent reply other threads:[~2019-04-08 19:15 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-07 14:07 Removing pcap? Alexandre Belloni
2019-04-07 18:03 ` Harald Welte
2019-04-08 6:58 ` Removing pcap? (EZX support in mainline linux) Antonio Ospite
2019-04-08 14:02 ` Arnd Bergmann
2019-04-08 19:14 ` Robert Jarzmik [this message]
2019-04-08 21:27 ` Antonio Ospite
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=87d0lwpavv.fsf@belgarion.home \
--to=robert.jarzmik@free.fr \
--cc=alexandre.belloni@bootlin.com \
--cc=ao2@ao2.it \
--cc=arnd@arndb.de \
--cc=laforge@gnumonks.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rtc@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).