Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1280004 > unrolled thread
| Started by | Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> |
|---|---|
| First post | 2015-11-30 16:30 +0100 |
| Last post | 2015-11-30 20:50 +0100 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] spi/nor: Fix error message with unrecognized JEDEC Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> - 2015-11-30 16:30 +0100
Re: [PATCH] spi/nor: Fix error message with unrecognized JEDEC Marek Vasut <marex@denx.de> - 2015-11-30 17:50 +0100
Re: [PATCH] spi/nor: Fix error message with unrecognized JEDEC Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> - 2015-11-30 20:50 +0100
| From | Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> |
|---|---|
| Date | 2015-11-30 16:30 +0100 |
| Subject | [PATCH] spi/nor: Fix error message with unrecognized JEDEC |
| Message-ID | <qAyyT-5lO-35@gated-at.bofh.it> |
The error message was: m25p80 spi32766.0: unrecognized JEDEC id bytes: 00, 0, 0 The new error message: m25p80 spi32766.0: unrecognized JEDEC id bytes: 00, 00, 00 Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> --- drivers/mtd/spi-nor/spi-nor.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c index b7c038c75684..7d2b96d5f350 100644 --- a/drivers/mtd/spi-nor/spi-nor.c +++ b/drivers/mtd/spi-nor/spi-nor.c @@ -890,7 +890,7 @@ static const struct flash_info *spi_nor_read_id(struct spi_nor *nor) return &spi_nor_ids[tmp]; } } - dev_err(nor->dev, "unrecognized JEDEC id bytes: %02x, %2x, %2x\n", + dev_err(nor->dev, "unrecognized JEDEC id bytes: %.2x, %.2x, %.2x\n", id[0], id[1], id[2]); return ERR_PTR(-ENODEV); } -- 2.6.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Marek Vasut <marex@denx.de> |
|---|---|
| Date | 2015-11-30 17:50 +0100 |
| Message-ID | <qAzOi-66r-11@gated-at.bofh.it> |
| In reply to | #1280004 |
On Monday, November 30, 2015 at 04:25:11 PM, Ricardo Ribalda Delgado wrote: > The error message was: > > m25p80 spi32766.0: unrecognized JEDEC id bytes: 00, 0, 0 > > The new error message: > > m25p80 spi32766.0: unrecognized JEDEC id bytes: 00, 00, 00 > > Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> > --- > drivers/mtd/spi-nor/spi-nor.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c > index b7c038c75684..7d2b96d5f350 100644 > --- a/drivers/mtd/spi-nor/spi-nor.c > +++ b/drivers/mtd/spi-nor/spi-nor.c > @@ -890,7 +890,7 @@ static const struct flash_info *spi_nor_read_id(struct > spi_nor *nor) return &spi_nor_ids[tmp]; > } > } > - dev_err(nor->dev, "unrecognized JEDEC id bytes: %02x, %2x, %2x\n", > + dev_err(nor->dev, "unrecognized JEDEC id bytes: %.2x, %.2x, %.2x\n", You can change it to %02x to make it consistent, no ? What do you think ? > id[0], id[1], id[2]); > return ERR_PTR(-ENODEV); > } Best regards, Marek Vasut -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> |
|---|---|
| Date | 2015-11-30 20:50 +0100 |
| Message-ID | <qACCu-7Qy-17@gated-at.bofh.it> |
| In reply to | #1280094 |
Hello Marek On Mon, Nov 30, 2015 at 5:46 PM, Marek Vasut <marex@denx.de> wrote: > On Monday, November 30, 2015 at 04:25:11 PM, Ricardo Ribalda Delgado wrote: >> The error message was: >> >> m25p80 spi32766.0: unrecognized JEDEC id bytes: 00, 0, 0 >> >> The new error message: >> >> m25p80 spi32766.0: unrecognized JEDEC id bytes: 00, 00, 00 >> >> Signed-off-by: Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com> >> --- >> drivers/mtd/spi-nor/spi-nor.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c >> index b7c038c75684..7d2b96d5f350 100644 >> --- a/drivers/mtd/spi-nor/spi-nor.c >> +++ b/drivers/mtd/spi-nor/spi-nor.c >> @@ -890,7 +890,7 @@ static const struct flash_info *spi_nor_read_id(struct >> spi_nor *nor) return &spi_nor_ids[tmp]; >> } >> } >> - dev_err(nor->dev, "unrecognized JEDEC id bytes: %02x, %2x, %2x\n", >> + dev_err(nor->dev, "unrecognized JEDEC id bytes: %.2x, %.2x, %.2x\n", > > You can change it to %02x to make it consistent, no ? What do you think ? Sure, no problem, it is just that I am more used to %.2 than %02 :) > >> id[0], id[1], id[2]); >> return ERR_PTR(-ENODEV); >> } > > Best regards, > Marek Vasut -- Ricardo Ribalda -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web