On Wed, Apr 10, 2024 at 10:21:02AM +0300, Andy Shevchenko wrote:
> Convert driver to use memory mapped IO accessors.
>
> Signed-off-by: Andy Shevchenko <andy.shevchenko@xxxxxxxxx>
Acked-by: William Breathitt Gray <wbg@xxxxxxxxxx>
A minor suggestion below, but I find this patch accepted as-is.
> static int sch_gpio_probe(struct platform_device *pdev)
> {
> + struct device *dev = &pdev->dev;
In general I think this is a good variable to define to simplify all the
&pdev->dev appearing throughout this callback, but I'd rather have seen
it as its own patch so we could change all the other uses of &pdev->dev
at once without distracting from the memory-mapped I/O change of this
particular patch. Not really necessary, but maybe at some point in the
future a follow-up patch doing such a cleanup would be nice.
William Breathitt Gray
Attachment:
signature.asc
Description: PGP signature
