Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Jonathan Corbet <corbet@lwn.net>
To:     hjh <huangjianghui@uniontech.com>
Cc:     Harry Wei <harryxiyou@gmail.com>,
        Alex Shi <alex.shi@linux.alibaba.com>,
        linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/1] [PATCH] Documentation/translations: Translate
 sound/hd-audio/controls.rst into Chinese
In-Reply-To: <20210219144856.27362-1-huangjianghui@uniontech.com>
References: <20210219144856.27362-1-huangjianghui@uniontech.com>
Date:   Fri, 19 Feb 2021 11:28:53 -0700
Message-ID: <874ki7oqsa.fsf@meer.lwn.net>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

hjh <huangjianghui@uniontech.com> writes:

> Signed-off-by: hjh <huangjianghui@uniontech.com>
> ---
>  .../zh_CN/sound/hd-audio/controls.txt         | 117 ++++++++++++++++++
>  1 file changed, 117 insertions(+)
>  create mode 100644 Documentation/translations/zh_CN/sound/hd-audio/controls.txt

Thanks for working to improve the documentation.

When you add a new file, you need to put it into an index.rst file
somewhere so that it gets built with the rest of the kernel docs.

As for *which* index.rst...I kind of wonder how you chose that document,
in particular, to translate?  If we want to start translating the sound
documentation, I think it would be better to start with the top-level
page and build the structure downward.  So can I please ask you to start
in Documentation/translations/zh_CN/sound, with some sort of basic
index.rst file there, then add onto it from there?

Thanks,

jon