Re: [U-Boot] [PATCH] cmd: host: fix seg fault at "host info"
On Thu, Aug 22, 2019 at 04:47:39PM +0900, AKASHI Takahiro wrote: > With the patch below applied, host_block_dev structure was switched > to be placed in platdata rather than priv. The command "host info" > must be aligned with this change. Otherwise, we will see "Segmentation > Fault." > > Fixes: 8f994c860d91 ("sandbox: blk: Switch to use platdata_auto_alloc_size > for the driver data") > Signed-off-by: AKASHI Takahiro > Reviewed-by: Bin Meng > Tested-by: Bin Meng > Reviewed-by: Heinrich Schuchardt Applied to u-boot/master, thanks! -- Tom signature.asc Description: PGP signature ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] cmd: host: fix seg fault at "host info"
+Tom On Thu, Oct 3, 2019 at 2:00 PM AKASHI Takahiro wrote: > > Ping, > > This patch has not been merged yet. > Simon is on vacation. Tom, could you please take this patch directly? This needs to be in v2019.10. Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] cmd: host: fix seg fault at "host info"
Ping, This patch has not been merged yet. -Takahiro Akashi On Thu, Aug 22, 2019 at 04:47:39PM +0900, AKASHI Takahiro wrote: > With the patch below applied, host_block_dev structure was switched > to be placed in platdata rather than priv. The command "host info" > must be aligned with this change. Otherwise, we will see "Segmentation > Fault." > > Fixes: 8f994c860d91 ("sandbox: blk: Switch to use platdata_auto_alloc_size > for the driver data") > Signed-off-by: AKASHI Takahiro > --- > cmd/host.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/cmd/host.c b/cmd/host.c > index f7d3eae5b1ad..98c4d2a099e9 100644 > --- a/cmd/host.c > +++ b/cmd/host.c > @@ -89,7 +89,7 @@ static int do_host_info(cmd_tbl_t *cmdtp, int flag, int > argc, > struct host_block_dev *host_dev; > > #ifdef CONFIG_BLK > - host_dev = dev_get_priv(blk_dev->bdev); > + host_dev = dev_get_platdata(blk_dev->bdev); > #else > host_dev = blk_dev->priv; > #endif > -- > 2.21.0 > ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] cmd: host: fix seg fault at "host info"
On 8/22/19 9:47 AM, AKASHI Takahiro wrote: With the patch below applied, host_block_dev structure was switched to be placed in platdata rather than priv. The command "host info" must be aligned with this change. Otherwise, we will see "Segmentation Fault." Fixes: 8f994c860d91 ("sandbox: blk: Switch to use platdata_auto_alloc_size for the driver data") Signed-off-by: AKASHI Takahiro Reviewed-by: Heinrich Schuchardt --- cmd/host.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/host.c b/cmd/host.c index f7d3eae5b1ad..98c4d2a099e9 100644 --- a/cmd/host.c +++ b/cmd/host.c @@ -89,7 +89,7 @@ static int do_host_info(cmd_tbl_t *cmdtp, int flag, int argc, struct host_block_dev *host_dev; #ifdef CONFIG_BLK - host_dev = dev_get_priv(blk_dev->bdev); + host_dev = dev_get_platdata(blk_dev->bdev); #else host_dev = blk_dev->priv; #endif ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] cmd: host: fix seg fault at "host info"
On Thu, Aug 22, 2019 at 3:45 PM AKASHI Takahiro wrote: > > With the patch below applied, host_block_dev structure was switched > to be placed in platdata rather than priv. The command "host info" > must be aligned with this change. Otherwise, we will see "Segmentation > Fault." > > Fixes: 8f994c860d91 ("sandbox: blk: Switch to use platdata_auto_alloc_size > for the driver data") > Signed-off-by: AKASHI Takahiro > --- > cmd/host.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > Reviewed-by: Bin Meng Tested-by: Bin Meng ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH] cmd: host: fix seg fault at "host info"
With the patch below applied, host_block_dev structure was switched to be placed in platdata rather than priv. The command "host info" must be aligned with this change. Otherwise, we will see "Segmentation Fault." Fixes: 8f994c860d91 ("sandbox: blk: Switch to use platdata_auto_alloc_size for the driver data") Signed-off-by: AKASHI Takahiro --- cmd/host.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/host.c b/cmd/host.c index f7d3eae5b1ad..98c4d2a099e9 100644 --- a/cmd/host.c +++ b/cmd/host.c @@ -89,7 +89,7 @@ static int do_host_info(cmd_tbl_t *cmdtp, int flag, int argc, struct host_block_dev *host_dev; #ifdef CONFIG_BLK - host_dev = dev_get_priv(blk_dev->bdev); + host_dev = dev_get_platdata(blk_dev->bdev); #else host_dev = blk_dev->priv; #endif -- 2.21.0 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot