linux-kernel - Re: [PATCH 00/12] Re-introduce Exynos4212 support and add Samsung Galaxy Tab 3 8.0 boards

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <e50f5980-b1f6-4495-08b7-7e3f4f78cc67@linaro.org>
Date:   Sun, 16 Apr 2023 14:42:15 +0200
From:   Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To:     Artur Weber <aweber.kernel@...il.com>
Cc:     Alim Akhtar <alim.akhtar@...sung.com>,
        Rob Herring <robh+dt@...nel.org>,
        Sylwester Nawrocki <s.nawrocki@...sung.com>,
        Tomasz Figa <tomasz.figa@...il.com>,
        Chanwoo Choi <cw00.choi@...sung.com>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>, Arnd Bergmann <arnd@...db.de>,
        Olof Johansson <olof@...om.net>, soc@...nel.org,
        Russell King <linux@...linux.org.uk>,
        Daniel Lezcano <daniel.lezcano@...aro.org>,
        Kukjin Kim <kgene@...nel.org>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        Vinod Koul <vkoul@...nel.org>,
        Kishon Vijay Abraham I <kishon@...nel.org>,
        linux-arm-kernel@...ts.infradead.org,
        linux-samsung-soc@...r.kernel.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-clk@...r.kernel.org,
        linux-pm@...r.kernel.org, linux-media@...r.kernel.org,
        linux-phy@...ts.infradead.org,
        ~postmarketos/upstreaming@...ts.sr.ht
Subject: Re: [PATCH 00/12] Re-introduce Exynos4212 support and add Samsung
 Galaxy Tab 3 8.0 boards

On 16/04/2023 14:37, Artur Weber wrote:
> On 16/04/2023 12:53, Krzysztof Kozlowski wrote:
>> On 16/04/2023 12:49, Artur Weber wrote:
>>> On 16/04/2023 12:34, Krzysztof Kozlowski wrote:
>>>> On 16/04/2023 12:26, Artur Weber wrote:
>>>>> On 16/04/2023 12:16, Artur Weber wrote:
>>>>>> This patches re-introduces the Exynos4212 platform and adds support
>>>>>> for the Samsung Galaxy Tab 3 8.0 series of tablets that uses it:
>>>>>>
>>>>>>     - Samsung Galaxy Tab 3 8.0 WiFi (SM-T310/lt01wifi)
>>>>>>     - Samsung Galaxy Tab 3 8.0 3G (SM-T311/lt013g)
>>>>>>     - Samsung Galaxy Tab 3 8.0 LTE (SM-T315/lt01lte)
>>>>>>
>>>>>> What works:
>>>>>>
>>>>>>     - Display and backlight
>>>>>>     - Touchscreen (without touchkeys)
>>>>>>     - GPIO buttons, hall sensor
>>>>>>     - WiFi and Bluetooth
>>>>>>     - USB, fuel gauge, charging (partial)
>>>>>>     - Accelerometer and magnetometer
>>>>>>     - WiFi model only: light sensor
>>>>>
>>>>> This patchset depends on "[PATCH 0/3] Add Samsung S6D7AA0 panel
>>>>> controller driver" for the display panel support for the Samsung Galaxy
>>>>> 3 8.0 boards.
>>>>
>>>> Why? DTS and ARM code cannot depend on driver changes. Please rework
>>>> your patchsets to remove any of such dependencies.
>>>
>>> Ah, that makes sense. I'll re-send the patchset in a second with the
>>> panel node removed.
>>
>> I am sorry, I don't understand. Why would you remove anything from DTS?
>> Are bindings NAKed?
> 
> The dependency display panel patchset introduces the panel and its 
> bindings, which in turn are included in the Tab3 DTSI. It was submitted 
> at roughly the same time as this series, and hasn't been fully reviewed 
> or merged as of writing. (I have seen your comments on that patchset, 
> and I will be addressing them shortly.) So the bindings haven't been 
> explicitly ACKed yet (assuming you mean the Acked-by reply).

So there is no dependency. DTS is (and must be) independent of driver.
You need to mention in the patch (under --- ) or cover letter that it
uses new bindings which are documented under the given link.

> 
> In response to:
> 
>  > Please rework your patchsets to remove any of such dependencies.
> 
> I suggested that I could remove the panel node from the DTSI for the 
> time being. The intent was to submit it in a separate patch later, once 
> the display is reviewed/merged, and thus actually available in the 
> kernel; this way, the two patches could be reviewed and merged separately.

No need, there is no dependency. Just remove such remarks from cover letter.


Best regards,
Krzysztof