Unverified Commit 1256eb42 authored by Mark Brown's avatar Mark Brown
Browse files

spi: microchip-core-qspi: Add regular transfers 

Merge series from Conor Dooley <conor@kernel.org>:

This is a v2 of a patchset I sent about this time last year, adding the
regular transfer_one_message op to the microchip-core-qspi driver. In that
v1 Mark expressed his dislike for that op, so v2 is using
prepare/unprepare/transfer_one instead. The unprepare implementation still
contains the 750 us delay that the driver had back in v1. I've heard a
suggestion internally as to why this is needed, but it was unsubstantiated,
so I still have no justification for it. I held off on sending a v2 because
of a lack of explanation for the delay, but I don't wanna hold off forever
for something I might never understand.
parents 5fc2c383 8f9cf02c

.mailmap

+10 −3
Original line number Diff line number Diff line
@@ -197,6 +197,7 @@ Daniel Borkmann <daniel@iogearbox.net> <daniel.borkmann@tik.ee.ethz.ch> 
Daniel Borkmann <daniel@iogearbox.net> <dborkmann@redhat.com>

Daniel Borkmann <daniel@iogearbox.net> <dborkman@redhat.com>

Daniel Borkmann <daniel@iogearbox.net> <dxchgb@gmail.com>

Danilo Krummrich <dakr@kernel.org> <dakr@redhat.com>

David Brownell <david-b@pacbell.net>

David Collins <quic_collinsd@quicinc.com> <collinsd@codeaurora.org>

David Heidelberg <david@ixit.cz> <d.okias@gmail.com>
@@ -282,6 +283,7 @@ Gustavo Padovan <gustavo@las.ic.unicamp.br> 
Gustavo Padovan <padovan@profusion.mobi>

Hamza Mahfooz <hamzamahfooz@linux.microsoft.com> <hamza.mahfooz@amd.com>

Hanjun Guo <guohanjun@huawei.com> <hanjun.guo@linaro.org>

Hans de Goede <hansg@kernel.org> <hdegoede@redhat.com>

Hans Verkuil <hverkuil@xs4all.nl> <hansverk@cisco.com>

Hans Verkuil <hverkuil@xs4all.nl> <hverkuil-cisco@xs4all.nl>

Harry Yoo <harry.yoo@oracle.com> <42.hyeyoo@gmail.com>
@@ -426,6 +428,9 @@ Krzysztof Wilczyński <kwilczynski@kernel.org> <krzysztof.wilczynski@linux.com> 
Krzysztof Wilczyński <kwilczynski@kernel.org> <kw@linux.com>

Kshitiz Godara <quic_kgodara@quicinc.com> <kgodara@codeaurora.org>

Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>

Kuniyuki Iwashima <kuniyu@google.com> <kuniyu@amazon.com>

Kuniyuki Iwashima <kuniyu@google.com> <kuniyu@amazon.co.jp>

Kuniyuki Iwashima <kuniyu@google.com> <kuni1840@gmail.com>

Kuogee Hsieh <quic_khsieh@quicinc.com> <khsieh@codeaurora.org>

Lee Jones <lee@kernel.org> <joneslee@google.com>

Lee Jones <lee@kernel.org> <lee.jones@canonical.com>
@@ -688,9 +693,10 @@ Serge Hallyn <sergeh@kernel.org> <serge.hallyn@canonical.com> 
Serge Hallyn <sergeh@kernel.org> <serue@us.ibm.com>

Seth Forshee <sforshee@kernel.org> <seth.forshee@canonical.com>

Shakeel Butt <shakeel.butt@linux.dev> <shakeelb@google.com>

Shannon Nelson <shannon.nelson@amd.com> <snelson@pensando.io>

Shannon Nelson <shannon.nelson@amd.com> <shannon.nelson@intel.com>

Shannon Nelson <shannon.nelson@amd.com> <shannon.nelson@oracle.com>

Shannon Nelson <sln@onemain.com> <shannon.nelson@amd.com>

Shannon Nelson <sln@onemain.com> <snelson@pensando.io>

Shannon Nelson <sln@onemain.com> <shannon.nelson@intel.com>

Shannon Nelson <sln@onemain.com> <shannon.nelson@oracle.com>

Sharath Chandra Vurukala <quic_sharathv@quicinc.com> <sharathv@codeaurora.org>

Shiraz Hashim <shiraz.linux.kernel@gmail.com> <shiraz.hashim@st.com>

Shuah Khan <shuah@kernel.org> <shuahkhan@gmail.com>
@@ -719,6 +725,7 @@ Srinivas Ramana <quic_sramana@quicinc.com> <sramana@codeaurora.org> 
Sriram R <quic_srirrama@quicinc.com> <srirrama@codeaurora.org>

Sriram Yagnaraman <sriram.yagnaraman@ericsson.com> <sriram.yagnaraman@est.tech>

Stanislav Fomichev <sdf@fomichev.me> <sdf@google.com>

Stanislav Fomichev <sdf@fomichev.me> <stfomichev@gmail.com>

Stefan Wahren <wahrenst@gmx.net> <stefan.wahren@i2se.com>

Stéphane Witzmann <stephane.witzmann@ubpmes.univ-bpclermont.fr>

Stephen Hemminger <stephen@networkplumber.org> <shemminger@linux-foundation.org>

