Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1530676

Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss gigabit ethernet driver

Show all headers | View raw

Hi, Lino,

...

> @@ -0,0 +1,28 @@
> +config NET_VENDOR_ALACRITECH
> +        bool "Alacritech devices"
> +        default y
> +        ---help---
> +          If you have a network (Ethernet) card belonging to this class, say Y.
> +
> +          Note that the answer to this question doesn't directly affect the
> +          kernel: saying N will just cause the configurator to skip all

Shouldn't it be "Alacritech devices" here, as appears earlier ?

> +          the questions about Renesas devices. If you say Y, you will be asked
> +          for your specific device in the following questions.
> +

...
...
...
> +struct slic_device {
> +       struct pci_dev *pdev;
...
> +       bool promisc;

Seems that the autoneg boolean is not used anywhere, apart from
setting it once to true in
the slic_set_link_autoneg() method. Apart from this member it is not
accessed anywhere, so it seems it should be removed.

> +       bool autoneg;
> +       int speed;
...
...

> +static int slic_load_rcvseq_firmware(struct slic_device *sdev)
> +{
> +       const struct firmware *fw;
> +       const char *file;
> +       u32 codelen;
> +       int idx = 0;
> +       u32 instr;
> +       u32 addr;
> +       int err;
> +
...
> +       /* Do an initial sanity check concerning firmware size now. A further
> +        * check follows below.
> +        */
> +       if (fw->size < SLIC_FIRMWARE_MIN_SIZE) {
> +               dev_err(&sdev->pdev->dev,
> +                       "invalid firmware size %zu (min %u expected)\n",
> +                       fw->size, SLIC_FIRMWARE_MIN_SIZE);
> +               err = -EINVAL;

in the release label, always 0 is returned:

> +               goto release;
> +       }
> +
> +       codelen = slic_read_dword_from_firmware(fw, &idx);
> +
> +       /* do another sanity check against firmware size */
> +       if ((codelen + 4) > fw->size) {
> +               dev_err(&sdev->pdev->dev,
> +                       "invalid rcv-sequencer firmware size %zu\n", fw->size);
> +               err = -EINVAL;

Again, in the release label, always 0 is returned:

> +               goto release;
> +       }
> +
>
> +release:
> +       release_firmware(fw);
> +
> +       return 0;
> +}
> +

Regards,
Rami Rosen

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Gigabit ethernet driver for Alacritechs SLIC devices (v3) Lino Sanfilippo <LinoSanfilippo@gmx.de> - 2016-11-26 13:30 +0100
  [PATCH v3 net-next 2/2] MAINTAINERS: add entry for slicoss ethernet driver Lino Sanfilippo <LinoSanfilippo@gmx.de> - 2016-11-26 13:30 +0100
  Re: Gigabit ethernet driver for Alacritechs SLIC devices (v3) Lino Sanfilippo <LinoSanfilippo@gmx.de> - 2016-11-26 13:40 +0100
  Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Rami Rosen <roszenrami@gmail.com> - 2016-11-26 16:50 +0100
    Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Lino Sanfilippo <LinoSanfilippo@gmx.de> - 2016-11-27 02:10 +0100
  Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Markus Böhme <markus.boehme@mailbox.org> - 2016-11-27 19:10 +0100
    Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Lino Sanfilippo <LinoSanfilippo@gmx.de> - 2016-11-28 21:50 +0100
      Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Markus Böhme <markus.boehme@mailbox.org> - 2016-11-29 13:20 +0100
  Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Florian Fainelli <f.fainelli@gmail.com> - 2016-11-28 06:00 +0100
    Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Lino Sanfilippo <LinoSanfilippo@gmx.de> - 2016-11-28 22:50 +0100
      Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Florian Fainelli <f.fainelli@gmail.com> - 2016-11-29 18:20 +0100
        Re: [PATCH v3 net-next 1/2] net: ethernet: slicoss: add slicoss  gigabit ethernet driver Lino Sanfilippo <LinoSanfilippo@gmx.de> - 2016-11-30 21:00 +0100

csiph-web