diff mbox series

[v2,3/6] dt-bindings: omap: Add Samaung Galaxy Tab 2 7.0

Message ID 20241030211847.413-1-bavishimithil@gmail.com (mailing list archive)
State Superseded
Headers show
Series Initial support for Samsung Galaxy Tab 2 series | expand

Commit Message

Mithil Bavishi Oct. 30, 2024, 9:18 p.m. UTC
Add samsung-espresso7 codename for the 7 inch variant

Signed-off-by: Mithil Bavishi <bavishimithil@gmail.com>
---
 Documentation/devicetree/bindings/arm/ti/omap.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski Oct. 31, 2024, 8:18 a.m. UTC | #1
On Wed, Oct 30, 2024 at 09:18:43PM +0000, Mithil Bavishi wrote:
> Add samsung-espresso7 codename for the 7 inch variant
> 
> Signed-off-by: Mithil Bavishi <bavishimithil@gmail.com>
> ---
>  Documentation/devicetree/bindings/arm/ti/omap.yaml | 1 +
>  1 file changed, 1 insertion(+)
>

This is v2, so where is the changelog? Same for all other patches. No
cover letter, no changelogs in patches.

Best regards,
Krzysztof
Andreas Kemnade Oct. 31, 2024, 8:24 a.m. UTC | #2
Am Thu, 31 Oct 2024 09:18:49 +0100
schrieb Krzysztof Kozlowski <krzk@kernel.org>:

> On Wed, Oct 30, 2024 at 09:18:43PM +0000, Mithil Bavishi wrote:
> > Add samsung-espresso7 codename for the 7 inch variant
> > 
> > Signed-off-by: Mithil Bavishi <bavishimithil@gmail.com>
> > ---
> >  Documentation/devicetree/bindings/arm/ti/omap.yaml | 1 +
> >  1 file changed, 1 insertion(+)
> >  
> 
> This is v2, so where is the changelog? Same for all other patches. No
> cover letter, no changelogs in patches.
> 
The cover letter including changelog is here:
https://lore.kernel.org/linux-omap/20241030211215.347710-1-bavishimithil@gmail.com/T/#t

seems like it was ripped apart somehow.

Regards,
Andreas
Krzysztof Kozlowski Oct. 31, 2024, 9:49 a.m. UTC | #3
On 31/10/2024 09:24, Andreas Kemnade wrote:
> Am Thu, 31 Oct 2024 09:18:49 +0100
> schrieb Krzysztof Kozlowski <krzk@kernel.org>:
> 
>> On Wed, Oct 30, 2024 at 09:18:43PM +0000, Mithil Bavishi wrote:
>>> Add samsung-espresso7 codename for the 7 inch variant
>>>
>>> Signed-off-by: Mithil Bavishi <bavishimithil@gmail.com>
>>> ---
>>>  Documentation/devicetree/bindings/arm/ti/omap.yaml | 1 +
>>>  1 file changed, 1 insertion(+)
>>>  
>>
>> This is v2, so where is the changelog? Same for all other patches. No
>> cover letter, no changelogs in patches.
>>
> The cover letter including changelog is here:
> https://lore.kernel.org/linux-omap/20241030211215.347710-1-bavishimithil@gmail.com/T/#t
> 
> seems like it was ripped apart somehow.

It's not attached to this thread. I don't have it in my mailbox. Neither
did b4 when applying entire thread for review. Sending something
separately or making it not accessible for review means it does not exist.

Best regards,
Krzysztof
Mithil Bavishi Oct. 31, 2024, 10:46 a.m. UTC | #4
> It's not attached to this thread. I don't have it in my mailbox. Neither
> did b4 when applying entire thread for review. Sending something
> separately or making it not accessible for review means it does not
> exist.

I am not sure why was sent seperately, I'm trying to learn this workflow
so it may have been a ametuer mistake from my side, apologies for that.
I have notes the changes you requested and will try to apply them in v3
along with others mentioned.

Thanks a lot for your patience.

Best Regards,
Mithil
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/arm/ti/omap.yaml b/Documentation/devicetree/bindings/arm/ti/omap.yaml
index 93e04a109..3d09ec1ad 100644
--- a/Documentation/devicetree/bindings/arm/ti/omap.yaml
+++ b/Documentation/devicetree/bindings/arm/ti/omap.yaml
@@ -136,6 +136,7 @@  properties:
               - motorola,droid-bionic   # Motorola Droid Bionic XT875
               - motorola,xyboard-mz609
               - motorola,xyboard-mz617
+              - samsung,espresso7
               - ti,omap4-panda
               - ti,omap4-sdp
           - const: ti,omap4430