Documentation/admin-guide/cifs/usage.rst

+2 −0
Original line number Diff line number Diff line
@@ -270,6 +270,8 @@ configured for Unix Extensions (and the client has not disabled 
illegal Windows/NTFS/SMB characters to a remap range (this mount parameter

is the default for SMB3). This remap (``mapposix``) range is also

compatible with Mac (and "Services for Mac" on some older Windows).

When POSIX Extensions for SMB 3.1.1 are negotiated, remapping is automatically

disabled.



CIFS VFS Mount Options

======================

Documentation/arch/arm64/booting.rst

+1 −1
Original line number Diff line number Diff line
@@ -234,7 +234,7 @@ Before jumping into the kernel, the following conditions must be met: 


  - If the kernel is entered at EL1:



      - ICC.SRE_EL2.Enable (bit 3) must be initialised to 0b1

      - ICC_SRE_EL2.Enable (bit 3) must be initialised to 0b1

      - ICC_SRE_EL2.SRE (bit 0) must be initialised to 0b1.



  - The DT or ACPI tables must describe a GICv3 interrupt controller.

Documentation/block/ublk.rst

+77 −0
Original line number Diff line number Diff line
@@ -352,6 +352,83 @@ For reaching best IO performance, ublk server should align its segment 
parameter of `struct ublk_param_segment` with backend for avoiding

unnecessary IO split, which usually hurts io_uring performance.



Auto Buffer Registration

------------------------



The ``UBLK_F_AUTO_BUF_REG`` feature automatically handles buffer registration

and unregistration for I/O requests, which simplifies the buffer management

process and reduces overhead in the ublk server implementation.



This is another feature flag for using zero copy, and it is compatible with

``UBLK_F_SUPPORT_ZERO_COPY``.



Feature Overview

~~~~~~~~~~~~~~~~



This feature automatically registers request buffers to the io_uring context

before delivering I/O commands to the ublk server and unregisters them when

completing I/O commands. This eliminates the need for manual buffer

registration/unregistration via ``UBLK_IO_REGISTER_IO_BUF`` and

``UBLK_IO_UNREGISTER_IO_BUF`` commands, then IO handling in ublk server

can avoid dependency on the two uring_cmd operations.



IOs can't be issued concurrently to io_uring if there is any dependency

among these IOs. So this way not only simplifies ublk server implementation,

but also makes concurrent IO handling becomes possible by removing the

dependency on buffer registration & unregistration commands.



Usage Requirements

~~~~~~~~~~~~~~~~~~



1. The ublk server must create a sparse buffer table on the same ``io_ring_ctx``

   used for ``UBLK_IO_FETCH_REQ`` and ``UBLK_IO_COMMIT_AND_FETCH_REQ``. If

   uring_cmd is issued on a different ``io_ring_ctx``, manual buffer

   unregistration is required.



2. Buffer registration data must be passed via uring_cmd's ``sqe->addr`` with the

   following structure::



    struct ublk_auto_buf_reg {

        __u16 index;      /* Buffer index for registration */

        __u8 flags;       /* Registration flags */

        __u8 reserved0;   /* Reserved for future use */

        __u32 reserved1;  /* Reserved for future use */

    };



   ublk_auto_buf_reg_to_sqe_addr() is for converting the above structure into

   ``sqe->addr``.



3. All reserved fields in ``ublk_auto_buf_reg`` must be zeroed.



4. Optional flags can be passed via ``ublk_auto_buf_reg.flags``.



Fallback Behavior

~~~~~~~~~~~~~~~~~



If auto buffer registration fails:



1. When ``UBLK_AUTO_BUF_REG_FALLBACK`` is enabled:



   - The uring_cmd is completed

   - ``UBLK_IO_F_NEED_REG_BUF`` is set in ``ublksrv_io_desc.op_flags``

   - The ublk server must manually deal with the failure, such as, register

     the buffer manually, or using user copy feature for retrieving the data

     for handling ublk IO



2. If fallback is not enabled:



   - The ublk I/O request fails silently

   - The uring_cmd won't be completed



Limitations

~~~~~~~~~~~



- Requires same ``io_ring_ctx`` for all operations

- May require manual buffer management in fallback cases

- io_ring_ctx buffer table has a max size of 16K, which may not be enough

  in case that too many ublk devices are handled by this single io_ring_ctx

  and each one has very large queue depth



References

==========

Documentation/devicetree/bindings/i2c/nvidia,tegra20-i2c.yaml

+23 −1
Original line number Diff line number Diff line
@@ -97,7 +97,10 @@ properties: 


  resets:

    items:

      - description: module reset

      - description:

          Module reset. This property is optional for controllers in Tegra194,

          Tegra234 etc where an internal software reset is available as an

          alternative.



  reset-names:

    items:
@@ -116,6 +119,13 @@ properties: 
      - const: rx

      - const: tx



required:

  - compatible

  - reg

  - interrupts

  - clocks

  - clock-names



allOf:

  - $ref: /schemas/i2c/i2c-controller.yaml

  - if:
@@ -169,6 +179,18 @@ allOf: 
      properties:

        power-domains: false



  - if:

      not:

        properties:

          compatible:

            contains:

              enum:

                - nvidia,tegra194-i2c

    then:

      required:

        - resets

        - reset-names



unevaluatedProperties: false



examples: