linux-kernel - Re: [GIT PULL for v3.17] si2165 firmware changes

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 3 Oct 2014 13:04:48 -0700
From:	Linus Torvalds <torvalds@...ux-foundation.org>
To:	Mauro Carvalho Chehab <mchehab@....samsung.com>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	Linux Media Mailing List <linux-media@...r.kernel.org>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [GIT PULL for v3.17] si2165 firmware changes

Yeah, this is pure crap. It doesn't even compile.

  drivers/media/dvb-frontends/si2165.c:1063:17: error: expected ‘,’ or
‘;’ before ‘SI2165_FIRMWARE’
   MODULE_FIRMWARE(SI2165_FIRMWARE);

because it should presumably say "SI2165_FIRMWARE_REV_D" now.

Why the f*ck do you send me totally untested crap?

                 Linus

On Fri, Oct 3, 2014 at 11:05 AM, Mauro Carvalho Chehab
<mchehab@....samsung.com> wrote:
> Hi Linus,
>
> Please pull from:
>   git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/topic/si2165-v3.17-rc8
>
> For some changes at the si2165 firmware name and the removal of an extra
> unneeded header added artificially via the script that extracts it from
> the original driver provided by the manufacturer.
>
> The si2165 is a new driver that was added for v3.17. There are two issues
> with the current firmware format:
>
> - The firmware only covers one specific revision of the chipset
>   (Rev. D). We'll be adding support for another revision for v3.18, so
>   it would be better to rename the firmware file to reflect the revision
>   on its name:
>
>         -#define SI2165_FIRMWARE "dvb-demod-si2165.fw"
>         +#define SI2165_FIRMWARE_REV_D "dvb-demod-si2165-d.fw"
>
> - Instead of containing a single blob with the firmware, the file
>   also contains some meta-data that could be determined on some other way
>   directly by the driver.
>
> The script that gets the firmware from the Internet was also updated
> accordingly to not add the extra header.
>
> Thanks!
> Mauro
>
>
>
> The following changes since commit 90a5dbef1a66e9f55b76ccb83c0ef27c0bd87c27:
>
>   Revert "[media] media: em28xx - remove reset_resume interface" (2014-09-28 22:25:24 -0300)
>
> are available in the git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/mchehab/linux-media tags/media/topic/si2165-v3.17-rc8
>
> for you to fetch changes up to 3173fbdce9e41fc4fabe0b3dedd99c615f47dbdd:
>
>   [media] [V2,2/2] si2165: do load firmware without extra header (2014-10-02 18:18:52 -0300)
>
> ----------------------------------------------------------------
> topic/si2165 fixes for v3.17-rc8
>
> ----------------------------------------------------------------
> Matthias Schwarzott (2):
>       [media] [V2, 1/2] get_dvb_firmware: si2165: drop the extra header from the firmware
>       [media] [V2,2/2] si2165: do load firmware without extra header
>
>  Documentation/dvb/get_dvb_firmware        |  20 ++----
>  drivers/media/dvb-frontends/Kconfig       |   1 +
>  drivers/media/dvb-frontends/si2165.c      | 107 ++++++++++++++++++------------
>  drivers/media/dvb-frontends/si2165_priv.h |   2 +-
>  4 files changed, 71 insertions(+), 59 deletions(-)
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/