Re: [PATCH] staging: sm750fb: Fix printk() style warning

From: Edward Lipinsky
Date: Fri Oct 14 2016 - 01:18:40 EST


On Thu, Oct 13, 2016 at 05:39:22PM +0300, Mike Rapoport wrote:
> On Mon, Oct 10, 2016 at 07:31:46AM -0700, Edward Lipinsky wrote:
> >
> > Thanks, that makes sense. I tried deleting the if statement and printk()
> > from ddk750_help.c, and adding the following in lynxfb_pci_probe() after
> > calling hw_sm750_map() and checking for errors:
> >
> > if (sm750_dev->revid == 0xfe)
> > dev_info(&pdev->dev, "found sm750le\n");
> >
> > The driver compiles, loads and unloads ok with this code.
> >
> > Does this look correct?
>
> Yep, seems just right.
>
> > Regards,
> > Ed L.

Good, thanks. I will submit an updated patch.

Regards,
Ed L.