[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CANiq72=3jtTn97Kkh1LBhrM4Kcs76b-SZnJ3=b+bNjX-BQM6vw@mail.gmail.com>
Date: Sat, 11 Dec 2021 19:38:31 +0100
From: Miguel Ojeda <miguel.ojeda.sandonis@...il.com>
To: Charles Mirabile <cmirabil@...hat.com>
Cc: linux-kernel <linux-kernel@...r.kernel.org>,
Miguel Ojeda <ojeda@...nel.org>,
Serge Schneider <serge@...pberrypi.org>,
Stefan Wahren <stefan.wahren@...e.com>,
Nicolas Saenz Julienne <nsaenzju@...hat.com>,
Mattias Brugger <mbrugger@...e.com>,
linux-rpi-kernel@...ts.infradead.org,
Linux ARM <linux-arm-kernel@...ts.infradead.org>,
fedora-rpi@...glegroups.com, Mwesigwa Guma <mguma@...hat.com>,
Joel Savitz <jsavitz@...hat.com>
Subject: Re: [PATCH V5 3/6] drivers/auxdisplay: sensehat: Raspberry Pi Sense
HAT display driver
Hi Charles, Mwesigwa, Joel, Serge,
On Fri, Dec 10, 2021 at 11:11 PM Charles Mirabile <cmirabil@...hat.com> wrote:
>
> Signed-off-by: Charles Mirabile <cmirabil@...hat.com>
> Co-developed-by: Mwesigwa Guma <mguma@...hat.com>
> Signed-off-by: Mwesigwa Guma <mguma@...hat.com>
> Co-developed-by: Joel Savitz <jsavitz@...hat.com>
> Signed-off-by: Joel Savitz <jsavitz@...hat.com>
The "submitting author" should be the last one, i.e.:
Co-developed-by: Mwesigwa Guma <mguma@...hat.com>
Signed-off-by: Mwesigwa Guma <mguma@...hat.com>
Co-developed-by: Joel Savitz <jsavitz@...hat.com>
Signed-off-by: Joel Savitz <jsavitz@...hat.com>
Signed-off-by: Charles Mirabile <cmirabil@...hat.com>
> +config SENSEHAT_DISPLAY
> + tristate "Raspberry pi Sense HAT display driver"
pi -> Pi
> +static int sensehat_update_display(struct sensehat *sensehat);
Can the function be directly defined instead?
> + if (*f_pos >= VMEM_SIZE)
> + return 0;
> + if (*f_pos + count > VMEM_SIZE)
> + count = VMEM_SIZE - *f_pos;
`min` / `min_t`?
> + if (ret < 0)
> + dev_err(sensehat->dev,
> + "Update to 8x8 LED matrix display failed");
Could this happen a lot of times? Is it expected to happen under some
condition or should never happen?
Cheers,
Miguel
Powered by blists - more mailing lists