提交 640985ec 编写于 作者: A Alexander Aring 提交者: David S. Miller

mac802154: at86rf230: add hw flags and merge ops

This patch adds new mac802154 hw flags for transmit power, csma and
listen before transmit (lbt). These flags indicates that the transceiver
supports these features. If the flags are set and the driver doesn't
implement the necessary functions, then ieee802154_register_device
returns -ENOSYS "Function not implemented".

This patch merges also all at86rf230 operations into one operations structure
and set the right hw flags for the at86rf230 transceivers.
Signed-off-by: NAlexander Aring <alex.aring@gmail.com>
Signed-off-by: NDavid S. Miller <davem@davemloft.net>
上级 1598c36a
...@@ -784,7 +784,7 @@ at86rf230_set_hw_addr_filt(struct ieee802154_dev *dev, ...@@ -784,7 +784,7 @@ at86rf230_set_hw_addr_filt(struct ieee802154_dev *dev,
} }
static int static int
at86rf212_set_txpower(struct ieee802154_dev *dev, int db) at86rf230_set_txpower(struct ieee802154_dev *dev, int db)
{ {
struct at86rf230_local *lp = dev->priv; struct at86rf230_local *lp = dev->priv;
...@@ -803,7 +803,7 @@ at86rf212_set_txpower(struct ieee802154_dev *dev, int db) ...@@ -803,7 +803,7 @@ at86rf212_set_txpower(struct ieee802154_dev *dev, int db)
} }
static int static int
at86rf212_set_lbt(struct ieee802154_dev *dev, bool on) at86rf230_set_lbt(struct ieee802154_dev *dev, bool on)
{ {
struct at86rf230_local *lp = dev->priv; struct at86rf230_local *lp = dev->priv;
...@@ -811,7 +811,7 @@ at86rf212_set_lbt(struct ieee802154_dev *dev, bool on) ...@@ -811,7 +811,7 @@ at86rf212_set_lbt(struct ieee802154_dev *dev, bool on)
} }
static int static int
at86rf212_set_cca_mode(struct ieee802154_dev *dev, u8 mode) at86rf230_set_cca_mode(struct ieee802154_dev *dev, u8 mode)
{ {
struct at86rf230_local *lp = dev->priv; struct at86rf230_local *lp = dev->priv;
...@@ -819,7 +819,7 @@ at86rf212_set_cca_mode(struct ieee802154_dev *dev, u8 mode) ...@@ -819,7 +819,7 @@ at86rf212_set_cca_mode(struct ieee802154_dev *dev, u8 mode)
} }
static int static int
at86rf212_set_cca_ed_level(struct ieee802154_dev *dev, s32 level) at86rf230_set_cca_ed_level(struct ieee802154_dev *dev, s32 level)
{ {
struct at86rf230_local *lp = dev->priv; struct at86rf230_local *lp = dev->priv;
int desens_steps; int desens_steps;
...@@ -833,7 +833,7 @@ at86rf212_set_cca_ed_level(struct ieee802154_dev *dev, s32 level) ...@@ -833,7 +833,7 @@ at86rf212_set_cca_ed_level(struct ieee802154_dev *dev, s32 level)
} }
static int static int
at86rf212_set_csma_params(struct ieee802154_dev *dev, u8 min_be, u8 max_be, at86rf230_set_csma_params(struct ieee802154_dev *dev, u8 min_be, u8 max_be,
u8 retries) u8 retries)
{ {
struct at86rf230_local *lp = dev->priv; struct at86rf230_local *lp = dev->priv;
...@@ -854,7 +854,7 @@ at86rf212_set_csma_params(struct ieee802154_dev *dev, u8 min_be, u8 max_be, ...@@ -854,7 +854,7 @@ at86rf212_set_csma_params(struct ieee802154_dev *dev, u8 min_be, u8 max_be,
} }
static int static int
at86rf212_set_frame_retries(struct ieee802154_dev *dev, s8 retries) at86rf230_set_frame_retries(struct ieee802154_dev *dev, s8 retries)
{ {
struct at86rf230_local *lp = dev->priv; struct at86rf230_local *lp = dev->priv;
int rc = 0; int rc = 0;
...@@ -878,22 +878,12 @@ static struct ieee802154_ops at86rf230_ops = { ...@@ -878,22 +878,12 @@ static struct ieee802154_ops at86rf230_ops = {
.start = at86rf230_start, .start = at86rf230_start,
.stop = at86rf230_stop, .stop = at86rf230_stop,
.set_hw_addr_filt = at86rf230_set_hw_addr_filt, .set_hw_addr_filt = at86rf230_set_hw_addr_filt,
}; .set_txpower = at86rf230_set_txpower,
.set_lbt = at86rf230_set_lbt,
static struct ieee802154_ops at86rf212_ops = { .set_cca_mode = at86rf230_set_cca_mode,
.owner = THIS_MODULE, .set_cca_ed_level = at86rf230_set_cca_ed_level,
.xmit = at86rf230_xmit, .set_csma_params = at86rf230_set_csma_params,
.ed = at86rf230_ed, .set_frame_retries = at86rf230_set_frame_retries,
.set_channel = at86rf230_channel,
.start = at86rf230_start,
.stop = at86rf230_stop,
.set_hw_addr_filt = at86rf230_set_hw_addr_filt,
.set_txpower = at86rf212_set_txpower,
.set_lbt = at86rf212_set_lbt,
.set_cca_mode = at86rf212_set_cca_mode,
.set_cca_ed_level = at86rf212_set_cca_ed_level,
.set_csma_params = at86rf212_set_csma_params,
.set_frame_retries = at86rf212_set_frame_retries,
}; };
static void at86rf230_irqwork(struct work_struct *work) static void at86rf230_irqwork(struct work_struct *work)
...@@ -1048,7 +1038,6 @@ static int at86rf230_probe(struct spi_device *spi) ...@@ -1048,7 +1038,6 @@ static int at86rf230_probe(struct spi_device *spi)
work_func_t irq_worker; work_func_t irq_worker;
int rc, irq_type; int rc, irq_type;
const char *chip; const char *chip;
struct ieee802154_ops *ops = NULL;
if (!spi->irq) { if (!spi->irq) {
dev_err(&spi->dev, "no IRQ specified\n"); dev_err(&spi->dev, "no IRQ specified\n");
...@@ -1084,9 +1073,25 @@ static int at86rf230_probe(struct spi_device *spi) ...@@ -1084,9 +1073,25 @@ static int at86rf230_probe(struct spi_device *spi)
usleep_range(120, 240); usleep_range(120, 240);
} }
dev = ieee802154_alloc_device(sizeof(*lp), &at86rf230_ops);
if (!dev)
return -ENOMEM;
lp = dev->priv;
lp->dev = dev;
lp->part = part;
lp->vers = version;
lp->spi = spi;
dev->parent = &spi->dev;
dev->extra_tx_headroom = 0;
dev->flags = IEEE802154_HW_OMIT_CKSUM | IEEE802154_HW_AACK |
IEEE802154_HW_TXPOWER | IEEE802154_HW_CSMA;
rc = __at86rf230_detect_device(spi, &man_id, &part, &version); rc = __at86rf230_detect_device(spi, &man_id, &part, &version);
if (rc < 0) if (rc < 0)
return rc; goto free_dev;
if (man_id != 0x001f) { if (man_id != 0x001f) {
dev_err(&spi->dev, "Non-Atmel dev found (MAN_ID %02x %02x)\n", dev_err(&spi->dev, "Non-Atmel dev found (MAN_ID %02x %02x)\n",
...@@ -1097,44 +1102,31 @@ static int at86rf230_probe(struct spi_device *spi) ...@@ -1097,44 +1102,31 @@ static int at86rf230_probe(struct spi_device *spi)
switch (part) { switch (part) {
case 2: case 2:
chip = "at86rf230"; chip = "at86rf230";
rc = -ENOTSUPP;
/* FIXME: should be easy to support; */ /* FIXME: should be easy to support; */
break; break;
case 3: case 3:
chip = "at86rf231"; chip = "at86rf231";
ops = &at86rf230_ops;
break; break;
case 7: case 7:
chip = "at86rf212"; chip = "at86rf212";
if (version == 1) if (version == 1)
ops = &at86rf212_ops; dev->flags |= IEEE802154_HW_LBT;
else
rc = -ENOTSUPP;
break; break;
case 11: case 11:
chip = "at86rf233"; chip = "at86rf233";
ops = &at86rf230_ops;
break; break;
default: default:
chip = "UNKNOWN"; chip = "UNKNOWN";
rc = -ENOTSUPP;
break; break;
} }
dev_info(&spi->dev, "Detected %s chip version %d\n", chip, version); dev_info(&spi->dev, "Detected %s chip version %d\n", chip, version);
if (!ops) if (rc < 0)
return -ENOTSUPP; goto free_dev;
dev = ieee802154_alloc_device(sizeof(*lp), ops);
if (!dev)
return -ENOMEM;
lp = dev->priv;
lp->dev = dev;
lp->part = part;
lp->vers = version;
lp->spi = spi;
dev->parent = &spi->dev;
dev->extra_tx_headroom = 0;
dev->flags = IEEE802154_HW_OMIT_CKSUM | IEEE802154_HW_AACK;
irq_type = irq_get_trigger_type(spi->irq); irq_type = irq_get_trigger_type(spi->irq);
if (!irq_type) if (!irq_type)
...@@ -1185,6 +1177,7 @@ static int at86rf230_probe(struct spi_device *spi) ...@@ -1185,6 +1177,7 @@ static int at86rf230_probe(struct spi_device *spi)
err_hw_init: err_hw_init:
flush_work(&lp->irqwork); flush_work(&lp->irqwork);
mutex_destroy(&lp->bmux); mutex_destroy(&lp->bmux);
free_dev:
ieee802154_free_device(lp->dev); ieee802154_free_device(lp->dev);
return rc; return rc;
......
...@@ -80,6 +80,25 @@ struct ieee802154_dev { ...@@ -80,6 +80,25 @@ struct ieee802154_dev {
#define IEEE802154_HW_OMIT_CKSUM 0x00000001 #define IEEE802154_HW_OMIT_CKSUM 0x00000001
/* Indicates that receiver will autorespond with ACK frames. */ /* Indicates that receiver will autorespond with ACK frames. */
#define IEEE802154_HW_AACK 0x00000002 #define IEEE802154_HW_AACK 0x00000002
/* Indicates that transceiver will support transmit power setting. */
#define IEEE802154_HW_TXPOWER 0x00000004
/* Indicates that transceiver will support listen before transmit. */
#define IEEE802154_HW_LBT 0x00000008
/* Indicates that transceiver will support cca mode setting. */
#define IEEE802154_HW_CCA_MODE 0x00000010
/* Indicates that transceiver will support cca ed level setting. */
#define IEEE802154_HW_CCA_ED_LEVEL 0x00000020
/* Indicates that transceiver will support csma (max_be, min_be, csma retries)
* settings. */
#define IEEE802154_HW_CSMA_PARAMS 0x00000040
/* Indicates that transceiver will support ARET frame retries setting. */
#define IEEE802154_HW_FRAME_RETRIES 0x00000080
/* This groups the most common CSMA support fields into one. */
#define IEEE802154_HW_CSMA (IEEE802154_HW_CCA_MODE | \
IEEE802154_HW_CCA_ED_LEVEL | \
IEEE802154_HW_CSMA_PARAMS | \
IEEE802154_HW_FRAME_RETRIES)
/* struct ieee802154_ops - callbacks from mac802154 to the driver /* struct ieee802154_ops - callbacks from mac802154 to the driver
* *
......
...@@ -304,29 +304,61 @@ EXPORT_SYMBOL(ieee802154_free_device); ...@@ -304,29 +304,61 @@ EXPORT_SYMBOL(ieee802154_free_device);
int ieee802154_register_device(struct ieee802154_dev *dev) int ieee802154_register_device(struct ieee802154_dev *dev)
{ {
struct mac802154_priv *priv = mac802154_to_priv(dev); struct mac802154_priv *priv = mac802154_to_priv(dev);
int rc = -ENOMEM; int rc = -ENOSYS;
if (dev->flags & IEEE802154_HW_TXPOWER) {
if (!priv->ops->set_txpower)
goto out;
priv->phy->set_txpower = mac802154_set_txpower;
}
if (dev->flags & IEEE802154_HW_LBT) {
if (!priv->ops->set_lbt)
goto out;
priv->phy->set_lbt = mac802154_set_lbt;
}
if (dev->flags & IEEE802154_HW_CCA_MODE) {
if (!priv->ops->set_cca_mode)
goto out;
priv->phy->set_cca_mode = mac802154_set_cca_mode;
}
if (dev->flags & IEEE802154_HW_CCA_ED_LEVEL) {
if (!priv->ops->set_cca_ed_level)
goto out;
priv->phy->set_cca_ed_level = mac802154_set_cca_ed_level;
}
if (dev->flags & IEEE802154_HW_CSMA_PARAMS) {
if (!priv->ops->set_csma_params)
goto out;
priv->phy->set_csma_params = mac802154_set_csma_params;
}
if (dev->flags & IEEE802154_HW_FRAME_RETRIES) {
if (!priv->ops->set_frame_retries)
goto out;
priv->phy->set_frame_retries = mac802154_set_frame_retries;
}
priv->dev_workqueue = priv->dev_workqueue =
create_singlethread_workqueue(wpan_phy_name(priv->phy)); create_singlethread_workqueue(wpan_phy_name(priv->phy));
if (!priv->dev_workqueue) if (!priv->dev_workqueue) {
rc = -ENOMEM;
goto out; goto out;
}
wpan_phy_set_dev(priv->phy, priv->hw.parent); wpan_phy_set_dev(priv->phy, priv->hw.parent);
priv->phy->add_iface = mac802154_add_iface; priv->phy->add_iface = mac802154_add_iface;
priv->phy->del_iface = mac802154_del_iface; priv->phy->del_iface = mac802154_del_iface;
if (priv->ops->set_txpower)
priv->phy->set_txpower = mac802154_set_txpower;
if (priv->ops->set_lbt)
priv->phy->set_lbt = mac802154_set_lbt;
if (priv->ops->set_cca_mode)
priv->phy->set_cca_mode = mac802154_set_cca_mode;
if (priv->ops->set_cca_ed_level)
priv->phy->set_cca_ed_level = mac802154_set_cca_ed_level;
if (priv->ops->set_csma_params)
priv->phy->set_csma_params = mac802154_set_csma_params;
if (priv->ops->set_frame_retries)
priv->phy->set_frame_retries = mac802154_set_frame_retries;
rc = wpan_phy_register(priv->phy); rc = wpan_phy_register(priv->phy);
if (rc < 0) if (rc < 0)
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册