Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <17a50d0d-6b00-8966-ae37-d8407c0a89fb@intel.com>
Date:   Tue, 11 Apr 2023 08:38:22 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.9.1
Subject: Re: [PATCH] net: ti/cpsw: Add explicit platform_device.h and
 of_platform.h includes
Content-Language: en-US
To:     Rob Herring <robh@kernel.org>,
        Grygorii Strashko <grygorii.strashko@ti.com>,
        "David S. Miller" <davem@davemloft.net>,
        "Eric Dumazet" <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>
CC:     <linux-omap@vger.kernel.org>, <netdev@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>
References: <20230410232719.1561950-1-robh@kernel.org>
From:   Jesse Brandeburg <jesse.brandeburg@intel.com>
In-Reply-To: <20230410232719.1561950-1-robh@kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Y29pd0YzbkR1Ylc4NFNrTC9Mc09tcnEvRGhQbWlKYy8vRThWNXEzYkZMOWVX?=
 =?utf-8?B?ekVyb3M3ejF5U2hDekVCKzMvZXBoWjU4ait6M1FPbEhmYVRRWFBSaFJGcm12?=
 =?utf-8?B?NGd2ZjlaWUlEakZOQmxzM2pKR2w1WXo1aGZYQXV1ZW1Za2ltd0hJUUwrV3E2?=
 =?utf-8?B?ZERZRytrZER2eGZwSndqTk9oWnpMSTROZWhUUGFCWU1LQnpMVEZFZTVYTExv?=
 =?utf-8?B?Y2RHa0ZGamErSzRkWHNMcnF5dWg4T0ZKc2RibVR5K0lVRXBpNW5oRXJFQVdw?=
 =?utf-8?B?VkUwRTYxVzhPRmNFazZoajhOdjJSaDNZZW1lZUZYdHJOYXRKaVdwZzNYZUI0?=
 =?utf-8?B?V1R0Z1g4T0dMNERuMXQ2L3AwN1c5VHJwb2ZSa0kzV3hMbVdNYko1RVJoME8r?=
 =?utf-8?B?ZkZYVGpCZ1JIV09uTkVJU29rSmNWK1V6UkZSeDNJclZRUVk1LzhXbVhjVnBq?=
 =?utf-8?B?TWhrMDR5cnZSRGpVem5zaDNMZm5BWFIzaE52blFLZzBPS0tGdUQyTERIeHlX?=
 =?utf-8?B?ZkVxZTk0V1poM3owenMwWnhXWUlDK2FBd1V0RWpwS3NEeTBHTWowbG10Lzgw?=
 =?utf-8?B?ZGREUEZMVWtSOGNpRmw4eTJ3UGNpY0wxMVM0YzY3Z1ZFOXB0blhBV1JkZldl?=
 =?utf-8?B?cUIrQzh0V051RWtlVjZLcXNkQlMrd01lSG93QnNMRjVnL1l5V3FOUHdlM2pn?=
 =?utf-8?B?SjUrTDhSSzdPcU9ndU1ocmtNelBidUtmZGxKei9JMUlOVHgyWG1yYllPRUxL?=
 =?utf-8?B?L09za1lVTENQVVRpWXVua1ZjKzMzWkxKQUVDUWpQY0tiVEdCOEdtS2t2Z010?=
 =?utf-8?B?YjBwaFArT3FrM05jUTFpdFljRytsTmxEOHA0ZjdZRHNHVGdFYzBndlg5b0Y3?=
 =?utf-8?B?SUhEUkdwL0RUVDVQTXEwckNOWmp4WkE0dHhUOXg5N3gxQ3JNV3JINmttcXBx?=
 =?utf-8?B?bGV2TitnQlFsL2Fjc1NtZ3FGcWVMdmpFN3BDUWM2WlNRTGhYTlN1VmNTRlkz?=
 =?utf-8?B?WSt5UHNBU1BCYjBTdVliVmNreHZENkFyTCsvNkdoVGc0WkhrL0p5ckpNajRr?=
 =?utf-8?B?RVF5UlNYbHgwVzNUTTlZTENhbXUrUDJ4b2RzZlhYUDU3U1J3eFJkYU9qMXVl?=
 =?utf-8?B?c0ZSZHB1MytBTUZwV2JvUnV4MlpRRnBqTS8vbjBJSktHc2ErdlRpRHdnbjZv?=
 =?utf-8?B?QldjZXU1bTJhZWhUL3FTdEg1Rk1FRGRKVFZPVHZoNlZ6TGxCMHViUTJuU2x6?=
 =?utf-8?B?NWxOSUQremhlcE4xLzU5S0swZ1lvQ2Y0QXVJUzBNYndVQ21JVkU5RmRGYXUv?=
 =?utf-8?B?K3Q0aGxlcXVnN0lTNXc0UlgvQlVkWlN4QWhHTFlrWDVEVlRLZHV6MzZ1aUYv?=
 =?utf-8?B?UzNzdnJsWjE2UjRJVmxiZ0JFMUpNZWpvOUE5cll0bnM5NGlLaDBveE80OUgw?=
 =?utf-8?B?R0RQaklDUXAwWWhlNnFqcjk5OENGUEtpWWcwMFIwdXF0Z2RlRy8yMFQvYXgy?=
 =?utf-8?B?bElVOUhpQitYNWxJNlNmOWlnQ3AyREVPN0JtS2pLSGVGaUJxL0xrRXJpYWlx?=
 =?utf-8?B?b3czMkl5dEE5RzRLWG9RSENuVGIybGJ1cTM3VTVRM3J2bXY2SUx6RjJrWFhN?=
 =?utf-8?B?KzJnMWdkL2dIRngxeGpNcFlpMnRybzFjQUVZTGF5a0RocXhobVNzREREVlZH?=
 =?utf-8?B?QjQ3Ni9SYThYZC9VaGN4dW0xRUcxOXZPK0RXOGlLTFc4VnhNUUFxOGxMQTN6?=
 =?utf-8?B?MGYvVGlUMzZHMlMzZEFkSUtaY3Z5SGVXc29tVUdSQXVpSmJUSGxnMHg4ZUVa?=
 =?utf-8?B?cDdQRk80VXc5RzFyUUhyd21SUU04UXhJOGRPdlZYSkxUTFdKWi9GUUlVNE1M?=
 =?utf-8?B?VHhUbURSUW9GV1ZtUDhLTTU2WjJ5N005bktNQWhpN3J3ZHVxcU9xWVArc04z?=
 =?utf-8?B?UDVJYzAwNHorUUdDd1JEa2k2cjhtajNpWXUwakhyWFFrR2ZibGVQd29QOVps?=
 =?utf-8?B?eTNGOFRrOW1DeGVUb2RLR3g5dmQyM0tmNU9ibkFMYzlpSS9udytpWm12WWJx?=
 =?utf-8?B?V3ZhNXhSR2ErYmhSUXBwa2xTRzloazBYS1NxdEUvS3dvV3A3S2t5YjZaNTk3?=
 =?utf-8?B?VDQyYkVNWmxuV0orek5COGhKSGxkaVlMMEJZNXBzejJTQmVMYlEyQnNpS1N2?=
 =?utf-8?B?bXc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 0f6468fb-54f3-456d-2e3c-08db3aa2c973
X-MS-Exchange-CrossTenant-AuthSource: CO1PR11MB4914.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 11 Apr 2023 15:38:24.4110
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 2dauERgHfMwZSWju3J99RFhq+bF0kAB/J21UDcBImoUzCpaairVkdLmcWokXnB24k+O7ec8nJDIUoo4NApUPdi+MxWeKzy2h2bcnd1+mDR0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB5022
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.7 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_MED,
        RCVD_IN_MSPIKE_H2,SPF_HELO_NONE,SPF_NONE autolearn=unavailable
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 4/10/2023 4:27 PM, Rob Herring wrote:
> TI CPSW uses of_platform_* functions which are declared in of_platform.h.
> of_platform.h gets implicitly included by of_device.h, but that is going
> to be removed soon. Nothing else depends on of_device.h so it can be
> dropped. of_platform.h also implicitly includes platform_device.h, so
> add an explicit include for it, too.
> 
> Signed-off-by: Rob Herring <robh@kernel.org>

Reviewed-by: Jesse Brandeburg <jesse.brandeburg@intel.com>