From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from phobos.denx.de (phobos.denx.de [85.214.62.61]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5C055ECAAD8 for ; Wed, 14 Sep 2022 02:09:16 +0000 (UTC) Received: from h2850616.stratoserver.net (localhost [IPv6:::1]) by phobos.denx.de (Postfix) with ESMTP id 1755E847EF; Wed, 14 Sep 2022 04:09:14 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=u-boot-bounces@lists.denx.de Authentication-Results: phobos.denx.de; dkim=pass (2048-bit key; unprotected) header.d=linaro.org header.i=@linaro.org header.b="SUGUO6K5"; dkim-atps=neutral Received: by phobos.denx.de (Postfix, from userid 109) id 013DD8454A; Wed, 14 Sep 2022 04:09:12 +0200 (CEST) Received: from mail-pg1-x530.google.com (mail-pg1-x530.google.com [IPv6:2607:f8b0:4864:20::530]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits)) (No client certificate requested) by phobos.denx.de (Postfix) with ESMTPS id A3B37847EF for ; Wed, 14 Sep 2022 04:09:05 +0200 (CEST) Authentication-Results: phobos.denx.de; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: phobos.denx.de; spf=pass smtp.mailfrom=takahiro.akashi@linaro.org Received: by mail-pg1-x530.google.com with SMTP id t65so13026707pgt.2 for ; Tue, 13 Sep 2022 19:09:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date:from:to:cc :subject:date; bh=Onhm8bduCA5bOOMIJ1hUORngVPchthHx4UKoIncygeo=; b=SUGUO6K5IkK3MHTPFVx/+RxEHPKBVzzHv0LJwtUOYP6Cd4zcIGGRrxhpHJQC6rC46v fmD6I0uNShDs2HLdYdKmmMR+Qi1IWTQBg0Khqg4vf78HpwA5QMzlcEg5gds2C+YA/OCU JOWMXBSgVsqekNP+4w+BOTvNjS966r0f+REXJJoiHhl7CBxyb5GeMfuT37hq8RMFUB8S 5MmZLKXFiknHqXNcrlqT+UuPlqzspL5omPM/sUajRLDJw1U1iw4ggJ5LIzRu3qSMZkBa qDlsX6J7y3xtB4Y9+TW/DvGl7EK9qTSsO6gIgrZVcUTDFO4caQ5ddWMnYy9v2QKZKknl LaaA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references :mail-followup-to:message-id:subject:cc:to:from:date :x-gm-message-state:from:to:cc:subject:date; bh=Onhm8bduCA5bOOMIJ1hUORngVPchthHx4UKoIncygeo=; b=V3Ghe+VCZzR/Ue5cAk41QoEj/NwPzIJGiu4LKPEbifYKqoKGAGLN4yCbhZqhY8Oxw0 mrvIyZUost9lxVbCdQoamo29XFLG56xkYHmpy79ffM7wnv+eGNmlZN04TBqEkancI8CM OIhsg6QdMGcZWlledV0F4cxqf6zCkcWD+DAQwlLe+m6iwEwb+wF1KHMXVtgm1EwxYzu/ 8cROsLHXAzm/Cw3Mr+lX3ctFIKp+14NV5fca8LdnYRrAQpBvbIHsMx2dlqQa9UF2cbJ+ v6XXQJvvkRPlb9X8+cMby5gQy6OA/WJD8PxwgWK2Iml8Yi88t2zqOI3YuiFyeLzfvr1v kP4A== X-Gm-Message-State: ACgBeo0br/i2gBtzKfH6l3Ts5C7NZ6YbzIrcoAkrKLkPAQc8Ziw0VR6v 57I1cXZbZHS0btKA7bnTfWwzPg== X-Google-Smtp-Source: AA6agR7lKmzUMdb4Vpugylmq2gxjluuxiXG5v/pX5p3mA/mS7+rXa4llTYmJ2vDPCp0v+FNZWD+9gA== X-Received: by 2002:a65:6cc8:0:b0:3fe:2b89:cc00 with SMTP id g8-20020a656cc8000000b003fe2b89cc00mr30510915pgw.599.1663121343723; Tue, 13 Sep 2022 19:09:03 -0700 (PDT) Received: from laputa ([2400:4050:c3e1:100:cdcf:865e:fea5:fea9]) by smtp.gmail.com with ESMTPSA id t20-20020a62d154000000b0053653c6b9f9sm8586268pfl.204.2022.09.13.19.09.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 13 Sep 2022 19:09:02 -0700 (PDT) Date: Wed, 14 Sep 2022 11:08:58 +0900 From: AKASHI Takahiro To: Heinrich Schuchardt Cc: Simon Glass , Tom Rini , Ilias Apalodimas , Masahisa Kojima , U-Boot Mailing List Subject: Re: [PATCH v2 22/24] blk: Drop if_type Message-ID: <20220914020858.GA42806@laputa> Mail-Followup-To: AKASHI Takahiro , Heinrich Schuchardt , Simon Glass , Tom Rini , Ilias Apalodimas , Masahisa Kojima , U-Boot Mailing List References: <20220812013503.1724919-1-sjg@chromium.org> <20220812013503.1724919-23-sjg@chromium.org> <91a37375-fc9e-4099-95f5-63d02d3bb7cc@gmx.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <91a37375-fc9e-4099-95f5-63d02d3bb7cc@gmx.de> X-BeenThere: u-boot@lists.denx.de X-Mailman-Version: 2.1.39 Precedence: list List-Id: U-Boot discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: u-boot-bounces@lists.denx.de Sender: "U-Boot" X-Virus-Scanned: clamav-milter 0.103.6 at phobos.denx.de X-Virus-Status: Clean On Tue, Sep 13, 2022 at 06:27:37PM +0200, Heinrich Schuchardt wrote: > On 8/12/22 03:35, Simon Glass wrote: > > Use the uclass ID instead. > > > > Signed-off-by: Simon Glass > > --- > > > > (no changes since v1) > > > > cmd/blk_common.c | 2 +- > > drivers/block/blk-uclass.c | 34 +++++++++++++++---------------- > > drivers/block/blk_legacy.c | 20 +++++++++---------- > > include/blk.h | 41 ++++++++++++++++++-------------------- > > include/efi_loader.h | 2 +- > > 5 files changed, 48 insertions(+), 51 deletions(-) > > > > diff --git a/cmd/blk_common.c b/cmd/blk_common.c > > index 4e442f2918b..369c5ae4bbe 100644 > > --- a/cmd/blk_common.c > > +++ b/cmd/blk_common.c > > @@ -12,7 +12,7 @@ > > #include > > #include > > > > -int blk_common_cmd(int argc, char *const argv[], enum if_type if_type, > > +int blk_common_cmd(int argc, char *const argv[], enum uclass_id if_type, > > int *cur_devnump) > > { > > const char *if_name = blk_get_if_type_name(if_type); > > diff --git a/drivers/block/blk-uclass.c b/drivers/block/blk-uclass.c > > index 436af764f91..a9a85aa37f3 100644 > > --- a/drivers/block/blk-uclass.c > > +++ b/drivers/block/blk-uclass.c > > @@ -34,7 +34,7 @@ static struct { > > { UCLASS_PVBLOCK, "pvblock" }, > > }; > > > > -static enum if_type if_typename_to_iftype(const char *if_typename) > > +static enum uclass_id if_typename_to_iftype(const char *if_typename) > > { > > int i; > > > > @@ -46,7 +46,7 @@ static enum if_type if_typename_to_iftype(const char *if_typename) > > return UCLASS_INVALID; > > } > > > > -static enum uclass_id if_type_to_uclass_id(enum if_type if_type) > > +static enum uclass_id if_type_to_uclass_id(enum uclass_id if_type) > > { > > /* > > * This strange adjustment is used because we use UCLASS_MASS_STORAGE > > @@ -71,7 +71,7 @@ static enum uclass_id if_type_to_uclass_id(enum if_type if_type) > > return if_type; > > } > > > > -const char *blk_get_if_type_name(enum if_type if_type) > > +const char *blk_get_if_type_name(enum uclass_id if_type) > > { > > int i; > > > > @@ -83,7 +83,7 @@ const char *blk_get_if_type_name(enum if_type if_type) > > return "(none)"; > > } > > > > -struct blk_desc *blk_get_devnum_by_type(enum if_type if_type, int devnum) > > +struct blk_desc *blk_get_devnum_by_type(enum uclass_id if_type, int devnum) > > { > > struct blk_desc *desc; > > struct udevice *dev; > > @@ -105,7 +105,7 @@ struct blk_desc *blk_get_devnum_by_type(enum if_type if_type, int devnum) > > struct blk_desc *blk_get_devnum_by_typename(const char *if_typename, int devnum) > > { > > enum uclass_id uclass_id; > > - enum if_type type; > > + enum uclass_id type; > > struct udevice *dev; > > struct uclass *uc; > > int ret; > > @@ -185,7 +185,7 @@ struct blk_desc *blk_get_by_device(struct udevice *dev) > > * with a higher device number, -ENOENT if there is no such device but there > > * is one with a higher number, or other -ve on other error. > > */ > > -static int get_desc(enum if_type if_type, int devnum, struct blk_desc **descp) > > +static int get_desc(enum uclass_id if_type, int devnum, struct blk_desc **descp) > > { > > bool found_more = false; > > struct udevice *dev; > > @@ -218,7 +218,7 @@ static int get_desc(enum if_type if_type, int devnum, struct blk_desc **descp) > > return found_more ? -ENOENT : -ENODEV; > > } > > > > -int blk_select_hwpart_devnum(enum if_type if_type, int devnum, int hwpart) > > +int blk_select_hwpart_devnum(enum uclass_id if_type, int devnum, int hwpart) > > { > > struct udevice *dev; > > int ret; > > @@ -230,7 +230,7 @@ int blk_select_hwpart_devnum(enum if_type if_type, int devnum, int hwpart) > > return blk_select_hwpart(dev, hwpart); > > } > > > > -int blk_list_part(enum if_type if_type) > > +int blk_list_part(enum uclass_id if_type) > > { > > struct blk_desc *desc; > > int devnum, ok; > > @@ -255,7 +255,7 @@ int blk_list_part(enum if_type if_type) > > return 0; > > } > > > > -int blk_print_part_devnum(enum if_type if_type, int devnum) > > +int blk_print_part_devnum(enum uclass_id if_type, int devnum) > > { > > struct blk_desc *desc; > > int ret; > > @@ -270,7 +270,7 @@ int blk_print_part_devnum(enum if_type if_type, int devnum) > > return 0; > > } > > > > -void blk_list_devices(enum if_type if_type) > > +void blk_list_devices(enum uclass_id if_type) > > { > > struct blk_desc *desc; > > int ret; > > @@ -289,7 +289,7 @@ void blk_list_devices(enum if_type if_type) > > } > > } > > > > -int blk_print_device_num(enum if_type if_type, int devnum) > > +int blk_print_device_num(enum uclass_id if_type, int devnum) > > { > > struct blk_desc *desc; > > int ret; > > @@ -303,7 +303,7 @@ int blk_print_device_num(enum if_type if_type, int devnum) > > return 0; > > } > > > > -int blk_show_device(enum if_type if_type, int devnum) > > +int blk_show_device(enum uclass_id if_type, int devnum) > > { > > struct blk_desc *desc; > > int ret; > > @@ -324,7 +324,7 @@ int blk_show_device(enum if_type if_type, int devnum) > > return 0; > > } > > > > -ulong blk_read_devnum(enum if_type if_type, int devnum, lbaint_t start, > > +ulong blk_read_devnum(enum uclass_id if_type, int devnum, lbaint_t start, > > lbaint_t blkcnt, void *buffer) > > { > > struct blk_desc *desc; > > @@ -341,7 +341,7 @@ ulong blk_read_devnum(enum if_type if_type, int devnum, lbaint_t start, > > return n; > > } > > > > -ulong blk_write_devnum(enum if_type if_type, int devnum, lbaint_t start, > > +ulong blk_write_devnum(enum uclass_id if_type, int devnum, lbaint_t start, > > lbaint_t blkcnt, const void *buffer) > > { > > struct blk_desc *desc; > > @@ -525,7 +525,7 @@ const char *blk_get_devtype(struct udevice *dev) > > return uclass_get_name(device_get_uclass_id(parent)); > > }; > > > > -int blk_find_max_devnum(enum if_type if_type) > > +int blk_find_max_devnum(enum uclass_id if_type) > > { > > struct udevice *dev; > > int max_devnum = -ENODEV; > > @@ -545,7 +545,7 @@ int blk_find_max_devnum(enum if_type if_type) > > return max_devnum; > > } > > > > -int blk_next_free_devnum(enum if_type if_type) > > +int blk_next_free_devnum(enum uclass_id if_type) > > { > > int ret; > > > > @@ -631,7 +631,7 @@ int blk_count_devices(enum blk_flag_t flag) > > return count; > > } > > > > -static int blk_claim_devnum(enum if_type if_type, int devnum) > > +static int blk_claim_devnum(enum uclass_id if_type, int devnum) > > { > > struct udevice *dev; > > struct uclass *uc; > > diff --git a/drivers/block/blk_legacy.c b/drivers/block/blk_legacy.c > > index bd8a17df6a9..8c6f9cb208e 100644 > > --- a/drivers/block/blk_legacy.c > > +++ b/drivers/block/blk_legacy.c > > @@ -39,7 +39,7 @@ static struct blk_driver *blk_driver_lookup_typename(const char *if_typename) > > return NULL; > > } > > > > -const char *blk_get_if_type_name(enum if_type if_type) > > +const char *blk_get_if_type_name(enum uclass_id if_type) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > > > @@ -70,7 +70,7 @@ static int get_desc(struct blk_driver *drv, int devnum, struct blk_desc **descp) > > return drv->get_dev(devnum, descp); > > } > > > > -int blk_list_part(enum if_type if_type) > > +int blk_list_part(enum uclass_id if_type) > > { > > struct blk_driver *drv; > > struct blk_desc *desc; > > @@ -97,7 +97,7 @@ int blk_list_part(enum if_type if_type) > > return 0; > > } > > > > -int blk_print_part_devnum(enum if_type if_type, int devnum) > > +int blk_print_part_devnum(enum uclass_id if_type, int devnum) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > struct blk_desc *desc; > > @@ -115,7 +115,7 @@ int blk_print_part_devnum(enum if_type if_type, int devnum) > > return 0; > > } > > > > -void blk_list_devices(enum if_type if_type) > > +void blk_list_devices(enum uclass_id if_type) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > struct blk_desc *desc; > > @@ -133,7 +133,7 @@ void blk_list_devices(enum if_type if_type) > > } > > } > > > > -int blk_print_device_num(enum if_type if_type, int devnum) > > +int blk_print_device_num(enum uclass_id if_type, int devnum) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > struct blk_desc *desc; > > @@ -150,7 +150,7 @@ int blk_print_device_num(enum if_type if_type, int devnum) > > return 0; > > } > > > > -int blk_show_device(enum if_type if_type, int devnum) > > +int blk_show_device(enum uclass_id if_type, int devnum) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > struct blk_desc *desc; > > @@ -174,7 +174,7 @@ int blk_show_device(enum if_type if_type, int devnum) > > return 0; > > } > > > > -struct blk_desc *blk_get_devnum_by_type(enum if_type if_type, int devnum) > > +struct blk_desc *blk_get_devnum_by_type(enum uclass_id if_type, int devnum) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > struct blk_desc *desc; > > @@ -214,7 +214,7 @@ struct blk_desc *blk_get_devnum_by_typename(const char *if_typename, int devnum) > > return desc; > > } > > > > -ulong blk_read_devnum(enum if_type if_type, int devnum, lbaint_t start, > > +ulong blk_read_devnum(enum uclass_id if_type, int devnum, lbaint_t start, > > lbaint_t blkcnt, void *buffer) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > @@ -234,7 +234,7 @@ ulong blk_read_devnum(enum if_type if_type, int devnum, lbaint_t start, > > return n; > > } > > > > -ulong blk_write_devnum(enum if_type if_type, int devnum, lbaint_t start, > > +ulong blk_write_devnum(enum uclass_id if_type, int devnum, lbaint_t start, > > lbaint_t blkcnt, const void *buffer) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > @@ -249,7 +249,7 @@ ulong blk_write_devnum(enum if_type if_type, int devnum, lbaint_t start, > > return desc->block_write(desc, start, blkcnt, buffer); > > } > > > > -int blk_select_hwpart_devnum(enum if_type if_type, int devnum, int hwpart) > > +int blk_select_hwpart_devnum(enum uclass_id if_type, int devnum, int hwpart) > > { > > struct blk_driver *drv = blk_driver_lookup_type(if_type); > > struct blk_desc *desc; > > diff --git a/include/blk.h b/include/blk.h > > index 548cd09d6bb..279f9ea4a9d 100644 > > --- a/include/blk.h > > +++ b/include/blk.h > > @@ -27,9 +27,6 @@ static inline bool blk_enabled(void) > > return CONFIG_IS_ENABLED(BLK) || IS_ENABLED(CONFIG_SPL_LEGACY_BLOCK); > > } > > > > -/* Interface types: */ > > -#define if_type uclass_id > > - > > #define BLK_VEN_SIZE 40 > > #define BLK_PRD_SIZE 20 > > #define BLK_REV_SIZE 8 > > @@ -57,7 +54,7 @@ struct blk_desc { > > * TODO: With driver model we should be able to use the parent > > * device's uclass instead. > > */ > > - enum if_type if_type; /* type of the interface */ > > + enum uclass_id if_type; /* type of the interface */ > > int devnum; /* device number */ > > unsigned char part_type; /* partition type */ > > unsigned char target; /* target SCSI ID */ > > @@ -282,7 +279,7 @@ unsigned long blk_derase(struct blk_desc *block_dev, lbaint_t start, > > * This function does not activate the device. The device will be returned > > * whether or not it is activated. > > * > > - * @if_type: Interface type (enum if_type_t) > > + * @if_type: Interface type (enum uclass_id_t) > > * @devnum: Device number (specific to each interface type) > > * @devp: the device, if found > > * Return: 0 if found, -ENODEV if no device found, or other -ve error value > > @@ -292,7 +289,7 @@ int blk_find_device(int if_type, int devnum, struct udevice **devp); > > /** > > * blk_get_device() - Find and probe a block device ready for use > > * > > - * @if_type: Interface type (enum if_type_t) > > + * @if_type: Interface type (enum uclass_id_t) > > * @devnum: Device number (specific to each interface type) > > * @devp: the device, if found > > * Return: 0 if found, -ENODEV if no device found, or other -ve error value > > @@ -330,7 +327,7 @@ int blk_next_device(struct udevice **devp); > > * @parent: Parent of the new device > > * @drv_name: Driver name to use for the block device > > * @name: Name for the device > > - * @if_type: Interface type (enum if_type_t) > > + * @if_type: Interface type (enum uclass_id_t) > > * @devnum: Device number, specific to the interface type, or -1 to > > * allocate the next available number > > * @blksz: Block size of the device in bytes (typically 512) > > @@ -347,7 +344,7 @@ int blk_create_device(struct udevice *parent, const char *drv_name, > > * @parent: Parent of the new device > > * @drv_name: Driver name to use for the block device > > * @name: Name for the device (parent name is prepended) > > - * @if_type: Interface type (enum if_type_t) > > + * @if_type: Interface type (enum uclass_id_t) > > * @devnum: Device number, specific to the interface type, or -1 to > > * allocate the next available number > > * @blksz: Block size of the device in bytes (typically 512) > > @@ -390,7 +387,7 @@ int blk_unbind_all(int if_type); > > * Return: maximum device number found, or -ENODEV if none, or other -ve on > > * error > > */ > > -int blk_find_max_devnum(enum if_type if_type); > > +int blk_find_max_devnum(enum uclass_id if_type); > > > > /** > > * blk_next_free_devnum() - get the next device number for an interface type > > @@ -401,7 +398,7 @@ int blk_find_max_devnum(enum if_type if_type); > > * @if_type: Interface type to scan > > * Return: next device number safe to use, or -ve on error > > */ > > -int blk_next_free_devnum(enum if_type if_type); > > +int blk_next_free_devnum(enum uclass_id if_type); > > > > /** > > * blk_select_hwpart() - select a hardware partition > > @@ -496,7 +493,7 @@ static inline ulong blk_derase(struct blk_desc *block_dev, lbaint_t start, > > */ > > struct blk_driver { > > const char *if_typename; > > - enum if_type if_type; > > + enum uclass_id if_type; > > int max_devs; > > struct blk_desc *desc; > > /** > > @@ -557,7 +554,7 @@ struct blk_driver *blk_driver_lookup_type(int if_type); > > * @devnum: Device number > > * Return: point to block device descriptor, or NULL if not found > > */ > > -struct blk_desc *blk_get_devnum_by_type(enum if_type if_type, int devnum); > > +struct blk_desc *blk_get_devnum_by_type(enum uclass_id if_type, int devnum); > > > > /** > > * blk_get_devnum_by_type() - Get a block device by type name, and number > > @@ -594,7 +591,7 @@ int blk_dselect_hwpart(struct blk_desc *desc, int hwpart); > > * @if_type: Block device type > > * Return: 0 if OK, -ENODEV if there is none of that type > > */ > > -int blk_list_part(enum if_type if_type); > > +int blk_list_part(enum uclass_id if_type); > > > > /** > > * blk_list_devices() - list the block devices of a given type > > @@ -604,7 +601,7 @@ int blk_list_part(enum if_type if_type); > > * > > * @if_type: Block device type > > */ > > -void blk_list_devices(enum if_type if_type); > > +void blk_list_devices(enum uclass_id if_type); > > > > /** > > * blk_show_device() - show information about a given block device > > @@ -615,7 +612,7 @@ void blk_list_devices(enum if_type if_type); > > * @devnum: Device number > > * Return: 0 if OK, -ENODEV for invalid device number > > */ > > -int blk_show_device(enum if_type if_type, int devnum); > > +int blk_show_device(enum uclass_id if_type, int devnum); > > > > /** > > * blk_print_device_num() - show information about a given block device > > @@ -628,7 +625,7 @@ int blk_show_device(enum if_type if_type, int devnum); > > * Return: 0 if OK, -ENODEV for invalid device number, -ENOENT if the block > > * device is not connected > > */ > > -int blk_print_device_num(enum if_type if_type, int devnum); > > +int blk_print_device_num(enum uclass_id if_type, int devnum); > > > > /** > > * blk_print_part_devnum() - print the partition information for a device > > @@ -638,7 +635,7 @@ int blk_print_device_num(enum if_type if_type, int devnum); > > * Return: 0 if OK, -ENOENT if the block device is not connected, -ENOSYS if > > * the interface type is not supported, other -ve on other error > > */ > > -int blk_print_part_devnum(enum if_type if_type, int devnum); > > +int blk_print_part_devnum(enum uclass_id if_type, int devnum); > > > > /** > > * blk_read_devnum() - read blocks from a device > > @@ -649,7 +646,7 @@ int blk_print_part_devnum(enum if_type if_type, int devnum); > > * @buffer: Address to write data to > > * Return: number of blocks read, or -ve error number on error > > */ > > -ulong blk_read_devnum(enum if_type if_type, int devnum, lbaint_t start, > > +ulong blk_read_devnum(enum uclass_id if_type, int devnum, lbaint_t start, > > lbaint_t blkcnt, void *buffer); > > > > /** > > @@ -661,7 +658,7 @@ ulong blk_read_devnum(enum if_type if_type, int devnum, lbaint_t start, > > * @buffer: Address to read data from > > * Return: number of blocks written, or -ve error number on error > > */ > > -ulong blk_write_devnum(enum if_type if_type, int devnum, lbaint_t start, > > +ulong blk_write_devnum(enum uclass_id if_type, int devnum, lbaint_t start, > > lbaint_t blkcnt, const void *buffer); > > > > /** > > @@ -675,7 +672,7 @@ ulong blk_write_devnum(enum if_type if_type, int devnum, lbaint_t start, > > * @hwpart: Partition number to select > > * Return: 0 if OK, -ve on error > > */ > > -int blk_select_hwpart_devnum(enum if_type if_type, int devnum, int hwpart); > > +int blk_select_hwpart_devnum(enum uclass_id if_type, int devnum, int hwpart); > > > > /** > > * blk_get_if_type_name() - Get the name of an interface type > > @@ -683,7 +680,7 @@ int blk_select_hwpart_devnum(enum if_type if_type, int devnum, int hwpart); > > * @if_type: Interface type to check > > * Return: name of interface, or NULL if none > > */ > > -const char *blk_get_if_type_name(enum if_type if_type); > > +const char *blk_get_if_type_name(enum uclass_id if_type); > > > > /** > > * blk_common_cmd() - handle common commands with block devices > > @@ -694,7 +691,7 @@ const char *blk_get_if_type_name(enum if_type if_type); > > * @cur_devnump: Current device number for this interface type > > * Return: 0 if OK, CMD_RET_ERROR on error > > */ > > -int blk_common_cmd(int argc, char *const argv[], enum if_type if_type, > > +int blk_common_cmd(int argc, char *const argv[], enum uclass_id if_type, > > int *cur_devnump); > > > > enum blk_flag_t { > > diff --git a/include/efi_loader.h b/include/efi_loader.h > > index 3a63a1f75fd..ae58efb0036 100644 > > --- a/include/efi_loader.h > > +++ b/include/efi_loader.h > > @@ -156,7 +156,7 @@ extern bool efi_st_keep_devices; > > > > /* EFI system partition */ > > extern struct efi_system_partition { > > - enum if_type if_type; > > + enum uclass_id if_type; > > int devnum; > > Please, rename the field if_type to class_id in a later patch. > > Uclass ID and device number may refer to a different device after > commands like usb init. It would be better if we could use the partition > GUID to identify a partition in the long term. But that is beyond the > scope of this series. Well, as far as UEFI is concerned, we should use a device path (and/or a handle to it) to uniquely identify a given device. Then the whole code in efi_var_file.c can be rewritten by using SIMPLE_FILE_SYSTEM_PROTOCOL attached to it. -Takahiro Akashi > Acked-by: Heinrich Schuchardt > > > > u8 part; > > > > > } efi_system_partition; >