Skip to contentSkip to conversation

AMLogic Meson SoC RTC driver

PATCH Superseded Ben Dooks 2 participants 3 messages Jul 5, 2016 – Jul 12, 2016

Subsystems (2)

v3 of 4 Superseded Latest is v4 →

lore
Share & tools
lore

Ben Dooks

author
#0 lore raw

v3 cover AMLogic Meson SoC RTC driver

This series adds support for the on SoC RTC driver on the AMLogic Meson devices. The device-tree updates are not included as they depend on the reset-controller support in -next.

The series does not have any build dependencies, but does require the reset controller and device-tree entries for the relevant boards and has been tested on the ODroid-C1 board.

Changes since V1:
<[email protected]>

- Fixed checkpatch and sparse errors
- Fixed minor locking bug when writing static-control register
- Updated the code ordering in the rtc-meson.c driver
- Fixed alphanumeric ordering in on-cpu rtc block
- Added Rob Hering's ack for the DT binding documentation
- Added better description to the RTC driver header.

Ben Dooks (2):
rtc: support for amlogic meson rtc
meson-rtc: add device-tree binding

 .../devicetree/bindings/rtc/rtc-amlogic.txt        |  21 ++
 drivers/rtc/Kconfig                                |   9 +
 drivers/rtc/Makefile                               |   1 +
 drivers/rtc/rtc-meson.c                            | 349 +++++++++++++++++++++
 6 files changed, 392 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/rtc/rtc-amlogic.txt
 create mode 100644 drivers/rtc/rtc-meson.c
Signature · 1 line
2.8.1
maintainer
#1 lore raw

v3 cover AMLogic Meson SoC RTC driver

Quoted 2 lines · Ben Dooks

This series adds support for the on SoC RTC driver on the AMLogic Meson devices.

-ECONFUSED

In reply to this patch, there are two other "PATCH v3 0/2" replies, an then the two patches themselves, have "PATCH n/4" subjects when one would expect "PATCH n/2" subjects.

Might I recommend that you use 'git format-patch --cover-letter --subject-prefix="PATCH vX"' when formatting your series. Doing that will take care of all the numbering for you.

Anyways, I was able to test this version with the DTs from the previous version, and it works ok.

Can you respin one more time (including the Tested-by) so the RTC maintainers can pick it up?

Thanks,

Kevin

author
#2 lore raw

v3 cover AMLogic Meson SoC RTC driver

Quoted 10 lines · Kevin Hilman
Quoted 2 lines · Ben Dooks

This series adds support for the on SoC RTC driver on the AMLogic Meson devices.

-ECONFUSED

In reply to this patch, there are two other "PATCH v3 0/2" replies, an then the two patches themselves, have "PATCH n/4" subjects when one would expect "PATCH n/2" subjects.

This was my mistake of formatting the patch then thinking it would be easy to remove the other two patches I decided to keep separate (for reasons below)

Quoted 2 lines

Anyways, I was able to test this version with the DTs from the previous version, and it works ok.

Yes, no changes for the DT bits so I decided to remove them as firstly they depend on the reset-controller changes going in and secondly the rtc driver part doesn't need them to be there to be able to build.

Quoted 2 lines

Can you respin one more time (including the Tested-by) so the RTC maintainers can pick it up?

I'll do that now and try and post them tonight.
Thankyou for testing.

Signature · 2 lines
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius