On dim., juil. 30, 2023 at 11:15, Simon Glass <s...@chromium.org> wrote:
> This name is a little confusing since it suggests that it sets up the > sibling block device. In fact it sets up a bootdev for it. Rename the > function to make this clearer. > > Signed-off-by: Simon Glass <s...@chromium.org> Reviewed-by: Mattijs Korpershoek <mkorpersh...@baylibre.com> > --- > > Changes in v2: > - Fix the function name in the header as well > > boot/bootdev-uclass.c | 8 +++++--- > common/usb_storage.c | 2 +- > doc/develop/bootstd.rst | 4 ++-- > drivers/mmc/mmc-uclass.c | 2 +- > drivers/nvme/nvme.c | 2 +- > drivers/scsi/scsi.c | 2 +- > drivers/virtio/virtio-uclass.c | 2 +- > include/bootdev.h | 10 +++++----- > 8 files changed, 17 insertions(+), 15 deletions(-) > > diff --git a/boot/bootdev-uclass.c b/boot/bootdev-uclass.c > index 9660ff75676d..114853ffb72a 100644 > --- a/boot/bootdev-uclass.c > +++ b/boot/bootdev-uclass.c > @@ -262,7 +262,7 @@ static int bootdev_get_suffix_start(struct udevice *dev, > const char *suffix) > return len; > } > > -int bootdev_setup_sibling_blk(struct udevice *blk, const char *drv_name) > +int bootdev_setup_for_sibling_blk(struct udevice *blk, const char *drv_name) > { > struct udevice *parent, *dev; > char dev_name[50]; > @@ -305,7 +305,9 @@ int bootdev_get_sibling_blk(struct udevice *dev, struct > udevice **blkp) > if (device_get_uclass_id(dev) != UCLASS_BOOTDEV) > return -EINVAL; > > - /* This should always work if bootdev_setup_sibling_blk() was used */ > + /* > + * This should always work if bootdev_setup_for_sibling_blk() was used > + */ > len = bootdev_get_suffix_start(dev, ".bootdev"); > ret = device_find_child_by_namelen(parent, dev->name, len, &blk); > if (ret) { > @@ -335,7 +337,7 @@ static int bootdev_get_from_blk(struct udevice *blk, > struct udevice **bootdevp) > if (device_get_uclass_id(blk) != UCLASS_BLK) > return -EINVAL; > > - /* This should always work if bootdev_setup_sibling_blk() was used */ > + /* This should always work if bootdev_setup_for_sibling_blk() was used > */ > len = bootdev_get_suffix_start(blk, ".blk"); > snprintf(dev_name, sizeof(dev_name), "%.*s.%s", len, blk->name, > "bootdev"); > diff --git a/common/usb_storage.c b/common/usb_storage.c > index ac6427577379..85774220ef2a 100644 > --- a/common/usb_storage.c > +++ b/common/usb_storage.c > @@ -246,7 +246,7 @@ static int usb_stor_probe_device(struct usb_device *udev) > if (ret) > return ret; > > - ret = bootdev_setup_sibling_blk(dev, "usb_bootdev"); > + ret = bootdev_setup_for_sibling_blk(dev, "usb_bootdev"); > if (ret) { > int ret2; > > diff --git a/doc/develop/bootstd.rst b/doc/develop/bootstd.rst > index 7a2a69fdfcec..ec3136535783 100644 > --- a/doc/develop/bootstd.rst > +++ b/doc/develop/bootstd.rst > @@ -306,7 +306,7 @@ media device:: > > The bootdev device is typically created automatically in the media uclass' > `post_bind()` method by calling `bootdev_setup_for_dev()` or > -`bootdev_setup_sibling_blk()`. The code typically something like this:: > +`bootdev_setup_for_sibling_blk()`. The code typically something like this:: > > /* dev is the Ethernet device */ > ret = bootdev_setup_for_dev(dev, "eth_bootdev"); > @@ -316,7 +316,7 @@ The bootdev device is typically created automatically in > the media uclass' > or:: > > /* blk is the block device (child of MMC device) > - ret = bootdev_setup_sibling_blk(blk, "mmc_bootdev"); > + ret = bootdev_setup_for_sibling_blk(blk, "mmc_bootdev"); > if (ret) > return log_msg_ret("bootdev", ret); > > diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c > index 01d9b0201f2c..0e157672eae0 100644 > --- a/drivers/mmc/mmc-uclass.c > +++ b/drivers/mmc/mmc-uclass.c > @@ -421,7 +421,7 @@ int mmc_bind(struct udevice *dev, struct mmc *mmc, const > struct mmc_config *cfg) > mmc->cfg = cfg; > mmc->priv = dev; > > - ret = bootdev_setup_sibling_blk(bdev, "mmc_bootdev"); > + ret = bootdev_setup_for_sibling_blk(bdev, "mmc_bootdev"); > if (ret) > return log_msg_ret("bootdev", ret); > > diff --git a/drivers/nvme/nvme.c b/drivers/nvme/nvme.c > index a7add66ab4d1..20dc910d8a33 100644 > --- a/drivers/nvme/nvme.c > +++ b/drivers/nvme/nvme.c > @@ -910,7 +910,7 @@ int nvme_init(struct udevice *udev) > if (ret) > goto free_id; > > - ret = bootdev_setup_sibling_blk(ns_udev, "nvme_bootdev"); > + ret = bootdev_setup_for_sibling_blk(ns_udev, "nvme_bootdev"); > if (ret) > return log_msg_ret("bootdev", ret); > > diff --git a/drivers/scsi/scsi.c b/drivers/scsi/scsi.c > index 6caeb3fcdd0d..0a3420b7fbc2 100644 > --- a/drivers/scsi/scsi.c > +++ b/drivers/scsi/scsi.c > @@ -607,7 +607,7 @@ static int do_scsi_scan_one(struct udevice *dev, int id, > int lun, bool verbose) > /* TODO: undo create */ > return log_msg_ret("pro", ret); > > - ret = bootdev_setup_sibling_blk(bdev, "scsi_bootdev"); > + ret = bootdev_setup_for_sibling_blk(bdev, "scsi_bootdev"); > if (ret) > return log_msg_ret("bd", ret); > > diff --git a/drivers/virtio/virtio-uclass.c b/drivers/virtio/virtio-uclass.c > index 31bb21c534e5..87323ab6d193 100644 > --- a/drivers/virtio/virtio-uclass.c > +++ b/drivers/virtio/virtio-uclass.c > @@ -248,7 +248,7 @@ static int virtio_uclass_post_probe(struct udevice *udev) > device_set_name_alloced(vdev); > > if (uc_priv->device == VIRTIO_ID_BLOCK && !IS_ENABLED(CONFIG_SANDBOX)) { > - ret = bootdev_setup_sibling_blk(vdev, "virtio_bootdev"); > + ret = bootdev_setup_for_sibling_blk(vdev, "virtio_bootdev"); > if (ret) > return log_msg_ret("bootdev", ret); > } > diff --git a/include/bootdev.h b/include/bootdev.h > index 1533adfe5065..848233187f87 100644 > --- a/include/bootdev.h > +++ b/include/bootdev.h > @@ -371,7 +371,7 @@ int bootdev_next_prio(struct bootflow_iter *iter, struct > udevice **devp); > /** > * bootdev_setup_for_dev() - Bind a new bootdev device (deprecated) > * > - * Please use bootdev_setup_sibling_blk() instead since it supports multiple > + * Please use bootdev_setup_for_sibling_blk() instead since it supports > multiple > * (child) block devices for each media device. > * > * Creates a bootdev device as a child of @parent. This should be called from > @@ -386,7 +386,7 @@ int bootdev_next_prio(struct bootflow_iter *iter, struct > udevice **devp); > int bootdev_setup_for_dev(struct udevice *parent, const char *drv_name); > > /** > - * bootdev_setup_for_blk() - Bind a new bootdev device for a blk device > + * bootdev_setup_for_sibling_blk() - Bind a new bootdev device for a blk > device > * > * Creates a bootdev device as a sibling of @blk. This should be called from > * the driver's bind() method or its uclass' post_bind() method, at the same > @@ -398,7 +398,7 @@ int bootdev_setup_for_dev(struct udevice *parent, const > char *drv_name); > * @drv_name: Name of bootdev driver to bind > * Return: 0 if OK, -ve on error > */ > -int bootdev_setup_sibling_blk(struct udevice *blk, const char *drv_name); > +int bootdev_setup_for_sibling_blk(struct udevice *blk, const char *drv_name); > > /** > * bootdev_get_sibling_blk() - Locate the block device for a bootdev > @@ -428,8 +428,8 @@ static inline int bootdev_setup_for_dev(struct udevice > *parent, > return 0; > } > > -static inline int bootdev_setup_sibling_blk(struct udevice *blk, > - const char *drv_name) > +static inline int bootdev_setup_for_sibling_blk(struct udevice *blk, > + const char *drv_name) > { > return 0; > } > -- > 2.41.0.487.g6d72f3e995-goog