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
| ||
|
Message-ID: <7eb2e242-1adf-e813-f1cf-f32b672e7459@wedev4u.fr> Date: Wed, 27 Dec 2017 21:28:39 +0100 From: Cyrille Pitchen <cyrille.pitchen@...ev4u.fr> To: Romain Porte <romain.porte@...ia.com> Cc: linux-mtd@...ts.infradead.org, linux-kernel@...r.kernel.org, alexander.sverdlin@...ia.com, pascal.fabreges@...ia.com Subject: Re: [PATCH v2] mtd: spi-nor: Added ISSI is25lp080d support Hi Romain, Le 27/12/2017 à 15:46, Romain Porte a écrit : > On 27/12/2017 14:58, Romain Porte wrote: >> diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c >> index 8bafd462f..18e826d36 100644 >> --- a/drivers/mtd/spi-nor/spi-nor.c >> +++ b/drivers/mtd/spi-nor/spi-nor.c >> @@ -1050,6 +1050,8 @@ static const struct flash_info spi_nor_ids[] = { >> SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) }, >> { "is25lp128", INFO(0x9d6018, 0, 64 * 1024, 256, >> SECT_4K | SPI_NOR_DUAL_READ) }, >> + { "is25lp080d", INFO(0x9d6014, 0, 64 * 1024, 16, >> + SECT_4K | SPI_NOR_DUAL_READ | SPI_NOR_QUAD_READ) }, >> > Please note that the new SPI_NOR_DUAL_READ and SPI_NOR_QUAD_READ options have not been tested on our hardware for the moment. The Tested-by flag applies only for the previous configuration without these two new flags. We may be able to test these additional flags in the future, but they were added only from your (Cyrille's) review from previous patch. > > Please let me know if you want to wait for our hardware test results before merging into spi-nor/next, or if you are confident enough to merge it as this. Based on the manufacturer's datasheet, I'd rather add the SPI_NOR_{DUAL,QUAD}_READ flags now than later, even if not tested yet. Otherwise, it would never be tested. Of course, if you can test on your side, it would be even better! However, who is the author of this patch? There is no Signed-off-by tag. So I shouldn't take this patch without this tag. Besides, the subject line should be something like: "mtd: spi-nor: Add ISSI is25lp080d support" >From Documentation/process/submitting-patches.rst: """ Describe your changes in imperative mood, e.g. "make xyzzy do frotz" instead of "[This patch] makes xyzzy do frotz" or "[I] changed xyzzy to do frotz", as if you are giving orders to the codebase to change its behaviour. """ Finally, just to let you know if you're not very familiar with posting to linux mailing lists: some may complain that you didn't configure your mail client to truncate lines at the 75th or 80th column ;) It's not a big deal but it's still better to follow the guide lines. Best regards, Cyrille > > Romain. >
Powered by blists - more mailing lists