On Mon, 2014-09-15 at 22:55 +0800, Huang Shijie wrote:
> On Sun, Sep 14, 2014 at 06:11:16PM +0100, Ben Hutchings wrote:
> > There is currently no useful way to override the default
> > implementation of this operation. The returned struct spi_device_id
> > must have a pointer to struct flash_info in its private data, but this
> > structure is defined inside spi-nor.
> >
> > Signed-off-by: Ben Hutchings <ben@xxxxxxxxxxxxxxx>
> > ---
> > drivers/mtd/spi-nor/spi-nor.c | 4 +---
> > include/linux/mtd/spi-nor.h | 3 ---
> > 2 files changed, 1 insertion(+), 6 deletions(-)
> >
> > diff --git a/drivers/mtd/spi-nor/spi-nor.c b/drivers/mtd/spi-nor/spi-nor.c
> > index 6b1bda2..6f699c5 100644
> > --- a/drivers/mtd/spi-nor/spi-nor.c
> > +++ b/drivers/mtd/spi-nor/spi-nor.c
> > @@ -902,8 +902,6 @@ static int spi_nor_check(struct spi_nor *nor)
> > return -EINVAL;
> > }
> >
> > - if (!nor->read_id)
> > - nor->read_id = spi_nor_read_id;
> > if (!nor->wait_till_ready)
> > nor->wait_till_ready = spi_nor_wait_till_ready;
> >
> > @@ -952,7 +950,7 @@ int spi_nor_scan(struct spi_nor *nor, const struct spi_device_id *id,
> > if (info->jedec_id) {
> > const struct spi_device_id *jid;
> >
> > - jid = nor->read_id(nor);
> > + jid = spi_nor_read_id(nor);
> > if (IS_ERR(jid)) {
> > return PTR_ERR(jid);
> > } else if (jid != id) {
> > diff --git a/include/linux/mtd/spi-nor.h b/include/linux/mtd/spi-nor.h
> > index 5ec84cc..66af67a 100644
> > --- a/include/linux/mtd/spi-nor.h
> > +++ b/include/linux/mtd/spi-nor.h
> > @@ -139,8 +139,6 @@ enum spi_nor_ops {
> > * @write_xfer: [OPTIONAL] the writefundamental primitive
> > * @read_reg: [DRIVER-SPECIFIC] read out the register
> > * @write_reg: [DRIVER-SPECIFIC] write data to the register
> > - * @read_id: [REPLACEABLE] read out the ID data, and find
> > - * the proper spi_device_id
> If we remove this hook, i guess we will add it back in the later.
> Other new drivers will use this hook.
>
> so i suggest to keep it as it is.
It may well be needed, but it can't be implemented in its current form.
Ben.
--
Ben Hutchings
Make three consecutive correct guesses and you will be considered an expert.
Attachment:
signature.asc
Description: This is a digitally signed message part
