[PATCH linux v4 0/3] Seven segment display support
Joel Stanley
joel at jms.id.au
Wed Nov 16 12:45:32 AEDT 2016
Hello Jagha,
On Wed, Nov 16, 2016 at 7:07 AM, Jaghathiswari Rankappagounder
Natarajan <jaghu at google.com> wrote:
> Hi Joel,
>
> Could you please provide feedback for this driver.
You have not addressed all of the comments from Rick in v4 of your
patches. The first thing I noticed is he pointed out you need to
separate out the device tree documentation from the code. I stopped
reading at this point.
Please go back and re-read the previous review and ensure your patches
address all comments.
Cheers,
Joel
>
> Thanks,
> Jagha
>
>
> On Wed, Nov 2, 2016 at 1:17 PM, Jaghathiswari Rankappagounder Natarajan
> <jaghu at google.com> wrote:
>>
>> This patchset includes:
>>
>> Character device driver which 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.
>>
>> Platform device driver which 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.
>>
>> Adding clock, data and clear signal lines in the devicetree to control
>> seven segment display on zaius platform.
>>
>> A changelog between versions is included in the individual patches. Please
>> provide feedback.
>>
>> Thanks,
>> Jagha
>>
>> Jaghathiswari Rankappagounder Natarajan (3):
>> drivers: misc: Character device driver for seven segment display
>> drivers: misc: Platform driver for seven segment display support
>> devicetree: Add devicetree changes to support seven segment display on
>> zaius
>>
>> .../bindings/misc/seven_segment_gpio.txt | 23 +++
>> arch/arm/boot/dts/aspeed-bmc-opp-zaius.dts | 8 +
>> drivers/misc/Kconfig | 16 ++
>> drivers/misc/Makefile | 2 +
>> drivers/misc/seven_seg_disp.c | 198
>> ++++++++++++++++++
>> drivers/misc/seven_seg_disp.h | 35 ++++
>> drivers/misc/seven_seg_gpio.c | 225
>> +++++++++++++++++++++
>> 7 files changed, 507 insertions(+)
>> create mode 100644
>> Documentation/devicetree/bindings/misc/seven_segment_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