[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: does drive_get_next(IF_NONE) make sense?
From: |
Markus Armbruster |
Subject: |
Re: does drive_get_next(IF_NONE) make sense? |
Date: |
Mon, 15 Nov 2021 06:31:24 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Peter Maydell <peter.maydell@linaro.org> writes:
> On Fri, 12 Nov 2021 at 13:34, Markus Armbruster <armbru@redhat.com> wrote:
>>
>> Thomas Huth <thuth@redhat.com> writes:
>>
>> > On 03/11/2021 09.41, Markus Armbruster wrote:
>> >> Peter Maydell <peter.maydell@linaro.org> writes:
>> >>
>> >>> Does it make sense for a device/board to do drive_get_next(IF_NONE) ?
>> >> Short answer: hell, no! ;)
>> >
>> > Would it make sense to add an "assert(type != IF_NONE)" to drive_get()
>> > to avoid such mistakes in the future?
>>
>> Worth a try.
>
> You need to fix the sifive_u_otp device first :-)
And for that, we may want Hao Wu's "[PATCH v4 5/7] blockdev: Add a new
IF type IF_OTHER" first.
- does drive_get_next(IF_NONE) make sense?, Peter Maydell, 2021/11/02
- Re: does drive_get_next(IF_NONE) make sense?, Philippe Mathieu-Daudé, 2021/11/02
- Re: does drive_get_next(IF_NONE) make sense?, Markus Armbruster, 2021/11/03
- Re: does drive_get_next(IF_NONE) make sense?, Thomas Huth, 2021/11/03
- Re: does drive_get_next(IF_NONE) make sense?, Markus Armbruster, 2021/11/12
- Re: does drive_get_next(IF_NONE) make sense?, Peter Maydell, 2021/11/14
- Re: does drive_get_next(IF_NONE) make sense?,
Markus Armbruster <=
- Re: does drive_get_next(IF_NONE) make sense?, Alistair Francis, 2021/11/15
- Re: does drive_get_next(IF_NONE) make sense?, Thomas Huth, 2021/11/15
- Re: does drive_get_next(IF_NONE) make sense?, Alistair Francis, 2021/11/18
- Re: does drive_get_next(IF_NONE) make sense?, Peter Maydell, 2021/11/18
- Re: does drive_get_next(IF_NONE) make sense?, Kevin Wolf, 2021/11/15
- Re: does drive_get_next(IF_NONE) make sense?, Peter Maydell, 2021/11/15
- Re: does drive_get_next(IF_NONE) make sense?, Kevin Wolf, 2021/11/15
- Re: does drive_get_next(IF_NONE) make sense?, Markus Armbruster, 2021/11/15