[PATCH linux v5 0/4] Seven segment display support
Joel Stanley
joel at jms.id.au
Tue Dec 13 11:31:46 AEDT 2016
Hi Jagha,
On Tue, Dec 13, 2016 at 10:58 AM, Jaghathiswari Rankappagounder
Natarajan <jaghu at google.com> wrote:
> Hi Joel,
> Could you please help review this patch series. Would like to have the seven
> segment display driver for testing out on zaius machine.
I have spent a fair amount of time reading your code. It would be nice
to see other developers take a look as well. Have you asked for review
from your peers?
> (I am trying to get the fan tach driver to work and I hope to send that a> along with pwm driver for review soon.)
If you think this driver is ready to go, I suggest you send it
upstream for wider community review. Same for the pwm driver. Make
sure you cc me so I can help out.
> Also you had mentioned about writing the LPC driver. Is that checked in?
Andrew sent a proposed driver some time ago. Did you have time to review it?
http://patchwork.ozlabs.org/patch/693032/
I think we may prefer an ioctl based approach, but this is up for debate.
Cheers,
Joel
>
> Thanks,
> Jagha
>
> On Tue, Nov 22, 2016 at 6:45 PM, Joel Stanley <joel at jms.id.au> wrote:
>>
>> We spoke on IRC about the clock changes. Lets wait until those issues
>> are addressed before reviewing this series.
>>
>> Cheers,
>>
>> Joel
>>
>> On Sun, Nov 20, 2016 at 5:57 PM, Jaghathiswari Rankappagounder
>> Natarajan <jaghu at google.com> wrote:
>> > This patchset includes:
>> >
>> > Documentation for the binding which provides an interface for adding
>> > clock,
>> > data and clear signal GPIO lines to control seven segment display.
>> >
>> > The platform device driver provides an API for displaying on two
>> > 7-segment
>> > displays, and implements the required bit-banging. The hardware assumed
>> > is
>> > 74HC164 wired to two 7-segment displays.
>> >
>> > The character device driver implements the user-space API for letting a
>> > user
>> > write to two 7-segment displays including any conversion methods
>> > necessary
>> > to map the user input to two 7-segment displays.
>> >
>> > Adding clock, data and clear signal GPIO lines in the devicetree to
>> > control
>> > seven segment display on zaius platform.
>> >
>> > The platform driver matches on the device tree node; the platform driver
>> > also
>> > initializes the character device.
>> >
>> > Tested that the seven segment display works properly by writing to the
>> > character device file on EVB AST2500 board and on Zaius board.
>> >
>> > A changelog between versions is included in the individual patches.
>> > Please
>> > provide feedback.
>> >
>> > Jaghathiswari Rankappagounder Natarajan (4):
>> > Documentation: dt-bindings: Document bindings for seven segment
>> > display support
>> > drivers: misc: Character device driver for seven segment display
>> > drivers: misc: Platform driver for seven segment display support
>> > arm: dts: Add dt-binding to support seven segment display on zaius
>> >
>> > .../devicetree/bindings/misc/seven-seg-gpio.txt | 27 +++
>> > arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts | 8 +
>> > drivers/misc/Kconfig | 16 ++
>> > drivers/misc/Makefile | 2 +
>> > drivers/misc/seven_seg_disp.c | 197
>> > ++++++++++++++++++++
>> > drivers/misc/seven_seg_disp.h | 34 ++++
>> > drivers/misc/seven_seg_gpio.c | 206
>> > +++++++++++++++++++++
>> > 7 files changed, 490 insertions(+)
>> > create mode 100644
>> > Documentation/devicetree/bindings/misc/seven-seg-gpio.txt
>> > create mode 100644 drivers/misc/seven_seg_disp.c
>> > create mode 100644 drivers/misc/seven_seg_disp.h
>> > create mode 100644 drivers/misc/seven_seg_gpio.c
>> >
>> > --
>> > 2.8.0.rc3.226.g39d4020
>> >
>
>
More information about the openbmc
mailing list